linux.git/drivers/net/hyperv/netvsc.c, branch v4.9.4 Clone of https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git hv_netvsc: use consume_skb 2016-09-23T12:39:48+00:00 Stephen Hemminger sthemmin@microsoft.com 2016-09-22T23:56:29+00:00 17db4bcef3c3c45b95b3b3d8577f725df1b2c0a0 Packets that are transmitted in normal path should use consume_skb instead of kfree_skb. This allows for better tracing of packet drops. Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com> Signed-off-by: David S. Miller <davem@davemloft.net> 
Packets that are transmitted in normal path should use consume_skb
instead of kfree_skb. This allows for better tracing of packet drops.

Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Revert "hv_netvsc: make inline functions static" 2016-09-11T04:23:00+00:00 Stephen Hemminger sthemmin@microsoft.com 2016-09-09T19:45:24+00:00 3a8963acc70e69606729404713cfa9a03b58b18c These functions are used by other code misc-next tree. This reverts commit 30d1de08c87ddde6f73936c3350e7e153988fe02. Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com> Signed-off-by: David S. Miller <davem@davemloft.net> 
These functions are used by other code misc-next tree.

This reverts commit 30d1de08c87ddde6f73936c3350e7e153988fe02.

Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
hv_netvsc: make variable local 2016-08-23T19:05:37+00:00 Stephen Hemminger sthemmin@microsoft.com 2016-08-23T19:17:55+00:00 6c4c137e5035e0e17fa40c223fa0a3167e0f65fa The variable m_ret is only used in one basic block. Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com> Signed-off-by: David S. Miller <davem@davemloft.net> 
The variable m_ret is only used in one basic block.

Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
hv_netvsc: make netvsc_destroy_buf void 2016-08-23T19:05:37+00:00 Stephen Hemminger sthemmin@microsoft.com 2016-08-23T19:17:54+00:00 7a2a0a84fda33062200decf5e201b182591bf0ec No caller checks the return value. Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com> Signed-off-by: David S. Miller <davem@davemloft.net> 
No caller checks the return value.

Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
hv_netvsc: refactor completion function 2016-08-23T19:05:37+00:00 Stephen Hemminger sthemmin@microsoft.com 2016-08-23T19:17:53+00:00 bc304dd3b4444b84082c483534a8dcac7a22cb9d Break the different cases, code is cleaner if broken up Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com> Signed-off-by: David S. Miller <davem@davemloft.net> 
Break the different cases, code is cleaner if broken up

Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
hv_netvsc: init completion during alloc 2016-08-23T19:05:37+00:00 Stephen Hemminger sthemmin@microsoft.com 2016-08-23T19:17:51+00:00 fd612602d6a7919982779fda914bd521e5778593 Move initialization to allocate where other fields are initialized. Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com> Signed-off-by: David S. Miller <davem@davemloft.net> 
Move initialization to allocate where other fields are initialized.

Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
hv_netvsc: make device_remove void 2016-08-23T19:05:37+00:00 Stephen Hemminger sthemmin@microsoft.com 2016-08-23T19:17:50+00:00 e08f3ea586d4145e36c77f0dd1602374b5d7e928 Always returns 0 and no callers check. Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com> Signed-off-by: David S. Miller <davem@davemloft.net> 
Always returns 0 and no callers check.

Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
hv_netvsc: use ARRAY_SIZE() for NDIS versions 2016-08-23T19:05:36+00:00 Stephen Hemminger sthemmin@microsoft.com 2016-08-23T19:17:49+00:00 e5a78fad4f1eb49064e4c35d0a4263424b12124c Don't hard code size of array of NDIS versions. Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com> Signed-off-by: David S. Miller <davem@davemloft.net> 
Don't hard code size of array of NDIS versions.

Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
hv_netvsc: make inline functions static 2016-08-23T19:05:36+00:00 Stephen Hemminger sthemmin@microsoft.com 2016-08-23T19:17:48+00:00 30d1de08c87ddde6f73936c3350e7e153988fe02 Several new functions were introduced into hyperv.h but only used in one file. Move them and let compiler decide on inline. Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com> Signed-off-by: David S. Miller <davem@davemloft.net> 
Several new functions were introduced into hyperv.h but only used in one file.
Move them and let compiler decide on inline.

Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
hv_netvsc: style cleanups 2016-08-23T19:05:36+00:00 Stephen Hemminger sthemmin@microsoft.com 2016-08-23T19:17:47+00:00 796cc88c32c1bd1f833d596448ac785a8736e57c Fix most of the complaints about the style of the code. Things like extra blank lines and return statements. Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com> Signed-off-by: David S. Miller <davem@davemloft.net> 
Fix most of the complaints about the style of the code.
Things like extra blank lines and return statements.

Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
Signed-off-by: David S. Miller <davem@davemloft.net>