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

2016-10-20 Thread Haiyang Zhang via Virtualization
.org; KY Srinivasan > <k...@microsoft.com>; Haiyang Zhang <haiya...@microsoft.com>; Michael S. > Tsirkin <m...@redhat.com>; Shrikrishna Khare <skh...@vmware.com>; VMware, > Inc. <pv-driv...@vmware.com>; Wei Liu <wei.l...@citrix.com>; Paul > Durrant <paul.durr...@

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

2016-10-19 Thread Haiyang Zhang via Virtualization
.org; KY Srinivasan > <k...@microsoft.com>; Haiyang Zhang <haiya...@microsoft.com>; Michael S. > Tsirkin <m...@redhat.com>; Shrikrishna Khare <skh...@vmware.com>; VMware, > Inc. <pv-driv...@vmware.com> > Subject: [PATCH net-next 5/6] net: use core MTU range checking in v

[PATCH 00/10] netvsc bug fixes and cleanups

2011-09-01 Thread Haiyang Zhang
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_loglevel to netdev_loglevel in netvsc staging: hv: fix a kernel

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

2011-09-01 Thread Haiyang Zhang
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 haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc.c |1 - drivers/staging/hv/netvsc_drv.c |1

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

2011-09-01 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc.c | 58 +- 1 files changed, 29 insertions(+), 29 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging

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

2011-09-01 Thread Haiyang Zhang
Reported-by: Joe Perches j...@perches.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers

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

2011-09-01 Thread Haiyang Zhang
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 haiya...@microsoft.com Signed-off-by: K. Y

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

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

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

2011-09-01 Thread Haiyang Zhang
to stable kernels 2.6.32 and later. Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: stable sta...@kernel.org --- drivers/staging/hv/netvsc_drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv

[PATCH 03/10] staging: hv: convert dev_loglevel to netdev_loglevel in netvsc

2011-09-01 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc.c | 68 + drivers/staging/hv/rndis_filter.c | 17 + 2 files changed, 49 insertions(+), 36 deletions(-) diff

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

2011-09-01 Thread Haiyang Zhang
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 haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers

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

2011-09-01 Thread Haiyang Zhang
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 haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc_drv.c | 32

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

2011-09-01 Thread Haiyang Zhang
Fix the return status, so the upper layer will retry if transmission fails. Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc_drv.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git

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

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

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

2011-07-22 Thread Haiyang Zhang
-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(struct

[PATCH 0/9] netvsc bug fixes and cleanups

2011-07-21 Thread Haiyang Zhang
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_loglevel to netdev_loglevel in netvsc staging: hv: fix a kernel

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

2011-07-21 Thread Haiyang Zhang
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 haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc.c |1 - drivers/staging/hv/netvsc_drv.c |1

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

2011-07-21 Thread Haiyang Zhang
Fix the return status, so the upper layer will retry if transmission fails. Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc_drv.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git

[PATCH 3/9] staging: hv: convert dev_loglevel to netdev_loglevel in netvsc

2011-07-21 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc.c | 68 + drivers/staging/hv/rndis_filter.c | 17 + 2 files changed, 49 insertions(+), 36 deletions(-) diff

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

2011-07-21 Thread Haiyang Zhang
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 haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers

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

2011-07-21 Thread Haiyang Zhang
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 haiya...@microsoft.com Signed-off-by: K. Y

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

2011-07-21 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc.c | 58 +++--- drivers/staging/hv/netvsc_drv.c |2 +- 2 files changed, 30 insertions(+), 30 deletions(-) diff --git

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

2011-07-21 Thread Haiyang Zhang
to stable kernels 2.6.32 and later. Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: stable sta...@kernel.org --- drivers/staging/hv/netvsc_drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv

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

2011-07-21 Thread Haiyang Zhang
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 haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc_drv.c | 32

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

2011-07-05 Thread Haiyang Zhang
-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...@linuxdriverproject.org; virtualizat

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

2011-06-17 Thread Haiyang Zhang
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 haiya...@microsoft.com Signed-off-by: Abhishek Kane v-abk...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com Signed-off-by: K. Y. Srinivasan k

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

2011-06-17 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Abhishek Kane v-abk...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc.c | 60 +++--- drivers

[PATCH 4/5] staging: hv: convert dev_loglevel to netdev_loglevel in netvsc

2011-06-17 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Abhishek Kane v-abk...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc.c | 71 +--- drivers

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

2011-06-17 Thread Haiyang Zhang
2.6.32 and later. Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: stable sta...@kernel.org --- 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

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

2011-06-17 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Abhishek Kane v-abk...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc.c |8 drivers/staging/hv/netvsc_drv.c |2

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

2011-05-27 Thread Haiyang Zhang
Instead of sleeping in a scheduled work, we now use delayed_work for netvsc_send_garp(). Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: Stephen Hemminger shemmin...@vyatta.com

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

2011-05-27 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: Stephen Hemminger shemmin...@vyatta.com --- drivers/staging/hv/rndis_filter.c | 29 - 1 files changed, 16

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

2011-05-25 Thread Haiyang Zhang
netvsc send buffer is not used, so remove it. Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/hyperv_net.h | 10 --- drivers/staging/hv/netvsc.c | 161

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

2011-05-23 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Abhishek Kane v-abk...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/rndis_filter.c | 18 -- 1 files changed, 0

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

2011-05-23 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Abhishek Kane v-abk...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/netvsc_drv.c |3 --- 1 files changed, 0 insertions(+), 3

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

2011-05-23 Thread Haiyang Zhang
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 haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Abhishek Kane v-abk...@microsoft.com Signed-off

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

2011-05-23 Thread Haiyang Zhang
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 haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Abhishek Kane v-abk

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

2011-05-23 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Abhishek Kane v-abk...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/netvsc_drv.c |1 - 1 files changed, 0 insertions(+), 1

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

2011-05-23 Thread Haiyang Zhang
rename rndis_filte_device_add to rndis_filter_device_add Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Abhishek Kane v-abk...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv

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

2011-04-21 Thread Haiyang Zhang
From: haiyangz haiya...@hz-dev.msft.interop.novell.com 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

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

2011-04-21 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc.c | 176 +-- 1 files changed, 87 insertions(+), 89 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging

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

2011-04-21 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc.c | 102 +-- 1 files changed, 50 insertions(+), 52 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging

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

2011-04-21 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc.c | 138 +- 1 files changed, 69 insertions(+), 69 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging

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

2011-04-21 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- 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

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

2011-04-21 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- 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

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

2011-04-21 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/netvsc.c | 87 +-- 1 files changed, 42 insertions(+), 45 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers/staging

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

2011-04-05 Thread Haiyang Zhang
-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 will

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

2011-04-05 Thread Haiyang Zhang
to Mike Surcouf m...@surcouf.co.uk 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 haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com Signed-off-by: Abhishek Kane v-abk

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

2011-04-05 Thread Haiyang Zhang
-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 1/4] staging: hv: Fix the code depending on struct vmbus_driver_context data order

2011-02-23 Thread Haiyang Zhang
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 haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k

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

2011-02-23 Thread Haiyang Zhang
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 haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k

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

2011-02-23 Thread Haiyang Zhang
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 haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k

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

2011-02-23 Thread Haiyang Zhang
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 device_driver'

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

2011-02-18 Thread Haiyang Zhang
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 haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com

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

2011-02-18 Thread Haiyang Zhang
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 haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers

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

2011-02-16 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- 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

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

2011-02-02 Thread Haiyang Zhang
to stable kernel 2.6.32 and later. Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com Cc: stable sta...@kernel.org --- drivers/staging/hv/netvsc_drv.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/staging/hv

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

2011-01-26 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- 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

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

2011-01-26 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- 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

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

2011-01-26 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c |4 ++-- drivers/staging/hv/channel_mgmt.c| 20 ++-- drivers/staging/hv/vmbus_channel_interface.h | 26

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

2011-01-26 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c | 12 +- drivers/staging/hv/channel_mgmt.c |4 +- drivers/staging/hv/connection.c| 39 +++ drivers

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

2011-01-26 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c | 22 +- drivers/staging/hv/channel_mgmt.c | 16 drivers/staging/hv/connection.c| 74

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

2011-01-26 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- 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

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

2011-01-26 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/blkvsc_drv.c |2 +- drivers/staging/hv/channel.c | 56 +++--- drivers/staging/hv/channel_mgmt.c|2 +- drivers/staging/hv

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

2011-01-26 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- 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

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

2011-01-26 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- 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

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

2011-01-26 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c | 12 +- drivers/staging/hv/channel_mgmt.c |4 +- drivers/staging/hv/connection.c| 39 +++ drivers

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

2011-01-26 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- 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

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

2011-01-26 Thread Haiyang Zhang
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 8/8] staging: hv: Convert camel cased struct fields in vmbus_private.h to lower cases

2011-01-26 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c | 22 +- drivers/staging/hv/channel_mgmt.c | 16 drivers/staging/hv/connection.c| 74

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

2011-01-26 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/blkvsc_drv.c |2 +- drivers/staging/hv/channel.c | 56 +++--- drivers/staging/hv/channel_mgmt.c|2 +- drivers/staging/hv

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

2011-01-26 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c |4 ++-- drivers/staging/hv/channel_mgmt.c| 20 ++-- drivers/staging/hv/vmbus_channel_interface.h | 26

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

2010-12-10 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- 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

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

2010-12-10 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/netvsc.c | 173 --- drivers/staging/hv/netvsc_api.h |2 +- drivers/staging/hv/netvsc_drv.c |2 +- 3 files changed, 90

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

2010-12-10 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/rndis_filter.c | 582 +++-- 1 files changed, 296 insertions(+), 286 deletions(-) diff --git a/drivers/staging/hv/rndis_filter.c b

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

2010-12-10 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/rndis.h| 353 +++-- drivers/staging/hv/rndis_filter.c | 204 +++--- 2 files changed, 279 insertions(+), 278

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

2010-12-10 Thread Haiyang Zhang
Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/netvsc.c | 852 ++- 1 files changed, 445 insertions(+), 407 deletions(-) diff --git a/drivers/staging/hv/netvsc.c b/drivers

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

2010-11-08 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com staging: hv: Convert camel cased struct fields in ring_buffer.h to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c | 10 +++--- drivers/staging

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

2010-11-08 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com staging: hv: Convert camel cased local variables in ring_buffer.c to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c |4 +- drivers/staging/hv

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

2010-11-08 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com staging: hv: Convert camel cased local variables in osd.c to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/osd.c | 40

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

2010-11-08 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com staging: hv: Convert camel cased local variables in hv.c to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/hv.c | 172

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

2010-11-08 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com staging: hv: Convert camel cased functions in hv.c to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/connection.c |4 +- drivers/staging/hv/hv.c

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

2010-11-08 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com staging: hv: Convert camel cased functions in ring_buffer.c to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c | 40 ++-- drivers/staging

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

2010-11-08 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com staging: hv: Convert camel cased struct fields in hv.h to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/hv.c| 95

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

2010-11-08 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com staging: hv: Convert camel cased struct fields in hv_api.h to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c | 32 ++-- drivers/staging/hv

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

2010-11-08 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com staging: hv: Convert camel cased functions in osd.c to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c | 20 +- drivers/staging

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

2010-11-03 Thread Haiyang Zhang
From: Greg KH [g...@kroah.com] Sent: Wednesday, November 03, 2010 7:36 AM But you should. Remember, be consistent. Care to redo these again? Sure, I will update the patches. Thanks, - Haiyang ___ Virtualization mailing list

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

2010-11-02 Thread Haiyang Zhang
From: Brandon Philips [mailto:bran...@ifup.org] Sent: Tuesday, November 02, 2010 1:04 PM -static int HvQueryHypervisorInfo(void) +static int hvquery_hypervisor_info(void) -static u64 HvDoHypercall(u64 control, void *input, void *output) +static u64 hvdo_hypercall(u64 control, void

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

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com Convert camel cased struct fields in ring_buffer.h to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c | 10 +++--- drivers/staging/hv

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

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com Convert camel cased struct fields in hv_api.h to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c | 32 ++-- drivers/staging/hv/channel_mgmt.c

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

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com Convert camel cased local variables in hv.c to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/hv.c | 170 +++--- 1

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

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com Convert camel cased struct fields in hv.h to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/hv.c| 95 +++- drivers

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

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com Convert camel cased local variables in ring_buffer.c to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c |4 +- drivers/staging/hv/ring_buffer.c

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

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com Convert camel cased local variables in osd.c to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/osd.c | 40 1 files

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

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com Convert camel cased functions in osd.c to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c | 20 +- drivers/staging/hv

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

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com Convert camel cased local variables in hv.c to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/hv.c | 170 +++--- 1

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

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com Convert camel cased local variables in ring_buffer.c to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c |4 +- drivers/staging/hv/ring_buffer.c

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

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com Convert camel cased struct fields in hv.h to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/hv.c| 95 +++- drivers

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

2010-11-01 Thread Haiyang Zhang
From: Greg KH [mailto:gre...@suse.de] Sent: Monday, November 01, 2010 1:43 PM Why did I get two different copies of some of these patches? Are they different? They are the same. Initially, I sent all 10 patches out at the same time, but I only saw 5 of them in my mail box. So I re-sent the

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

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang haiya...@microsoft.com Convert camel cased struct fields in ring_buffer.h to lower cases Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c | 10 +++--- drivers/staging/hv

  1   2   >