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

2012-09-25 Thread Aaron Lu
On Mon, Sep 24, 2012 at 11:40:18PM +0200, Rafael J. Wysocki wrote: > On Monday, September 24, 2012, Aaron Lu wrote: > > On Mon, Sep 24, 2012 at 02:55:31PM +0200, Rafael J. Wysocki wrote: > > > And I'd add a comment about the next poll. > > > > > > This appears somewhat racy, though, because in the

Re: [PATCH v7 0/6] ZPODD patches

2012-09-25 Thread Aaron Lu
On Mon, Sep 24, 2012 at 11:46:03PM +0200, Rafael J. Wysocki wrote: > On Monday, September 24, 2012, Aaron Lu wrote: > > On Mon, Sep 24, 2012 at 03:06:11PM +0200, Rafael J. Wysocki wrote: > > > On Monday, September 24, 2012, Aaron Lu wrote: > > > > need_eject: > > > > First consider how the device w

Re: [SCSI PATCH] sd: max-retries becomes configurable

2012-09-25 Thread James Bottomley
On Tue, 2012-09-25 at 01:21 -0400, Jeff Garzik wrote: > On 09/25/2012 12:06 AM, James Bottomley wrote: > > On Mon, 2012-09-24 at 17:00 -0400, Jeff Garzik wrote: > >> > >> drivers/scsi/sd.c |4 > >> drivers/scsi/sd.h |2 +- > >> 2 files changed, 5 insertions(+), 1 deletion(-) > > >

Re: [PATCH v7 0/6] ZPODD patches

2012-09-25 Thread James Bottomley
On Tue, 2012-09-25 at 16:18 +0800, Aaron Lu wrote: > A example patch would be something like the following, I didn't seperate > these ACPI calls in sr.c as this is just a concept proof, if this is the > right thing to do, I will separate them into another file sr-acpi.c and > make empty stubs for t

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

2012-09-25 Thread Rafael J. Wysocki
On Tuesday, September 25, 2012, Aaron Lu wrote: > On Mon, Sep 24, 2012 at 11:40:18PM +0200, Rafael J. Wysocki wrote: > > On Monday, September 24, 2012, Aaron Lu wrote: > > > On Mon, Sep 24, 2012 at 02:55:31PM +0200, Rafael J. Wysocki wrote: > > > > And I'd add a comment about the next poll. > > > >

[PATCH] aic94xx: fix handling of default CTRL-A settings

2012-09-25 Thread Paul Bolle
Compiling aic94xx_sds.o (part of the aic94xx driver) triggers this GCC warning: drivers/scsi/aic94xx/aic94xx_sds.c: In function 'asd_read_flash': drivers/scsi/aic94xx/aic94xx_sds.c:597:21: warning: 'offs' may be used uninitialized in this function [-Wmaybe-uninitialized] drivers/scsi/a

Re: [PATCH v7 0/6] ZPODD patches

2012-09-25 Thread Aaron Lu
On Tue, Sep 25, 2012 at 03:02:29PM +0400, James Bottomley wrote: > On Tue, 2012-09-25 at 16:18 +0800, Aaron Lu wrote: > > A example patch would be something like the following, I didn't seperate > > these ACPI calls in sr.c as this is just a concept proof, if this is the > > right thing to do, I wi

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

2012-09-25 Thread Aaron Lu
On Tue, Sep 25, 2012 at 01:47:52PM +0200, Rafael J. Wysocki wrote: > On Tuesday, September 25, 2012, Aaron Lu wrote: > > I'm thinking of enabling this GPE in sr_suspend once we decided that it > > is ready to be powered off, so the time frame between sr_suspend and > > when the power is actually re

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

2012-09-25 Thread Oliver Neukum
On Tuesday 25 September 2012 22:20:21 Aaron Lu wrote: > On Tue, Sep 25, 2012 at 01:47:52PM +0200, Rafael J. Wysocki wrote: > > On Tuesday, September 25, 2012, Aaron Lu wrote: > > > I'm thinking of enabling this GPE in sr_suspend once we decided that it > > > is ready to be powered off, so the time

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

2012-09-25 Thread Aaron Lu
On 09/25/2012 10:23 PM, Oliver Neukum wrote: > On Tuesday 25 September 2012 22:20:21 Aaron Lu wrote: >> On Tue, Sep 25, 2012 at 01:47:52PM +0200, Rafael J. Wysocki wrote: >>> On Tuesday, September 25, 2012, Aaron Lu wrote: I'm thinking of enabling this GPE in sr_suspend once we decided that it

Re: [PATCH 00/20, v4] Make ib_srp better suited for H.A. purposes

2012-09-25 Thread Bart Van Assche
On 08/09/12 17:41, Bart Van Assche wrote: [ ... ] Hello Dave, More than six weeks have elapsed since I posted version four of this patch series. It would be appreciated if you could tell me when review comments for this patch series will be posted. I'd also like to remind you that some time

[PATCH v2 0/3] block: add queue-private command filter, editable via sysfs

2012-09-25 Thread Paolo Bonzini
The set of use cases for SG_IO is quite variable that no single filter can accomodate all of them. The current filter is tailored very much to CD burning, and includes many MMC-specific commands that may have other meanings in different standards. Someone may want to remove those commands; at the

[PATCH v2 3/3] block: add back command filter modification via sysfs

2012-09-25 Thread Paolo Bonzini
This adds two new sysfs attributes to the queue kobject. The attributes allow reading and writing the whitelist of unprivileged commands. This is again a bit different from what was removed in commit 018e044 (block: get rid of queue-private command filter, 2009-06-26), but the idea is the same.

[PATCH v2 2/3] scsi: create an all-zero filter for scanners

2012-09-25 Thread Paolo Bonzini
Using /dev/sg for scanners is blocked from unprivileged users. Reimplement this using customizable command filters, so that the sysfs knobs will work in this case too. Cc: linux-scsi@vger.kernel.org Signed-off-by: Paolo Bonzini --- v1->v2: OOM check [Alan Cox] use GFP_ATO

[PATCH v2 1/3] block: add back queue-private command filter

2012-09-25 Thread Paolo Bonzini
The command filter used to be mutable via sysfs, but this was broken and backed out. Let's add it back. This patch adds the infrastructure for filtering, but unlike the old code this one just adds a pointer to request_queue, so as to make it cheaper in the majority of cases where no special filte

RE: [PATCH 2/3] libfc: fix lun reset failure bugs in fc_fcp_resp handling of FCP_RSP_INFO

2012-09-25 Thread Zou, Yi
> > On 09/24/12 20:52, Robert Love wrote: > > In LUN RESET testing involving NetApp targets, it is observed that LUN > > RESET is failing. The fc_fcp_resp() is not completing the completiong > > It looks like there is a typo in the patch description (in the last word > of the second line) ? > >

Re: usb3 fails to write when using usb3 hub in usb3 port

2012-09-25 Thread Sarah Sharp
On Tue, Sep 25, 2012 at 09:26:00AM +0300, Adrian Sandu wrote: > On Tue, Sep 25, 2012 at 12:38 AM, Sarah Sharp > wrote: > > Ok, so 3.4.11 doesn't work, and the log file was from 3.5. > > If you want I can provide a 3.4 log... Hmm, does a 3.3 stable kernel work for you? I have a hypothesis. Alan

Re: usb3 fails to write when using usb3 hub in usb3 port

2012-09-25 Thread Andiry Xu
On Tue, Sep 25, 2012 at 2:19 PM, Sarah Sharp wrote: > On Tue, Sep 25, 2012 at 09:26:00AM +0300, Adrian Sandu wrote: >> On Tue, Sep 25, 2012 at 12:38 AM, Sarah Sharp >> wrote: >> > Ok, so 3.4.11 doesn't work, and the log file was from 3.5. >> >> If you want I can provide a 3.4 log... > > Hmm, does

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

2012-09-25 Thread Rafael J. Wysocki
On Tuesday, September 25, 2012, Aaron Lu wrote: > On 09/25/2012 10:23 PM, Oliver Neukum wrote: > > On Tuesday 25 September 2012 22:20:21 Aaron Lu wrote: > >> On Tue, Sep 25, 2012 at 01:47:52PM +0200, Rafael J. Wysocki wrote: > >>> On Tuesday, September 25, 2012, Aaron Lu wrote: > I'm thinking

[PATCH] MAINTAINERS: update Intel C600 SAS driver maintainers

2012-09-25 Thread Dave Jiang
Cc: Lukasz Dorau Cc: Maciej Patelczyk Signed-off-by: Dave Jiang --- MAINTAINERS |7 --- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index b17587d..162f602 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -3552,11 +3552,12 @@ K: \b(ABS|SYN)

Re: next-20120925: BUG at drivers/scsi/scsi_lib.c:640!

2012-09-25 Thread Andrew Morton
i_intel > microcode sg xhci_hcd button ext3 jbd mbcache sd_mod crc_t10dif\ > elper cryptd lrw aes_x86_64 xts gf128mul ahci libahci pata_acpi ata_generic > dm_mirror dm_region_hash dm_log dm_mod > CPU 0 > Pid: 753, comm: fsck.ext3 Not tainted 3.6.0-rc7-next-20120925+ #4

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

2012-09-25 Thread Aaron Lu
On Tue, Sep 25, 2012 at 11:45:34PM +0200, Rafael J. Wysocki wrote: > On Tuesday, September 25, 2012, Aaron Lu wrote: > > On 09/25/2012 10:23 PM, Oliver Neukum wrote: > > > On Tuesday 25 September 2012 22:20:21 Aaron Lu wrote: > > >> On Tue, Sep 25, 2012 at 01:47:52PM +0200, Rafael J. Wysocki wrote: