linux-stable.git/drivers/hv, branch linux-3.4.y

Drivers: hv: vmbus: Don't wait after requesting offers

2015-09-18T01:20:26+00:00

commit 73cffdb65e679b98893f484063462c045adcf212 upstream.

Don't wait after sending request for offers to the host. This wait is
unnecessary and simply adds 5 seconds to the boot time.

Signed-off-by: K. Y. Srinivasan 
Signed-off-by: Greg Kroah-Hartman 
[lizf: Backported to 3.4: adjust context]
Signed-off-by: Zefan Li

Drivers: hv: vmbus: Fix a bug in the error path in vmbus_open()

2015-09-18T01:20:23+00:00

commit 40384e4bbeb9f2651fe9bffc0062d9f31ef625bf upstream.

Correctly rollback state if the failure occurs after we have handed over
the ownership of the buffer to the host.

Signed-off-by: K. Y. Srinivasan 
Signed-off-by: Greg Kroah-Hartman 
Signed-off-by: Zefan Li

Drivers: hv: vmbus: incorrect device name is printed when child device is unregistered

2015-04-14T09:34:00+00:00

commit 84672369ffb98a51d4ddf74c20a23636da3ad615 upstream.

Whenever a device is unregistered in vmbus_device_unregister (drivers/hv/vmbus_drv.c), the device name in the log message may contain garbage as the memory has already been freed by the time pr_info is called. Log example:
 [ 3149.170475] hv_vmbus: child device àõsèè0_5 unregistered

By logging the message just before calling device_unregister, the correct device name is printed:
[ 3145.034652] hv_vmbus: child device vmbus_0_5 unregistered

Also changing register & unregister messages to debug to avoid unnecessarily cluttering the kernel log.

Signed-off-by: Fernando M Soto 
Signed-off-by: K. Y. Srinivasan 
Signed-off-by: Greg Kroah-Hartman 
Cc: Joseph Salisbury 
Signed-off-by: Zefan Li

Drivers: hv: vmbus: Cleanup vmbus_close_internal()

2015-02-02T09:04:41+00:00

commit 98d731bb064a9d1817a6ca9bf8b97051334a7cfe upstream.

Eliminate calls to BUG_ON() in vmbus_close_internal().
We have chosen to potentially leak memory, than crash the guest
in case of failures.

In this version of the patch I have addressed comments from
Dan Carpenter (dan.carpenter@oracle.com).

Signed-off-by: K. Y. Srinivasan 
Tested-by: Sitsofe Wheeler 
Signed-off-by: Greg Kroah-Hartman 
[lizf: Backported to 3.4: s/return ret/return/g]
Signed-off-by: Zefan Li

Drivers: hv: vmbus: Fix a bug in vmbus_open()

2015-02-02T09:04:40+00:00

commit 45d727cee9e200f5b351528b9fb063b69cf702c8 upstream.

Fix a bug in vmbus_open() and properly propagate the error. I would
like to thank Dexuan Cui  for identifying the
issue.

Signed-off-by: K. Y. Srinivasan 
Tested-by: Sitsofe Wheeler 
Signed-off-by: Greg Kroah-Hartman 
Signed-off-by: Zefan Li

Drivers: hv: vmbus: Cleanup vmbus_establish_gpadl()

2015-02-02T09:04:40+00:00

commit 72c6b71c245dac8f371167d97ef471b367d0b66b upstream.

Eliminate the call to BUG_ON() by waiting for the host to respond. We are
trying to reclaim the ownership of memory that was given to the host and so
we will have to wait until the host responds.

Signed-off-by: K. Y. Srinivasan 
Tested-by: Sitsofe Wheeler 
Signed-off-by: Greg Kroah-Hartman 
Signed-off-by: Zefan Li

Drivers: hv: vmbus: Cleanup vmbus_teardown_gpadl()

2015-02-02T09:04:40+00:00

commit 66be653083057358724d56d817e870e53fb81ca7 upstream.

Eliminate calls to BUG_ON() by properly handling errors. In cases where
rollback is possible, we will return the appropriate error to have the
calling code decide how to rollback state. In the case where we are
transferring ownership of the guest physical pages to the host,
we will wait for the host to respond.

Signed-off-by: K. Y. Srinivasan 
Tested-by: Sitsofe Wheeler 
Signed-off-by: Greg Kroah-Hartman 
Signed-off-by: Zefan Li

Drivers: hv: vmbus: Cleanup vmbus_post_msg()

2015-02-02T09:04:39+00:00

commit fdeebcc62279119dbeafbc1a2e39e773839025fd upstream.

Posting messages to the host can fail because of transient resource
related failures. Correctly deal with these failures and increase the
number of attempts to post the message before giving up.

In this version of the patch, I have normalized the error code to
Linux error code.

Signed-off-by: K. Y. Srinivasan 
Tested-by: Sitsofe Wheeler 
Signed-off-by: Greg Kroah-Hartman 
Signed-off-by: Zefan Li

drivers: hv: switch to use mb() instead of smp_mb()

2014-06-07T23:02:10+00:00

commit 35848f68b07df3f917cb13fc3c134718669f569b upstream.

Even if guest were compiled without SMP support, it could not assume that host
wasn't. So switch to use mb() instead of smp_mb() to force memory barriers for
UP guest.

Signed-off-by: Jason Wang 
Cc: Haiyang Zhang 
Signed-off-by: K. Y. Srinivasan 
Signed-off-by: Greg Kroah-Hartman 
[bwh: Backported to 3.2:
 - Drop changes to functions that don't exist here
 - hv_ringbuffer_write() has only a write memory barrier]
Signed-off-by: Ben Hutchings 
[hq: Backported to 3.4:
 - Add the change in hv_ringbuffer_read]
Signed-off-by: Qiang Huang 
Signed-off-by: Greg Kroah-Hartman

Drivers: hv: Cleanup error handling in vmbus_open()

2012-10-31T17:02:58+00:00

commit 1392550240aaa72ce3a094a38bd23525cd67ce60 upstream.

Fix a memory leak  in the error handling path in the function vmbus_open().

Signed-off-by: K. Y. Srinivasan 
Reviewed-by: Haiyang Zhang 
Reported-by: Jason Wang 
Acked-by: Jason Wang 
Signed-off-by: Greg Kroah-Hartman