[PATCH 00/40] Staging: hv: Driver cleanup · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 01/40] Staging: hv: storvsc: Do not aquire an unnecessary reference on stor_device · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 04/40] Staging: hv: storvsc: Cleanup alloc_stor_device() · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 05/40] Staging: hv: storvsc: Introduce state to manage the lifecycle of stor device · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 09/40] Staging: hv: storvsc: Get rid of release_stor_device() by inlining the code · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 02/40] Staging: hv: storvsc: Rename must_get_stor_device() · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 07/40] Staging: hv: storvsc: Use the newly introduced lock in accessing ext field · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 15/40] Staging: hv: netvsc: Cleanup alloc_net_device() · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 14/40] Staging: hv: netvsc: Inline the code for free_net_device() · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 10/40] Staging: hv: storvsc: Get rid of final_release_stor_device() by inlining code · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 13/40] Staging: hv: netvsc: Introduce code to properly manage outstanding sends · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 03/40] Staging: hv: storvsc: Rename get_stor_device() · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 25/40] Staging: hv: storvsc: Add state to manage the lifecycle of emulated HBA · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
Re: [PATCH 25/40] Staging: hv: storvsc: Add state to manage the lifecycle of emulated HBA · Christoph Hellwig <hch@infradead.org> · 2011-06-30
[PATCH 27/40] Staging: hv: storvsc: Handle IDE devices correctly in storvsc_remove() · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 22/40] Staging: hv: netvsc: Further cleanup reference counting of netvsc_device · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 24/40] Staging: hv: storvsc: On I/O get the correct IDE device · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
Re: [PATCH 24/40] Staging: hv: storvsc: On I/O get the correct IDE device · Christoph Hellwig <hch@infradead.org> · 2011-06-30
RE: [PATCH 24/40] Staging: hv: storvsc: On I/O get the correct IDE device · KY Srinivasan <kys@microsoft.com> · 2011-06-30
Re: [PATCH 24/40] Staging: hv: storvsc: On I/O get the correct IDE device · Christoph Hellwig <hch@infradead.org> · 2011-07-01
RE: [PATCH 24/40] Staging: hv: storvsc: On I/O get the correct IDE device · KY Srinivasan <kys@microsoft.com> · 2011-07-01
Re: [PATCH 24/40] Staging: hv: storvsc: On I/O get the correct IDE device · Christoph Hellwig <hch@infradead.org> · 2011-07-01
[PATCH 39/40] Staging: hv: storvsc: In case of scsi errors offline the device · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 34/40] Staging: hv: storvsc: Add the contents of hyperv_storage.h to storvsc_drv.c · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
Re: [PATCH 34/40] Staging: hv: storvsc: Add the contents of hyperv_storage.h to storvsc_drv.c · Christoph Hellwig <hch@infradead.org> · 2011-06-30
RE: [PATCH 34/40] Staging: hv: storvsc: Add the contents of hyperv_storage.h to storvsc_drv.c · KY Srinivasan <kys@microsoft.com> · 2011-06-30
Re: [PATCH 34/40] Staging: hv: storvsc: Add the contents of hyperv_storage.h to storvsc_drv.c · Christoph Hellwig <hch@infradead.org> · 2011-07-01
RE: [PATCH 34/40] Staging: hv: storvsc: Add the contents of hyperv_storage.h to storvsc_drv.c · KY Srinivasan <kys@microsoft.com> · 2011-07-01
Re: [PATCH 34/40] Staging: hv: storvsc: Add the contents of hyperv_storage.h to storvsc_drv.c · Greg KH <hidden> · 2011-07-05
[PATCH 40/40] Staging: hv: storvsc: No need to copy from bounce buffer in case of failure · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 38/40] Staging: hv: storvsc: Fixup srb_status for INQUIRY and MODE_SENSE · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
Re: [PATCH 38/40] Staging: hv: storvsc: Fixup srb_status for INQUIRY and MODE_SENSE · Christoph Hellwig <hch@infradead.org> · 2011-06-30
RE: [PATCH 38/40] Staging: hv: storvsc: Fixup srb_status for INQUIRY and MODE_SENSE · KY Srinivasan <kys@microsoft.com> · 2011-06-30
Re: [PATCH 38/40] Staging: hv: storvsc: Fixup srb_status for INQUIRY and MODE_SENSE · Christoph Hellwig <hch@infradead.org> · 2011-07-01
[PATCH 37/40] Staging: hv: storvsc: Make storvsc_do_io() a static function · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 36/40] Staging: hv: storvsc: Make storvsc_dev_remove() a static function · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 35/40] Staging: hv: storvsc: Make storvsc_dev_add() a static function · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
Re: [PATCH 35/40] Staging: hv: storvsc: Make storvsc_dev_add() a static function · Christoph Hellwig <hch@infradead.org> · 2011-06-30
[PATCH 32/40] Staging: hv: storvsc: Include storvsc.c in storvsc_drv.c · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 31/40] Staging: hv: storvsc: Get rid of blkvsc_drv.c as this code is not used · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 33/40] Staging: hv: storvsc: Cleanup storvsc_drv.c after adding the contents of storvsc.c · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 30/40] Staging: hv: storvsc: Optimize the bounce buffer handling in the "read" case · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 29/40] Staging: hv: storvsc: Optimize bounce buffer handling for the "write" case · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 26/40] Staging: hv: storvsc: Handle probing IDE devices · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 23/40] Staging: hv: storvsc: Introduce code to manage IDE devices using storvsc HBA · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
Re: [PATCH 23/40] Staging: hv: storvsc: Introduce code to manage IDE devices using storvsc HBA · Christoph Hellwig <hch@infradead.org> · 2011-06-30
RE: [PATCH 23/40] Staging: hv: storvsc: Introduce code to manage IDE devices using storvsc HBA · KY Srinivasan <kys@microsoft.com> · 2011-06-30
Re: [PATCH 23/40] Staging: hv: storvsc: Introduce code to manage IDE devices using storvsc HBA · Christoph Hellwig <hch@infradead.org> · 2011-07-01
[PATCH 28/40] Staging: hv: storvsc: Handle IDE devices using the storvsc driver · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 21/40] Staging: hv: netvsc: Leverage the spinlock to manage ref_cnt · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 19/40] Staging: hv: netvsc: Get rid of release_outbound_net_device() by inlining the code · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 18/40] Staging: hv: netvsc: Prevent outgoing traffic when netvsc dev is destroyed · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 17/40] Staging: hv: netvsc: Use the newly introduced lock in accessing ext field · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 16/40] Staging: hv: netvsc: Introduce state to manage the lifecycle of net device · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 12/40] Staging: hv: storvsc: Further cleanup reference counting of stor_device · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 11/40] Staging: hv: storvsc: Leverage the spinlock to manage ref_cnt · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 06/40] Staging: hv: vmbus: Introduce a lock to protect the ext field in hv_device · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 08/40] Staging: hv: storvsc: Prevent outgoing traffic when stor dev is destroyed · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
[PATCH 20/40] Staging: hv: netvsc: Get rid of release_inbound_net_device() by inlining the code · "K. Y. Srinivasan" <kys@microsoft.com> · 2011-06-29
Re: [PATCH 01/40] Staging: hv: storvsc: Do not aquire an unnecessary reference on stor_device · Greg KH <hidden> · 2011-07-05
RE: [PATCH 01/40] Staging: hv: storvsc: Do not aquire an unnecessary reference on stor_device · KY Srinivasan <kys@microsoft.com> · 2011-07-05
Re: [PATCH 00/40] Staging: hv: Driver cleanup · Christoph Hellwig <hch@infradead.org> · 2011-06-30
RE: [PATCH 00/40] Staging: hv: Driver cleanup · KY Srinivasan <kys@microsoft.com> · 2011-06-30
Re: [PATCH 00/40] Staging: hv: Driver cleanup · Christoph Hellwig <hch@infradead.org> · 2011-07-01
RE: [PATCH 00/40] Staging: hv: Driver cleanup · KY Srinivasan <kys@microsoft.com> · 2011-07-01
RE: [PATCH 00/40] Staging: hv: Driver cleanup · KY Srinivasan <kys@microsoft.com> · 2011-06-30
Re: [PATCH 00/40] Staging: hv: Driver cleanup · Stephen Hemminger <hidden> · 2011-06-30
RE: [PATCH 00/40] Staging: hv: Driver cleanup · KY Srinivasan <kys@microsoft.com> · 2011-07-01
Re: [PATCH 00/40] Staging: hv: Driver cleanup · Stephen Hemminger <hidden> · 2011-07-01
RE: [PATCH 00/40] Staging: hv: Driver cleanup · KY Srinivasan <kys@microsoft.com> · 2011-07-01

Re: [PATCH 00/40] Staging: hv: Driver cleanup

From: Christoph Hellwig <hch@infradead.org>
Date: 2011-06-30 19:33:51
Also in: lkml

On Wed, Jun 29, 2011 at 07:38:21AM -0700, K. Y. Srinivasan wrote:

Further cleanup of the hv drivers:

	1) Cleanup the reference counting mess for both stor and net devices.

I really don't understand the need for reference counting on the storage
side, especially now that you only have a SCSI driver.  The SCSI
midlayer does proper counting on it's objects (Scsi_Host, scsi_device,
scsi_cmnd), so you'll get that for free given that SCSI drivers just
piggyback on the midlayer lifetime rules.

For now your patches should probably go in as-is, but mid-term you
should be able to completely remove that code on the storage side.

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help