[PATCH] dpt_i2o depends on virt_to_bus

2007-07-19 Thread Stephen Rothwell
Signed-off-by: Stephen Rothwell <[EMAIL PROTECTED]> --- drivers/scsi/Kconfig |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) -- Cheers, Stephen Rothwell[EMAIL PROTECTED] diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig index 3727231..a947257 100644 --- a

Re: [PATCH 8/9] qla2xxx: Add ISP25XX support.

2007-07-19 Thread Andrew Vasquez
On Thu, 19 Jul 2007, James Bottomley wrote: > On Thu, 2007-07-19 at 15:06 -0700, Andrew Vasquez wrote: > > + dmp_reg = (uint32_t __iomem *)((uint8_t __iomem *)reg + 0xF0); > > + WRT_REG_DWORD(dmp_reg, 0xB010); > > + dmp_reg = (uint32_t __iomem *)((uint8_t __iomem

Re: [PATCH 8/9] qla2xxx: Add ISP25XX support.

2007-07-19 Thread James Bottomley
On Thu, 2007-07-19 at 15:06 -0700, Andrew Vasquez wrote: > + dmp_reg = (uint32_t __iomem *)((uint8_t __iomem *)reg + 0xF0); > + WRT_REG_DWORD(dmp_reg, 0xB010); > + dmp_reg = (uint32_t __iomem *)((uint8_t __iomem *)reg + 0xFC); > + fw->shadow_reg[1

Re: [PATCH] update mpt Kconfig help

2007-07-19 Thread James Bottomley
On Thu, 2007-07-19 at 12:26 -0700, Gwendal Grignou wrote: > Update help in Kconfig for mptfc driver to indicate the driver supports > Brocade FC 4G HBA. This patch won't apply because of whitespace problems ... it looks like your mailer has converted all the tabs to spaces. James - To unsubscr

[PATCH 8/9] qla2xxx: Add ISP25XX support.

2007-07-19 Thread Andrew Vasquez
Large code-reuse from ISP24xx, consolidate RISC memory extraction routines during firmware-dump. Signed-off-by: Andrew Vasquez <[EMAIL PROTECTED]> --- drivers/scsi/qla2xxx/qla_attr.c |2 +- drivers/scsi/qla2xxx/qla_dbg.c | 919 +++ drivers/scsi/qla2xxx/ql

[PATCH 9/9] qla2xxx: Update version number to 8.02.00-k2.

2007-07-19 Thread Andrew Vasquez
Signed-off-by: Andrew Vasquez <[EMAIL PROTECTED]> --- drivers/scsi/qla2xxx/qla_version.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/qla2xxx/qla_version.h b/drivers/scsi/qla2xxx/qla_version.h index fd2f10a..dd1f8ce 100644 --- a/drivers/scsi/qla2xxx/qla_v

[PATCH 5/9] qla2xxx: Re-factor isp_operations to static structures.

2007-07-19 Thread Andrew Vasquez
In preparation for new ISP types. Signed-off-by: Andrew Vasquez <[EMAIL PROTECTED]> --- drivers/scsi/qla2xxx/qla_attr.c | 23 ++-- drivers/scsi/qla2xxx/qla_def.h|2 +- drivers/scsi/qla2xxx/qla_gs.c | 33 +++-- drivers/scsi/qla2xxx/qla_init.c | 54 drivers/scsi/qla2x

[PATCH 6/9] qla2xxx: Use PCI-X/PCI-Express read control interfaces.

2007-07-19 Thread Andrew Vasquez
Original from Peter Oruba <[EMAIL PROTECTED]>. Additional cleanups included. Signed-off-by: Andrew Vasquez <[EMAIL PROTECTED]> --- drivers/scsi/qla2xxx/qla_init.c | 25 - 1 files changed, 4 insertions(+), 21 deletions(-) diff --git a/drivers/scsi/qla2xxx/qla_init.c b/d

[PATCH 7/9] qla2xxx: Use pci_try_set_mwi().

2007-07-19 Thread Andrew Vasquez
As the "must-check" return-value of pci_set_msi() is never really checked. Signed-off-by: Andrew Vasquez <[EMAIL PROTECTED]> --- drivers/scsi/qla2xxx/qla_init.c |9 +++-- 1 files changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/scsi/qla2xxx/qla_init.c b/drivers/scsi/qla2xxx/

[PATCH 3/9] qla2xxx: Correct setting of 'current' and 'supported' speeds during FDMI registration.

2007-07-19 Thread Andrew Vasquez
Signed-off-by: Andrew Vasquez <[EMAIL PROTECTED]> --- drivers/scsi/qla2xxx/qla_def.h |8 drivers/scsi/qla2xxx/qla_gs.c | 29 - 2 files changed, 28 insertions(+), 9 deletions(-) diff --git a/drivers/scsi/qla2xxx/qla_def.h b/drivers/scsi/qla2xxx/qla_def.h

[PATCH 4/9] qla2xxx: Validate mid-layer 'underflow' during check-condition handling.

2007-07-19 Thread Andrew Vasquez
From: Shyam Sundar <[EMAIL PROTECTED]> Signed-off-by: Andrew Vasquez <[EMAIL PROTECTED]> --- drivers/scsi/qla2xxx/qla_isr.c | 19 +++ 1 files changed, 19 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/qla2xxx/qla_isr.c b/drivers/scsi/qla2xxx/qla_isr.c index fa21cd8..4a

[PATCH 2/9] qla2xxx: Generalize iIDMA support.

2007-07-19 Thread Andrew Vasquez
In preparation for new ISP types. Signed-off-by: Andrew Vasquez <[EMAIL PROTECTED]> --- drivers/scsi/qla2xxx/qla_def.h |2 ++ drivers/scsi/qla2xxx/qla_gs.c |4 ++-- drivers/scsi/qla2xxx/qla_init.c |2 +- drivers/scsi/qla2xxx/qla_mbx.c |4 ++-- drivers/scsi/qla2xxx/qla_os.c |

[PATCH 1/9] qla2xxx: Generalize FW-Interface-2 support.

2007-07-19 Thread Andrew Vasquez
In preparation for new ISP types. Signed-off-by: Andrew Vasquez <[EMAIL PROTECTED]> --- drivers/scsi/qla2xxx/qla_attr.c |8 +++--- drivers/scsi/qla2xxx/qla_def.h|2 + drivers/scsi/qla2xxx/qla_gs.c | 10 +++--- drivers/scsi/qla2xxx/qla_init.c |8 +++--- drivers/scsi/qla2x

[PATCH 0/9] qla2xxx: updates for 2.6.23 [8.02.00-k2].

2007-07-19 Thread Andrew Vasquez
This patchset updates the qla2xxx driver to 8.02.00-k2. drivers/scsi/qla2xxx/qla_attr.c| 33 +- drivers/scsi/qla2xxx/qla_dbg.c | 919 drivers/scsi/qla2xxx/qla_dbg.h | 38 ++ drivers/scsi/qla2xxx/qla_def.h | 22 +- drivers/scsi/qla2xxx/ql

RE: [PATCH] update mpt Kconfig help

2007-07-19 Thread Moore, Eric
On Thursday, July 19, 2007 1:27 PM, Gwendal Grignou wrote: > Update help in Kconfig for mptfc driver to indicate the > driver supports > Brocade FC 4G HBA. > > signed-off-by: Gwendal Grignou <[EMAIL PROTECTED]> ACK Eric Moore - To unsubscribe from this list: send the line "unsubscribe linux-s

[PATCH] update mpt Kconfig help

2007-07-19 Thread Gwendal Grignou
Update help in Kconfig for mptfc driver to indicate the driver supports Brocade FC 4G HBA. signed-off-by: Gwendal Grignou <[EMAIL PROTECTED]> --- diff --git a/drivers/message/fusion/Kconfig b/drivers/message/fusion/Kconfig index c88cc75..4494e0f 100644 --- a/drivers/message/fusion/Kconfig +++ b/

Re: [RFC] Fix device not ready printk

2007-07-19 Thread Martin K. Petersen
> "Matthew" == Matthew Wilcox <[EMAIL PROTECTED]> writes: Matthew> After three weeks with no discussion, I guess it may as well Matthew> be applied. Matthew> scsi: Better error messages when device not ready Looks good to me. Acked-by: Martin K. Petersen <[EMAIL PROTECTED]> -- Martin K. P

Re: [RFC] Fix device not ready printk

2007-07-19 Thread Matthew Wilcox
On Fri, Jun 29, 2007 at 07:21:26AM -0600, Matthew Wilcox wrote: > I'm in two minds about printing the 'Device not ready' twice. On the > one hand, it's redundant information. On the other hand, it helps the > harried sysadmin with multiple simultaneous disc failures tie the two > messages togethe

Re: [patch 2/3] ps3: BD/DVD/CD-ROM Storage Driver

2007-07-19 Thread Rene Herman
On 07/19/2007 11:47 AM, Andrew Morton wrote: On Thu, 19 Jul 2007 11:39:32 +0200 (CEST) Geert Uytterhoeven <[EMAIL PROTECTED]> wrote: Oh dear. ps3rom_priv(dev) = host; that's 'orrid. We have an identifier pretending to be a function, only we go and treat it as an lvalue. I mean, C

Re: [patch 0/4] aha152x.c - Cleanup, need help in testing and auditing

2007-07-19 Thread Randy Dunlap
On Thu, 19 Jul 2007 18:57:50 +0300 Boaz Harrosh wrote: > Randy Dunlap wrote: > > > > Yes, this problem has been around forever AFAIK. You didn't add > > to it. > Do we need to file a bug report in Bugzilla or something, so people have a > documentation of the work-around, until it is fixed? Wo

Re: [PATCH] bsg: separate bsg and SCSI (so SCSI can be modular)

2007-07-19 Thread Jens Axboe
On Thu, Jul 19 2007, James Bottomley wrote: > This patch moves the bsg registration into SCSI so that bsg no longer > has a dependency on the scsi_interface_register API. > > This can be viewed as a temporary expedient until we can get universal > bsg binding sorted out properly. Acked-by: Jens A

Re: [patch 0/4] aha152x.c - Cleanup, need help in testing and auditing

2007-07-19 Thread Boaz Harrosh
Randy Dunlap wrote: > > Yes, this problem has been around forever AFAIK. You didn't add > to it. Do we need to file a bug report in Bugzilla or something, so people have a documentation of the work-around, until it is fixed? I think that for now I will wrap it up as it is. Could you send me the

Re: Someone ( bsg merge ? ) broke {sd,hd}parm on current git

2007-07-19 Thread Gabriel C
James Bottomley wrote: > On Thu, 2007-07-19 at 16:31 +0200, Gabriel C wrote: > >> No is not an SATA controller. >> >> sda and sdb are SCSI disks connected to an Adaptec AIC-7892P U160/m >> controller using the aic7xxx driver. >> > > OK, this definitely works for me, that's what my root disk

Re: [patch 0/4] aha152x.c - Cleanup, need help in testing and auditing

2007-07-19 Thread Randy Dunlap
On Thu, 19 Jul 2007 16:04:45 +0300 Boaz Harrosh wrote: > Randy Dunlap wrote: > > On Wed, 18 Jul 2007 19:43:35 -0700 Randy Dunlap wrote: > > > >> On Wed, 18 Jul 2007 14:58:18 -0700 Randy Dunlap wrote: > >> > >>> On Wed, 18 Jul 2007 10:33:20 -0700 Randy Dunlap wrote: > >>> > Hi, > >

Re: Someone ( bsg merge ? ) broke {sd,hd}parm on current git

2007-07-19 Thread James Bottomley
On Thu, 2007-07-19 at 16:31 +0200, Gabriel C wrote: > No is not an SATA controller. > > sda and sdb are SCSI disks connected to an Adaptec AIC-7892P U160/m > controller using the aic7xxx driver. OK, this definitely works for me, that's what my root disk is on. I think this isn't a kernel problem

[PATCH] bsg: separate bsg and SCSI (so SCSI can be modular)

2007-07-19 Thread James Bottomley
This patch moves the bsg registration into SCSI so that bsg no longer has a dependency on the scsi_interface_register API. This can be viewed as a temporary expedient until we can get universal bsg binding sorted out properly. James --- Index: BUILD-2.6/block/Kconfig

[PATCH scsi-misc resend] Make scsi_host_template::proc_name const char * instead of char *.

2007-07-19 Thread Stefan Richter
From:Kristian Høgsberg <[EMAIL PROTECTED]> Date: Wed, 9 May 2007 19:23:12 -0400 Signed-off-by: Kristian Høgsberg <[EMAIL PROTECTED]> collapsed with fw-sbp2 patch "Drop cast to non-const char * in host template initialization." from Kristian Høgsberg Signed-off-by: Stefan Richter <[EMAIL PROTECT

Re: Someone ( bsg merge ? ) broke {sd,hd}parm on current git

2007-07-19 Thread Gabriel C
James Bottomley wrote: > On Thu, 2007-07-19 at 08:25 +0900, FUJITA Tomonori wrote: > >> From: Gabriel C <[EMAIL PROTECTED]> >> Subject: Re: Someone ( bsg merge ? ) broke {sd,hd}parm on current git >> Date: Tue, 17 Jul 2007 03:40:58 +0200 >> >> >>> FUJITA Tomonori wrote: >>> From

Re: Someone ( bsg merge ? ) broke {sd,hd}parm on current git

2007-07-19 Thread James Bottomley
On Thu, 2007-07-19 at 08:25 +0900, FUJITA Tomonori wrote: > From: Gabriel C <[EMAIL PROTECTED]> > Subject: Re: Someone ( bsg merge ? ) broke {sd,hd}parm on current git > Date: Tue, 17 Jul 2007 03:40:58 +0200 > > > FUJITA Tomonori wrote: > > > From: Gabriel C <[EMAIL PROTECTED]> > > > Subject: Re:

[patch 4/4] ps3flash: updates after final review

2007-07-19 Thread Geert Uytterhoeven
ps3flash: updates after final review: o Kill confusing ps3flash_priv() macro, open code it instead o ps3flash_llseek(): Add missing locking (using i_mutex) o Replace do_div_llr() by `/' and `%' o ps3flash_read(): Use a common return for the error path o Minor coding style fixes not report

[patch 3/4] ps3rom: updates after final review

2007-07-19 Thread Geert Uytterhoeven
ps3rom: updates after final review: o Kill confusing ps3rom_priv() macro, open code it instead o kmap_atomic() cannot fail Signed-off-by: Geert Uytterhoeven <[EMAIL PROTECTED]> --- drivers/scsi/ps3rom.c | 15 +-- 1 files changed, 5 insertions(+), 10 deletions(-) --- a/drivers/s

[patch 2/4] ps3disk: updates after final review

2007-07-19 Thread Geert Uytterhoeven
ps3disk: updates after final review: o Kill KERNEL_SECTOR_SIZE, just hardcode `512' or `>> 9' o Kill confusing ps3disk_priv() macro, open code it instead o ps3disk_scatter_gather(): Kill unused variable `sectors' o Introduce ps3disk_mask_mutex to protect ps3disk_mask o Minor coding style

[patch 1/4] ps3disk: use correct bio vector size

2007-07-19 Thread Geert Uytterhoeven
ps3disk: use correct bio vector size This fixes the O_DIRECT corruptions, as reported by Olaf Hering (triggered by e.g. parted >= 1.8.0). Signed-off-by: Geert Uytterhoeven <[EMAIL PROTECTED]> --- drivers/block/ps3disk.c |2 +- 1 files changed, 1 insertion(+), 1 deletion(-) --- a/drivers/blo

[patch 0/4] PS3 storage driver updates (was: Re: [patch 1/3] ps3: Disk Storage Driver)

2007-07-19 Thread Geert Uytterhoeven
Hi Andrew, On Thu, 19 Jul 2007, Andrew Morton wrote: > On Thu, 19 Jul 2007 10:57:53 +0200 (CEST) Geert Uytterhoeven <[EMAIL > PROTECTED]> wrote: > > On Wed, 18 Jul 2007, Andrew Morton wrote: > > > Your patchset significantly hits powerpc, scsi and block. So who gets to > > > merge this?

Re: [PATCH] aacraid: add vpd to inquiry

2007-07-19 Thread Hannes Reinecke
On Mon, Jul 16, 2007 at 05:08:05PM -0400, Salyzyn, Mark wrote: > Report VPD inquiry page 0x80 with an unique array creation serial number > (CUID). When an array is created, the metadata stored on the physical > drives gets an unique serial number. This serial number remains constant > through arra

Re: [patch 0/4] aha152x.c - Cleanup, need help in testing and auditing

2007-07-19 Thread Boaz Harrosh
Randy Dunlap wrote: > On Wed, 18 Jul 2007 19:43:35 -0700 Randy Dunlap wrote: > >> On Wed, 18 Jul 2007 14:58:18 -0700 Randy Dunlap wrote: >> >>> On Wed, 18 Jul 2007 10:33:20 -0700 Randy Dunlap wrote: >>> Hi, I don't see the reset problem after applying this patch, but I'm still

Re: [patch 1/3] ps3: Disk Storage Driver

2007-07-19 Thread Cornelia Huck
On Thu, 19 Jul 2007 14:13:33 +0200 (CEST), Geert Uytterhoeven <[EMAIL PROTECTED]> wrote: > Any chance this will change? I already added a mutex to ps3disk to protect > against this. Probably not in the near future. A mutex looks like a good idea though, since one never knows (and the driver core

Re: [patch 1/3] ps3: Disk Storage Driver

2007-07-19 Thread Geert Uytterhoeven
On Thu, 19 Jul 2007, Cornelia Huck wrote: > On Thu, 19 Jul 2007 11:36:31 +0200 (CEST), > Geert Uytterhoeven <[EMAIL PROTECTED]> wrote: > > > We have a probe thread that checks for new storage devices and adds them to > > the > > bus with ps3_system_bus_device_register(), which calls device_regist

Re: [patch 1/3] ps3: Disk Storage Driver

2007-07-19 Thread Cornelia Huck
On Thu, 19 Jul 2007 11:36:31 +0200 (CEST), Geert Uytterhoeven <[EMAIL PROTECTED]> wrote: > We have a probe thread that checks for new storage devices and adds them to > the > bus with ps3_system_bus_device_register(), which calls device_register(). > > I guess the actual bus probe() routine gets

Re: [patch 3/3] ps3: FLASH ROM Storage Driver

2007-07-19 Thread Geert Uytterhoeven
Hi Andrew, On Wed, 18 Jul 2007, Andrew Morton wrote: > > +struct ps3flash_private { > > + struct mutex mutex; /* Bounce buffer mutex */ > > +}; > > +#define ps3flash_priv(dev) ((dev)->sbd.core.driver_data) > > bzzzt! Same defense as ps3rom ;-) > > +static loff_t ps3flash_llseek(st

Re: [patch 2/3] ps3: BD/DVD/CD-ROM Storage Driver

2007-07-19 Thread Andrew Morton
On Thu, 19 Jul 2007 11:39:32 +0200 (CEST) Geert Uytterhoeven <[EMAIL PROTECTED]> wrote: > > Oh dear. > > > > ps3rom_priv(dev) = host; > > > > that's 'orrid. We have an identifier pretending to be a function, only we > > go and treat it as an lvalue. > > > > I mean, C code should look like

Re: [patch 2/3] ps3: BD/DVD/CD-ROM Storage Driver

2007-07-19 Thread Jens Axboe
On Thu, Jul 19 2007, Geert Uytterhoeven wrote: > On Thu, 19 Jul 2007, Andrew Morton wrote: > > On Thu, 19 Jul 2007 11:02:07 +0200 (CEST) Geert Uytterhoeven <[EMAIL > > PROTECTED]> wrote: > > > > > On Wed, 18 Jul 2007, Andrew Morton wrote: > > > > > +struct ps3rom_private { > > > > > + struct

Re: [patch 2/3] ps3: BD/DVD/CD-ROM Storage Driver

2007-07-19 Thread Geert Uytterhoeven
On Thu, 19 Jul 2007, Andrew Morton wrote: > On Thu, 19 Jul 2007 11:02:07 +0200 (CEST) Geert Uytterhoeven <[EMAIL > PROTECTED]> wrote: > > > On Wed, 18 Jul 2007, Andrew Morton wrote: > > > > +struct ps3rom_private { > > > > + struct ps3_storage_device *dev; > > > > + struct scsi_cmnd *

Re: [patch 1/3] ps3: Disk Storage Driver

2007-07-19 Thread Geert Uytterhoeven
On Thu, 19 Jul 2007, Cornelia Huck wrote: > On Thu, 19 Jul 2007 10:57:53 +0200 (CEST), > Geert Uytterhoeven <[EMAIL PROTECTED]> wrote: > > > Were .probe()/.remove() made concurrent again? I thought that idea was > > dropped > > because it caused too many problems? > > Well, for a given device, -

Re: [patch 1/3] ps3: Disk Storage Driver

2007-07-19 Thread Cornelia Huck
On Thu, 19 Jul 2007 10:57:53 +0200 (CEST), Geert Uytterhoeven <[EMAIL PROTECTED]> wrote: > Were .probe()/.remove() made concurrent again? I thought that idea was dropped > because it caused too many problems? Well, for a given device, ->probe()/->remove() are locked by dev->sem, so that there can

Re: [patch 2/3] ps3: BD/DVD/CD-ROM Storage Driver

2007-07-19 Thread Andrew Morton
On Thu, 19 Jul 2007 11:02:07 +0200 (CEST) Geert Uytterhoeven <[EMAIL PROTECTED]> wrote: > On Wed, 18 Jul 2007, Andrew Morton wrote: > > > +struct ps3rom_private { > > > + struct ps3_storage_device *dev; > > > + struct scsi_cmnd *curr_cmd; > > > +}; > > > +#define ps3rom_priv(dev) ((dev)->sbd.core

Re: [patch 1/3] ps3: Disk Storage Driver

2007-07-19 Thread Andrew Morton
On Thu, 19 Jul 2007 10:57:53 +0200 (CEST) Geert Uytterhoeven <[EMAIL PROTECTED]> wrote: > Hi Andrew, > > On Wed, 18 Jul 2007, Andrew Morton wrote: > > On Mon, 16 Jul 2007 18:15:40 +0200 > > Geert Uytterhoeven <[EMAIL PROTECTED]> wrote: > > > > > From: Geert Uytterhoeven <[EMAIL PROTECTED]

Re: [patch 2/3] ps3: BD/DVD/CD-ROM Storage Driver

2007-07-19 Thread Geert Uytterhoeven
On Wed, 18 Jul 2007, Andrew Morton wrote: > > +struct ps3rom_private { > > + struct ps3_storage_device *dev; > > + struct scsi_cmnd *curr_cmd; > > +}; > > +#define ps3rom_priv(dev) ((dev)->sbd.core.driver_data) > > + > > Someone should invent a keyboard which delivers an electric shock when

Re: [patch 1/3] ps3: Disk Storage Driver

2007-07-19 Thread Geert Uytterhoeven
Hi Andrew, On Wed, 18 Jul 2007, Andrew Morton wrote: > On Mon, 16 Jul 2007 18:15:40 +0200 > Geert Uytterhoeven <[EMAIL PROTECTED]> wrote: > > > From: Geert Uytterhoeven <[EMAIL PROTECTED]> > > > > Add a Disk Storage Driver for the PS3: > > Your patchset significantly hits powerpc, scsi

Re: [patch 1/3] ps3: Disk Storage Driver

2007-07-19 Thread Geert Uytterhoeven
On Thu, 19 Jul 2007, Jens Axboe wrote: > On Wed, Jul 18 2007, Andrew Morton wrote: > > On Mon, 16 Jul 2007 18:15:40 +0200 > > Geert Uytterhoeven <[EMAIL PROTECTED]> wrote: > > > > > From: Geert Uytterhoeven <[EMAIL PROTECTED]> > > > > > > Add a Disk Storage Driver for the PS3: > > > > Your patch