Re: [PATCH net-next] net: ena: Add a driver for Amazon Elastic Network Adapters (ENA)

From: Matt Wilson
Date: Wed Jun 15 2016 - 14:07:36 EST


On Mon, Jun 13, 2016 at 11:46:13AM +0300, Netanel Belgazal wrote:
> This is a driver for the forthcoming ENA family of networking devices.
>
> Signed-off-by: Netanel Belgazal <netanel@xxxxxxxxxxxxxxxxx>

[...]

> +
> +struct ena_napi {
> + struct napi_struct napi ____cacheline_aligned;
> + struct ena_ring *tx_ring;
> + struct ena_ring *rx_ring;
> +#ifndef HAVE_NETDEV_NAPI_LIST
> + struct net_device poll_dev;
> +#endif /* HAVE_NETDEV_NAPI_LIST */
> + u32 qid;
> +};

Netanel,

Sorry I missed this when I read through the patch earlier this week
(though I pointed it out before...). This "#ifndef
HAVE_NETDEV_NAPI_LIST" bit needs to be removed for the in-tree
version.

You also had a few typos in the email addresses on the Cc: line that
I've corrected (davm@xxxxxxxxxxxxx -> davem@xxxxxxxxxxxxxx,
aligouri@xxxxxxxxxx -> aliguori@xxxxxxxxxx). Please make sure to
correct in your v2 patch.

--msw