[PATCH 00/12] Staging: hv: Cleanup vmbus driver - Phase II

2011-03-15 Thread K. Y. Srinivasan
This patch-set fixes the following issues in the vmbus driver (vmbus_drv.c): Make vmbus driver a platform pci device and cleanup root device management and irq allocation (patches 1/12 through 3/12): 1) Make vmbus driver a platform pci driver. 2) Cleanup root device management.

[PATCH 01/12] Staging: hv: Make vmbus driver a pci driver

2011-03-15 Thread K. Y. Srinivasan
Make vmbus driver a pci driver. This is in preparation to cleaning up the root device management as well as the irq allocation for this driver. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Mike Sterling

[PATCH 10/12] Staging: hv: Get rid of the forward declaration for vmbus_device_release

2011-03-15 Thread K. Y. Srinivasan
Get rid of the forward declaration of vmbus_device_release by moving the code around. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Mike Sterling mike.sterl...@microsoft.com Signed-off-by: Abhishek Kane v-abk...@microsoft.com

[PATCH 02/12] Staging: hv: Cleanup root device handling

2011-03-15 Thread K. Y. Srinivasan
Now we can complete the cleanup of the root device management - use the pci device as the root device for all Hyper-V devices. As part of this cleanup get rid of the root device object from vmbus_driver_context. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang

[PATCH 09/12] Staging: hv: Get rid of the forward declaration for vmbus_shutdown

2011-03-15 Thread K. Y. Srinivasan
Get rid of the forward declaration of vmbus_shutdown by moving the code around. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Mike Sterling mike.sterl...@microsoft.com Signed-off-by: Abhishek Kane v-abk...@microsoft.com

[PATCH 11/12] Staging: hv: Get rid of the forward declaration for vmbus_isr

2011-03-15 Thread K. Y. Srinivasan
Get rid of the forward declaration of vmbus_isr by moving the code around. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Mike Sterling mike.sterl...@microsoft.com Signed-off-by: Abhishek Kane v-abk...@microsoft.com

[PATCH 06/12] Staging: hv: Get rid of the forward declaration for vmbus_match

2011-03-15 Thread K. Y. Srinivasan
Get rid of the forward declaration of vmbus_match by moving the code around. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Mike Sterling mike.sterl...@microsoft.com Signed-off-by: Abhishek Kane v-abk...@microsoft.com

[PATCH 05/12] Get rid of the forward declaration for vmbus_uevent

2011-03-15 Thread K. Y. Srinivasan
Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/staging/hv/vmbus_drv.c | 151 1 files changed, 75 insertions(+), 76 deletions(-) diff --git a/drivers/staging/hv/vmbus_drv.c b/drivers/staging/hv/vmbus_drv.c index 1331a6f..42066e9 100644

[PATCH 12/12] Staging: hv: Get rid of the forward declaration for vmbus_show_device_attr

2011-03-15 Thread K. Y. Srinivasan
Get rid of the forward declaration of vmbus_show_device_attr by moving the code around. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Mike Sterling mike.sterl...@microsoft.com Signed-off-by: Abhishek Kane

[PATCH 07/12] Staging: hv: Get rid of the forward declaration for vmbus_probe

2011-03-15 Thread K. Y. Srinivasan
Get rid of the forward declaration of vmbus_probe by moving the code around. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Mike Sterling mike.sterl...@microsoft.com Signed-off-by: Abhishek Kane v-abk...@microsoft.com

[PATCH 04/12] Staging: hv: Rename vmbus_driver_context structure

2011-03-15 Thread K. Y. Srinivasan
Now that struct vmbus_driver_context is properly cleaned up, rename this structure appropriately and cleanup the code. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Mike Sterling mike.sterl...@microsoft.com Signed-off-by:

Re: [PATCH 00/12] Staging: hv: Cleanup vmbus driver - Phase II

2011-03-15 Thread Greg KH
On Tue, Mar 15, 2011 at 03:02:07PM -0700, K. Y. Srinivasan wrote: This patch-set fixes the following issues in the vmbus driver (vmbus_drv.c): snip Thanks for the patches, but as the .39 merge window is closed, I'll be holding on to these until after .39-rc1 is out before I can do anything with

[PATCH 08/12] Staging: hv: Get rid of the forward declaration for vmbus_remove

2011-03-15 Thread K. Y. Srinivasan
Get rid of the forward declaration of vmbus_remove by moving the code around. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Mike Sterling mike.sterl...@microsoft.com Signed-off-by: Abhishek Kane v-abk...@microsoft.com

[PATCH 05/12] Staging: hv: Get rid of the forward declaration for vmbus_uevent

2011-03-15 Thread K. Y. Srinivasan
Get rid of the forward declaration of vmbus_uevent by moving the code around. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Haiyang Zhang haiya...@microsoft.com Signed-off-by: Mike Sterling mike.sterl...@microsoft.com Signed-off-by: Abhishek Kane v-abk...@microsoft.com

[PATCH 03/12] Staging: hv: Cleanup irq management

2011-03-15 Thread K. Y. Srinivasan
Now that vmbus_driver is a pci driver, cleanup the irq allocation mess by using the standard irq allocation mechanisms. Note that this patch generates an error when the checkpatch script is run because of the IRQF_SAMPLE_RANDOM flag used in request_irq() function. This interrupt may be the only

RE: [PATCH 00/12] Staging: hv: Cleanup vmbus driver - Phase II

2011-03-15 Thread KY Srinivasan
-Original Message- From: Greg KH [mailto:gre...@suse.de] Sent: Tuesday, March 15, 2011 6:05 PM To: KY Srinivasan Cc: linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; virtualizat...@lists.osdl.org Subject: Re: [PATCH 00/12] Staging: hv: Cleanup vmbus driver - Phase II

Re: [PATCH 05/12] Staging: hv: Get rid of the forward declaration for vmbus_uevent

2011-03-15 Thread Greg KH
On Tue, Mar 15, 2011 at 03:03:37PM -0700, K. Y. Srinivasan wrote: Get rid of the forward declaration of vmbus_uevent by moving the code around. There are 2 05/12 patches and they are different. confused, greg k-h ___ Virtualization mailing list

Re: [PATCH 05/12] Staging: hv: Get rid of the forward declaration for vmbus_uevent

2011-03-15 Thread Greg KH
On Tue, Mar 15, 2011 at 03:22:46PM -0700, Greg KH wrote: On Tue, Mar 15, 2011 at 03:03:37PM -0700, K. Y. Srinivasan wrote: Get rid of the forward declaration of vmbus_uevent by moving the code around. There are 2 05/12 patches and they are different. Ah, one has the proper subject and

Re: [PATCH 00/12] Staging: hv: Cleanup vmbus driver - Phase II

2011-03-15 Thread Greg KH
On Tue, Mar 15, 2011 at 10:24:41PM +, KY Srinivasan wrote: -Original Message- From: Greg KH [mailto:gre...@suse.de] Sent: Tuesday, March 15, 2011 6:05 PM To: KY Srinivasan Cc: linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; virtualizat...@lists.osdl.org

RE: [PATCH 05/12] Staging: hv: Get rid of the forward declaration for vmbus_uevent

2011-03-15 Thread KY Srinivasan
-Original Message- From: Greg KH [mailto:g...@kroah.com] Sent: Tuesday, March 15, 2011 6:23 PM To: KY Srinivasan Cc: gre...@suse.de; linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; virtualizat...@lists.osdl.org; Haiyang Zhang; Mike Sterling; Abhishek Kane (Mindtree

Re: [PATCH 05/12] Staging: hv: Get rid of the forward declaration for vmbus_uevent

2011-03-15 Thread Greg KH
On Tue, Mar 15, 2011 at 10:39:07PM +, KY Srinivasan wrote: -Original Message- From: Greg KH [mailto:g...@kroah.com] Sent: Tuesday, March 15, 2011 6:23 PM To: KY Srinivasan Cc: gre...@suse.de; linux-ker...@vger.kernel.org; de...@linuxdriverproject.org;

Re: [PATCH 0/2] Fix hot-unplug: device removal while port in use

2011-03-15 Thread Rusty Russell
On Fri, 11 Mar 2011 16:46:28 +0530, Amit Shah amit.s...@redhat.com wrote: Ideally virtio_pci_release_dev() shouldn't be needed at all; all that work can be moved to virtio_pci_remove(). virtio_pci_release_dev() was added in 29f9f12e to curb a warning: virtio: add PCI device release()

Re: [PATCH 00/02] virtio: Virtio platform driver

2011-03-15 Thread Rusty Russell
On Thu, 10 Mar 2011 16:05:41 +0900, Magnus Damm magnus.d...@gmail.com wrote: virtio: Virtio platform driver [PATCH 01/02] virtio: Break out lguest virtio code to virtio_lguest.c [PATCH 02/02] virtio: Add virtio platform driver I have no problem with these patches, but it's just churn until

Re: [PATCH] virtio_pci: Prevent double-free of pci regions after device hot-unplug

2011-03-15 Thread Rusty Russell
On Mon, 14 Mar 2011 17:45:02 +0530, Amit Shah amit.s...@redhat.com wrote: In the case where a virtio-console port is in use (opened by a program) and a virtio-console device is removed, the port is kept around but all the virtio-related state is assumed to be gone. When the port is finally

Re: [PATCH] virtio: console: Enable call to hvc_remove() on console port remove

2011-03-15 Thread Rusty Russell
On Mon, 14 Mar 2011 17:45:48 +0530, Amit Shah amit.s...@redhat.com wrote: This call was disabled as hot-unplugging one virtconsole port led to another virtconsole port freezing. Upon testing it again, this now works, so enable it. Applied, Rusty.