[PATCH] qla1280: convert to use the data buffer accessors

2007-10-17 Thread FUJITA Tomonori
lists and the parameters. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/scsi/qla1280.c | 376 1 files changed, 157 insertions(+), 219 deletions(-) diff --git a/drivers/scsi/qla1280.c b/drivers/scsi/qla1280.c index 76089cf..1

Re: [PATCH] SPARC64: fix iommu sg chaining

2007-10-17 Thread FUJITA Tomonori
On Wed, 17 Oct 2007 20:57:17 +0900 FUJITA Tomonori <[EMAIL PROTECTED]> wrote: > On Wed, 17 Oct 2007 13:41:17 +0200 > Jens Axboe <[EMAIL PROTECTED]> wrote: > > > On Wed, Oct 17 2007, FUJITA Tomonori wrote: > > > On Wed, 17 Oct 2007 13:01:42 +0200 > &

Re: [PATCH] SPARC64: fix iommu sg chaining

2007-10-17 Thread FUJITA Tomonori
On Wed, 17 Oct 2007 13:41:17 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > On Wed, Oct 17 2007, FUJITA Tomonori wrote: > > On Wed, 17 Oct 2007 13:01:42 +0200 > > Jens Axboe <[EMAIL PROTECTED]> wrote: > > > > > On Wed, Oct 17 2007, Jens Axboe wrote: >

Re: [PATCH] SPARC64: fix iommu sg chaining

2007-10-17 Thread FUJITA Tomonori
On Wed, 17 Oct 2007 13:01:42 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > On Wed, Oct 17 2007, Jens Axboe wrote: > > On Wed, Oct 17 2007, David Miller wrote: > > > From: Jens Axboe <[EMAIL PROTECTED]> > > > Date: Wed, 17 Oct 2007 11:16:29 +0200 > > > > > > > On Wed, Oct 17 2007, David Miller wro

Re: [PATCH] SPARC64: fix iommu sg chaining

2007-10-17 Thread FUJITA Tomonori
On Wed, 17 Oct 2007 02:45:47 -0700 (PDT) David Miller <[EMAIL PROTECTED]> wrote: > From: FUJITA Tomonori <[EMAIL PROTECTED]> > Date: Wed, 17 Oct 2007 18:24:01 +0900 > > > On Wed, 17 Oct 2007 11:16:29 +0200 > > Jens Axboe <[EMAIL PROTECTED]> wrote: >

Re: [PATCH] SPARC64: fix iommu sg chaining

2007-10-17 Thread FUJITA Tomonori
worried about this specific change. But > there's only been one piece of fallout because if it until now - well > two, with the sparc64 stuff. > > The problem is that you cannot zero the entire sg entry, because then > you'd potentially overwrite the chain pointer. > > I&#

Re: [PATCH] SPARC64: fix iommu sg chaining

2007-10-17 Thread FUJITA Tomonori
On Wed, 17 Oct 2007 11:16:29 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > On Wed, Oct 17 2007, David Miller wrote: > > From: Jens Axboe <[EMAIL PROTECTED]> > > Date: Wed, 17 Oct 2007 10:45:28 +0200 > > > > > Righto, it's invalid to call sg_next() on the last entry! > > > > Unfortunately, that's

[PATCH] IA64: iommu uses sg_next with an invalid sg element

2007-10-17 Thread FUJITA Tomonori
sg list elements might not be continuous. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- arch/ia64/hp/common/sba_iommu.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/ia64/hp/common/sba_iommu.c b/arch/ia64/hp/common/sba_iommu.c index 4338f41..3

[PATCH] swiotlb: fix map_sg failure handling

2007-10-17 Thread FUJITA Tomonori
sg list elements might not be continuous. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- lib/swiotlb.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/swiotlb.c b/lib/swiotlb.c index c419ecf..752fd95 100644 --- a/lib/swiotlb.c +++ b/lib/swiotlb.c @@

[PATCH] cciss: fix scsi_dma_map failure handiling

2007-10-16 Thread FUJITA Tomonori
This fixes the scsi_dma_map failure path, which could lead to data corruption. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/block/cciss_scsi.c | 17 + 1 files changed, 13 insertions(+), 4 deletions(-) diff --git a/drivers/block/cciss_scsi.c b/drivers

[PATCH] SPARC64: fix iommu sg chaining

2007-10-16 Thread FUJITA Tomonori
Commit 2c941a204070ab32d92d40318a3196a7fb994c00 looks incomplete. The helper functions like prepare_sg() need to support sg chaining too. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- arch/sparc64/kernel/iommu.c|1 - arch/sparc64/kernel/iommu_common.c

Re: [GIT PATCH] SCSI updates for 2.6.24

2007-10-16 Thread FUJITA Tomonori
On Mon, 15 Oct 2007 00:09:41 -0400 James Bottomley <[EMAIL PROTECTED]> wrote: > This is the accumulated updates queued for 2.6.24. It contains the > usual slew of driver updates, plus some gdth and advansys rewrites. We > still have some outstanding bugs in gdth and fc4 for which I'm hoping to >

Re: [PATCH] Save some bytes in scsi_cmnd by rearranging members

2007-10-10 Thread FUJITA Tomonori
On Tue, 9 Oct 2007 12:06:01 -0600 Matthew Wilcox <[EMAIL PROTECTED]> wrote: > On Tue, Oct 09, 2007 at 11:42:43AM +0200, Boaz Harrosh wrote: > > Please don't do this patch for now. This is because of the > > soon to come scsi_data_buffer patch that rearages most of the members above > > and puts t

Re: 2.6.23-rc9 boot failure (megaraid?)

2007-10-04 Thread FUJITA Tomonori
On Thu, 4 Oct 2007 12:48:58 +0200 Adrian Bunk <[EMAIL PROTECTED]> wrote: > On Thu, Oct 04, 2007 at 09:28:34AM +0200, Jens Axboe wrote: > >... > > Tomo, this is the patch I added. > > Please excuse my comment in case this was already clear: > > You are aware that this bug is a regression in 2.6.2

Re: 2.6.23-rc9 boot failure (megaraid?)

2007-10-04 Thread FUJITA Tomonori
On Thu, 4 Oct 2007 09:28:34 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > On Thu, Oct 04 2007, FUJITA Tomonori wrote: > > On Wed, 3 Oct 2007 17:32:55 -0600 > > "Patro, Sumant" <[EMAIL PROTECTED]> wrote: > > > > > > > > > > &

Re: [PATCH 4/5] qla2xxx: add target mode support

2007-10-03 Thread FUJITA Tomonori
On Wed, 03 Oct 2007 12:47:26 -0700 Seokmann Ju <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > On Mon, 01 Oct 2007 11:00:44 -0700 > > Seokmann Ju <[EMAIL PROTECTED]> wrote: > >> atl-01:/lib/modules/2.6.23-rc3-smp-tgt/kernel/drivers/scsi/qla2xxx # >

RE: 2.6.23-rc9 boot failure (megaraid?)

2007-10-03 Thread FUJITA Tomonori
On Wed, 3 Oct 2007 17:32:55 -0600 "Patro, Sumant" <[EMAIL PROTECTED]> wrote: > > > > -Original Message----- > > From: FUJITA Tomonori [mailto:[EMAIL PROTECTED] > > Sent: Tuesday, October 02, 2007 5:01 PM > > To: [EMAIL PROTECTED] > > C

Re: [PATCH 3/5] add sg segment limitation info to device structure

2007-10-03 Thread FUJITA Tomonori
On Tue, 2 Oct 2007 09:44:13 -0700 Greg KH <[EMAIL PROTECTED]> wrote: > On Tue, Oct 02, 2007 at 09:25:34AM -0600, Matthew Wilcox wrote: > > On Tue, Oct 02, 2007 at 05:23:39PM +0200, Kay Sievers wrote: > > > Just looking at the number of devices, it seems that allocating it > > > dynamically would b

Re: [PATCH 4/5] qla2xxx: add target mode support

2007-10-03 Thread FUJITA Tomonori
On Mon, 01 Oct 2007 11:00:44 -0700 Seokmann Ju <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > On Thu, 27 Sep 2007 07:34:52 -0700 > > Seokmann Ju <[EMAIL PROTECTED]> wrote: > > > >> FUJITA Tomonori wrote: > >>> On Fri, 21 Sep 2

Re: 2.6.23-rc9 boot failure (megaraid?)

2007-10-02 Thread FUJITA Tomonori
On Tue, 02 Oct 2007 15:38:13 -0500 James Bottomley <[EMAIL PROTECTED]> wrote: > On Tue, 2007-10-02 at 20:15 +0200, Adrian Bunk wrote: > > Cc's added, the complete bug report is at > > http://lkml.org/lkml/2007/10/2/243 > > > > On Tue, Oct 02, 2007 at 12:48:26PM -0400, Burton Windle wrote: > > >

Re: 2.6.23-rc9 boot failure (megaraid?)

2007-10-02 Thread FUJITA Tomonori
On Tue, 02 Oct 2007 15:38:13 -0500 James Bottomley <[EMAIL PROTECTED]> wrote: > On Tue, 2007-10-02 at 20:15 +0200, Adrian Bunk wrote: > > Cc's added, the complete bug report is at > > http://lkml.org/lkml/2007/10/2/243 > > > > On Tue, Oct 02, 2007 at 12:48:26PM -0400, Burton Windle wrote: > > >

Re: [PATCH 4/5] qla2xxx: add target mode support

2007-09-30 Thread FUJITA Tomonori
On Sun, 30 Sep 2007 03:57:07 -0700 Seokmann Ju <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > On Thu, 27 Sep 2007 07:34:52 -0700 > > Seokmann Ju <[EMAIL PROTECTED]> wrote: > > > >> FUJITA Tomonori wrote: > >>> On Fri, 21 Sep 2

Re: [PATCH 1/5] add dma_max_segment_size option to scsi_host_template

2007-09-28 Thread FUJITA Tomonori
On Thu, 27 Sep 2007 02:31:14 -0400 Jeff Garzik <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > You are right. scsi_debug's pseudo-bus works. > > > > But probabaly, scsi_debug doesn't need to call > > blk_queue_max_segment_size now. > &

Re: [PATCH 4/5] qla2xxx: add target mode support

2007-09-28 Thread FUJITA Tomonori
On Thu, 27 Sep 2007 07:34:52 -0700 Seokmann Ju <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > On Fri, 21 Sep 2007 07:34:18 -0700 > > Seokmann Ju <[EMAIL PROTECTED]> wrote: > > > >> Andrew Vasquez wrote: > >>> On Sat, 01 Sep 2007, FUJ

Re: [PATCH] x86-64: pci-gart iommu sg chaining zeroes wrong sg.

2007-09-27 Thread FUJITA Tomonori
On Fri, 28 Sep 2007 01:38:27 +0900 FUJITA Tomonori <[EMAIL PROTECTED]> wrote: > This patch is for Jens' block tree (sg chaining branch). > > I don't have the hardware but this looks like a bug. > > --- > From: FUJITA Tomonori <[EMAIL PROTECTED]> >

[PATCH] x86-64: pci-gart iommu sg chaining zeroes wrong sg.

2007-09-27 Thread FUJITA Tomonori
This patch is for Jens' block tree (sg chaining branch). I don't have the hardware but this looks like a bug. --- From: FUJITA Tomonori <[EMAIL PROTECTED]> Subject: [PATCH] x86-64: pci-gart iommu sg chaining zeroes a wrong sg's dma_length Needs to zero the end of the

Re: [PATCH v3 2/2][BNX2]: Add iSCSI support to BNX2 devices.

2007-09-27 Thread FUJITA Tomonori
CC'ed Jens, James, and linux-scsi again. On Thu, 27 Sep 2007 04:22:15 -0400 Jeff Garzik <[EMAIL PROTECTED]> wrote: > Benjamin Herrenschmidt wrote: > > On Thu, 2007-09-27 at 03:49 -0400, Jeff Garzik wrote: > >> Benjamin Herrenschmidt wrote: > >>> On Thu, 2007-09-27 at 03:31 -0400, Jeff Garzik wrot

Re: [PATCH v3 2/2][BNX2]: Add iSCSI support to BNX2 devices.

2007-09-27 Thread FUJITA Tomonori
CC'ed Jens, James, and linux-scsi. On Thu, 27 Sep 2007 03:31:55 -0400 Jeff Garzik <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > Yeah, we could nicely handle lld's restrictions (especially with > > stacking devices). But iommu code needs only max_segment

Re: [PATCH 3/5] add sg segment limitation info to device structure

2007-09-26 Thread FUJITA Tomonori
On Wed, 26 Sep 2007 09:05:58 -0700 Greg KH <[EMAIL PROTECTED]> wrote: > On Wed, Sep 26, 2007 at 05:58:01PM +0900, FUJITA Tomonori wrote: > > iommu code merges sg segments without considering lld's sg segment > > restrictions. iommu code can't access to the limi

Re: [PATCH 1/5] add dma_max_segment_size option to scsi_host_template

2007-09-26 Thread FUJITA Tomonori
On Wed, 26 Sep 2007 10:42:06 -0400 Jeff Garzik <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > On Wed, 26 Sep 2007 06:11:45 -0400 > > Jeff Garzik <[EMAIL PROTECTED]> wrote: > > > >> FUJITA Tomonori wrote: > >>> This patch mov

Re: [PATCH 0/5] fix iommu sg list merge problem

2007-09-26 Thread FUJITA Tomonori
On Wed, 26 Sep 2007 09:38:32 -0500 James Bottomley <[EMAIL PROTECTED]> wrote: > On Wed, 2007-09-26 at 23:34 +0900, FUJITA Tomonori wrote: > > Yeah, I thought about it and it's possible (but not so easy). parisc > > has IOMMU merging helper code and two parisc IOMMUs use

Re: [PATCH 0/5] fix iommu sg list merge problem

2007-09-26 Thread FUJITA Tomonori
On Wed, 26 Sep 2007 12:02:21 +0200 Muli Ben-Yehuda <[EMAIL PROTECTED]> wrote: > On Wed, Sep 26, 2007 at 05:57:57PM +0900, FUJITA Tomonori wrote: > > > iommu code merges sg lists without considering lld's restrictions so > > some llds need a workaround to split

Re: [PATCH 1/5] add dma_max_segment_size option to scsi_host_template

2007-09-26 Thread FUJITA Tomonori
On Wed, 26 Sep 2007 06:11:45 -0400 Jeff Garzik <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > This patch moves blk_queue_max_segment_size to scsi_alloc_queue from > > llds. It enables scsi_add_host to tells iommu lld's > > dma_max_segment_size. If a

[PATCH 1/5] add dma_max_segment_size option to scsi_host_template

2007-09-26 Thread FUJITA Tomonori
ned-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/scsi/hosts.c |5 + drivers/scsi/scsi_lib.c |1 + include/scsi/scsi_host.h |6 ++ 3 files changed, 12 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/hosts.c b/drivers/scsi/hosts.c index 96bc312..2287

[PATCH 4/5] scsi_add_host sets device max_segment_size

2007-09-26 Thread FUJITA Tomonori
scsi_add_host sets shost_gendev.parent->max_segment_size to enables iommu code to merge sg lists properly about lld's max segment size limit. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/scsi/hosts.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-)

[PATCH 0/5] fix iommu sg list merge problem

2007-09-26 Thread FUJITA Tomonori
iommu code merges sg lists without considering lld's restrictions so some llds need a workaround to split sg lists again. This patchset fixes iommu to handle lld's max segment size limit properly. The problem is that iommu code can't access to the restrictions because they are in request_queue. io

[PATCH 3/5] add sg segment limitation info to device structure

2007-09-26 Thread FUJITA Tomonori
iommu code merges sg segments without considering lld's sg segment restrictions. iommu code can't access to the limitations because they are in request_queue. This patch adds max_segment_size to device structure. seg_boundary_mask will be added too later. Signed-off-by: FUJITA Tomono

[PATCH 5/5] x86-64: pci-gart iommu uses max_segment_size

2007-09-26 Thread FUJITA Tomonori
This enables pci-gart iommu to merge sg lists properly about lld's max segment size limit. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- arch/x86_64/kernel/pci-gart.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/arch/x86_64/kernel/pci-ga

[PATCH 2/5] sata_inic162x: use dma_max_segment_size in scsi_host_template

2007-09-26 Thread FUJITA Tomonori
This removes blk_queue_max_segment_size and uses dma_max_segment_size in scsi_host_template. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/ata/sata_inic162x.c | 19 +++ 1 files changed, 7 insertions(+), 12 deletions(-) diff --git a/drivers/ata/sata_inic

Re: queued patches for SCSI for 2.6.24

2007-09-25 Thread FUJITA Tomonori
On Tue, 25 Sep 2007 22:45:53 -0500 James Bottomley <[EMAIL PROTECTED]> wrote: > On Tue, 2007-09-25 at 23:34 -0400, Jeff Garzik wrote: > > Matthew Wilcox wrote: > > > On Tue, Sep 25, 2007 at 10:37:33PM -0400, Jeff Garzik wrote: > > >> Are there any const-ness worries for scsi_host_template, or plan

Re: queued patches for SCSI for 2.6.24

2007-09-25 Thread FUJITA Tomonori
On Tue, 25 Sep 2007 23:01:53 -0500 James Bottomley <[EMAIL PROTECTED]> wrote: > On Wed, 2007-09-26 at 12:55 +0900, FUJITA Tomonori wrote: > > On Tue, 25 Sep 2007 22:45:53 -0500 > > James Bottomley <[EMAIL PROTECTED]> wrote: > > > > > On Tue, 20

Re: queued patches for SCSI for 2.6.24

2007-09-25 Thread FUJITA Tomonori
On Tue, 25 Sep 2007 22:45:53 -0500 James Bottomley <[EMAIL PROTECTED]> wrote: > On Tue, 2007-09-25 at 23:34 -0400, Jeff Garzik wrote: > > Matthew Wilcox wrote: > > > On Tue, Sep 25, 2007 at 10:37:33PM -0400, Jeff Garzik wrote: > > >> Are there any const-ness worries for scsi_host_template, or plan

Re: queued patches for SCSI for 2.6.24

2007-09-25 Thread FUJITA Tomonori
On Tue, 25 Sep 2007 22:37:33 -0400 Jeff Garzik <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > diff --git a/drivers/scsi/hosts.c b/drivers/scsi/hosts.c > > index adc9559..694015d 100644 > > --- a/drivers/scsi/hosts.c > > +++ b/drivers/scsi/hosts.c > >

Re: queued patches for SCSI for 2.6.24

2007-09-25 Thread FUJITA Tomonori
On Tue, 25 Sep 2007 20:42:35 -0500 James Bottomley <[EMAIL PROTECTED]> wrote: > On Wed, 2007-09-26 at 10:28 +0900, FUJITA Tomonori wrote: > > On Tue, 25 Sep 2007 20:00:02 -0500 > > James Bottomley <[EMAIL PROTECTED]> wrote: > > > > > Andrew asked that I

Re: queued patches for SCSI for 2.6.24

2007-09-25 Thread FUJITA Tomonori
On Tue, 25 Sep 2007 20:00:02 -0500 James Bottomley <[EMAIL PROTECTED]> wrote: > Andrew asked that I provide a status report of pending updates. The > list is attached below. It's pretty much driver updates and minor bug > fixes. The main functionality changes are Kay's sysfs updates and the > s

Re: [PATCH 4/5] qla2xxx: add target mode support

2007-09-23 Thread FUJITA Tomonori
On Fri, 21 Sep 2007 07:34:18 -0700 Seokmann Ju <[EMAIL PROTECTED]> wrote: > Andrew Vasquez wrote: > > On Sat, 01 Sep 2007, FUJITA Tomonori wrote: > > > >> This adds target mode support to qla2xxx. > >> > >> With set ql2enable_target_mode module par

Re: What's in linux-2.6-block.git for 2.6.24

2007-09-23 Thread FUJITA Tomonori
On Sun, 23 Sep 2007 15:19:13 +0200 "Torsten Kaiser" <[EMAIL PROTECTED]> wrote: > On 9/21/07, Jens Axboe <[EMAIL PROTECTED]> wrote: > > SG chaining bits: > > - This is the bulk of the patchset. It consists of three major > > components: > > > > - sglist-core, which add helpers for iterati

Re: 2.6.23-rc4-mm1

2007-09-18 Thread FUJITA Tomonori
On Tue, 18 Sep 2007 12:18:40 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > On Mon, Sep 17 2007, FUJITA Tomonori wrote: > > On Mon, 17 Sep 2007 15:28:19 +0200 > > Jens Axboe <[EMAIL PROTECTED]> wrote: > > > > > On Sat, Sep 15 2007, FUJITA Tomonori wrote:

Re: 2.6.23-rc4-mm1

2007-09-17 Thread FUJITA Tomonori
On Mon, 17 Sep 2007 15:28:19 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > On Sat, Sep 15 2007, FUJITA Tomonori wrote: > > On Fri, 14 Sep 2007 21:16:35 -0700 > > Paul Jackson <[EMAIL PROTECTED]> wrote: > > > > > FUJITA Tomonori wrote: > > &g

Re: [PATCH 16/22] advansys: Eliminate prototypes

2007-09-16 Thread FUJITA Tomonori
On Sun, 16 Sep 2007 17:03:50 -0600 Matthew Wilcox <[EMAIL PROTECTED]> wrote: > On Mon, Sep 17, 2007 at 07:41:06AM +0900, FUJITA Tomonori wrote: > > On Sun, 16 Sep 2007 09:31:34 -0600 > > Matthew Wilcox <[EMAIL PROTECTED]> wrote: > > > > > On Sat, Sep 15,

Re: [PATCH 16/22] advansys: Eliminate prototypes

2007-09-16 Thread FUJITA Tomonori
ext+0x210): undefined reference to `isa_unregister_driver' advansys.c:(.exit.text+0x21a): undefined reference to `isa_unregister_driver' make[1]: *** [.tmp_vmlinux1] Error 1 make[1]: Leaving directory `/home/fujita/git/linux-2.6' make: *** [debian/stamp-build-kernel] Error 2 Is this alread

Re: [PATCH] set supported_mode for lots of llds

2007-09-15 Thread FUJITA Tomonori
On Sat, 15 Sep 2007 23:07:10 -0400 Jeff Garzik <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > The majority of llds don't set supported_mode in scsi_host_template so > > we get: > > > > luce:/sys/class/scsi_host/host0$ cat supported_mode > > un

[PATCH] set supported_mode for lots of llds

2007-09-15 Thread FUJITA Tomonori
The majority of llds don't set supported_mode in scsi_host_template so we get: luce:/sys/class/scsi_host/host0$ cat supported_mode unknown It's harmless but it would be better to get: luce:/sys/class/scsi_host/host0$ cat supported_mode Initiator Signed-off-by: FUJITA Tomono

Re: command failing at iSCSI disconnect

2007-09-15 Thread FUJITA Tomonori
On Fri, 14 Sep 2007 14:43:43 -0700 Dave Jiang <[EMAIL PROTECTED]> wrote: > I'm using the latest linus git tree. This is in fileio mode with > IOMode=wb. It seems that if I do I/O and then immediately disconnect > then the cache sync commands fail. Is this expected behavior or should > the connecti

Re: [PATCH update] SCSI: update Kconfig help text to indicate SCSI core's widespread usage

2007-09-15 Thread FUJITA Tomonori
On Sat, 15 Sep 2007 14:30:10 +0200 Stefan Richter <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > On Sat, 15 Sep 2007 08:16:03 +0200 > > Stefan Richter <[EMAIL PROTECTED]> wrote: > >> Or would be "for newer SCSI transports such as Fibre Channel, &g

Re: 2.6.23-rc4-mm1

2007-09-15 Thread FUJITA Tomonori
On Fri, 14 Sep 2007 21:16:35 -0700 Paul Jackson <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > Can you try this patch (against 2.6.23-rc4-mm1)? > > > > >From 592bd2049cb3e6e1f1dde7cf631879f26ddffeaa Mon Sep 17 00:00:00 2001 > > From: FUJITA Tomonori &

Re: [PATCH update] SCSI: update Kconfig help text to indicate SCSI core's widespread usage

2007-09-15 Thread FUJITA Tomonori
On Sat, 15 Sep 2007 08:16:03 +0200 Stefan Richter <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > On Fri, 14 Sep 2007 23:14:21 +0200 (CEST) > > Stefan Richter <[EMAIL PROTECTED]> wrote: > ... > >> And one more update: > >> There is SAS

Re: [PATCH update] SCSI: update Kconfig help text to indicate SCSI core's widespread usage

2007-09-14 Thread FUJITA Tomonori
On Fri, 14 Sep 2007 23:14:21 +0200 (CEST) Stefan Richter <[EMAIL PROTECTED]> wrote: > Signed-off-by: Stefan Richter <[EMAIL PROTECTED]> > --- > > And one more update: > There is SAS too, and I forgot 'is' in "on a disk which __ accessed via". > > drivers/scsi/Kconfig | 67

Re: [0/22] Advansys updates 2007-09-09

2007-09-10 Thread FUJITA Tomonori
16/22 Eliminate prototypes. Boring and large. > 19/22 Fix simultaneous calls to ->queuecommand This is on the top of the patchset. Can I get your ACK on it? This depends on scsi-misc, the patchset, and sg chaining stuff. -- From: FUJITA Tomonori <[EMAIL PROTECTED]> Subject: [PATC

Re: 2.6.23-rc4-mm1

2007-09-10 Thread FUJITA Tomonori
On Mon, 10 Sep 2007 12:20:38 -0700 Andrew Morton <[EMAIL PROTECTED]> wrote: > On Mon, 10 Sep 2007 20:59:49 +0200 "Torsten Kaiser" <[EMAIL PROTECTED]> wrote: > > > On 9/10/07, Andrew Morton <[EMAIL PROTECTED]> wrote: > > > On Mon, 10 Sep 2007 18:49:26 +0100 Andy Whitcroft <[EMAIL PROTECTED]> > >

Re: 2.6.23-rc4-mm1

2007-09-10 Thread FUJITA Tomonori
t; end_request: I/O error, dev sda, sector 28411047 > > sd 0:0:0:0: [sda] Result: hostbyte=0x07 driverbyte=0x00 > > end_request: I/O error, dev sda, sector 31135687 > > sd 0:0:0:0: [sda] Result: hostbyte=0x07 driverbyte=0x00 > > end_request: I/O error,

Re: 2.6.23-rc4-mm1

2007-09-10 Thread FUJITA Tomonori
rror, dev sda, sector 31080855 > > sd 0:0:0:0: [sda] Result: hostbyte=0x07 driverbyte=0x00 > > end_request: I/O error, dev sda, sector 31080919 > > Buffer I/O error on device sda1, logical block 3885107 > > sd 0:0:0:0: [sda] Result: hostbyte=0x07 driverbyte=0x00 &g

Re: [PATCH 0/9] scsi_data_buffer structure (in preparation for bidi)

2007-09-09 Thread FUJITA Tomonori
On Sun, 09 Sep 2007 17:09:15 +0300 Boaz Harrosh <[EMAIL PROTECTED]> wrote: > On Sun, Sep 09 2007 at 16:47 +0300, FUJITA Tomonori <[EMAIL PROTECTED]> wrote: > > On Sun, 09 Sep 2007 11:25:57 +0300 > > Boaz Harrosh <[EMAIL PROTECTED]> wrote: > > > &g

Re: [PATCH 6/9] tgt: convert ibmvstgt and libsrp to use scsi_data_buffer

2007-09-09 Thread FUJITA Tomonori
On Sun, 9 Sep 2007 23:38:55 +0900 FUJITA Tomonori <[EMAIL PROTECTED]> wrote: > On Sun, 09 Sep 2007 17:28:41 +0300 > Boaz Harrosh <[EMAIL PROTECTED]> wrote: > > > On Sun, Sep 09 2007 at 16:47 +0300, FUJITA Tomonori <[EMAIL PROTECTED]> > > wrote: &

Re: [PATCH 6/9] tgt: convert ibmvstgt and libsrp to use scsi_data_buffer

2007-09-09 Thread FUJITA Tomonori
On Sun, 09 Sep 2007 17:28:41 +0300 Boaz Harrosh <[EMAIL PROTECTED]> wrote: > On Sun, Sep 09 2007 at 16:47 +0300, FUJITA Tomonori <[EMAIL PROTECTED]> wrote: > > On Sun, 09 Sep 2007 13:12:03 +0300 > > Boaz Harrosh <[EMAIL PROTECTED]> wrote: > > > &g

Re: [PATCH 6/9] tgt: convert ibmvstgt and libsrp to use scsi_data_buffer

2007-09-09 Thread FUJITA Tomonori
On Sun, 09 Sep 2007 13:12:03 +0300 Boaz Harrosh <[EMAIL PROTECTED]> wrote: > On Fri, Sep 07 2007 at 0:50 +0300, FUJITA Tomonori <[EMAIL PROTECTED]> wrote: > > Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> > > --- > > drivers/scsi/ibmvscsi/ibmvstgt

Re: [PATCH 0/9] scsi_data_buffer structure (in preparation for bidi)

2007-09-09 Thread FUJITA Tomonori
On Sun, 09 Sep 2007 11:25:57 +0300 Boaz Harrosh <[EMAIL PROTECTED]> wrote: > On Fri, Sep 07 2007 at 0:50 +0300, FUJITA Tomonori <[EMAIL PROTECTED]> wrote: > > This patchset, which I submitted before, adds a new data structure, > > scsi_data_buffer, including e

Re: [PATCH 4/9] scsi_debug: convert to use the data buffer accessors

2007-09-08 Thread FUJITA Tomonori
On Fri, 07 Sep 2007 15:12:06 -0400 Douglas Gilbert <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > From: Boaz Harrosh <[EMAIL PROTECTED]> > > > > - remove the unnecessary map_single path. > > > > - convert to use the new accessors for the sg

Re: [PATCH 0/9] scsi_data_buffer structure (in preparation for bidi)

2007-09-06 Thread FUJITA Tomonori
On Fri, 7 Sep 2007 06:50:08 +0900 FUJITA Tomonori <[EMAIL PROTECTED]> wrote: > This patchset, which I submitted before, adds a new data structure, > scsi_data_buffer, including everything for data transfer: This patch is against Jens' sglist branch with the following patches:

[PATCH 3/9] convert scsi_error.c to use scsi_data_buffer

2007-09-06 Thread FUJITA Tomonori
From: Boaz Harrosh <[EMAIL PROTECTED]> Signed-off-by: Boaz Harrosh <[EMAIL PROTECTED]> --- drivers/scsi/scsi_error.c | 29 +++-- 1 files changed, 11 insertions(+), 18 deletions(-) diff --git a/drivers/scsi/scsi_error.c b/drivers/scsi/scsi_error.c index 8a525ab..af39a39

[PATCH 4/9] scsi_debug: convert to use the data buffer accessors

2007-09-06 Thread FUJITA Tomonori
From: Boaz Harrosh <[EMAIL PROTECTED]> - remove the unnecessary map_single path. - convert to use the new accessors for the sg lists and the parameters. Signed-off-by: Boaz Harrosh <[EMAIL PROTECTED]> --- drivers/scsi/scsi_debug.c | 36 ++-- 1 files changed, 10

[PATCH 8/9] ide-scsi: convert to use the data buffer accessors

2007-09-06 Thread FUJITA Tomonori
This just blindly converts ide-scsi to use the new accessors for the sg lists and the parameters. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/scsi/ide-scsi.c | 23 --- 1 files changed, 12 insertions(+), 11 deletions(-) diff --git a/drivers/sc

[PATCH 2/9] convert sd and sr to use scsi_data_buffer

2007-09-06 Thread FUJITA Tomonori
From: Boaz Harrosh <[EMAIL PROTECTED]> Signed-off-by: Boaz Harrosh <[EMAIL PROTECTED]> --- drivers/scsi/sd.c |6 +++--- drivers/scsi/sr.c | 25 + 2 files changed, 16 insertions(+), 15 deletions(-) diff --git a/drivers/scsi/sd.c b/drivers/scsi/sd.c index 2c6116f..aba

[PATCH 7/9] qla1280: convert to use the data buffer accessors

2007-09-06 Thread FUJITA Tomonori
- remove the unnecessary map_single path. - convert to use the new accessors for the sg lists and the parameters. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/scsi/qla1280.c | 369 1 files changed, 154 insertions(+

[PATCH 9/9] libata-scsi: convert to use the data buffer accessors

2007-09-06 Thread FUJITA Tomonori
From: Boaz Harrosh <[EMAIL PROTECTED]> This is a minimal patch needed if we want to use the new scsi_data_buffer implementation, by useing of the new data accessors. But it is not a complete clean up of the !use_sg path. Libata-core still has the qc->flags & ATA_QCFLAG_SG and !qc->n_elem code path

[PATCH 0/9] scsi_data_buffer structure (in preparation for bidi)

2007-09-06 Thread FUJITA Tomonori
This patchset, which I submitted before, adds a new data structure, scsi_data_buffer, including everything for data transfer: struct scsi_data_buffer { unsigned length; int resid; short sg_count; short __sg_count; struct scatterlist *sglist; }; One scsi_dat

[PATCH 1/9] add scsi_data_buffer structure

2007-09-06 Thread FUJITA Tomonori
This adds a new data structure, scsi_data_buffer, including scsi data transfer. One scsi_data_buffer structure is embedded in struct scsi_cmnd for uni-directional transfer. This is a preparation for bidirectional support. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- driver

[PATCH 6/9] tgt: convert ibmvstgt and libsrp to use scsi_data_buffer

2007-09-06 Thread FUJITA Tomonori
Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/scsi/ibmvscsi/ibmvstgt.c |2 +- drivers/scsi/libsrp.c| 27 ++- 2 files changed, 15 insertions(+), 14 deletions(-) diff --git a/drivers/scsi/ibmvscsi/ibmvstgt.c b/drivers/scsi/ib

[PATCH 5/9] tgt: convert to use scsi_data_buffer

2007-09-06 Thread FUJITA Tomonori
Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/scsi/scsi_tgt_if.c |2 +- drivers/scsi/scsi_tgt_lib.c | 27 ++- 2 files changed, 15 insertions(+), 14 deletions(-) diff --git a/drivers/scsi/scsi_tgt_if.c b/drivers/scsi/scsi_tgt_if.c index c

[PATCH] remove sglist_len

2007-09-06 Thread FUJITA Tomonori
to change more about the sg chaining's scsi_lib.c? -- >From b887de7c48a674ace622544addd70ebc8ae31fd2 Mon Sep 17 00:00:00 2001 From: FUJITA Tomonori <[EMAIL PROTECTED]> Date: Thu, 6 Sep 2007 08:41:33 +0100 Subject: [PATCH] remove sglist_len Signed-off-by: FUJITA Tomonori <[EMA

Re: [PATCH 4/5] qla2xxx: add target mode support

2007-09-05 Thread FUJITA Tomonori
On Wed, 5 Sep 2007 08:05:34 -0700 Andrew Vasquez <[EMAIL PROTECTED]> wrote: > On Sat, 01 Sep 2007, FUJITA Tomonori wrote: > > > This adds target mode support to qla2xxx. > > > > With set ql2enable_target_mode module parameter to 1, the driver runs

Re: Fw: data disclosure in ioctl sg inquiry

2007-09-03 Thread FUJITA Tomonori
On Mon, 03 Sep 2007 13:00:43 -0400 Douglas Gilbert <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > On Sun, 2 Sep 2007 04:56:01 -0700 > > Andrew Morton <[EMAIL PROTECTED]> wrote: > > > >> > >> Begin forwarded message: > >> &

[PATCH 2/2] libata: remove blk_queue_max_phys_segments

2007-09-03 Thread FUJITA Tomonori
Jeff, can I get your ACK on this patch? Whatever in a request queue we set, iommu code ignores all the sg list limitations. --- >From 703d5158361bb6a4ecdc5cd9a6961a8cfb419f73 Mon Sep 17 00:00:00 2001 From: FUJITA Tomonori <[EMAIL PROTECTED]> Date: Mon, 3 Sep 2007 06:49:11 +0100 Subject:

[PATCH 1/2] revert sg segment size ifdefs

2007-09-03 Thread FUJITA Tomonori
00:00:00 2001 From: FUJITA Tomonori <[EMAIL PROTECTED]> Date: Sat, 18 Aug 2007 19:00:47 +0900 Subject: [PATCH 1/2] revert sg segment size ifdefs This reverts sg segment size ifdefs that the current code has in order to provide a way to reduce sgpool memory consumption. Signed-off-by: FUJITA To

Re: Fw: data disclosure in ioctl sg inquiry

2007-09-02 Thread FUJITA Tomonori
0 00 64 69 66 66 20 2d 75 72 Y...diff -ur 20 2d 2d 65 78 63 6c 75 64 65 20 2e 73 76 6e 20 --exclude .svn INQUIRY duration=3 millisecs, resid=60 Note that resid is 60. So, in your case, only the first 36 bytes are valid. But I guess that it's not good to leak random kernel data to user-sp

[PATCH 5/5] srp_transport: convert to use supported_mode attribute

2007-08-31 Thread FUJITA Tomonori
srp transport works for target drivers without supported_mode attribute but it would be better to use it explicitly. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/scsi/ibmvscsi/ibmvstgt.c |1 + drivers/scsi/scsi_transport_srp.c | 10 ++ 2 files chan

[PATCH 3/5] fc_transport: add target driver support

2007-08-31 Thread FUJITA Tomonori
This adds minimum target driver support like the srp transport does: - fc_remote_port_{rolechg,delete} calls scsi_tgt_it_nexus_{create,destroy} for target drivers. - add callbacks to notify target drivers of the nexus and tmf operation results to fc_function_template. Signed-off-by: FUJITA

[PATCH 2/5] add supported_mode and active_mode attributes

2007-08-31 Thread FUJITA Tomonori
scsi_host. We would add a hook to a scsi_host_template to change the active_mode attribute dynamically. But now there is no hook since no lld supports that feature. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> Signed-off-by: Mike Christie <[EMAIL PROTECTED]> --- drivers/scsi/hosts.c

[PATCH 0/5] tgt: add target mode support to fc transport and qla2xxx

2007-08-31 Thread FUJITA Tomonori
This patchset adds target mode support to fc transport class and qla2xxx driver. The pathset is against scsi-misc. The target mode support for fc transport class works in the same way as srp transport class. fc_remote_port_{rolechg,delete} calls scsi_tgt_it_nexus_{create,destroy} for target driver

[PATCH 1/5] tgt: fix can_queue bug

2007-08-31 Thread FUJITA Tomonori
should use host->can_queue instead of host->hostt->can_queue. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> Signed-off-by: Mike Christie <[EMAIL PROTECTED]> --- drivers/scsi/scsi_tgt_lib.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/driver

Re: [PATCH] add use_sg_chaining option to scsi_host_template

2007-08-20 Thread FUJITA Tomonori
On Mon, 20 Aug 2007 15:05:22 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > On Mon, Aug 20 2007, FUJITA Tomonori wrote: > > On Mon, 20 Aug 2007 09:10:31 +0200 > > Jens Axboe <[EMAIL PROTECTED]> wrote: > > > > > On Sat, Aug 18 2007, FUJITA Tomonori wrote:

Re: [PATCH] add use_sg_chaining option to scsi_host_template

2007-08-20 Thread FUJITA Tomonori
On Mon, 20 Aug 2007 09:10:31 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > On Sat, Aug 18 2007, FUJITA Tomonori wrote: > > On Fri, 17 Aug 2007 01:47:59 +0900 > > FUJITA Tomonori <[EMAIL PROTECTED]> wrote: > > > > > This is for Jens' sglist branch in

Re: [PATCH] add use_sg_chaining option to scsi_host_template

2007-08-18 Thread FUJITA Tomonori
On Fri, 17 Aug 2007 01:47:59 +0900 FUJITA Tomonori <[EMAIL PROTECTED]> wrote: > This is for Jens' sglist branch in the block git tree. > > It enables sg chaining support for the LLDs that use scsi_for_each_sg > accessor properly. > > --- > >From a6e50a3b476bc

[PATCH] libata sg chaining support fix

2007-08-18 Thread FUJITA Tomonori
=2 I guess that it would be better to leave sglist libata conversion in the maintainers' hands. --- >From 86d7a796417bde84fc3cbe6ac6ebaaa524266092 Mon Sep 17 00:00:00 2001 From: FUJITA Tomonori <[EMAIL PROTECTED]> Date: Sat, 18 Aug 2007 18:27:36 +0900 Subject: [PATCH] libata sg c

[PATCH] add use_sg_chaining option to scsi_host_template

2007-08-16 Thread FUJITA Tomonori
This is for Jens' sglist branch in the block git tree. It enables sg chaining support for the LLDs that use scsi_for_each_sg accessor properly. --- >From a6e50a3b476bc193de103e8c1d95877ced38918e Mon Sep 17 00:00:00 2001 From: FUJITA Tomonori <[EMAIL PROTECTED]> Date: Fri, 17 Aug

[PATCH] ppc sg chaining support fixes

2007-08-15 Thread FUJITA Tomonori
This is for the sglist branch in Jens' block git tree. --- >From 445923d1a8ff272293af3aadb6e58f82e6439e98 Mon Sep 17 00:00:00 2001 From: FUJITA Tomonori <[EMAIL PROTECTED]> Date: Thu, 16 Aug 2007 02:25:32 +0900 Subject: [PATCH] ppc sg chaining support fixes Signed-off-by: FUJITA To

RE: [PATCH] microtek: convert to use the data buffer accessors

2007-08-12 Thread FUJITA Tomonori
On Sun, 12 Aug 2007 11:01:20 -0400 "Harrosh, Boaz" <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote ... > > Subject: [PATCH] microtek: convert to use the data buffer accessors > > > > - remove the unnecessary map_single path. > > > > - convert

[PATCH] microtek: convert to use the data buffer accessors

2007-08-12 Thread FUJITA Tomonori
The patch is only compile tested. --- >From d5ad824bce0e5e3894a102fee74f1f7c36d639b7 Mon Sep 17 00:00:00 2001 From: FUJITA Tomonori <[EMAIL PROTECTED]> Date: Sun, 12 Aug 2007 15:33:23 +0900 Subject: [PATCH] microtek: convert to use the data buffer accessors - remove the unnecessary m

Re: [PATCH] sg: increase sglist_len of the sg_scatter_hold structure

2007-08-09 Thread FUJITA Tomonori
On Wed, 08 Aug 2007 12:20:43 -0500 Mike Christie <[EMAIL PROTECTED]> wrote: > Mike Christie wrote: > > For drivers like sg and st, do mean the the sg list that is passed to > > functions like scsi_execute_async? If we kill that argument, and instead > > have sg.c and other scsi_execute_async cal

Re: [PATCH] sg: increase sglist_len of the sg_scatter_hold structure

2007-08-09 Thread FUJITA Tomonori
On Wed, 08 Aug 2007 11:58:14 -0500 Mike Christie <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > On Tue, 07 Aug 2007 12:13:41 -0500 > > Mike Christie <[EMAIL PROTECTED]> wrote: > > > >> FUJITA Tomonori wrote: > >>> Allocating 64K

<    1   2   3   4   5   6   >