[PATCH 124/206] Staging: hv: Get rid of unnecessary code/comments

Get rid of unnecessary code/comments. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/blkvsc_drv.c | 15 --- 1 files changed, 0 insertions(+), 15 deletions(-) diff --git a/drivers/stagi

[PATCH 098/206] Staging: hv: Rename ringbuffer_peek

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c |4 ++-- drivers/staging/hv/ring_buffer.c |4 ++-- drivers/staging/hv/ring_buffer.h |2 +- 3 fil

[PATCH 079/206] Staging: hv: Get rid of some unnecessary code in rndis_filter.c

Now that we have gotten rid of the indirection; get rid of some unnecessary code. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/rndis_filter.c |7 --- 1 files changed, 0 insertions(+), 7 del

[PATCH 080/206] Staging: hv: Get rid of the indirection to invoke mousevsc_on_device_add()

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/hv_mouse.c |5 + 1 files changed, 1 insertions(+), 4 deletions(-) diff --git a/drivers/staging/hv/hv_mouse.c b/dr

[PATCH 096/206] Staging: hv: Rename ringbuffer_cleanup

Rename ringbuffer_cleanup. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c |8 drivers/staging/hv/ring_buffer.c |4 ++-- drivers/staging/hv/ring_buffer.h |2 +- 3 f

[PATCH 077/206] Staging: hv: Get rid of call to cleanup()

cleanup() is an empty function; get rid of it. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/netvsc_drv.c |4 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/drivers/staging/

[PATCH 117/206] Staging: hv: Get rid of some dead code in channel.c

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 71 -- 1 files changed, 0 insertions(+), 71 deletions(-) diff --git

[PATCH 092/206] Staging: hv: Rename the variable g_mousevsc_drv

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/hv_mouse.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/hv/hv_mouse.c b

[PATCH 128/206] Staging: hv: Now get rid of the empty function

Now, get rid of the empty function. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/blkvsc_drv.c | 15 --- 1 files changed, 0 insertions(+), 15 deletions(-) diff --git a/drivers/staging

[PATCH 067/206] Staging: hv: Make the function netvsc_device_add() non static

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/netvsc.c |2 +- drivers/staging/hv/netvsc_api.h |1 + 2 files changed, 2 insertions(+), 1 deletions(-) diff -

[PATCH 089/206] Staging: hv: Rename the variable g_netvsc_drv

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/netvsc_drv.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/hv/netvsc_drv

[PATCH 086/206] Staging: hv: Cleanup calls to cleanup in storvsc_drv.c

cleanup() is an empty function; get rid of calls to this. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c |4 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/driv

Re: various vmbus review comments

On Mon, May 09, 2011 at 02:56:52PM +, KY Srinivasan wrote: > I will address this. Greg had a concern about module reference counting > and looking at the current code, it did not appear to be an issue. The > change you are suggesting will not affect the vmbus core which is what I want > to focu

[PATCH 102/206] Staging: hv: Rename ringbuffer_get_debuginfo

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c |4 ++-- drivers/staging/hv/ring_buffer.c |4 ++-- drivers/staging/hv/ring_buffer.h |2 +- 3 fil

[PATCH 105/206] Staging: hv: Get rid of the forward declaration of hv_copyfrom_ringbuffer

Get rid of the forward declaration by moving code around. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/ring_buffer.c | 83 +- 1 files changed, 37 insertions(+

[PATCH 106/206] Staging: hv: Get rid of the forward declaration of hv_copyto_ringbuffer()

Get rid of the forward declaration by moving the code around. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/ring_buffer.c | 66 +++-- 1 files changed, 27 insertions

[PATCH 100/206] Staging: hv: Rename get_ringbuffer_interrupt_mask

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c |6 +++--- drivers/staging/hv/ring_buffer.c |4 ++-- drivers/staging/hv/ring_buffer.h |2 +- 3 f

[PATCH 074/206] Staging: hv: Get rid of the indirection to invoke netvsc_device_add()

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/rndis_filter.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv/rndis_filter.c

[PATCH 097/206] Staging: hv: Rename ringbuffer_write

Rename ringbuffer_write. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c |6 +++--- drivers/staging/hv/ring_buffer.c |4 ++-- drivers/staging/hv/ring_buffer.h |2 +- 3 files

[PATCH 081/206] Staging: hv: Get rid of the indirection to invoke mousevsc_on_device_remove

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/hv_mouse.c |8 +--- 1 files changed, 1 insertions(+), 7 deletions(-) diff --git a/drivers/staging/hv/hv_mouse.c b

[PATCH 072/206] Staging: hv: Get rid of rndis_filter_cleanup()

rndis_filter_cleanup() is an empty function; get rid of it. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/rndis_filter.c |6 -- 1 files changed, 0 insertions(+), 6 deletions(-) diff --git a

[PATCH 094/206] Staging: hv: Statically initialize probe/remove

Statically initialize probe/remove. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/hv_mouse.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/hv/hv_mou

[PATCH 068/206] Staging: hv: Make netvsc_device_remove non static

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/netvsc.c |2 +- drivers/staging/hv/netvsc_api.h |1 + 2 files changed, 2 insertions(+), 1 deletions(-) diff -

[PATCH 121/206] Staging: hv: Change Cleanup to cleanup

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/hv.c | 18 +- 1 files changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/hv/hv.c b/d

[PATCH 101/206] Staging: hv: Rename dump_ring_info

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c |4 ++-- drivers/staging/hv/ring_buffer.c |4 ++-- drivers/staging/hv/ring_buffer.h |2 +- 3 fil

[PATCH 111/206] Staging: hv: Rename the variable messageType

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/channel_mgmt.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv/channel_mgmt.c

[PATCH 090/206] Staging: hv: Move the declaration of the variable netvsc_drv

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/netvsc_drv.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/hv/netvsc_drv.c

[PATCH 084/206] Staging: hv: Get rid of dev_add from struct hv_driver

Now that we have gotten rid of the indirection, get rid of the unnecessary state struct hv_driver. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/vmbus_api.h |1 - 1 files changed, 0 insertions(+

[PATCH 103/206] Staging: hv: Rename all static functions in ring_buffer.c

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/ring_buffer.c | 88 +++--- 1 files changed, 44 insertions(+), 44 deletions(-) diff --gi

[PATCH 032/206] Staging: hv: Rename g_driver_name

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/storvsc_drv.c

[PATCH 095/206] Staging: hv: Rename the function ringbuffer_init

Rename the function ringbuffer_init to not pollute the namespace. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c |7 +-- drivers/staging/hv/ring_buffer.c |4 ++-- drivers/st

[PATCH 041/206] Staging: hv: Get rid of the forward declaration for storvsc_device_configure()

Get rid of the forward declaration by moving the code around. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 39 ++--- 1 files changed, 19 insertions

[PATCH 118/206] Staging: hv: Change Cleanup to cleanup

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/connection.c | 20 ++-- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/staging

[PATCH 075/206] Staging: hv: Get rid of the indirection in invoking rndis_filter_device_remove()

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/netvsc_drv.c | 11 ++- 1 files changed, 2 insertions(+), 9 deletions(-) diff --git a/drivers/staging/hv/netvsc_

[PATCH 120/206] Staging: hv: Get rid of some dead code from hv.c

Get rid of some commented code. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/hv.c | 31 --- 1 files changed, 0 insertions(+), 31 deletions(-) diff --git a/drivers/sta

[PATCH 065/206] Staging: hv: Get rid of the code initializing the dispatch table

Now that we have eliminated the indirection, we can get rid of the code to initialize the dispatch table. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c |5 - 1 files changed,

[PATCH 108/206] Staging: hv: Cleanup ring_buffer.h

Access to these variables is already serialized via a spin lock. Get rid of the unnecessary volatile qualifier. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/ring_buffer.h |7 +++ 1 files ch

[PATCH 058/206] Staging: hv: Rename the driver variable

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/hv/storvsc

[PATCH 115/206] Staging: hv: Use completion abstraction in struct netvsc_device

Use completion abstraction in struct netvsc_device instead of struct wait_queue_head_t to synchronize. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/netvsc.c | 32 +---

[PATCH 099/206] Staging: hv: Rename ringbuffer_read

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c |4 ++-- drivers/staging/hv/ring_buffer.c |4 ++-- drivers/staging/hv/ring_buffer.h |2 +- 3 fil

[PATCH 114/206] Staging: hv: Get rid of some unnecessary state from struct vmbus_msginfo

This state is not used anywhere; get rid of it. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/vmbus_private.h |4 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/drivers/stag

[PATCH 061/206] Staging: hv: Get rid of unnecessary comments/dead code

Get rid of unnecessary comments/dead code. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 17 - 1 files changed, 0 insertions(+), 17 deletions(-) diff --git a/drive

[PATCH 112/206] Staging: hv: Rename the variable gChannelMessageTable

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/channel_mgmt.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/hv/channel_mg

[PATCH 109/206] Staging: hv: Rename the variable gSupportedDeviceClasses

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/channel_mgmt.c |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/channel_mgm

[PATCH 038/206] Staging: hv: Get rid of some DPRINTS

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 65 +++-- 1 files changed, 6 insertions(+), 59 deletions(-) diff --git

[PATCH 028/206] Staging: hv: Rename stor_vsc_on_channel_callback

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/storvsc.c b/drive

[PATCH 116/206] Staging: hv: Use the completion abstraction in struct rndis_request

Use the completion abstraction in struct rndis_request to synchronize. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/rndis_filter.c | 33 ++--- 1 files changed, 14 inse

[PATCH 119/206] Staging: hv: Get rid of some dead code in connection.c

Get rid some commented code and dated comments.T Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/connection.c | 10 -- 1 files changed, 4 insertions(+), 6 deletions(-) diff --git a/drivers/

[PATCH 069/206] Staging: hv: Get rid of netvsc_cleanup()

netvsc_cleanup() is an empty function; get rid of it. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/netvsc.c |8 1 files changed, 0 insertions(+), 8 deletions(-) diff --git a/drivers/s

[PATCH 054/206] Staging: hv: Move module parameters to earlier in the file

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/hv/storvsc_drv

[PATCH 037/206] Staging: hv: Get rid of some dated comments

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/drivers/staging/hv/storvsc_drv.c

[PATCH 082/206] Staging: hv: Get rid of the mouse cleanup() function

mousevsc_on_cleanup() is an empty function; get rid of it. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/hv_mouse.c |4 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/drive

[PATCH 031/206] Staging: hv: Get rid of some unnecessary DPRINTS

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc.c | 116 ++ 1 files changed, 16 insertions(+), 100 deletions(-) diff --g

[PATCH 056/206] Staging: hv: Get rid of the forward declaration of storvsc_queuecommand()

Get rid of the forward declaration by moving the code around. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 338 +++--- 1 files changed, 169 insertio

[PATCH 034/206] Staging: hv: Rename variables pointing to struct hv_host_device

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 50 +++--- 1 files changed, 25 insertions(+), 25 deletions(-) diff --gi

[PATCH 071/206] Staging: hv: Make rndis_filter_device_remove() non static

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/netvsc_api.h |1 + drivers/staging/hv/rndis_filter.c |4 +--- 2 files changed, 2 insertions(+), 3 deletions(-)

[PATCH 110/206] Staging: hv: Rename the variable messageHandler

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/channel_mgmt.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/hv/channel_mg

[PATCH 048/206] Staging: hv: Get rid of the forward declaration of storvsc_get_chs()

Get rid of the forward declaration by moving the code around. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 164 ++--- 1 files changed, 80 insertions

[PATCH 050/206] Staging: hv: Rename stor_vsc_initialize()

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/hv/storvsc_drv

[PATCH 029/206] Staging: hv: Rename stor_vsc_connect_to_vsp

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/storvsc.c b/drive

[PATCH 026/206] Staging: hv: Rename stor_vsc_on_io_completion

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/storvsc.c b/drive

Re: [PATCH 077/206] Staging: hv: Get rid of call to cleanup()

On Mon, May 09, 2011 at 11:53:35PM +, KY Srinivasan wrote: > > Note, I wouldn't recommend sending that many patches to lkml, it's not > > needed, just stick to the de...@linuxdriverproject.org list as I really > > doubt anyone else cares about this code at the moment. > > Agreed; I will do tha

RE: [PATCH 077/206] Staging: hv: Get rid of call to cleanup()

> -Original Message- > From: Greg KH [mailto:g...@kroah.com] > Sent: Monday, May 09, 2011 6:53 PM > To: KY Srinivasan > Cc: gre...@suse.de; linux-ker...@vger.kernel.org; > de...@linuxdriverproject.org; virtualizat...@lists.osdl.org; Haiyang Zhang; > Abhishek Kane (Mindtree Consulting PVT

[PATCH 039/206] Staging: hv: Get rid of the forward declaration for storvsc_device_alloc()

Get rid of the forward declaration by moving the code around. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 24 ++-- 1 files changed, 10 insertions(+), 14 delet

[PATCH 052/206] Staging: hv: Get rid of the forward declaration for storvsc_host_reset_handler()

Get rid of the forward declaration by moving the code around. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 49 ++--- 1 files changed, 24 insertions

[PATCH 047/206] Staging: hv: Get rid of the forward declaration of storvsc_remove()

Get rid of the forward declaration by moving the code around. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 62 +++--- 1 files changed, 31 insertion

[PATCH 053/206] Staging: hv: Move the definition of storvsc_ringbuffer_size to earlier in the file

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/hv/storvsc_drv.c b

[PATCH 024/206] Staging: hv: Cleanup blkvsc_shutdown()

Use the newly introduced function to wait for draining I/Os to cleanup blkvsc_shutdown(). Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/blkvsc_drv.c | 12 +--- 1 files changed, 5 insertion

[PATCH 059/206] Staging: hv: Move the declaration of the driver variable

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c |8 +--- 1 files changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/staging/hv/storvsc_d

[PATCH 027/206] Staging: hv: Rename stor_vsc_on_receive

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/storvsc.c b/drive

[PATCH 023/206] Staging: hv: Cleanup blkvsc_remove()

The function storvsc_dev_remove() already deals with draining of the outstanding I/O. Cleanup blkvsc_remove() keeping this in mind. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/blkvsc_drv.c | 18

[PATCH 036/206] Staging: hv: Rename variables pointing to struct hv_device

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 15 +++ 1 files changed, 7 insertions(+), 8 deletions(-) diff --git a/drivers/staging/hv/st

Re: [PATCH 077/206] Staging: hv: Get rid of call to cleanup()

On Mon, May 09, 2011 at 02:55:59PM -0700, K. Y. Srinivasan wrote: > cleanup() is an empty function; get rid of it. > > Signed-off-by: K. Y. Srinivasan > Signed-off-by: Haiyang Zhang > Signed-off-by: Abhishek Kane > Signed-off-by: Hank Janssen > --- > drivers/staging/hv/netvsc_drv.c |4 ---

[PATCH 066/206] Staging: hv: Simplify the code for getting the drive parameters

We are making up the drive parameters; simplify the code. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 84 +++--- 1 files changed, 15 insertions(+)

[PATCH 043/206] Staging: hv: Get rid of the forward declaration for do_bounce_buffer()

Get rid of the forward declaration by moving the code around. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 55 ++--- 1 files changed, 27 insertions

[PATCH 063/206] Staging: hv: Get rid of the indirection in invoking storvsc_dev_add()

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c |7 +-- 1 files changed, 1 insertions(+), 6 deletions(-) diff --git a/drivers/staging/hv/storvsc_dr

[PATCH 021/206] Staging: hv: Introduce a function to wait to drain outgoing I/O

Rather than busy loop waiting to drain I/O, introduce a function that does not burn CPU cycles waiting. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc.c |6 +- drivers/staging/hv

[PATCH 051/206] Staging: hv: Move the definition of storvsc_host_reset() to earlier in the file

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 99 +++--- 1 files changed, 50 insertions(+), 49 deletions(-) diff --gi

[PATCH 033/206] Staging: hv: Rename struct host_device_context

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 27 ++- 1 files changed, 14 insertions(+), 13 deletions(-) diff --git a/drivers

[PATCH 044/206] Staging: hv: Get rid of the forward declaration for create_bounce_buffer()

Get rid of the forward declaration by moving the code around. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 61 ++ 1 files changed, 29 insertion

[PATCH 046/206] Staging: hv: Get rid of the forward declaration for copy_to_bounce_buffer()

Get rid of the forward declaration by moving the code around. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 132 +++--- 1 files changed, 66 insertion

[PATCH 042/206] Staging: hv: Get rid of the forwrd declaration of destroy_bounce_buffer

Get rid of the forward declaration by moving the code around. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 32 +++- 1 files changed, 15 insertions(+),

[PATCH 045/206] Staging: hv: Get rid of the forward declaration of copy_from_bounce_buffer()

Get rid of the forward declaration by moving the code around. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 129 +++--- 1 files changed, 65 insertion

[PATCH 025/206] Staging: hv: Rename stor_vsc_channel_init()

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/storvsc.c b/drive

[PATCH 030/206] Staging: hv: Get rid of/change some dated comments

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc.c | 12 +--- 1 files changed, 1 insertions(+), 11 deletions(-) diff --git a/drivers/staging/hv/storvsc.

[PATCH 049/206] Staging: hv: Staging: hv: Rename the function stor_vsc_on_host_reset()

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/hv/storvsc_drv.c

[PATCH 022/206] Staging: hv: Use the newly introduced I/O drain synch method

Use the newly introduced I/O drain synch method. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc.c |7 ++- 1 files changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/stagin

[PATCH 064/206] Staging: hv: Get rid of the indirection in invoking storvsc_do_io()

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c |6 ++ 1 files changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/hv/storvsc_drv

[PATCH 011/206] Staging: hv: Rename the function stor_vsc_device_add

The subject says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/blkvsc_drv.c |2 +- drivers/staging/hv/storvsc.c |2 +- drivers/staging/hv/storvsc_api.h |2 +- drivers/stagin

[PATCH 015/206] Staging: hv: Get rid of the indirection for invoking cleanup function

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/blkvsc_drv.c |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/drivers/staging/hv/blkvsc_drv.c b

[PATCH 002/206] Staging: hv: Cleanup spin lock usage

This spin lock is potentially acquired from interrupt context. Ensure that the interrupts are blocked whenever the lock is held. The current code was not consistent with regards to blocking interrupts - the same lock would be acquired without blocking interrupts in some instance while the interrupt

[PATCH 001/206] Staging: hv: Get rid of blkvsc_mutex

The blkvsc_mutex is unnecessary as the state it is protecting (blkdev->users) is already protected by the spin lock. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/blkvsc_drv.c |5 - 1 files

[PATCH 013/206] Staging: hv: Get rid of the indirection for removing stor device

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/blkvsc_drv.c | 10 ++ 1 files changed, 2 insertions(+), 8 deletions(-) diff --git a/drivers/staging/hv/blkvsc_d

[PATCH 035/206] Staging: hv: Rename the struct hv_device * element of struct hv_host_device

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/hv/storvsc_d

[PATCH 018/206] Staging: hv: Statically initialize pointers for probe etc

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/blkvsc_drv.c | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/hv/blkvsc_d

[PATCH 006/206] Staging: hv: Change the name of struct storvsc_driver_object

Rename the struct storvsc_driver_object. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/blkvsc_drv.c | 14 +++--- drivers/staging/hv/storvsc.c |2 +- drivers/staging/hv/storvsc_ap

[PATCH 062/206] Staging: hv: Get rid of the indirection in invoking storvsc_dev_remove()

Get rid of the indirection in invoking storvsc_dev_remove() Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c |6 ++ 1 files changed, 2 insertions(+), 4 deletions(-) diff --git a/

[PATCH 003/206] Staging: hv: Fix some locking issues

Additional lock related cleanup. Properly serialize access to state even for command related operations. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/blkvsc_drv.c | 31 +++

[PATCH 055/206] Staging: hv: Move the function storvsc_commmand_completion() to earlier in the file

The subject line says it all. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/storvsc_drv.c | 115 +++--- 1 files changed, 58 insertions(+), 57 deletions(-) diff --gi

[PATCH 019/206] Staging: hv: Get rid of the calls to storvsc_cleanup

The storvsc_cleanup() is an empty function; get rid of it. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/blkvsc_drv.c |3 --- drivers/staging/hv/storvsc.c |7 --- drivers/staging/hv

  1   2   >