Re: [PATCH] ahci_xgene: Fix the dma state machine lockup for the ATA_CMD_SMART PIO mode command.

2015-01-27 Thread Tejun Heo
On Tue, Jan 27, 2015 at 11:26:11PM +0530, Suman Tripathi wrote: This patch addresses the issue with ATA_CMD_SMART pio mode command for enumeration and device detection with ATA devices.It is the same issue as in patch www.spinics.net/lists/linux-ide/msg49092.html The link is broken. Can

Re: old linux scsi headers

2015-01-27 Thread Andy Grover
On 01/10/2015 02:10 AM, Christoph Hellwig wrote: On Fri, Jan 09, 2015 at 09:38:50AM -0800, Andy Grover wrote: Hello glibc people, This concerns sysdeps/unix/sysv/linux/scsi/{scsi, scsi_ioctl, sg}.h They define common SCSI values, as well as Linux's SCSI-related ioctls. Apparently they were

[PATCH 10/10] qla2xxx: Comment out dead code

2015-01-27 Thread Bart Van Assche
Comment out the code that is never reached in qlt_set_data_offset() to avoid that static source code analysis tools report warnings for this code. Signed-off-by: Bart Van Assche bart.vanass...@sandisk.com Cc: Quinn Tran quinn.t...@qlogic.com Cc: Saurav Kashyap saurav.kash...@qlogic.com ---

[PATCH 09/10] qla2xxx: Remove dead code

2015-01-27 Thread Bart Van Assche
The return QLA_SUCCESS statement just above the fw_load_failed label cannot be reached, hence remove it. Additionally remove the else keyword since the code block below the if-statement ends with a return statement. Signed-off-by: Bart Van Assche bart.vanass...@sandisk.com Cc: Quinn Tran

[PATCH 03/10] qla2xxx: Remove set-but-not-used variables

2015-01-27 Thread Bart Van Assche
Signed-off-by: Bart Van Assche bart.vanass...@sandisk.com Cc: Quinn Tran quinn.t...@qlogic.com Cc: Saurav Kashyap saurav.kash...@qlogic.com --- drivers/scsi/qla2xxx/qla_attr.c | 2 -- drivers/scsi/qla2xxx/qla_bsg.c| 5 - drivers/scsi/qla2xxx/qla_dbg.c| 24 ++--

[PATCH 02/10] qla2xxx: Declare local functions static

2015-01-27 Thread Bart Van Assche
Signed-off-by: Bart Van Assche bart.vanass...@sandisk.com Cc: Quinn Tran quinn.t...@qlogic.com Cc: Saurav Kashyap saurav.kash...@qlogic.com --- drivers/scsi/qla2xxx/qla_bsg.c | 2 +- drivers/scsi/qla2xxx/qla_iocb.c | 4 ++-- drivers/scsi/qla2xxx/qla_nx.c | 2 +- drivers/scsi/qla2xxx/qla_nx2.c

[PATCH 01/10] qla2xxx: Report both rsp_info and rsp_info_len

2015-01-27 Thread Bart Van Assche
Let the debug statement in qlafx00_tm_iocb_entry() report both rsp_info and rsp_info_len instead of reporting rsp_info_len twice. Signed-off-by: Bart Van Assche bart.vanass...@sandisk.com Cc: Quinn Tran quinn.t...@qlogic.com Cc: Saurav Kashyap saurav.kash...@qlogic.com ---

[PATCH 04/10] qla2xxx: Replace two macros with an inline function

2015-01-27 Thread Bart Van Assche
Replace the QLA82XX_ADDR_IN_RANGE() and QLA8044_ADDR_IN_RANGE() macros with the inline function addr_in_range(). This avoids that the compiler reports the following warning when building with W=1: comparison of unsigned expression = 0 is always true. Signed-off-by: Bart Van Assche

[PATCH 05/10] qla2xxx: Remove __constant_ prefix

2015-01-27 Thread Bart Van Assche
Whether htonl() or __constant_htonl() is used, if the argument is a constant the conversion happens at compile time. Hence leave out the __constant_ prefix for this and other endianness conversion functions. This improves source code readability. Signed-off-by: Bart Van Assche

[PATCH 06/10] qla2xxx: Avoid that sparse complains about duplicate [noderef] attributes

2015-01-27 Thread Bart Van Assche
Signed-off-by: Bart Van Assche bart.vanass...@sandisk.com Cc: Quinn Tran quinn.t...@qlogic.com Cc: Saurav Kashyap saurav.kash...@qlogic.com --- drivers/scsi/qla2xxx/qla_dbg.c | 2 +- drivers/scsi/qla2xxx/qla_init.c | 2 +- drivers/scsi/qla2xxx/qla_iocb.c | 4 ++-- 3 files changed, 4

[PATCH 07/10] qla2xxx: Fix sparse annotations

2015-01-27 Thread Bart Van Assche
This patch removes 21 casts between an __iomem pointer type and another data type but also introduces five new casts (see also the casts with __force). Although this patch does not change any functionality, IMHO the code with __force casts needs further review. Signed-off-by: Bart Van Assche

[PATCH 08/10] qla2xxx: Remove a superfluous test

2015-01-27 Thread Bart Van Assche
Avoid that smatch reports the following warning: drivers/scsi/qla2xxx/qla_attr.c:1081: qla2x00_model_desc_show() warn: this array is probably non-NULL. 'vha-hw-model_desc' Signed-off-by: Bart Van Assche bart.vanass...@sandisk.com Cc: Quinn Tran quinn.t...@qlogic.com Cc: Saurav Kashyap

Re: [PATCH 08/17] ibmvscsi: Fix bidi command test

2015-01-27 Thread Tyrel Datwyler
On 01/23/2015 04:10 AM, Bart Van Assche wrote: Signed-off-by: Bart Van Assche bart.vanass...@sandisk.com Cc: Brian King brk...@linux.vnet.ibm.com Cc: Nathan Fontenot nf...@linux.vnet.ibm.com --- drivers/scsi/ibmvscsi/ibmvscsi.c | 10 ++ 1 file changed, 6 insertions(+), 4

[PATCH] scsi: use kzalloc instead of kmalloc/memset

2015-01-27 Thread Liviu Gheorghisan
Hint given by Coccinelle. Signed-off-by: Liviu Gheorghisan liv...@gmail.com --- drivers/scsi/aic7xxx/aic7xxx_osm.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/scsi/aic7xxx/aic7xxx_osm.c b/drivers/scsi/aic7xxx/aic7xxx_osm.c index 8836011..a2f2c77 100644 ---

Re: [PATCH] csiostor:T5 Firmware fix and cleanup.

2015-01-27 Thread David Miller
From: Praveen Madhavan prave...@chelsio.com Date: Tue, 27 Jan 2015 19:46:37 +0530 This patch fix is to use default firmware configuration files present in the adapter incase if not available in standard /lib/firmware/* dir. Additional cleanup is done to reuse flash related defines from cxgb4

[PATCH] csiostor:T5 Firmware fix and cleanup.

2015-01-27 Thread Praveen Madhavan
This patch fix is to use default firmware configuration files present in the adapter incase if not available in standard /lib/firmware/* dir. Additional cleanup is done to reuse flash related defines from cxgb4 header file. Please apply over net-next since it depends on previous commit.

Re: Backport of a fix for HPSA (Disabling a disabled device problem during kdump) driver

2015-01-27 Thread Greg KH
On Tue, Jan 06, 2015 at 05:15:19PM +0100, Tomas Henzl wrote: On 01/05/2015 07:41 PM, Masoud Sharbiani wrote: Dear stable maintainers, Can you please backport commitid 132aa220b45d60e9b20def1e9d8be9422eed9616 (hpsa: refine the pci enable/disable handling) to 3.10 stable (and earlier, if

Re: [PATCH] ahci_xgene: Fix the dma state machine lockup for the ATA_CMD_SMART PIO mode command.

2015-01-27 Thread Suman Tripathi
On Tue, Jan 27, 2015 at 11:26:11PM +0530, Suman Tripathi wrote: This patch addresses the issue with ATA_CMD_SMART pio mode command for enumeration and device detection with ATA devices.It is the same issue as in patch www.spinics.net/lists/linux-ide/msg49092.html The link is broken. Can you

[PATCH 3.4 100/177] scsi: Fix error handling in SCSI_IOCTL_SEND_COMMAND

2015-01-27 Thread lizf
From: Jan Kara j...@suse.cz 3.4.106-rc1 review patch. If anyone has any objections, please let me know. -- commit 84ce0f0e94ac97217398b3b69c21c7a62ebeed05 upstream. When sg_scsi_ioctl() fails to prepare request to submit in blk_rq_map_kern() we jump to a label where we just

Re: [PATCH v2 48/48] hpsa: Use local workqueues instead of system workqueues

2015-01-27 Thread Tomas Henzl
On 01/23/2015 11:45 PM, Don Brace wrote: Suggested-by: Tomas Henzl the...@redhat.com Yes i have suggested something but it was something different I wanted to move to his own workqueue the hpsa_monitor_ctlr_worker and exactly that is what you have left out in this patch. I'm going to post a

[PATCH 0/2 v2] hpsa: workqueue + minor fix

2015-01-27 Thread Tomas Henzl
I've adapted previously posted changes on top of the latest hpsa series. And once again. Tomas -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH 2/2] hpsa: remove a needless call - pci_get_drvdata

2015-01-27 Thread Tomas Henzl
Signed-off-by: Tomas Henzl the...@redhat.com --- drivers/scsi/hpsa.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index af32962259..6e4abba285 100644 --- a/drivers/scsi/hpsa.c +++ b/drivers/scsi/hpsa.c @@ -7102,14 +7102,13 @@

[PATCH 1/2] hpsa: use a driver's own workqueue instead of a shared wq

2015-01-27 Thread Tomas Henzl
hpsa driver uses a shared wq, max sleep time in function spent hpsa_wait_for_clear_event_notify_ack may take up to 40sec and that is too much for a shared workqueue. This patch takes the easiest approach and just creates a driver's own workqueue. Signed-off-by: Tomas Henzl the...@redhat.com ---

Re: [PATCH 1/2] ata: ahci_platform: fix owner module reference mismatch for scsi host

2015-01-27 Thread Tejun Heo
Hello, On Tue, Jan 27, 2015 at 08:34:58AM +0900, Akinobu Mita wrote: The owner module reference of the ahci platform's scsi_host is initialized to libahci_platform's one, because these drivers use a scsi_host_template defined in libahci_platform. So these drivers can be unloaded even if the

Re: [PATCH v2 48/48] hpsa: Use local workqueues instead of system workqueues

2015-01-27 Thread Tomas Henzl
On 01/27/2015 05:55 PM, Tomas Henzl wrote: On 01/23/2015 11:45 PM, Don Brace wrote: Suggested-by: Tomas Henzl the...@redhat.com Yes i have suggested something but it was something different I wanted to move to his own workqueue the hpsa_monitor_ctlr_worker and exactly that is what you have

Re: [PATCH 0/2 v2] hpsa: workqueue + minor fix

2015-01-27 Thread Tomas Henzl
On 01/27/2015 05:56 PM, Tomas Henzl wrote: I've adapted previously posted changes on top of the latest hpsa series. And once again. And because I haven't read the code in the last hpsa series properly I haven't realised that the body of this patch is already there. Please ignore this series.

[PATCH] Fix the dma state machine lockup for the ATA_CMD_SMART PIO mode command.

2015-01-27 Thread Suman Tripathi
Signed-off-by: Suman Tripathi stripa...@apm.com --- Suman Tripathi (1): ahci_xgene: Fix the dma state machine lockup for the ATA_CMD_SMART PIO mode command. drivers/ata/ahci_xgene.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) -- 1.8.2.1 -- To unsubscribe from this list: send

[PATCH] ahci_xgene: Fix the dma state machine lockup for the ATA_CMD_SMART PIO mode command.

2015-01-27 Thread Suman Tripathi
This patch addresses the issue with ATA_CMD_SMART pio mode command for enumeration and device detection with ATA devices.It is the same issue as in patch www.spinics.net/lists/linux-ide/msg49092.html Signed-off-by: Suman Tripathi stripa...@apm.com --- --- drivers/ata/ahci_xgene.c | 3 ++- 1