diff options
author | 2017-09-01 14:30:06 -0700 | |
---|---|---|
committer | 2017-09-01 20:39:12 -0700 | |
commit | 06be580ac7b650938cb3f2cf8cd02d73ff96d3e4 (patch) | |
tree | a4e33abdbe35cbb5028968e615c130b247280d01 | |
parent | hv_netvsc: Simplify num_chn checking in rndis_filter_device_add() (diff) | |
download | linux-06be580ac7b650938cb3f2cf8cd02d73ff96d3e4.tar.gz linux-06be580ac7b650938cb3f2cf8cd02d73ff96d3e4.tar.bz2 linux-06be580ac7b650938cb3f2cf8cd02d73ff96d3e4.zip |
hv_netvsc: Simplify the limit check in netvsc_set_channels()
Because of the following code, net->num_tx_queues equals to
VRSS_CHANNEL_MAX, and max_chn is less than or equals to VRSS_CHANNEL_MAX.
netvsc_drv.c:
alloc_etherdev_mq(sizeof(struct net_device_context),
VRSS_CHANNEL_MAX);
rndis_filter.c:
net_device->max_chn = min_t(u32, VRSS_CHANNEL_MAX, num_possible_rss_qs);
So this patch removes the unnecessary limit check before comparing
with "max_chn".
Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/hyperv/netvsc_drv.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/net/hyperv/netvsc_drv.c b/drivers/net/hyperv/netvsc_drv.c index 992a48ec0206..fd65d32bf24d 100644 --- a/drivers/net/hyperv/netvsc_drv.c +++ b/drivers/net/hyperv/netvsc_drv.c @@ -830,9 +830,6 @@ static int netvsc_set_channels(struct net_device *net, channels->rx_count || channels->tx_count || channels->other_count) return -EINVAL; - if (count > net->num_tx_queues || count > VRSS_CHANNEL_MAX) - return -EINVAL; - if (!nvdev || nvdev->destroy) return -ENODEV; |