RE: [PATCH net-next v2 6/9] net: use core MTU range checking in virt drivers

> -Original Message- > From: Jarod Wilson [mailto:ja...@redhat.com] > Sent: Thursday, October 20, 2016 1:55 PM > To: linux-ker...@vger.kernel.org > Cc: Jarod Wilson ; net...@vger.kernel.org; > virtualization@lists.linux-foundation.org; KY Srinivasan > ; Haiy

RE: [PATCH net-next 5/6] net: use core MTU range checking in virt drivers

> -Original Message- > From: Jarod Wilson [mailto:ja...@redhat.com] > Sent: Tuesday, October 18, 2016 10:34 PM > To: linux-ker...@vger.kernel.org > Cc: Jarod Wilson ; net...@vger.kernel.org; > virtualization@lists.linux-foundation.org; KY Srinivasan > ; Haiy

[PATCH 08/10] staging: hv: fix the return status of netvsc_start_xmit()

Fix the return status, so the upper layer will retry if transmission fails. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc_drv.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/hv/netvsc_drv.c b/drivers

[PATCH 05/10] staging: hv: re-order the code in netvsc_probe()

Re-order the code in netvsc_probe() to prevent a guest crash caused by packets possibly received from NetVSP before call to register_netdev(). Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc_drv.c | 32 +--- 1 files

[PATCH 09/10] staging: hv: fix the page buffer when rndis data go across page boundary

In rndis_filter_receive_data(), we need to drop the 0th page and move the rest of pages forward if the rndis data go across page boundary, otherwise the page offset will overflow. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/rndis_filter.c | 10

[PATCH 03/10] staging: hv: convert dev_ to netdev_ in netvsc

Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c | 68 + drivers/staging/hv/rndis_filter.c | 17 + 2 files changed, 49 insertions(+), 36 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b

[PATCH 04/10] staging: hv: fix a kernel warning in netvsc_linkstatus_callback()

ack-ported to stable kernels 2.6.32 and later. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan Cc: stable --- drivers/staging/hv/netvsc_drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv/netvsc_drv.c b/drivers/staging/hv/netvsc_d

[PATCH 06/10] staging: hv: fix counting of #outstanding-sends in failed sends

If the packet failed to be sent, we shouldn't count it as the number of outstanding sends. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv/netvs

[PATCH 07/10] staging: hv: fix counting of available buffer slots when send fails

Because the number of available buffer slots doesn't decrease for failed sends, we should not call netvsc_xmit_completion(), which increase the count of available slots. In this failed case, just free the memory is enough. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srini

[PATCH 10/10] staging: hv: fix some typos in netvsc.c

Reported-by: Joe Perches Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging/hv/netvsc.c index efbc8a0..115629f 100644 --- a

[PATCH 02/10] staging: hv: add newline to log messages in netvsc

Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c | 58 +- 1 files changed, 29 insertions(+), 29 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging/hv/netvsc.c index 9a9228c..ef764e6

[PATCH 01/10] staging: hv: remove unnecessary includes in netvsc

hyperv.h is included by hyperv_net.h already, so no need to include it again in these C files. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c |1 - drivers/staging/hv/netvsc_drv.c |1 - drivers/staging/hv/rndis_filter.c |1 - 3

[PATCH 00/10] netvsc bug fixes and cleanups

Fixed a set of bugs in netvsc module, and cleaned up some coding style issues. Haiyang Zhang (10): staging: hv: remove unnecessary includes in netvsc staging: hv: add newline to log messages in netvsc staging: hv: convert dev_ to netdev_ in netvsc staging: hv: fix a kernel warning in

[PATCH] staging: hv: fix a memory leak in adj_guesttime()

The allocated struct adj_time_work needs to be freed if we are not using it. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/hv_util.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/drivers/staging/hv/hv_util.c b/drivers/staging

[PATCH] staging: hv: fix some typos in netvsc.c

Reported-by: Joe Perches Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging/hv/netvsc.c index 2b33e13..23328aa 100644 --- a

RE: [PATCH 2/9] staging: hv: add newline to log messages in netvsc

> -Original Message- > From: Joe Perches [mailto:j...@perches.com] > Sent: Thursday, July 21, 2011 9:36 PM > Just noticing some trivial typos... > > > diff --git a/drivers/staging/hv/netvsc.c > b/drivers/staging/hv/netvsc.c > [] > > @@ -246,7 +246,7 @@ static int netvsc_init_recv_buf(struc

[PATCH 5/9] staging: hv: re-order the code in netvsc_probe()

Re-order the code in netvsc_probe() to prevent a guest crash caused by packets possibly received from NetVSP before call to register_netdev(). Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc_drv.c | 32 +--- 1 files

[PATCH 4/9] staging: hv: fix a kernel warning in netvsc_linkstatus_callback()

ack-ported to stable kernels 2.6.32 and later. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan Cc: stable --- drivers/staging/hv/netvsc_drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv/netvsc_drv.c b/drivers/staging/hv/netvsc_d

[PATCH 2/9] staging: hv: add newline to log messages in netvsc

Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c | 58 +++--- drivers/staging/hv/netvsc_drv.c |2 +- 2 files changed, 30 insertions(+), 30 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers

[PATCH 7/9] staging: hv: fix counting of available buffer slots when send fails

Because the number of available buffer slots doesn't decrease for failed sends, we should not call netvsc_xmit_completion(), which increase the count of available slots. In this failed case, just free the memory is enough. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srini

[PATCH 6/9] staging: hv: fix counting of #outstanding-sends in failed sends

If the packet failed to be sent, we shouldn't count it as the number of outstanding sends. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv/netvs

[PATCH 9/9] staging: hv: fix the page buffer when rndis data go across page boundary

In rndis_filter_receive_data(), we need to drop the 0th page and move the rest of pages forward if the rndis data go across page boundary, otherwise the page offset will overflow. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/rndis_filter.c | 10

[PATCH 3/9] staging: hv: convert dev_ to netdev_ in netvsc

Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c | 68 + drivers/staging/hv/rndis_filter.c | 17 + 2 files changed, 49 insertions(+), 36 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b

[PATCH 8/9] staging: hv: fix the return status of netvsc_start_xmit()

Fix the return status, so the upper layer will retry if transmission fails. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc_drv.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/hv/netvsc_drv.c b/drivers

[PATCH 1/9] staging: hv: remove unnecessary includes in netvsc

hyperv.h is included by hyperv_net.h already, so no need to include it again in these C files. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c |1 - drivers/staging/hv/netvsc_drv.c |1 - drivers/staging/hv/rndis_filter.c |1 - 3

[PATCH 0/9] netvsc bug fixes and cleanups

Fixed a set of bugs in netvsc module, and cleaned up some coding style issues. Haiyang Zhang (9): staging: hv: remove unnecessary includes in netvsc staging: hv: add newline to log messages in netvsc staging: hv: convert dev_ to netdev_ in netvsc staging: hv: fix a kernel warning in

RE: [PATCH 5/5] staging: hv: fix a kernel warning in netvsc_linkstatus_callback()

> -Original Message- > From: Greg KH [mailto:g...@kroah.com] > Sent: Tuesday, July 05, 2011 12:10 PM > To: Haiyang Zhang > Cc: Hank Janssen; KY Srinivasan; Abhishek Kane (Mindtree Consulting PVT > LTD); gre...@suse.de; linux-ker...@vger.kernel.org; > de...@l

RE: [PATCH 1/5] staging: hv: fix some white spaces in netvsc driver

> -Original Message- > From: Nicolas Kaiser [mailto:ni...@nikai.net] > Sent: Friday, June 17, 2011 12:03 PM > To: Haiyang Zhang > Cc: Hank Janssen; KY Srinivasan; Abhishek Kane (Mindtree Consulting > PVT LTD); gre...@suse.de; linux-ker...@vger.kernel.org; > de...@l

[PATCH 1/5] staging: hv: fix some white spaces in netvsc driver

Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c |8 drivers/staging/hv/netvsc_drv.c |2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging

[PATCH 5/5] staging: hv: fix a kernel warning in netvsc_linkstatus_callback()

2.6.32 and later. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan Cc: stable --- drivers/staging/hv/netvsc_drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv/netvsc_drv.c b/drivers/staging/hv/netvsc_drv.c index c9ed19a..2299b88 100644

[PATCH 4/5] staging: hv: convert dev_ to netdev_ in netvsc

Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c | 71 +--- drivers/staging/hv/rndis_filter.c | 17 + 2 files changed, 51 insertions(+), 37

[PATCH 3/5] staging: hv: add newline to log messages in netvsc

Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c | 60 +++--- drivers/staging/hv/netvsc_drv.c |2 +- 2 files changed, 31 insertions(+), 31 deletions

[PATCH 2/5] staging: hv: remove unnecessary includes in netvsc

hyperv.h is included by hyperv_net.h already, so no need to include it again in these C files. Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c |1 - drivers/staging/hv/netvsc_drv.c

[PATCH 1/2] staging: hv: convert DPRINT_DBG() to netdev_dbg() in dump_rndis_message()

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen Signed-off-by: K. Y. Srinivasan Cc: Stephen Hemminger --- drivers/staging/hv/rndis_filter.c | 29 - 1 files changed, 16 insertions(+), 13 deletions(-) diff --git a/drivers/staging/hv/rndis_filter.c b

[PATCH 2/2] staging: hv: use delayed_work for netvsc_send_garp()

Instead of sleeping in a scheduled work, we now use delayed_work for netvsc_send_garp(). Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen Signed-off-by: K. Y. Srinivasan Cc: Stephen Hemminger --- drivers/staging/hv/netvsc_drv.c | 15 +-- 1 files changed, 9 insertions

[PATCH 1/1] staging: hv: remove netvsc send buffer and related functions

netvsc send buffer is not used, so remove it. Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/hyperv_net.h | 10 --- drivers/staging/hv/netvsc.c | 161 --- 2 files changed, 0 insertions

[PATCH 5/6] staging: hv: fix typo in name rndis_filte_device_add()

rename rndis_filte_device_add to rndis_filter_device_add Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/hyperv_net.h |2 +- drivers/staging/hv/netvsc_drv.c |2 +- drivers/staging/hv

[PATCH 4/6] staging: hv: remove commented out code in netvsc_remove()

Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/netvsc_drv.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv/netvsc_drv.c b/drivers/staging/hv/netvsc_drv.c

[PATCH 1/6] staging: hv: remove unnecessary code in netvsc_probe().

netif_carrier_off() was called earlier in this function, and there is no other thread access this device yet. The status checking code is not necessary here. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers

[PATCH 3/6] staging: hv: change rndis_filter_device_remove() to void return type

rndis_filter_device_remove() always return 0, so change it to void return type. Also cleaned up the error checking in the caller. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/hyperv_net.h

[PATCH 2/6] staging: hv: remove commented out code from netvsc_drv.c

Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/netvsc_drv.c |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/drivers/staging/hv/netvsc_drv.c b/drivers/staging/hv

[PATCH 6/6] staging: hv: removed commented out code from rndis_filter_receive()

Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/rndis_filter.c | 18 -- 1 files changed, 0 insertions(+), 18 deletions(-) diff --git a/drivers/staging/hv/rndis_filter.c b/drivers

[PATCH 3/8] staging: hv: move netvsc_send_recv_completion() to clean up forward declaration

Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c | 87 +-- 1 files changed, 42 insertions(+), 45 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging/hv/netvsc.c index edd2f4e..ea3c72e

[PATCH 8/8] staging: hv: convert function name NetVscDisconnectFromVsp to lower case

Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging/hv/netvsc.c index 5838ed4..b3e6497 100644 --- a/drivers/staging/hv/netvsc.c

[PATCH 7/8] staging: hv: clean up unused forward declarations

Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c | 12 1 files changed, 0 insertions(+), 12 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging/hv/netvsc.c index 1c3bdda..5838ed4 100644 --- a/drivers/staging/hv

[PATCH 4/8] staging: hv: move netvsc_destroy_recv_buf() to clean up forward declaration

Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c | 138 +- 1 files changed, 69 insertions(+), 69 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging/hv/netvsc.c index ea3c72e..8b2defd

[PATCH 2/8] staging: hv: move netvsc_receive_completion() to clean up forward declaration

Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c | 102 +-- 1 files changed, 50 insertions(+), 52 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging/hv/netvsc.c index 6eaecc1..edd2f4e

[PATCH 1/8] staging: hv: move netvsc_initialize() to clean up forward declaration

Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c | 51 +- 1 files changed, 21 insertions(+), 30 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging/hv/netvsc.c index e8c15d2..6eaecc1

[PATCH 0/8] staging: hv: clean up forward declarations and camel cases in netvsc.c

From: haiyangz This patch series cleans up forward declarations and the remaining camel cases in netvsc.c Haiyang Zhang (8): staging: hv: move netvsc_initialize() to clean up forward declaration staging: hv: move netvsc_receive_completion() to clean up forward declaration staging: hv

[PATCH 5/8] staging: hv: move netvsc_destroy_send_buf() to clean up forward declaration

Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c | 130 +-- 1 files changed, 64 insertions(+), 66 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging/hv/netvsc.c index 8b2defd..75640ea

[PATCH 6/8] staging: hv: move netvsc_device_add() to clean up forward declaration

Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c | 176 +-- 1 files changed, 87 insertions(+), 89 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging/hv/netvsc.c index 75640ea..1c3bdda

[PATCH] staging: hv: use sync_bitops when interacting with the hypervisor

From: Olaf Hering Locking is required when tweaking bits located in a shared page, use the sync_ version of bitops. Without this change vmbus_on_event() will miss events and as a result, vmbus_isr() will not schedule the receive tasklet. Backported to 2.6.32 stable kernel by Haiyang Zhang

[PATCH] staging: hv: Fix GARP not sent after Quick Migration

. Thanks to Mike Surcouf for reporting the bug and testing the patch. This patch should also be backported to stable kernel 2.6.32 and later. Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen Signed-off-by: Abhishek Kane Signed-off-by: K. Y. Srinivasan Cc: Mike Surcouf Cc: stable

RE: [PATCH] staging: hv: Fix GARP not sent after Quick Migration

> -Original Message- > From: Greg KH [mailto:g...@kroah.com] > Sent: Tuesday, April 05, 2011 12:23 PM > Instead, use a proper timer as you do know how long you need to wait, > right? Don't abuse workqueues for something they were not designed for. Thanks for the suggestion, I will add an

[PATCH] staging: hv: Fix GARP not sent after Quick Migration

to Mike Surcouf for reporting the bug and testing the patch. This patch should also be backported to stable kernel 2.6.32 and later. Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen Signed-off-by: Abhishek Kane Signed-off-by: K. Y. Srinivasan Cc: Mike Surcouf Cc: stable

RE: [PATCH] staging: hv: Fix GARP not sent after Quick Migration

> -Original Message- > From: Greg KH [mailto:gre...@suse.de] > Sent: Tuesday, April 05, 2011 11:42 AM > But that's a hypervisor bug, right? It shouldn't be sending a "connect" > event on the ethernet device if the network really isn't present. How > about getting it fixed there, which wil

[PATCH] staging: hv: Fix GARP not sent after Quick Migration

to Mike Surcouf for reporting the bug and testing the patch. This patch should also be backported to stable kernel 2.6.32 and later. Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen Signed-off-by: Abhishek Kane Signed-off-by: K. Y. Srinivasan Cc: Mike Surcouf Cc: stable

RE: [PATCH 1/4] staging: hv: Fix the code depending on struct vmbus_driver_context data order

> From: Greg KH [mailto:g...@kroah.com] > The layering is almost ok, there is still one more layer here than is > needed, and it should be removed (I already removed lots of layers that > were not needed, just didn't get to this one.) But the naming also > needs to be fixed up as it is wrong from

RE: [PATCH 1/4] staging: hv: Fix the code depending on struct vmbus_driver_context data order

> From: Greg KH [mailto:g...@kroah.com] > Sent: Wednesday, February 23, 2011 4:27 PM > > struct driver_context { > > struct hv_guid class_id; > > - > > struct device_driver driver; > > + struct hv_driver *hv_drv; > > If you have a pointer to hv_driver, why do you need a full 'struct > d

[PATCH 2/4] staging: hv: Fix the code depending on struct netvsc_driver_context data order

The patch fixed the code depending on the exact order of fields in the struct netvsc_driver_context. Now, we use container_of() instead of type casting from the first field to the container struct. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan Signed-off-by: Hank Janssen

[PATCH 4/4] staging: hv: Fix the code depending on struct storvsc_driver_context data order

The patch fixed the code depending on the exact order of fields in the struct storvsc_driver_context. Now, we use container_of() instead of type casting from the first field to the container struct. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan Signed-off-by: Hank Janssen

[PATCH 1/4] staging: hv: Fix the code depending on struct vmbus_driver_context data order

The patch fixed the code depending on the exact order of fields in the struct vmbus_driver_context, so the unused field drv_ctx can be removed, and drv_obj doesn't have to be the second field in this structure. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan Signed-off-by:

[PATCH 3/4] staging: hv: Fix the code depending on struct blkvsc_driver_context data order

The patch fixed the code depending on the exact order of fields in the struct blkvsc_driver_context. Now, we use container_of() instead of type casting from the first field to the container struct. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan Signed-off-by: Hank Janssen

[PATCH 2/2] staging: hv: Fix the WARN_ON condition in free_net_device()

In a previous commit, 7a09876d, ASSERT was changed to WARN_ON, but the condition wasn't updated. This patch fixed this error. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan Signed-off-by: Hank Janssen --- drivers/staging/hv/netvsc.c |2 +- 1 files changed, 1 inser

[PATCH 1/2] staging: hv: Remove unnecessary ASSERTs in netvsc_initialize()

These fields have been assigned in netvsc_drv_init() before calling netvsc_initialize(), so there is no need to check them. The ASSERTs were already commented out, and this patch removes them. Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan Signed-off-by: Hank Janssen

[PATCH 1/2] staging: hv: Remove dead code from netvsc.c

Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/netvsc.c | 34 -- 1 files changed, 0 insertions(+), 34 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging/hv/netvsc.c index fa46a7e..8c6d4ae 100644 --- a

[PATCH 2/2] staging: hv: Remove dead code from rndis_filter.c

Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/staging/hv/rndis_filter.c | 15 --- 1 files changed, 0 insertions(+), 15 deletions(-) diff --git a/drivers/staging/hv/rndis_filter.c b/drivers/staging/hv/rndis_filter.c index 9dde936..e7189cd 100644 --- a

[PATCH] staging: hv: Enable sending GARP packet after live migration

ed to stable kernel 2.6.32 and later. Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen Cc: stable --- drivers/staging/hv/netvsc_drv.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/staging/hv/netvsc_drv.c b/drivers/staging/hv/netvsc_drv.c index 54

[PATCH 6/8] staging: hv: Convert camel cased struct fields in vmbus_channel_interface.h to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c |4 ++-- drivers/staging/hv/channel_mgmt.c| 20 ++-- drivers/staging/hv/vmbus_channel_interface.h | 26 +- 3 files changed, 25

[PATCH 7/8] staging: hv: Convert camel cased struct fields in vmbus_packet_format.h to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/blkvsc_drv.c |2 +- drivers/staging/hv/channel.c | 56 +++--- drivers/staging/hv/channel_mgmt.c|2 +- drivers/staging/hv/hv_kvp.c |4 +- drivers

[PATCH 8/8] staging: hv: Convert camel cased struct fields in vmbus_private.h to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 22 +- drivers/staging/hv/channel_mgmt.c | 16 drivers/staging/hv/connection.c| 74 ++-- drivers/staging/hv/vmbus_drv.c |2

RE: [PATCH 7/8] staging: hv: Convert camel cased struct fields in vmbus_packet_format.h to lower cases

> From: Dan Carpenter [mailto:erro...@gmail.com] > > + VM_PKT_INVALID = 0x0, > > + VM_PKT_SYNCH= 0x1, > > + VM_PKT_ADD_XFER_PAGESET = 0x2, > > + VM_PKT_RM_XFER_PAGESET = 0x3, > ^^

[PATCH 5/8] staging: hv: Convert camel cased struct fields in vmbus_api.h to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/blkvsc.c | 20 ++-- drivers/staging/hv/blkvsc_drv.c | 56 ++-- drivers/staging/hv/channel.c | 20 ++-- drivers/staging/hv/channel_mgmt.c |2 +- drivers/staging/hv/netvsc.c | 58

[PATCH 1/8] staging: hv: Convert camel cased variables in connection.c to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 48 ++-- drivers/staging/hv/channel_mgmt.c | 48 ++-- drivers/staging/hv/connection.c| 154 ++-- drivers/staging/hv/vmbus_drv.c |2

[PATCH 2/8] staging: hv: Convert camel cased functions in connection.c to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 12 +- drivers/staging/hv/channel_mgmt.c |4 +- drivers/staging/hv/connection.c| 39 +++ drivers/staging/hv/vmbus_drv.c |6 ++-- drivers

[PATCH 4/8] staging: hv: Convert camel cased functions in vmbus_drv.c to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel_mgmt.c |4 ++-- drivers/staging/hv/vmbus_drv.c | 22 +++--- drivers/staging/hv/vmbus_private.h |6 +++--- 3 files changed, 16 insertions(+), 16 deletions(-) diff --git a

[PATCH 3/8] staging: hv: Convert camel cased variables in vmbus_drv.c to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/vmbus_drv.c | 50 1 files changed, 25 insertions(+), 25 deletions(-) diff --git a/drivers/staging/hv/vmbus_drv.c b/drivers/staging/hv/vmbus_drv.c index b33f497..99686f0

[PATCH 7/8] staging: hv: Convert camel cased struct fields in vmbus_packet_format.h to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/blkvsc_drv.c |2 +- drivers/staging/hv/channel.c | 56 +++--- drivers/staging/hv/channel_mgmt.c|2 +- drivers/staging/hv/hv_kvp.c |4 +- drivers

[PATCH 1/8] staging: hv: Convert camel cased variables in connection.c to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 48 ++-- drivers/staging/hv/channel_mgmt.c | 48 ++-- drivers/staging/hv/connection.c| 154 ++-- drivers/staging/hv/vmbus_drv.c |2

[PATCH 3/8] staging: hv: Convert camel cased variables in vmbus_drv.c to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/vmbus_drv.c | 50 1 files changed, 25 insertions(+), 25 deletions(-) diff --git a/drivers/staging/hv/vmbus_drv.c b/drivers/staging/hv/vmbus_drv.c index b33f497..99686f0

[PATCH 8/8] staging: hv: Convert camel cased struct fields in vmbus_private.h to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 22 +- drivers/staging/hv/channel_mgmt.c | 16 drivers/staging/hv/connection.c| 74 ++-- drivers/staging/hv/vmbus_drv.c |2

[PATCH 2/8] staging: hv: Convert camel cased functions in connection.c to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 12 +- drivers/staging/hv/channel_mgmt.c |4 +- drivers/staging/hv/connection.c| 39 +++ drivers/staging/hv/vmbus_drv.c |6 ++-- drivers

[PATCH 6/8] staging: hv: Convert camel cased struct fields in vmbus_channel_interface.h to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c |4 ++-- drivers/staging/hv/channel_mgmt.c| 20 ++-- drivers/staging/hv/vmbus_channel_interface.h | 26 +- 3 files changed, 25

[PATCH 5/8] staging: hv: Convert camel cased struct fields in vmbus_api.h to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/blkvsc.c | 20 ++-- drivers/staging/hv/blkvsc_drv.c | 56 ++-- drivers/staging/hv/channel.c | 20 ++-- drivers/staging/hv/channel_mgmt.c |2 +- drivers/staging/hv/netvsc.c | 58

[PATCH 4/8] staging: hv: Convert camel cased functions in vmbus_drv.c to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel_mgmt.c |4 ++-- drivers/staging/hv/vmbus_drv.c | 22 +++--- drivers/staging/hv/vmbus_private.h |6 +++--- 3 files changed, 16 insertions(+), 16 deletions(-) diff --git a

[PATCH 5/7] staging: hv: Convert camel cased struct fields in netvsc_api.h to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/netvsc.c | 104 +- drivers/staging/hv/netvsc_api.h | 58 ++-- drivers/staging/hv/netvsc_drv.c | 74 drivers/staging/hv

[PATCH 6/7] staging: hv: Convert camel cased struct fields in netvsc.h to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/netvsc.c | 340 +++-- drivers/staging/hv/netvsc.h | 164 +- drivers/staging/hv/netvsc_api.h |2 +- drivers/staging/hv/rndis_filter.c | 22 ++-- 4

[PATCH 1/7] staging: hv: Convert camel cased variables in netvsc.c to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/netvsc.c | 852 ++- 1 files changed, 445 insertions(+), 407 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging/hv/netvsc.c index 8022781..1c1ee57 100644

[PATCH 7/7] staging: hv: Convert camel cased struct fields in rndis.h to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/rndis.h| 353 +++-- drivers/staging/hv/rndis_filter.c | 204 +++--- 2 files changed, 279 insertions(+), 278 deletions(-) diff --git a/drivers/staging/hv

[PATCH 3/7] staging: hv: Convert camel cased variables in rndis_filter.c to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/rndis_filter.c | 582 +++-- 1 files changed, 296 insertions(+), 286 deletions(-) diff --git a/drivers/staging/hv/rndis_filter.c b/drivers/staging/hv/rndis_filter.c index b85c825

[PATCH 2/7] staging: hv: Convert camel cased functions in netvsc.c to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/netvsc.c | 173 --- drivers/staging/hv/netvsc_api.h |2 +- drivers/staging/hv/netvsc_drv.c |2 +- 3 files changed, 90 insertions(+), 87 deletions(-) diff --git a

[PATCH 4/7] staging: hv: Convert camel cased functions in rndis_filter.c to lower cases

Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/netvsc.c |2 +- drivers/staging/hv/netvsc_api.h |4 +- drivers/staging/hv/netvsc_drv.c |4 +- drivers/staging/hv/rndis_filter.c | 132 ++-- drivers/staging/hv

[PATCH 07/10] staging: hv: Convert camel cased functions in osd.c to lower cases

From: Haiyang Zhang staging: hv: Convert camel cased functions in osd.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 20 +- drivers/staging/hv/channel_mgmt.c | 12 +- drivers/staging/hv

[PATCH 02/10] staging: hv: Convert camel cased struct fields in hv_api.h to lower cases

From: Haiyang Zhang staging: hv: Convert camel cased struct fields in hv_api.h to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 32 ++-- drivers/staging/hv/channel_mgmt.c |6 +- drivers/staging/hv/connection.c |4

[PATCH 03/10] staging: hv: Convert camel cased struct fields in hv.h to lower cases

From: Haiyang Zhang staging: hv: Convert camel cased struct fields in hv.h to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/hv.c| 95 +++- drivers/staging/hv/hv.h| 20 +- drivers

[PATCH 10/10] staging: hv: Convert camel cased functions in ring_buffer.c to lower cases

From: Haiyang Zhang staging: hv: Convert camel cased functions in ring_buffer.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 40 ++-- drivers/staging/hv/ring_buffer.c | 123

[PATCH 05/10] staging: hv: Convert camel cased functions in hv.c to lower cases

From: Haiyang Zhang staging: hv: Convert camel cased functions in hv.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/connection.c |4 +- drivers/staging/hv/hv.c | 47 -- drivers/staging/hv

[PATCH 04/10] staging: hv: Convert camel cased local variables in hv.c to lower cases

From: Haiyang Zhang staging: hv: Convert camel cased local variables in hv.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/hv.c | 172 +++--- 1 files changed, 86 insertions(+), 86 deletions(-) diff

[PATCH 06/10] staging: hv: Convert camel cased local variables in osd.c to lower cases

From: Haiyang Zhang staging: hv: Convert camel cased local variables in osd.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/osd.c | 40 1 files changed, 20 insertions(+), 20 deletions(-) diff --git

[PATCH 09/10] staging: hv: Convert camel cased local variables in ring_buffer.c to lower cases

From: Haiyang Zhang staging: hv: Convert camel cased local variables in ring_buffer.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c |4 +- drivers/staging/hv/ring_buffer.c | 337

  1   2   >