Re: [PATCH][SCSI] mpt2sas: fix undefined reference to `__udivdi3' compilation errors

2014-09-22 Thread Christoph Hellwig
Looks good to me. Can I get a quick second review? On Fri, Sep 19, 2014 at 12:17:27PM +0530, Sreekanth Reddy wrote: This patch will fix the below compilation errors on i386 ARCH drivers/built-in.o: In function `_scsih_qcmd': mpt2sas_scsih.c:(.text+0x1e7b56): undefined reference to

[PATCH] ahci_xgene: Fix the error print invalid resource for APM X-Gene SoC AHCI SATA Host Controller driver.

2014-09-22 Thread Suman Tripathi
This patch fixes the error print invalid resource for the APM X-Gene SoC AHCI SATA Host Controller driver. This print was due to the fact that the controller 3 don't have a mux resource. This didn't result in any errors but the print seems like meaningless. Signed-off-by: Loc Ho l...@apm.com

Re: [PATCH][SCSI] mpt2sas: fix undefined reference to `__udivdi3' compilation errors

2014-09-22 Thread Martin K. Petersen
Sreekanth == Sreekanth Reddy sreekanth.re...@avagotech.com writes: Sreekanth This patch will fix the below compilation errors on i386 ARCH Sreekanth drivers/built-in.o: In function `_scsih_qcmd': Sreekanth mpt2sas_scsih.c:(.text+0x1e7b56): undefined reference to Sreekanth `__udivdi3'

two more fixes for block/for-linus

2014-09-22 Thread Christoph Hellwig
Fix two regressions with the timer/blk_mq_start_request changes found when using scsi-mq on an ATA device. Note that given how late we are in the cycle I wonder if we really should push the changes in block/for-linus to Linus in this cycle. The race part of the timer fixes is extremly narrow,

[PATCH 1/2] block: fix blk_abort_request on blk-mq

2014-09-22 Thread Christoph Hellwig
Signed-off-by: Christoph Hellwig h...@lst.de --- block/blk-mq.c | 2 +- block/blk-timeout.c | 5 - block/blk.h | 2 ++ 3 files changed, 7 insertions(+), 2 deletions(-) diff --git a/block/blk-mq.c b/block/blk-mq.c index 966c920..aeac4b7 100644 --- a/block/blk-mq.c +++

Re: two more fixes for block/for-linus

2014-09-22 Thread Jens Axboe
On 2014-09-22 07:59, Christoph Hellwig wrote: Fix two regressions with the timer/blk_mq_start_request changes found when using scsi-mq on an ATA device. Note that given how late we are in the cycle I wonder if we really should push the changes in block/for-linus to Linus in this cycle. The

Re: [PATCH 1/2] block: fix blk_abort_request on blk-mq

2014-09-22 Thread Jens Axboe
On 2014-09-22 07:59, Christoph Hellwig wrote: Signed-off-by: Christoph Hellwig h...@lst.de --- block/blk-mq.c | 2 +- block/blk-timeout.c | 5 - block/blk.h | 2 ++ 3 files changed, 7 insertions(+), 2 deletions(-) diff --git a/block/blk.h b/block/blk.h index

Re: [PATCH V3 01/16] scsi: support well known logical units

2014-09-22 Thread 'Christoph Hellwig'
On Thu, Sep 18, 2014 at 12:12:22PM -0700, Subhash Jadavani wrote: Hi Chris, While testing with another vendor's UFS devices, I realized that SELECT REPORT need not to be set to 0x02 for making the device report the W-LUs. Even when SELECT REPORT is set 0x00, this particular UFS device

Re: [PATCH RFC 0/8] IB/srp: Add multichannel support

2014-09-22 Thread Christoph Hellwig
Hi Bart, I like these changes modulo the minor comments we had. One thing that is missing is generation multiqueue-aware tags at the blk-mq level, which should be as simple as always adding a queue prefix in the tag allocation code. Did you consider switching srp to use the block layer provided

Re: [PATCH] scsi: fix kconfig dependency warnings for SCSI_FC_ATTRS

2014-09-22 Thread Christoph Hellwig
On Fri, Sep 19, 2014 at 04:50:59PM -0700, Randy Dunlap wrote: --- linux-next-20140918.orig/drivers/scsi/qla2xxx/Kconfig +++ linux-next-20140918/drivers/scsi/qla2xxx/Kconfig @@ -30,7 +30,7 @@ config SCSI_QLA_FC config TCM_QLA2XXX tristate TCM_QLA2XXX fabric module for Qlogic 2xxx

Re: [PATCH scsi] cxgb4i : Avoid holding mutex in interrupt context

2014-09-22 Thread Christoph Hellwig
Thanks, applied. -- 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

Re: [PATCH] scsi: fix kconfig dependency warnings for SCSI_FC_ATTRS

2014-09-22 Thread David Miller
From: Christoph Hellwig h...@infradead.org Date: Mon, 22 Sep 2014 07:42:13 -0700 On Fri, Sep 19, 2014 at 04:50:59PM -0700, Randy Dunlap wrote: --- linux-next-20140918.orig/drivers/scsi/qla2xxx/Kconfig +++ linux-next-20140918/drivers/scsi/qla2xxx/Kconfig @@ -30,7 +30,7 @@ config SCSI_QLA_FC

Re: [PATCH] scsi: fix kconfig dependency warnings for SCSI_FC_ATTRS

2014-09-22 Thread David Miller
From: Christoph Hellwig h...@infradead.org Date: Mon, 22 Sep 2014 07:42:13 -0700 On Fri, Sep 19, 2014 at 04:50:59PM -0700, Randy Dunlap wrote: --- linux-next-20140918.orig/drivers/scsi/qla2xxx/Kconfig +++ linux-next-20140918/drivers/scsi/qla2xxx/Kconfig @@ -30,7 +30,7 @@ config SCSI_QLA_FC

[PATCH] [SCSI] csiostor: Don't ignore the newly alloced lnode.

2014-09-22 Thread P Karthik Jayanth
In case of multiple FCFs, the newly allocated lnode by csio_lnode_alloc() is completely ignored during link up event. Fix it. Signed-off-by: P Karthik Jayanth kart...@chelsio.com --- drivers/scsi/csiostor/csio_lnode.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH] scsi: fix kconfig dependency warnings for SCSI_FC_ATTRS

2014-09-22 Thread David Miller
From: Randy Dunlap rdun...@infradead.org Date: Fri, 19 Sep 2014 16:50:59 -0700 From: Randy Dunlap rdun...@infradead.org Fix kconfig dependency warnings which can lead to build errors: warning: (SCSI_BNX2X_FCOE LIBFCOE TCM_QLA2XXX) selects LIBFC which has unmet direct dependencies

Re: [PATCH RFC 0/8] IB/srp: Add multichannel support

2014-09-22 Thread Jens Axboe
On 2014-09-22 10:25, Bart Van Assche wrote: On 22/09/2014 8:37, Christoph Hellwig wrote: One thing that is missing is generation multiqueue-aware tags at the blk-mq level, which should be as simple as always adding a queue prefix in the tag allocation code. Hello Christoph, Adding a queue

Re: SCSI test tool for linux (and others)

2014-09-22 Thread Bart Van Assche
On 21/09/2014 9:58, ronnie sahlberg wrote: For now I populated it with a bunch of tests I copied fro the SCSI family. Some, not all, tests. Missing are a whole bunch of tests that require that you have multiple I_T nexuses, such as all the persistent reservation tests, which would be difficult

Re: [RFC v2 3/6] kthread: warn on kill signal if not OOM

2014-09-22 Thread Luis R. Rodriguez
On Mon, Sep 8, 2014 at 7:57 PM, Luis R. Rodriguez mcg...@do-not-panic.com wrote: Why do we care about the priority of probing tasks? Does that actually make any meaningful difference? If so, how? As I noted before -- I have yet to provide clear metrics but at least changing both init paths

Re: [PATCH RFC 0/8] IB/srp: Add multichannel support

2014-09-22 Thread Jens Axboe
On 2014-09-22 10:31, Jens Axboe wrote: On 2014-09-22 10:25, Bart Van Assche wrote: On 22/09/2014 8:37, Christoph Hellwig wrote: One thing that is missing is generation multiqueue-aware tags at the blk-mq level, which should be as simple as always adding a queue prefix in the tag allocation

Re: SCSI test tool for linux (and others)

2014-09-22 Thread ronnie sahlberg
On Mon, Sep 22, 2014 at 9:34 AM, Bart Van Assche bvanass...@acm.org wrote: On 21/09/2014 9:58, ronnie sahlberg wrote: For now I populated it with a bunch of tests I copied fro the SCSI family. Some, not all, tests. Missing are a whole bunch of tests that require that you have multiple I_T

Re: [PATCH] scsi: fix kconfig dependency warnings for SCSI_FC_ATTRS

2014-09-22 Thread Christoph Hellwig
On Mon, Sep 22, 2014 at 11:56:58AM -0400, David Miller wrote: CONFIG_SCSI_QLA_FC currently selects SCSI_FC_ATTRS, so we shouldn't need this here. Also any reason not to switch that one over to a depends as well? Christoph, I understand your confusion now, in the networking tree

Re: [PATCH RFC 0/8] IB/srp: Add multichannel support

2014-09-22 Thread Bart Van Assche
On 22/09/2014 8:37, Christoph Hellwig wrote: One thing that is missing is generation multiqueue-aware tags at the blk-mq level, which should be as simple as always adding a queue prefix in the tag allocation code. Hello Christoph, Adding a queue prefix in the tag allocation code is an

[PATCH] scsi: sas_ata: change ata_foo_printk routines to return void

2014-09-22 Thread Joe Perches
The return value is not used by callers of this function so change the function to return void. Signed-off-by: Joe Perches j...@perches.com --- This change is associated to a desire to eventually change printk to return void. drivers/scsi/libsas/sas_ata.c | 11 --- 1 file changed, 4

[PATCH] mptfusion: enable no_write_same in scsi_host_template

2014-09-22 Thread Chris J Arges
When using a virtual SCSI disk in a VMWare VM if blkdev_issue_zeroout is used data can be improperly zeroed out using the mptfusion driver. This patch disables write_same for this driver which ensures that manual zeroing out is used instead. BugLink: http://bugs.launchpad.net/bugs/1371591

Re: [PATCH] mptfusion: enable no_write_same in scsi_host_template

2014-09-22 Thread Chris J Arges
On 09/22/2014 01:02 PM, Christoph Hellwig wrote: On Mon, Sep 22, 2014 at 12:56:59PM -0500, Chris J Arges wrote: When using a virtual SCSI disk in a VMWare VM if blkdev_issue_zeroout is used data can be improperly zeroed out using the mptfusion driver. This patch disables write_same for this

Re: [PATCH] mptfusion: enable no_write_same in scsi_host_template

2014-09-22 Thread Christoph Hellwig
On Mon, Sep 22, 2014 at 01:17:01PM -0500, Chris J Arges wrote: I've only been able to reproduce this on VMWare. There is a pretty straightforward reproducer in the BugLink if there is any interest in verifying on hardware. How would you recommending blacklisting only VMWare guests in this

Re: two more fixes for block/for-linus

2014-09-22 Thread Douglas Gilbert
On 14-09-22 10:06 AM, Jens Axboe wrote: On 2014-09-22 07:59, Christoph Hellwig wrote: Fix two regressions with the timer/blk_mq_start_request changes found when using scsi-mq on an ATA device. Note that given how late we are in the cycle I wonder if we really should push the changes in

Re: [PATCH] mptfusion: enable no_write_same in scsi_host_template

2014-09-22 Thread Chris J Arges
On 09/22/2014 01:19 PM, Christoph Hellwig wrote: On Mon, Sep 22, 2014 at 01:17:01PM -0500, Chris J Arges wrote: I've only been able to reproduce this on VMWare. There is a pretty straightforward reproducer in the BugLink if there is any interest in verifying on hardware. How would you

make ARCH=i386 allyesconfig build error with next-20140922, in drivers/scsi/mpt2sas/mpt2sas_scsih.c

2014-09-22 Thread Jim Davis
drivers/built-in.o: In function `_scsih_qcmd': mpt2sas_scsih.c:(.text+0xf43d7b): undefined reference to `__udivdi3' mpt2sas_scsih.c:(.text+0xf43db4): undefined reference to `__umoddi3' make: *** [vmlinux] Error 1 -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of

Re: [RFC v2 3/6] kthread: warn on kill signal if not OOM

2014-09-22 Thread Pavel Machek
On Thu 2014-09-11 13:23:54, Dmitry Torokhov wrote: On Thu, Sep 11, 2014 at 12:59:25PM -0700, James Bottomley wrote: On Tue, 2014-09-09 at 16:01 -0700, Dmitry Torokhov wrote: On Tuesday, September 09, 2014 03:46:23 PM James Bottomley wrote: On Wed, 2014-09-10 at 07:41 +0900, Tejun Heo

Re: [RFC v2 3/6] kthread: warn on kill signal if not OOM

2014-09-22 Thread Dmitry Torokhov
On Monday, September 22, 2014 09:49:06 PM Pavel Machek wrote: On Thu 2014-09-11 13:23:54, Dmitry Torokhov wrote: On Thu, Sep 11, 2014 at 12:59:25PM -0700, James Bottomley wrote: On Tue, 2014-09-09 at 16:01 -0700, Dmitry Torokhov wrote: On Tuesday, September 09, 2014 03:46:23 PM James

Re: [PATCH 4/4] target: Add a user-passthrough backstore

2014-09-22 Thread Nicholas A. Bellinger
On Fri, 2014-09-19 at 16:51 -0700, Alex Elsayed wrote: Nicholas A. Bellinger wrote: On Fri, 2014-09-19 at 14:43 -0700, Alex Elsayed wrote: Nicholas A. Bellinger wrote: snip So the idea of allowing the in-kernel CDB emulation to run after user-space has returned unsupported opcode

[PATCH v2] mptfusion: enable no_write_same for vmware scsi disks

2014-09-22 Thread Chris J Arges
When using a virtual SCSI disk in a VMWare VM if blkdev_issue_zeroout is used data can be improperly zeroed out using the mptfusion driver. This patch disables write_same for this driver and the vmware subsystem_vendor which ensures that manual zeroing out is used instead. BugLink:

Re: [PATCH 4/4] target: Add a user-passthrough backstore

2014-09-22 Thread Nicholas A. Bellinger
On Fri, 2014-09-19 at 17:35 -0700, Andy Grover wrote: On 09/19/2014 04:51 PM, Alex Elsayed wrote: Not sure I follow.. How does the proposed passthrough mode prevent someone from emulating OSDs, media changers, optical disks or anything else in userspace with TCMU..? The main thing

Re: [PATCH 4/4] target: Add a user-passthrough backstore

2014-09-22 Thread Andy Grover
On 09/22/2014 01:58 PM, Nicholas A. Bellinger wrote: So I'd still like to start for an initial merge with the two different modes mentioned earlier. The pure-passthrough mode where everything is handled by user-space, and an I/O passthrough mode where only SCF_SCSI_DATA_CDB is passed along to

Re: [PATCH 4/4] target: Add a user-passthrough backstore

2014-09-22 Thread Nicholas A. Bellinger
On Mon, 2014-09-22 at 14:00 -0700, Andy Grover wrote: On 09/22/2014 01:58 PM, Nicholas A. Bellinger wrote: So I'd still like to start for an initial merge with the two different modes mentioned earlier. The pure-passthrough mode where everything is handled by user-space, and an I/O

Re: boot stall regression due to blk-mq: use percpu_ref for mq usage count

2014-09-22 Thread Christoph Hellwig
Jens, can we simply get these commits reverted from now if there's no better fix? I'd hate to have this boot stall in the first kernel with blk-mq support for scsi. -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to majord...@vger.kernel.org More

Re: two more fixes for block/for-linus

2014-09-22 Thread Christoph Hellwig
On Mon, Sep 22, 2014 at 02:40:15PM -0400, Douglas Gilbert wrote: With these patches applied (actually a resync an hour ago with the for-linus tree which includes them), the freeze-during-boot-up problem that I have been seeing with an old SATA boot disk (perhaps 1.5 Gbps) for the last two

Re: boot stall regression due to blk-mq: use percpu_ref for mq usage count

2014-09-22 Thread Tejun Heo
On Tue, Sep 23, 2014 at 07:55:54AM +0200, Christoph Hellwig wrote: Jens, can we simply get these commits reverted from now if there's no better fix? I'd hate to have this boot stall in the first kernel with blk-mq support for scsi. Patches going out right now. Thanks. -- tejun -- To

Re: boot stall regression due to blk-mq: use percpu_ref for mq usage count

2014-09-22 Thread Tejun Heo
On Tue, Sep 23, 2014 at 01:56:48AM -0400, Tejun Heo wrote: On Tue, Sep 23, 2014 at 07:55:54AM +0200, Christoph Hellwig wrote: Jens, can we simply get these commits reverted from now if there's no better fix? I'd hate to have this boot stall in the first kernel with blk-mq support for