[GIT PULL -v3] target fixes for v3.12-rc0 (was v3.11)

2013-09-02 Thread Nicholas A. Bellinger
Hi Linus, Now with DKIM + SPF in place on linux-iscsi.org (thanks for the help Ted & Co), here is another attempt at the target fixes PULL request for v3.11 that was missed. Given that the last PULL emails hit your spam folder, if/when you get this please give me a quick response to confirm. Ple

[Bug 60758] module scsi_wait_scan not found kernel panic on boot

2013-09-02 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=60758 --- Comment #22 from Jeff Zhou --- (In reply to zakrzewskim from comment #21) > Here you are: http://www.upemax.user.icpnet.pl/3.10.4-1.el6.lst Yes then we see the scsi_scan_wait is not in 3.10.4 either, otherwise there will be a line :"drivers/s

[PATCH] ibmvscsi: Fix little endian issues

2013-09-02 Thread Anton Blanchard
The hypervisor is big endian, so little endian kernel builds need to byteswap. Signed-off-by: Anton Blanchard --- diff --git a/drivers/scsi/ibmvscsi/ibmvscsi.c b/drivers/scsi/ibmvscsi/ibmvscsi.c index d0fa4b6..62bdbc9 100644 --- a/drivers/scsi/ibmvscsi/ibmvscsi.c +++ b/drivers/scsi/ibmvscsi

[Bug 60758] module scsi_wait_scan not found kernel panic on boot

2013-09-02 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=60758 --- Comment #21 from zakrzews...@wp.pl --- Here you are: http://www.upemax.user.icpnet.pl/3.10.4-1.el6.lst -- You are receiving this mail because: You are the assignee for the bug. -- To unsubscribe from this list: send the line "unsubscribe linu

[Bug 60758] module scsi_wait_scan not found kernel panic on boot

2013-09-02 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=60758 --- Comment #20 from Jeff Zhou --- Yes, there could be extra bugs. Since this bug is about "scsi_wait_scan not found" in v3.10.5 and above, we can try to work on this one first. As suggested by Alan, the lsinitrd is available, it can be used to

Re: [PATCH 4/7] Disallow changing the device state via sysfs into "deleted"

2013-09-02 Thread Christoph Hellwig
On Mon, Sep 02, 2013 at 09:12:15PM +0200, Bart Van Assche wrote: > Hmm ... I think the second option would require to add an additional > case in sdev_set_state() to avoid a compiler warning. This is > something James objected against (on June 24, see also > http://thread.gmane.org/gmane.linux.scsi

Re: [PATCH 4/7] Disallow changing the device state via sysfs into "deleted"

2013-09-02 Thread Christoph Hellwig
On Mon, Sep 02, 2013 at 08:59:30PM +0200, Bart Van Assche wrote: > Do you agree with changing switch (state) into switch ((int)state) ? > Without that additional change gcc reports the following warning: > > drivers/scsi/scsi_sysfs.c: In function ?store_state_field?: > drivers/scsi/scsi_sysfs.c:64

Re: [PATCH 4/7] Disallow changing the device state via sysfs into "deleted"

2013-09-02 Thread Bart Van Assche
On 09/02/13 21:06, Christoph Hellwig wrote: On Mon, Sep 02, 2013 at 08:59:30PM +0200, Bart Van Assche wrote: Do you agree with changing switch (state) into switch ((int)state) ? Without that additional change gcc reports the following warning: drivers/scsi/scsi_sysfs.c: In function ?store_state

Re: [PATCH 4/7] Disallow changing the device state via sysfs into "deleted"

2013-09-02 Thread Bart Van Assche
On 09/01/13 18:49, Christoph Hellwig wrote: On Tue, Aug 20, 2013 at 02:08:39PM +0200, Bart Van Assche wrote: Changing the state of a SCSI device via sysfs into "cancel", "cancel-offline" or "deleted" prevents removal of these devices by scsi_remove_host(). Hence do not allow this. Looks good m

Re: [PATCH 3/9] scsi: improved eh timeout handler

2013-09-02 Thread Bart Van Assche
On 09/02/13 15:11, Hannes Reinecke wrote: On 09/02/2013 02:45 PM, Bart Van Assche wrote: On 09/02/13 09:12, Hannes Reinecke wrote: @@ -353,6 +354,8 @@ void scsi_put_command(struct scsi_cmnd *cmd) list_del_init(&cmd->list); spin_unlock_irqrestore(&cmd->device->list_lock, flags); +

Hallo..........

2013-09-02 Thread Mutoni Williams
Hello,How are you,I hope you're well,my name is Mutoni,I'm medium height and fair in complexion,i love,caring and I decided to contact you.I really want to have a good relationship with you.Next I have a special something I want to discuss with you,and tell you more about my self.Hope hear from

[Bug 60758] module scsi_wait_scan not found kernel panic on boot

2013-09-02 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=60758 --- Comment #19 from zakrzews...@wp.pl --- 1. All of them are separate. I just install kernel-ml via yum. 2. I can only show such log from kernel 3.10.4-1. I don't have KVM to see what's going on. Since this is a production machine inside datacent

Re: Odd behavior of a "SAS-2" backplane with SGPIO commands

2013-09-02 Thread Pasi Kärkkäinen
On Sun, Sep 01, 2013 at 08:13:02PM +0300, Pasi Kärkkäinen wrote: > On Fri, Aug 30, 2013 at 05:38:04PM -0400, Rich wrote: > >Apparently, only about 4 months. > >P17 firmware is out. > >Tested on a card which was demonstrating the incorrect behavior before, > > of > >model 9201-16i.

Re: [PATCH 3/9] scsi: improved eh timeout handler

2013-09-02 Thread Hannes Reinecke
On 09/02/2013 02:45 PM, Bart Van Assche wrote: > On 09/02/13 09:12, Hannes Reinecke wrote: >> @@ -353,6 +354,8 @@ void scsi_put_command(struct scsi_cmnd *cmd) >> list_del_init(&cmd->list); >> spin_unlock_irqrestore(&cmd->device->list_lock, flags); >> >> +cancel_delayed_work(&cmd->ab

Testing NPIV Feature with Qemu-KVM

2013-09-02 Thread cshastri
Hi All, I am testing NPIV feature on upstream Qemu, I have configured the zone and able to see the created vport on the storage array. Since, I am learning on how to setup the NPIV, I haven't created the different zone for the vport and the array, I just added in the existing zone. Now, how

Re: [PATCH 3/9] scsi: improved eh timeout handler

2013-09-02 Thread Bart Van Assche
On 09/02/13 09:12, Hannes Reinecke wrote: @@ -353,6 +354,8 @@ void scsi_put_command(struct scsi_cmnd *cmd) list_del_init(&cmd->list); spin_unlock_irqrestore(&cmd->device->list_lock, flags); + cancel_delayed_work(&cmd->abort_work); + __scsi_put_command(cmd->device->h

[PATCH 1/3] scsi: Fix erratic device offline during EH

2013-09-02 Thread Hannes Reinecke
Commit 18a4d0a22ed6c54b67af7718c305cd010f09ddf8 (Handle disk devices which can not process medium access commands) was introduced to offline any device which cannot process medium access commands. However, commit 3eef6257de48ff84a5d98ca533685df8a3beaeb8 (Reduce error recovery time by reducing use o

[PATCH 3/3] scsi_error: Update documentation

2013-09-02 Thread Hannes Reinecke
Update the documentation to cover asynchronous command aborts and clarify sections referring to SCSI timeout handling. Signed-off-by: Hannes Reinecke --- Documentation/scsi/scsi_eh.txt | 69 +++-- Documentation/scsi/scsi_mid_low_api.txt | 9 - drivers/sc

[PATCH 2/3] scsi: improved eh timeout handler

2013-09-02 Thread Hannes Reinecke
When a command runs into a timeout we need to send an 'ABORT TASK' TMF. This is typically done by the 'eh_abort_handler' LLDD callback. Conceptually, however, this function is a normal SCSI command, so there is no need to enter the error handler. This patch implements a new scsi_abort_command() f

[PATCHv6 0/3] New EH command timeout handler

2013-09-02 Thread Hannes Reinecke
Hi all, this patchset implements a new SCSI EH command timeout handler which will be sending command aborts inline without actually engaging SCSI EH. SCSI EH will only be invoked if command abort fails. In addition the commands will be returned directly if the command abort succeeded, cutting dow

[PATCH 3/5] uas: rename work list lock + list field

2013-09-02 Thread Gerd Hoffmann
This patch prepares for the addition of another list and renames the work list lock and the list_head field in struct uas_cmd_info. Signed-off-by: Gerd Hoffmann --- drivers/usb/storage/uas.c | 50 +++ 1 file changed, 25 insertions(+), 25 deletions(-)

[PATCH 2/5] uas: properly reinitialize in uas_eh_bus_reset_handler

2013-09-02 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann --- drivers/usb/storage/uas.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/usb/storage/uas.c b/drivers/usb/storage/uas.c index d966b59..f89202f 100644 --- a/drivers/usb/storage/uas.c +++ b/drivers/usb/storage/uas.c @@ -85,6 +85,8 @@ static int uas

[PATCH 4/5] uas: add dead request list

2013-09-02 Thread Gerd Hoffmann
This patch adds a new list where all requests which are canceled are added to, so we don't loose them. Then, after killing all inflight urbs on bus reset (and disconnect) we'll walk over the list and clean them up. Without this we can end up with aborted requests lingering around in case of statu

Re: [PATCHv5 0/9] New EH command timeout handler

2013-09-02 Thread Hannes Reinecke
On 09/02/2013 11:31 AM, Christoph Hellwig wrote: > On Mon, Sep 02, 2013 at 10:54:13AM +0200, Hannes Reinecke wrote: >> I don't mind. Having talked to the various SCSI folks everyone >> agreed that calling abort asynchronously shouldn't do any harm. >> At least as far as the SCSI spec goes. >> >> As

Re: [PATCHv5 0/9] New EH command timeout handler

2013-09-02 Thread Christoph Hellwig
On Mon, Sep 02, 2013 at 10:54:13AM +0200, Hannes Reinecke wrote: > I don't mind. Having talked to the various SCSI folks everyone > agreed that calling abort asynchronously shouldn't do any harm. > At least as far as the SCSI spec goes. > > As for documentation: I didn't document it it currently a

re: [SCSI] qla2xxx: Add support for ISP8044.

2013-09-02 Thread Dan Carpenter
Hello Atul Deshmukh, This is a semi-automatic email about new static checker warnings. The patch ef4647420025: "[SCSI] qla2xxx: Add support for ISP8044." from Aug 27, 2013, leads to the following Smatch complaint: drivers/scsi/qla2xxx/qla_os.c:2962 qla2x00_probe_one() error: we previou

Re: [PATCHv5 0/9] New EH command timeout handler

2013-09-02 Thread Hannes Reinecke
On 09/02/2013 10:27 AM, Christoph Hellwig wrote: > One thing I'm still wondering is why we can't enable this globally, > and if there is a reason it should be documented. > > As far as I can tell the actual calling context of the eh_abort_handler > doesn't change, so an LLDD would have to rely on

Re: [PATCHv5 0/9] New EH command timeout handler

2013-09-02 Thread Christoph Hellwig
One thing I'm still wondering is why we can't enable this globally, and if there is a reason it should be documented. As far as I can tell the actual calling context of the eh_abort_handler doesn't change, so an LLDD would have to rely on some very specific side effects of being in EH to break, an

[PATCH 5/9] libsas: Enable new EH timeout handler

2013-09-02 Thread Hannes Reinecke
Signed-off-by: Hannes Reinecke --- drivers/scsi/libsas/sas_scsi_host.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/libsas/sas_scsi_host.c b/drivers/scsi/libsas/sas_scsi_host.c index da3aee1..6eb2f0c 100644 --- a/drivers/scsi/libsas/sas_scsi_host.c +++ b/drive

[PATCH 9/9] scsi_transport_fc: Enable new EH timeout handler

2013-09-02 Thread Hannes Reinecke
Signed-off-by: Hannes Reinecke --- drivers/scsi/scsi_transport_fc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/scsi_transport_fc.c b/drivers/scsi/scsi_transport_fc.c index 4628fd5..012c801 100644 --- a/drivers/scsi/scsi_transport_fc.c +++ b/drivers/scsi/scsi_

[PATCHv5 0/9] New EH command timeout handler

2013-09-02 Thread Hannes Reinecke
Hi all, this patchset implements a new SCSI EH command timeout handler which will be sending command aborts inline without actually engaging SCSI EH. SCSI EH will only be invoked if command abort fails. In addition the commands will be returned directly if the command abort succeeded, cutting dow

[PATCH 2/9] blk-timeout: add BLK_EH_SCHEDULED return code

2013-09-02 Thread Hannes Reinecke
Add a 'BLK_EH_SCHEDULED' return code for blk-timeout to indicate that a delayed error recovery has been initiated. Signed-off-by: Hannes Reinecke --- drivers/scsi/scsi_error.c | 4 include/linux/blkdev.h| 1 + 2 files changed, 5 insertions(+) diff --git a/drivers/scsi/scsi_error.c b/dr

[PATCH 4/9] virtio_scsi: Enable new EH timeout handler

2013-09-02 Thread Hannes Reinecke
Signed-off-by: Hannes Reinecke --- drivers/scsi/virtio_scsi.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/scsi/virtio_scsi.c b/drivers/scsi/virtio_scsi.c index 74b88ef..5fca1ca 100644 --- a/drivers/scsi/virtio_scsi.c +++ b/drivers/scsi/virtio_scsi.c @@ -683,6 +683,7 @@ static st

[PATCH 1/9] scsi: Fix erratic device offline during EH

2013-09-02 Thread Hannes Reinecke
Commit 18a4d0a22ed6c54b67af7718c305cd010f09ddf8 (Handle disk devices which can not process medium access commands) was introduced to offline any device which cannot process medium access commands. However, commit 3eef6257de48ff84a5d98ca533685df8a3beaeb8 (Reduce error recovery time by reducing use o

[PATCH 7/9] mpt2sas: Enable new EH timeout handler

2013-09-02 Thread Hannes Reinecke
Signed-off-by: Hannes Reinecke --- drivers/scsi/mpt2sas/mpt2sas_scsih.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/scsi/mpt2sas/mpt2sas_scsih.c b/drivers/scsi/mpt2sas/mpt2sas_scsih.c index 389d792..9ede2c2 100644 --- a/drivers/scsi/mpt2sas/mpt2sas_scsih.c +++ b

[PATCH 8/9] mpt3sas: Enable new EH timeout handler

2013-09-02 Thread Hannes Reinecke
Signed-off-by: Hannes Reinecke --- drivers/scsi/mpt3sas/mpt3sas_scsih.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/scsi/mpt3sas/mpt3sas_scsih.c b/drivers/scsi/mpt3sas/mpt3sas_scsih.c index a961fe1..b893383 100644 --- a/drivers/scsi/mpt3sas/mpt3sas_scsih.c +++ b/drivers/scsi/mpt3

[PATCH 3/9] scsi: improved eh timeout handler

2013-09-02 Thread Hannes Reinecke
When a command runs into a timeout we need to send an 'ABORT TASK' TMF. This is typically done by the 'eh_abort_handler' LLDD callback. Conceptually, however, this function is a normal SCSI command, so there is no need to enter the error handler. This patch implements a new scsi_abort_command() f

[PATCH 6/9] mptsas: Enable new EH timeout handler

2013-09-02 Thread Hannes Reinecke
Signed-off-by: Hannes Reinecke --- drivers/message/fusion/mptsas.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/message/fusion/mptsas.c b/drivers/message/fusion/mptsas.c index dd239bd..d0aed77 100644 --- a/drivers/message/fusion/mptsas.c +++ b/drivers/message/fusi