[PATCH] netvsc: don't access netdev->num_rx_queues directly

Haiyang Zhang haiyangz at microsoft.com
Wed Jun 21 22:21:57 UTC 2017



> -----Original Message-----
> From: Arnd Bergmann [mailto:arnd at arndb.de]
> Sent: Wednesday, June 21, 2017 6:17 PM
> To: Stephen Hemminger <sthemmin at microsoft.com>
> Cc: Arnd Bergmann <arnd at arndb.de>; KY Srinivasan <kys at microsoft.com>;
> Haiyang Zhang <haiyangz at microsoft.com>; David S. Miller
> <davem at davemloft.net>; Simon Xiao <sixiao at microsoft.com>;
> devel at linuxdriverproject.org; netdev at vger.kernel.org; linux-
> kernel at vger.kernel.org
> Subject: [PATCH] netvsc: don't access netdev->num_rx_queues directly
> 
> This structure member is hidden behind CONFIG_SYSFS, and we
> get a build error when that is disabled:
> 
> drivers/net/hyperv/netvsc_drv.c: In function 'netvsc_set_channels':
> drivers/net/hyperv/netvsc_drv.c:754:49: error: 'struct net_device' has
> no member named 'num_rx_queues'; did you mean 'num_tx_queues'?
> drivers/net/hyperv/netvsc_drv.c: In function 'netvsc_set_rxfh':
> drivers/net/hyperv/netvsc_drv.c:1181:25: error: 'struct net_device' has
> no member named 'num_rx_queues'; did you mean 'num_tx_queues'?
> 
> As the value is only set once to the argument of alloc_netdev_mq(),
> we can compare against that constant directly.
> 
> Fixes: ff4a44199012 ("netvsc: allow get/set of RSS indirection table")
> Fixes: 2b01888d1b45 ("netvsc: allow more flexible setting of number of
> channels")
> Signed-off-by: Arnd Bergmann <arnd at arndb.de>

Reviewed-by: Haiyang Zhang <haiyangz at microsoft.com>

Thanks.


More information about the devel mailing list