RE: [PATCH 003/117] Staging: hv: Add struct hv_vmbus_device_id to mod_devicetable.h

2011-08-25 Thread KY Srinivasan
-Original Message- From: Greg KH [mailto:g...@kroah.com] Sent: Wednesday, August 24, 2011 10:40 PM To: KY Srinivasan Cc: de...@linuxdriverproject.org; Haiyang Zhang; gre...@suse.de; linux- ker...@vger.kernel.org; virtualizat...@lists.osdl.org Subject: Re: [PATCH 003/117] Staging:

[PATCH] virtio.h: correct comment for struct virtio_driver

2011-08-25 Thread Wang Sheng-Hui
The patch is against 3.0. Signed-off-by: Wang Sheng-Hui shh...@gmail.com --- include/linux/virtio.h |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/virtio.h b/include/linux/virtio.h index 7108857..e0a1973 100644 --- a/include/linux/virtio.h +++

[PATCH 16/59] Staging: hv: netvsc: Get rid of the PCI signature

2011-08-25 Thread K. Y. Srinivasan
Now that we have implemented a vmbus specific mechanism for auto-loading, get rid of the PCI signature. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/netvsc_drv.c |8 1 files changed, 0 insertions(+), 8

[PATCH 06/59] Staging: hv: blkvsc: Use the newly introduced vmbus ID in the blockvsc driver

2011-08-25 Thread K. Y. Srinivasan
Use the newly introduced vmbus ID in the blockvsc driver. Also, do the associated cleanup. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/blkvsc_drv.c | 23 ++- 1 files changed, 14 insertions(+),

[PATCH 15/59] Staging: hv: storvsc: Get rid of the DMI signature

2011-08-25 Thread K. Y. Srinivasan
Now that we have implemented a vmbus specific mechanism for auto-loading, get rid of the DMI signature. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/storvsc_drv.c | 22 -- 1 files changed, 0

[PATCH 08/59] Staging: hv: netvsc: Use the newly introduced vmbus ID in netvsc driver

2011-08-25 Thread K. Y. Srinivasan
Use the newly introduced vmbus ID in netvsc driver. Also, do the associated cleanup. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/netvsc.c | 10 -- drivers/staging/hv/netvsc_drv.c | 16

[PATCH 19/59] Staging: hv: util: Get rid of the PCI signature in hv_util.c

2011-08-25 Thread K. Y. Srinivasan
Now that we have implemented a vmbus specific mechanism for auto-loading, get rid of the PCI signature. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/hv_util.c |8 1 files changed, 0 insertions(+), 8

[PATCH 24/59] Staging: hv: vmbus: Cleanup unnecessary comments in hv.c

2011-08-25 Thread K. Y. Srinivasan
Cleanup unnecessary comments in hv.c. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/hv.c |8 1 files changed, 0 insertions(+), 8 deletions(-) diff --git a/drivers/staging/hv/hv.c

[PATCH 22/59] Staging: hv: vmbus: Get rid of the unused name field in struct hv_driver

2011-08-25 Thread K. Y. Srinivasan
Get rid of the unused name field in struct hv_driver. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/hyperv.h |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/hyperv.h

[PATCH 27/59] Staging: hv: vmbus: Get rid of the function dump_gpadl_body()

2011-08-25 Thread K. Y. Srinivasan
Get rid of the function dump_gpadl_body() since it adds no value and actually is in the data path. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/channel.c | 20 1 files changed, 0

[PATCH 30/59] Staging: hv: vmbus: Get rid of unnecessary comments in channel.c

2011-08-25 Thread K. Y. Srinivasan
Get rid of unnecessary comments in channel.c. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/channel.c |4 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/drivers/staging/hv/channel.c

[PATCH 20/59] Staging: hv: netvsc: Initialize the driver name directly

2011-08-25 Thread K. Y. Srinivasan
Initialize the driver name directly in netvsc_drv.c and do the necessary adjustments. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/netvsc.c |5 - drivers/staging/hv/netvsc_drv.c |4 +++- 2 files

[PATCH 05/59] Staging: hv: vmbus: Introduce vmbus ID space in struct hv_driver

2011-08-25 Thread K. Y. Srinivasan
In preparation for supporting auto-loading Hyper-V drivers using vmbus specific aliases, introduce vmbus ID space in struct hv_driver. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/hyperv.h |2 ++ 1 files

[PATCH 13/59] Staging: hv: vmbus: Support the notion of id tables in vmbus_match()

2011-08-25 Thread K. Y. Srinivasan
Introduce code to handle driver specific id tables to the vmbus core (vmbus_match). This would allow us to handle more than one device type with a given driver. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com ---

[PATCH 29/59] Staging: hv: vmbus: Rename openMsg to open_msg in channel.c

2011-08-25 Thread K. Y. Srinivasan
Rename openMsg to open_msg in channel.c. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/channel.c | 20 ++-- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git

[PATCH 25/59] Staging: hv: vmbus: Cleanup error handling in hv_init()

2011-08-25 Thread K. Y. Srinivasan
Use standard Linux error codes. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/hv.c |7 +++ 1 files changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/staging/hv/hv.c b/drivers/staging/hv/hv.c

[PATCH 36/59] Staging: hv: storvsc: Cleanup error handling in storvsc_dev_add()

2011-08-25 Thread K. Y. Srinivasan
Use standard Linux error values and cleanup error handling. I would like to acknowledge Andre Bartke (andre.bar...@gmail.com) for highlighting this problem. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/storvsc.c |

[PATCH 28/59] Staging: hv: vmbus: Get rid of the function dump_gpadl_header()

2011-08-25 Thread K. Y. Srinivasan
Get rid of the function dump_gpadl_header() as this adds no value. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/channel.c | 28 1 files changed, 0 insertions(+), 28 deletions(-) diff

[PATCH 12/59] Staging: hv: vmbus: Cleanup vmbus_uevent() code

2011-08-25 Thread K. Y. Srinivasan
Now generate appropriate uevent based on the modalias string. As part of this, cleanup the existing uevent code. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/vmbus_drv.c | 60

[PATCH 09/59] Staging: hv: mousevsc: Use the newly introduced vmbus ID in mouse driver

2011-08-25 Thread K. Y. Srinivasan
Use the newly introduced vmbus ID in mouse driver. Also, do the associated cleanup. Since the mouse driver is not functional, we disable the autoloading of this driver. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com ---

[PATCH 01/59] Staging: hv: vmbus: VMBUS is an ACPI enumerated device, get rid of the PCI signature

2011-08-25 Thread K. Y. Srinivasan
VMBUS is an ACPI enumerated device, get rid of the PCI signature. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/vmbus_drv.c | 13 - 1 files changed, 0 insertions(+), 13 deletions(-) diff --git

[PATCH 31/59] Staging: hv: vmbus: Change the variable name openInfo to open_info in channel.c

2011-08-25 Thread K. Y. Srinivasan
Change the variable name openInfo to open_info in channel.c. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/channel.c | 24 1 files changed, 12 insertions(+), 12 deletions(-) diff --git

[PATCH 33/59] Staging: hv: vmbus: Cleanup the error return value in vmbus_recvpacket_raw()

2011-08-25 Thread K. Y. Srinivasan
Use standard Linux errno values. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/channel.c |2 +- drivers/staging/hv/hv_mouse.c |2 +- drivers/staging/hv/netvsc.c |2 +- 3 files changed, 3 insertions(+),

[PATCH 07/59] Staging: hv: storvsc: Use the newly introduced vmbus ID in storvsc driver

2011-08-25 Thread K. Y. Srinivasan
Use the newly introduced vmbus ID in storvsc driver. Also, do the assciated cleanup. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/storvsc_drv.c | 25 ++--- 1 files changed, 14 insertions(+), 11

[PATCH 0000/0059] Staging: hv: Driver cleanup

2011-08-25 Thread K. Y. Srinivasan
Further cleanup of the hv drivers. 1) Implement code for autoloading the vmbus drivers without using PCI or DMI signatures. I have implemented this based on Greg's feedback on my earlier implementation. 2) Cleanup error handling across the board and use

[PATCH 04/59] Staging: hv: Add code to parse struct hv_vmbus_device_id table

2011-08-25 Thread K. Y. Srinivasan
Add code to parse struct hv_vmbus_device_id table. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- scripts/mod/file2alias.c | 25 + 1 files changed, 25 insertions(+), 0 deletions(-) diff --git

[PATCH 44/59] Staging: hv: storvsc: Cleanup returned error code in storvsc_drv_init()

2011-08-25 Thread K. Y. Srinivasan
Use standard Linux error codes. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/storvsc_drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv/storvsc_drv.c

[PATCH 56/59] Staging: hv: mouse: Change the jump label Cleanup to cleanup

2011-08-25 Thread K. Y. Srinivasan
Change the jump label Cleanup to cleanup. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/hv_mouse.c | 22 +++--- 1 files changed, 11 insertions(+), 11 deletions(-) diff --git

[PATCH 45/59] Staging: hv: netvsc: Cleanup the returned error code in netvsc_probe()

2011-08-25 Thread K. Y. Srinivasan
Use standard Linux error codes. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/netvsc_drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv/netvsc_drv.c

[PATCH 46/59] Staging: hv: netvsc: Cleanup error return codes in netvsc_destroy_recv_buf()

2011-08-25 Thread K. Y. Srinivasan
Cleanup error return codes in netvsc_destroy_recv_buf(). Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/netvsc.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/netvsc.c

[PATCH 40/59] Storage: hv: storvsc: Get rid of some unnecessary DPRINTs from storvsc.c

2011-08-25 Thread K. Y. Srinivasan
Get rid of some unnecessary DPRINTs from storvsc.c. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/storvsc.c | 16 1 files changed, 0 insertions(+), 16 deletions(-) diff --git

[PATCH 35/59] Staging: hv: vmbus: Retry vmbus_post_msg() before giving up

2011-08-25 Thread K. Y. Srinivasan
The function hv_post_msg() can fail because of transient resource conditions. It may be useful to retry the operation. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- arch/x86/include/asm/hyperv.h |1 +

[PATCH 54/59] Staging: hv: netvsc: Cleanup error returns in rndis_filter_init_device()

2011-08-25 Thread K. Y. Srinivasan
Use standard Linux error codes. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/rndis_filter.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/rndis_filter.c

[PATCH 39/59] Staging: hv: storvsc: Cleanup error handling in storvsc_do_io()

2011-08-25 Thread K. Y. Srinivasan
Use standard Linux error codes. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/storvsc.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv/storvsc.c

[PATCH 49/59] Staging: hv: netvsc: Cleanup error return values in netvsc_send()

2011-08-25 Thread K. Y. Srinivasan
Use standard Linux error codes. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/netvsc.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/netvsc.c

[PATCH 26/59] Staging: hv: vmbus: Get rid of unnecessay comments in connection.c

2011-08-25 Thread K. Y. Srinivasan
Get rid of unnecessay comments in connection.c. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/connection.c |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/connection.c

[PATCH 50/59] Staging: hv: netvsc: Cleanup error return codes in netvsc_device_add()

2011-08-25 Thread K. Y. Srinivasan
Use standard Linux error codes. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/netvsc.c |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/drivers/staging/hv/netvsc.c

[PATCH 18/59] Staging: hv: util: Get rid of the DMI signature in hv_util.c

2011-08-25 Thread K. Y. Srinivasan
Now that we have implemented a vmbus specific mechanism for auto-loading, get rid of the DMI signature. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/hv_util.c | 20 1 files changed, 0

[PATCH 32/59] Staging: hv: vmbus: Cleanup error values in ringbuffer.c

2011-08-25 Thread K. Y. Srinivasan
Use standard Linux errno values in ringbuffer.c and do the associated cleanup in the clients. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/blkvsc_drv.c |2 +- drivers/staging/hv/netvsc.c |2 +-

[PATCH 21/59] Staging: hv: netvsc: Get rid of the empty function netvsc_initialize()

2011-08-25 Thread K. Y. Srinivasan
Now, get rid of the empty function netvsc_initialize(). Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/hyperv_net.h |1 - drivers/staging/hv/netvsc.c |9 - drivers/staging/hv/netvsc_drv.c |3

[PATCH 34/59] Staging: hv: netvsc: Get rid of an unnecessary print statement in netvsc_probe()

2011-08-25 Thread K. Y. Srinivasan
Get rid of an unnecessary print statement in netvsc_probe(). Furthermore, this fixes a bug since netdev_err is being invoked after the device has been freed. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/netvsc_drv.c

[PATCH 58/59] Staging: hv: netvsc: Change the jump label Cleanup to cleanup

2011-08-25 Thread K. Y. Srinivasan
Change the jump label Cleanup to cleanup. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/rndis_filter.c | 28 ++-- 1 files changed, 14 insertions(+), 14 deletions(-) diff --git

[PATCH 47/59] Staging: hv: netvsc: Cleanup error return values in netvsc_init_recv_buf()

2011-08-25 Thread K. Y. Srinivasan
Use standard Linux error codes. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/netvsc.c | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/hv/netvsc.c

[PATCH 43/59] Staging: hv: storvsc: Cleanup error code returned in storvsc_probe()

2011-08-25 Thread K. Y. Srinivasan
Use standard Linux error codes. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/storvsc_drv.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/storvsc_drv.c

[PATCH 53/59] Staging: hv: netvsc: Cleanup error return values in rndis_filter_set_packet_filter()

2011-08-25 Thread K. Y. Srinivasan
Use standard Linux error codes and cleanup some error paths. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/rndis_filter.c |5 + 1 files changed, 1 insertions(+), 4 deletions(-) diff --git

[PATCH 52/59] Staging: hv: netvsc: Cleanup error code in rndis_filter_query_device()

2011-08-25 Thread K. Y. Srinivasan
Use standard Linux error codes. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/rndis_filter.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv/rndis_filter.c

[PATCH 37/59] Staging: hv: storvsc: Cleanup error handling in storvsc_channel_init()

2011-08-25 Thread K. Y. Srinivasan
Use standard Linux error codes. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/storvsc.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv/storvsc.c

[PATCH 59/59] Staging: hv: netvsc: Change the jump label Exit to exit

2011-08-25 Thread K. Y. Srinivasan
Change the jump lable Exit to exit. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/rndis_filter.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/rndis_filter.c

[PATCH 14/59] Staging: hv: vmbus: Get rid of an unnecessary include line in vmbus_drv.c

2011-08-25 Thread K. Y. Srinivasan
Get rid of an unnecessary include line in vmbus_drv.c. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/vmbus_drv.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv/vmbus_drv.c

[PATCH 42/59] Staging: hv: storvsc: Cleanup returned error code in storvsc_host_reset()

2011-08-25 Thread K. Y. Srinivasan
Use standard Linux error codes. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/storvsc_drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv/storvsc_drv.c

[PATCH 51/59] Staging: hv: netvsc: Cleanup error codes in rndis_filter_receive()

2011-08-25 Thread K. Y. Srinivasan
Use standard Linux error codes. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/rndis_filter.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/rndis_filter.c

[PATCH 38/59] Staging: hv: storvsc: Cleanup error handling in storvsc_connect_to_vsp()

2011-08-25 Thread K. Y. Srinivasan
Cleanup error handling in storvsc_connect_to_vsp(). Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/storvsc.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv/storvsc.c

[PATCH 11/59] Staging: hv: vmbus: Cleanup vmbus_match()

2011-08-25 Thread K. Y. Srinivasan
Use the preferred mechanism to compare guids in vmbus_match(). Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/vmbus_drv.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git

Re: [PATCH 22/59] Staging: hv: vmbus: Get rid of the unused name field in struct hv_driver

2011-08-25 Thread Greg KH
On Thu, Aug 25, 2011 at 02:24:28PM -0700, Greg KH wrote: On Thu, Aug 25, 2011 at 09:48:48AM -0700, K. Y. Srinivasan wrote: Get rid of the unused name field in struct hv_driver. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com ---

Re: [PATCH 22/59] Staging: hv: vmbus: Get rid of the unused name field in struct hv_driver

2011-08-25 Thread Greg KH
On Thu, Aug 25, 2011 at 09:48:48AM -0700, K. Y. Srinivasan wrote: Get rid of the unused name field in struct hv_driver. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com --- drivers/staging/hv/hyperv.h |2 -- 1 files changed, 0

Re: [PATCH 22/59] Staging: hv: vmbus: Get rid of the unused name field in struct hv_driver

2011-08-25 Thread Greg KH
On Thu, Aug 25, 2011 at 02:28:20PM -0700, Greg KH wrote: Man, if you want something done right, you have to do it yourself, let me go make these changes so you don't have to do any new work at this point in time, hopefully your other patches will apply... What,

RE: [PATCH 12/59] Staging: hv: vmbus: Cleanup vmbus_uevent() code

2011-08-25 Thread KY Srinivasan
-Original Message- From: Greg KH [mailto:g...@kroah.com] Sent: Thursday, August 25, 2011 5:00 PM To: KY Srinivasan Cc: gre...@suse.de; linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; virtualizat...@lists.osdl.org; Haiyang Zhang Subject: Re: [PATCH 12/59] Staging: hv:

RE: [PATCH 12/59] Staging: hv: vmbus: Cleanup vmbus_uevent() code

2011-08-25 Thread KY Srinivasan
-Original Message- From: Greg KH [mailto:g...@kroah.com] Sent: Thursday, August 25, 2011 5:15 PM To: KY Srinivasan Cc: de...@linuxdriverproject.org; Haiyang Zhang; gre...@suse.de; linux- ker...@vger.kernel.org; virtualizat...@lists.osdl.org Subject: Re: [PATCH 12/59] Staging: hv:

Re: [PATCH 12/59] Staging: hv: vmbus: Cleanup vmbus_uevent() code

2011-08-25 Thread Greg KH
On Thu, Aug 25, 2011 at 10:14:05PM +, KY Srinivasan wrote: -Original Message- From: Greg KH [mailto:g...@kroah.com] Sent: Thursday, August 25, 2011 5:00 PM To: KY Srinivasan Cc: gre...@suse.de; linux-ker...@vger.kernel.org; de...@linuxdriverproject.org;

RE: [PATCH 22/59] Staging: hv: vmbus: Get rid of the unused name field in struct hv_driver

2011-08-25 Thread KY Srinivasan
-Original Message- From: Greg KH [mailto:g...@kroah.com] Sent: Thursday, August 25, 2011 5:28 PM To: KY Srinivasan Cc: gre...@suse.de; linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; virtualizat...@lists.osdl.org; Haiyang Zhang Subject: Re: [PATCH 22/59] Staging: hv:

Re: [PATCH 12/59] Staging: hv: vmbus: Cleanup vmbus_uevent() code

2011-08-25 Thread Greg KH
On Thu, Aug 25, 2011 at 10:20:32PM +, KY Srinivasan wrote: I have to edit this to get it to work properly with the fact that I added the driver_data field to hv_vmbus_device_id. You should have a copy of the patch I applied in your inbox now, can you verify I didn't mess it up?

Re: [PATCH 22/59] Staging: hv: vmbus: Get rid of the unused name field in struct hv_driver

2011-08-25 Thread Greg KH
On Thu, Aug 25, 2011 at 10:27:40PM +, KY Srinivasan wrote: This was existing code and I realized it did not conform to the Linux Driver Model. I fixed it in the patch set I had sent earlier (110/117). Yesterday I only got up to 74 of the 117 patches that you commented on. I was planning to

RE: [PATCH 12/59] Staging: hv: vmbus: Cleanup vmbus_uevent() code

2011-08-25 Thread KY Srinivasan
-Original Message- From: Greg KH [mailto:g...@kroah.com] Sent: Thursday, August 25, 2011 6:28 PM To: KY Srinivasan Cc: de...@linuxdriverproject.org; Haiyang Zhang; gre...@suse.de; linux- ker...@vger.kernel.org; virtualizat...@lists.osdl.org Subject: Re: [PATCH 12/59] Staging: hv:

Re: [PATCH] virtio.h: correct comment for struct virtio_driver

2011-08-25 Thread Rusty Russell
On Thu, 25 Aug 2011 21:04:05 +0800, Wang Sheng-Hui shh...@gmail.com wrote: The patch is against 3.0. Signed-off-by: Wang Sheng-Hui shh...@gmail.com Thanks, applied! Cheers, Rusty. ___ Virtualization mailing list