[PATCH 7/7] scsi: storvsc: Set the tablesize based on the information given by the host

2015-03-26 Thread K. Y. Srinivasan
Set the tablesize based on the information given by the host. Signed-off-by: K. Y. Srinivasan Reviewed-by: Long Li Tested-by: Long Li --- drivers/scsi/storvsc_drv.c | 89 +++ 1 files changed, 64 insertions(+), 25 deletions(-) diff --git a/drivers/scsi

[PATCH V2 3/7] scsi: storvsc: Always send on the selected outgoing channel

2015-03-26 Thread K. Y. Srinivasan
The current code always sent packets without data on the primary channel. Properly distribute sending of packets with no data amongst all available channels. I would like to thank Long Li for noticing this problem. Signed-off-by: K. Y. Srinivasan Reviewed-by: Long Li --- drivers/scsi/storvsc_dr

[PATCH V2 6/7] scsi: storvsc: Don't assume that the scatterlist is not chained

2015-03-26 Thread K. Y. Srinivasan
The current code assumes that the scatterlists presented are not chained. Fix the code to not make this assumption. Signed-off-by: K. Y. Srinivasan Reviewed-by: Long Li --- V2: Got rid of double assignment. Olaf Hering drivers/scsi/storvsc_drv.c | 99 +

[PATCH V2 5/7] scsi: storvsc: Fix a bug in copy_from_bounce_buffer()

2015-03-26 Thread K. Y. Srinivasan
We may exit this function without properly freeing up the maapings we may have acquired. Fix the bug. Signed-off-by: K. Y. Srinivasan Reviewed-by: Long Li Cc: --- drivers/scsi/storvsc_drv.c | 15 --- 1 files changed, 8 insertions(+), 7 deletions(-) diff --git a/drivers/scsi/stor

[PATCH V2 2/7] scsi: storvsc: Size the queue depth based on the ringbuffer size

2015-03-26 Thread K. Y. Srinivasan
Size the queue depth based on the ringbuffer size. Also accomodate for the fact that we could have multiple channels (ringbuffers) per adaptor. Signed-off-by: K. Y. Srinivasan Reviewed-by: Long Li --- v2: Fixed a bug in computing queue depth: Venkatesh Srinivas drivers/scsi/storvsc_dr

[PATCH V2 4/7] scsi: storvsc: Retrieve information about the capability of the target

2015-03-26 Thread K. Y. Srinivasan
The storage protocol informs the guest of the I/O capabilities of the storage stack. Retrieve this information and use it in the guest. Signed-off-by: K. Y. Srinivasan Reviewed-by: Long Li --- drivers/scsi/storvsc_drv.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git

[PATCH V2 1/7] scsi: storvsc: Increase the ring buffer size

2015-03-26 Thread K. Y. Srinivasan
Increase the default ring buffer size as this can significantly improve performance especially on high latency storage back-ends. Signed-off-by: K. Y. Srinivasan Reviewed-by: Long Li --- drivers/scsi/storvsc_drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers

[PATCH V2 0/7] scsi: storvsc: Miscellaneous enhancements and fixes

2015-03-26 Thread K. Y. Srinivasan
This patch-set addresses perf issues discovered on the Azure storage stack. These patches also fix a couple of bugs. As in the first version of this patch-set, some of the patches are simply a resend. I have bumped up the version number of all patches though. In this version, I have addressed is

Re: [PATCH] Staging: rtl8192 Clean up function definition

2015-03-26 Thread Eddie Kovsky
On Thu, Mar 26, 2015 at 11:37:45AM +0300, Dan Carpenter wrote: > Huh. Weird. Please, could you just move it forward instead so we don't > have to have the prototype declaration? > > regars, > dan carpenter > Dan You're right. That's a better solution. But I already got a message from Greg tha

[PATCH] staging: media: lirc: Separate authors into own MODULE_AUTHOR

2015-03-26 Thread Boran Car
Fixes quoted string split accross lines checkpatch.pl warning. Signed-off-by: Boran Car --- drivers/staging/media/lirc/lirc_zilog.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/drivers/staging/media/lirc/lirc_zilog.c b/drivers/staging/media/lirc/lirc_zilog.c i

[PATCH net-next] hv_netvsc: remove vmbus_are_subchannels_present() in rndis_filter_device_add()

2015-03-26 Thread Haiyang Zhang
The vmbus_are_subchannels_present() also involves opening the channels, which may be too early at this point. Checking for subchannels is not necessary here. So this patch removes it. Subchannels will be opened when offer messages arrive. Signed-off-by: Haiyang Zhang --- drivers/net/hyperv/rndis

Re: [PATCH v2 2/2] Staging: slicoss: Get rid of redundant pointer variable

2015-03-26 Thread Joe Perches
On Thu, 2015-03-26 at 22:32 +0100, Greg KH wrote: > On Thu, Mar 26, 2015 at 01:52:33PM -0700, Joe Perches wrote: > > On Thu, 2015-03-26 at 19:52 +0100, Greg KH wrote: > > > On Thu, Mar 26, 2015 at 11:52:43PM +0530, Niranjan Dighe wrote: > > > > Replace string directly in place of format string and

Re: [PATCH v2 2/2] Staging: slicoss: Get rid of redundant pointer variable

2015-03-26 Thread Greg KH
On Thu, Mar 26, 2015 at 01:52:33PM -0700, Joe Perches wrote: > On Thu, 2015-03-26 at 19:52 +0100, Greg KH wrote: > > On Thu, Mar 26, 2015 at 11:52:43PM +0530, Niranjan Dighe wrote: > > > Replace string directly in place of format string and remove pointer > > > variable which was used just once. >

Re: [PATCH v2 2/2] Staging: slicoss: Get rid of redundant pointer variable

2015-03-26 Thread Joe Perches
On Thu, 2015-03-26 at 19:52 +0100, Greg KH wrote: > On Thu, Mar 26, 2015 at 11:52:43PM +0530, Niranjan Dighe wrote: > > Replace string directly in place of format string and remove pointer > > variable which was used just once. [] > > diff --git a/drivers/staging/slicoss/slicoss.c > > b/drivers/st

Re: [PATCH v2 2/2] Staging: slicoss: Get rid of redundant pointer variable

2015-03-26 Thread Greg KH
On Thu, Mar 26, 2015 at 11:52:43PM +0530, Niranjan Dighe wrote: > Replace string directly in place of format string and remove pointer > variable which was used just once. > > Signed-off-by: Niranjan Dighe > > diff --git a/drivers/staging/slicoss/slicoss.c > b/drivers/staging/slicoss/slicoss.c

Re: [PATCH v2 2/2] Staging: slicoss: Get rid of redundant pointer variable

2015-03-26 Thread Dan Carpenter
On Thu, Mar 26, 2015 at 11:52:43PM +0530, Niranjan Dighe wrote: > Replace string directly in place of format string and remove pointer > variable which was used just once. > > Signed-off-by: Niranjan Dighe > > diff --git a/drivers/staging/slicoss/slicoss.c > b/drivers/staging/slicoss/slicoss.c

[PATCH v2 2/2] Staging: slicoss: Get rid of redundant pointer variable

2015-03-26 Thread Niranjan Dighe
Replace string directly in place of format string and remove pointer variable which was used just once. Signed-off-by: Niranjan Dighe diff --git a/drivers/staging/slicoss/slicoss.c b/drivers/staging/slicoss/slicoss.c index c2bda1d..f3110f7 100644 --- a/drivers/staging/slicoss/slicoss.c +++ b/dr

[PATCH v2 1/2] Staging: slicoss: Remove redundant and disabled code block

2015-03-26 Thread Niranjan Dighe
Removing code guarded by undefined macro SLIC_TRACE_DUMP_ENABLED Signed-off-by: Niranjan Dighe diff --git a/drivers/staging/slicoss/slicoss.c b/drivers/staging/slicoss/slicoss.c index 3104cb0..c2bda1d 100644 --- a/drivers/staging/slicoss/slicoss.c +++ b/drivers/staging/slicoss/slicoss.c @@ -255

Re: [PATCH v5 1/5] staging: unisys: remove redundant variable

2015-03-26 Thread Ben Romer
On 03/26/2015 09:47 AM, Ben Romer wrote: On 03/26/2015 08:01 AM, Greg Kroah-Hartman wrote: I need an ack from Benjamin and/or David before I can take these, as they are the maintainers of the driver, and have the ability to test these patches. I'll just wait to apply them until that happens. B

Re: [PATCH] Drivers: hv: hv_balloon: eliminate jumps in piecewiese linear floor function

2015-03-26 Thread Laszlo Ersek
On 03/26/15 11:26, Vitaly Kuznetsov wrote: > Commit 79208c57da53 ("Drivers: hv: hv_balloon: Make adjustments in computing > the floor") was inacurate as it introduced a jump in our piecewiese linear > 'floor' function: > > At 2048MB we have: > Left limit: > 104 + 2048/8 = 360 > Right limit: > 256

Re: [PATCH] Drivers: hv: hv_balloon: survive ballooning request with num_pages=0

2015-03-26 Thread Laszlo Ersek
On 03/26/15 17:07, Vitaly Kuznetsov wrote: > ... and simplify alloc_balloon_pages() interface by removing redundant > alloc_error from it. > > If we happen to enter balloon_up() with balloon_wrk.num_pages = 0 we will > enter > infinite 'while (!done)' loop as alloc_balloon_pages() will be always

[PATCH] Drivers: hv: hv_balloon: survive ballooning request with num_pages=0

2015-03-26 Thread Vitaly Kuznetsov
... and simplify alloc_balloon_pages() interface by removing redundant alloc_error from it. If we happen to enter balloon_up() with balloon_wrk.num_pages = 0 we will enter infinite 'while (!done)' loop as alloc_balloon_pages() will be always returning 0 and not setting alloc_error. We will also be

[PATCH net-next] hv_netvsc: Implement batching in send buffer

2015-03-26 Thread Haiyang Zhang
With this patch, we can send out multiple RNDIS data packets in one send buffer slot and one VMBus message. It reduces the overhead associated with VMBus messages. Signed-off-by: Haiyang Zhang Reviewed-by: K. Y. Srinivasan --- drivers/net/hyperv/hyperv_net.h | 16 +++- drivers/net/hyperv/n

Re: [PATCH v5 1/5] staging: unisys: remove redundant variable

2015-03-26 Thread Ben Romer
On 03/26/2015 08:01 AM, Greg Kroah-Hartman wrote: I need an ack from Benjamin and/or David before I can take these, as they are the maintainers of the driver, and have the ability to test these patches. I'll just wait to apply them until that happens. Ben/David? I'll test them today. :) -- B

Re: [PATCH] Staging: slicoss: Fix checkpatch.pl issues

2015-03-26 Thread Greg KH
On Wed, Mar 25, 2015 at 01:28:43AM +0530, Niranjan Dighe wrote: > Removed unused block of code guarded by #ifdef SLIC_TRACE_DUMP_ENABLED > And removed redundant static char *slic_banner and replaced actual string in > place of format string. That's multiple things you are doing all in the same pat

Re: [PATCH v5 1/5] staging: unisys: remove redundant variable

2015-03-26 Thread Greg Kroah-Hartman
On Tue, Mar 24, 2015 at 08:47:26PM +0530, Sudip Mukherjee wrote: > remove the variable "registered", which was used in the cleanup() to > detect if the driver has successfully initialized. the cleanup() > is called from module_exit, so its obvious that the module has > successfully initialized. if

[PATCH] Drivers: hv: hv_balloon: eliminate jumps in piecewiese linear floor function

2015-03-26 Thread Vitaly Kuznetsov
Commit 79208c57da53 ("Drivers: hv: hv_balloon: Make adjustments in computing the floor") was inacurate as it introduced a jump in our piecewiese linear 'floor' function: At 2048MB we have: Left limit: 104 + 2048/8 = 360 Right limit: 256 + 2048/16 = 384 (so the right value is 232) We now have to m

Re: [PATCH v2 01/10] staging/lustre/osc: shorten IO calling path

2015-03-26 Thread Greg Kroah-Hartman
On Wed, Mar 25, 2015 at 10:04:53PM -0400, gr...@linuxhacker.ru wrote: > From: Bobi Jam Meta-comment: In the future, when doing a v2 patch, put the "v2" after the patch number, so I can properly sort things in my email client. In other words, the subject here should be: [PATCH 01/10 v2]

Re: [PATCHv2 0/5] ft1000 driver checkpatch.pl fixes

2015-03-26 Thread Greg Kroah-Hartman
On Wed, Mar 18, 2015 at 01:56:31PM -0700, Janakarajan Natarajan wrote: > Minor changes to fix ft1000 driver checkpatch.pl warnings > > Janakarajan Natarajan (5): > Drivers: Staging: ft1000: Single line if-statement changes > Drivers: Staging: ft1000: Fix extra parenthesis warnings > Drivers:

Re: [PATCH 0/1] Drivers: hv: hv_balloon: do not online pages in offline blocks

2015-03-26 Thread Vitaly Kuznetsov
KY Srinivasan writes: >> -Original Message- >> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] >> Sent: Wednesday, March 25, 2015 11:02 AM >> To: KY Srinivasan >> Cc: Haiyang Zhang; de...@linuxdriverproject.org; linux- >> ker...@vger.kernel.org; Dexuan Cui >> Subject: [PATCH 0/1] Driv

Re: [PATCH] Staging: rtl8192 Clean up function definition

2015-03-26 Thread Dan Carpenter
Huh. Weird. Please, could you just move it forward instead so we don't have to have the prototype declaration? regars, dan carpenter ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev