Re: [PATCH 01/21] lpfc: Fix the iteration count to match the 30 sec comment

2015-03-09 Thread James Bottomley
On Mon, 2015-03-09 at 10:07 -0400, James Smart wrote: Yes, I'll correct this. Great, thanks! Trying to make Christoph happy by using git to format all the patches. We're all happy you're using git to format the patches ... it's just a minor addition to your commit log to give me the

Re: [PATCH v3] xen-scsiback: define a pr_fmt macro with xen-pvscsi

2015-03-09 Thread Chentao (Boby)
On 2015/3/6 20:53, Juergen Gross wrote: On 03/06/2015 09:34 PM, Tao Chen wrote: Add the {xen-pvscsi: } prefix in pr_fmt and remove DPRINTK, then replace all DPRINTK with pr_debug. Also fixed up some comments just as eliminate redundant whitespace and format the code. These will make the

Re: [PATCH v2] xen-scsiback: use DRV_PFX in the pr macros and DPRINTK

2015-03-09 Thread Chentao (Boby)
On 2015/3/6 20:50, Joe Perches wrote: On Fri, 2015-03-06 at 17:33 +0800, Chentao (Boby) wrote: #ifdef pr_fmt #undef pr_fmt #endif #define pr_fmt(fmt) xen-pvscsi: fmt No, just use add #define pr_fmt(fmt) xen-pvscsi: fmt before the first #include. The #ifdef/#undef/#endif isn't necessary.

Re: [PATCH 2/2] bnx2fc: bnx2fc_io.c: match wait_for_completion_timeout return type

2015-03-09 Thread Chad Dupuis
Acked-by: Chad Dupuis chad.dup...@qlogic.com On Wed, 4 Mar 2015, Nicholas Mc Guire wrote: return type of wait_for_completion_timeout is unsigned long not int. An appropriately named variable of type unsigned long is added and the assignments fixed up. Signed-off-by: Nicholas Mc Guire

Re: [PATCH 01/21] lpfc: Fix the iteration count to match the 30 sec comment

2015-03-09 Thread James Smart
Yes, I'll correct this. Trying to make Christoph happy by using git to format all the patches. -- james On 3/7/2015 11:44 AM, James Bottomley wrote: On Thu, 2015-02-05 at 14:23 -0500, James Smart wrote: --- Could you put the signoffs (and any changelog) above here, please. You seem to

Re: [PATCH 0/5] Rename regulator_set_optimum_mode

2015-03-09 Thread Mark Brown
On Wed, Feb 11, 2015 at 07:35:26PM -0800, Bjorn Andersson wrote: Changing the name of the regulator_set_optimum_mode() to regulator_set_load() better reflects that the API is doing. Applied all, thanks. signature.asc Description: Digital signature

[PATCH] sym53c416: Fix nasty memory corruption

2015-03-09 Thread Ondrej Zary
In sym53c416_read(), the chip can (and does sometimes) return more bytes in the FIFO than we want to read. This causes buffer overflow, resulting in nasty memory and data corruption and oopses. I couldn't even read filesystem's root directory properly (and a simple dd with 1M blocksize crashed the

Re: [PATCH 3/3] tcm_qla2xxx: Expose per endpoint dynamic_sessions attribute

2015-03-09 Thread Saurav Kashyap
Acked-by: Saurav Kashyap saurav.kash...@qlogic.com Thanks, ~Saurav From: Nicholas Bellinger n...@linux-iscsi.org This patch exposes a new ../qla2xxx*/$WWPN/$TPGT/dynamic_sessions attribute to dump the currently active sessions by FC initiator WWPN that have been created with dynamically

[PATCH 1/3] scsi: storvsc: Retrieve information about the capability of the target

2015-03-09 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 k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com --- drivers/scsi/storvsc_drv.c |6 ++ 1 files changed, 6

[PATCH 3/3] scsi: storvsc: Enable clustering

2015-03-09 Thread K. Y. Srinivasan
Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c index a36c232..d7ebf00 100644 --- a/drivers/scsi/storvsc_drv.c +++

[PATCH 0/3] scsi: storvsc: Increase the tablesize based on host's capabilities

2015-03-09 Thread K. Y. Srinivasan
Presently, storvsc limits the I/O size arbitrarily. Make this configurable based on what the host advertises. K. Y. Srinivasan (3): scsi: storvsc: Retrieve information about the capability of the target scsi: storvsc: Set the tablesize based on the information given by the host

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

2015-03-09 Thread K. Y. Srinivasan
Set the tablesize based on what the host has informed us. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com --- drivers/scsi/storvsc_drv.c | 70 ++- 1 files changed, 55 insertions(+), 15 deletions(-) diff --git

Re: [PATCH] be2iscsi: Use pci_zalloc_consistent() where possible and get rid of memset(,0,)

2015-03-09 Thread Maurizio Lombardi
I forgot the signed-off-by, I'll send a new version. On Fri, 2015-03-06 at 14:52 +0100, Maurizio Lombardi wrote: --- drivers/scsi/be2iscsi/be_iscsi.c | 3 +-- drivers/scsi/be2iscsi/be_main.c | 3 +-- drivers/scsi/be2iscsi/be_mgmt.c | 3 +-- 3 files changed, 3 insertions(+), 6 deletions(-)

RFC scsi: aic94xx: use of complete() looks racy

2015-03-09 Thread Nicholas Mc Guire
Hi ! while reviewing some type missmatch of wait_for_completion_timeout in drivers/scsi/aic94xx/aic94xx_tmf.c:asd_clear_nexus() a question regarding the struct completion use came up: (line numbers are from 4.0-rc2 linux-next) 351 tascb-completion = completion; 352 353

[PATCH 0/3] NCR5380 patches from mailing list

2015-03-09 Thread Finn Thain
As requested by James, these are the NCR5380 patches that were reviewed on the linux-scsi mailing list. I've changed my acked-by tags to signed-off-by tags. git am and git format-patch helpfully moved the email From: headers to the message bodies, but not the Date: headers, and it chucked out the

[PATCH 1/3] ncr5380: Drop owner assignment from platform_drivers

2015-03-09 Thread Finn Thain
From: Wolfram Sang w...@the-dreams.de This platform_driver does not need to set an owner, it will be populated by the driver core. Signed-off-by: Wolfram Sang w...@the-dreams.de Signed-off-by: Finn Thain fth...@telegraphics.com.au --- drivers/scsi/atari_scsi.c | 1 - drivers/scsi/mac_scsi.c |

[PATCH 2/3] g_NCR5380: Kill compiler warning if builtin

2015-03-09 Thread Finn Thain
From: Geert Uytterhoeven ge...@linux-m68k.org If CONFIG_SCSI_GENERIC_NCR5380=y: drivers/scsi/g_NCR5380.c:727: warning: 'id_table' defined but not used In the non-modular case, MODULE_DEVICE_TABLE() expands to nothing, and id_table is not referenced. Correct the existing #ifdef to fix this.

[PATCH] aic94xx: match wait_for_completion_timeout return type

2015-03-09 Thread Nicholas Mc Guire
return type of wait_for_completion_timeout is unsigned long not int. As leftover is used exclusively for wait_for_completion_timeout its type is simply changed to unsigned long. Signed-off-by: Nicholas Mc Guire hof...@osadl.org --- This was only compile tested for x86_64_defconfig +