[GIT PULL] SCSI fixes for 3.6-rc6

2012-09-23 Thread James Bottomley
This is a set of four essential fixes: two oops related (bnx2i, virtio-scsi), one data corruption related (hpsa) and one failure to boot due to interrupt routing issues (mpt2ss). The patch is available here: git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git scsi-fixes The short changel

Re: [PATCHv2] st: remove st_mutex

2012-09-23 Thread Kai Makisara
On Mon, 10 Sep 2012, Lee Duncan wrote: > From: Hannes Reinecke > > The st_mutex was created when the BKL was removed, and > prevents simultaneous st_open calls. It is better to > protect just the necessary data. > > Signed-off-by: Hannes Reinecke > Reviewed-by: Lee Duncan Acked-by: Kai Mäkis

[PATCH 0/2 v2] Make blk_cleanup_queue() finish after request_fn

2012-09-23 Thread Bart Van Assche
Request processing functions like scsi_request_fn() that unlock the queue lock internally may cause blk_cleanup_queue() to finish while the request_fn is in progress. This two-patch series makes sure that blk_cleanup_queue() can't finish before all active request_fn calls have finished. Changes co

[PATCH 1/2] block: Rename queue dead flag

2012-09-23 Thread Bart Van Assche
This patch is the result of running the command below and manually fixing up indentation: git grep -lE 'blk_queue_dead|QUEUE_FLAG_DEAD' | xargs sed -i .tmp -e 's/blk_queue_dead/blk_queue_dying/g' \ -e 's/QUEUE_FLAG_DEAD/QUEUE_FLAG_DYING/g' Cc: James Bottomley Cc: Mike Christie Cc: Jen

[PATCH 2/2] Make blk_cleanup_queue() wait until request_fn finished

2012-09-23 Thread Bart Van Assche
Some request_fn implementations, e.g. scsi_request_fn(), unlock the queue lock. Make sure that blk_cleanup_queue() waits until all active request_fn invocations have finished. This fixes a potential use-after-free at the end of scsi_request_fn(). Also, make sure that the block layer doesn't invoke

[Bug 47781] ata2: exception Emask 0x10 SAct 0x0 SErr 0x4050002 action 0xe frozen

2012-09-23 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=47781 Alan changed: What|Removed |Added Component|IDE |SCSI AssignedTo|io_...@kernel-bugs.osdl.o

[Bug 47781] ata2: exception Emask 0x10 SAct 0x0 SErr 0x4050002 action 0xe frozen

2012-09-23 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=47781 Reartes Guillermo changed: What|Removed |Added CC||rtgui...@gmail.com --- Comment #5

Re: [PATCH v7 2/6] scsi: sr: support runtime pm

2012-09-23 Thread Aaron Lu
On Fri, Sep 21, 2012 at 10:49:32PM +0200, Rafael J. Wysocki wrote: > On Friday, September 21, 2012, Aaron Lu wrote: > > On Thu, Sep 20, 2012 at 10:48:10PM +0200, Rafael J. Wysocki wrote: > > > On Wednesday, September 12, 2012, Aaron Lu wrote: > > > > Place the ODD into runtime suspend state as soon

Re: [PATCH v7 0/6] ZPODD patches

2012-09-23 Thread Aaron Lu
On Fri, Sep 21, 2012 at 11:18:27PM +0200, Rafael J. Wysocki wrote: > On Friday, September 21, 2012, Aaron Lu wrote: > > On Thu, Sep 20, 2012 at 10:00:51PM +0200, Rafael J. Wysocki wrote: > > > On Wednesday, September 19, 2012, Aaron Lu wrote: > > > > Thanks Rafael, and if there is any question/prob

[PATCH] ahci: add a boot option to enable promise ahci

2012-09-23 Thread Tony Hung - PTT 洪瑞嶸
linux-3.2.0/drivers/ata/ahci.c: In function 'hptiop_host_request_callback': Use min_t(size_t, ...) to fix. Signed-off-by: Tony Hung --- diff --git a/linux-3.2.0/drivers/ata/ahci.c b/linux-3.2.0/drivers/ata/ahci.c --- a/linux-3.2.0/drivers/ata/ahci.c2012-08-29 13:40:26.046851693 +0800 +++ b/

[PATCH scsi] Add NULL checking of return value from scsi_cmd_to_driver()

2012-09-23 Thread Li Zhong
Just noticed that after commit 919f797, it is possible that scsi_cmd_to_driver() returns NULL. This patch adds the NULL checking for drv returned from the above function. Maybe it is not possible at run time, but from the code itself, we'd better have this check? This patch actually is a del

Re: [Bug 47781] ata2: exception Emask 0x10 SAct 0x0 SErr 0x4050002 action 0xe frozen

2012-09-23 Thread James Bottomley
On Sun, 2012-09-23 at 21:07 +, bugzilla-dae...@bugzilla.kernel.org wrote: > https://bugzilla.kernel.org/show_bug.cgi?id=47781 > > > Alan changed: > >What|Removed |Added > >

[Bug 47781] ata2: exception Emask 0x10 SAct 0x0 SErr 0x4050002 action 0xe frozen

2012-09-23 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=47781 --- Comment #6 from Anonymous Emailer 2012-09-24 05:38:59 --- Reply-To: james.bottom...@hansenpartnership.com On Sun, 2012-09-23 at 21:07 +, bugzilla-dae...@bugzilla.kernel.org wrote: > https://bugzilla.kernel.org/show_bug.cgi?id=47781

Re: [PATCH scsi] Add NULL checking of return value from scsi_cmd_to_driver()

2012-09-23 Thread James Bottomley
On Mon, 2012-09-24 at 13:30 +0800, Li Zhong wrote: > Just noticed that after commit 919f797, it is possible that > scsi_cmd_to_driver() returns NULL. This patch adds the NULL checking for drv > returned from the above function. > > Maybe it is not possible at run time, but from the code itself,