[PATCH] ide: build fix

2007-10-22 Thread FUJITA Tomonori
git-drivers/ide/ide-probe.c: In function 'hwif_init': drivers/ide/ide-probe.c:1327: error: implicit declaration of function 'sg_init_table' Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/ide/ide-probe.c |1 + 1 files changed, 1 insertions(+), 0

[PATCH] arm: build fix

2007-10-22 Thread FUJITA Tomonori
arch/arm/common/dmabounce.c: In function 'dma_map_sg': arch/arm/common/dmabounce.c:445: error: implicit declaration of function 'sg_page' Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- arch/arm/common/dmabounce.c |1 + 1 files changed, 1 insertions(+), 0

Re: IDE crash...

2007-10-23 Thread FUJITA Tomonori
On Tue, 23 Oct 2007 09:09:33 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > On Tue, Oct 23 2007, Jens Axboe wrote: > > On Mon, Oct 22 2007, David Miller wrote: > > > > > > I'm debugging a blk_rq_map_sg() crash that i'm getting on sparc64 as > > > root is mounted over IDE. I think I know what is h

Re: IDE crash...

2007-10-23 Thread FUJITA Tomonori
On Tue, 23 Oct 2007 00:43:21 -0700 (PDT) David Miller <[EMAIL PROTECTED]> wrote: > From: Jens Axboe <[EMAIL PROTECTED]> > Date: Tue, 23 Oct 2007 09:23:59 +0200 > > > On Tue, Oct 23 2007, David Miller wrote: > > > From: Jens Axboe <[EMAIL PROTECTED]> > > > Date: Tue, 23 Oct 2007 09:09:33 +0200 > >

Re: IDE crash...

2007-10-23 Thread FUJITA Tomonori
On Tue, 23 Oct 2007 12:58:11 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > On Tue, Oct 23 2007, Jens Axboe wrote: > > On Tue, Oct 23 2007, FUJITA Tomonori wrote: > > > On Tue, 23 Oct 2007 00:43:21 -0700 (PDT) > > > David Miller <[EMAIL PROTECTED]> wrote:

Re: [BUG] 2.6.23-git18 Kernel oops in sg helpers

2007-10-23 Thread FUJITA Tomonori
On Tue, 23 Oct 2007 20:49:40 +0530 Kamalesh Babulal <[EMAIL PROTECTED]> wrote: > Hi, > > Kernel oops is triggered while running fsx-linux test, followed by cpu > softlock > over the AMD box > > Unable to handle kernel NULL pointer dereference at 0018 RIP: > [] gart_map_sg+0x26c/0x

[PATCH -mm 05/11] IA64: make sba_iommu respect the segment size limits

2007-10-24 Thread FUJITA Tomonori
This patch makes sba iommu respect segment size limits when merging sg lists. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- arch/ia64/hp/common/sba_iommu.c |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/arch/ia64/hp/common/sba_iommu.c b/arch/i

[PATCH -mm 07/11] sparc64: make iommu respect the segment size limits

2007-10-24 Thread FUJITA Tomonori
This patch makes iommu respect segment size limits when merging sg lists. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- arch/sparc64/kernel/iommu.c|2 +- arch/sparc64/kernel/iommu_common.c |8 ++-- arch/sparc64/kernel/iommu_common.h |3 ++- arch/sparc64/

[PATCH -mm 09/11] call blk_queue_segment_boundary in __scsi_alloc_queue

2007-10-24 Thread FUJITA Tomonori
call device_dma_parameters accessors like pci_set_dma_max_seg_size when allocating scsi_host. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/scsi/scsi_lib.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/scsi_lib.c b/drivers/scsi/scsi_lib.c in

[PATCH -mm 10/11] sata_inic162x: use pci_set_dma_max_seg_size

2007-10-24 Thread FUJITA Tomonori
This sets the segment size limit properly via pci_set_dma_max_seg_size and remove blk_queue_max_segment_size because scsi-ml calls it. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/ata/sata_inic162x.c | 25 + 1 files changed, 13 insertions(

[PATCH -mm 08/11] parisc: make iommu respect the segment size limits

2007-10-24 Thread FUJITA Tomonori
This patch makes iommu respect segment size limits when merging sg lists. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/parisc/ccio-dma.c |2 +- drivers/parisc/iommu-helpers.h |7 ++- drivers/parisc/sba_iommu.c |2 +- 3 files changed, 8 insertions

[PATCH -mm 0/11] fix iommu sg merging problem

2007-10-24 Thread FUJITA Tomonori
IOMMUs merges scatter/gather segments without considering a low level driver's restrictions. The problem is that IOMMUs can't access to the limitations because they are in request_queue. This patchset introduces a new structure, device_dma_parameters, including dma information. A pointer to device

[PATCH -mm 11/11] aacraid: use pci_set_dma_max_seg_size

2007-10-24 Thread FUJITA Tomonori
This sets the segment size limit properly via pci_set_dma_max_seg_size and remove blk_queue_max_segment_size because scsi-ml calls it. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/scsi/aacraid/linit.c |9 ++--- 1 files changed, 6 insertions(+), 3 deletions(-)

[PATCH -mm 06/11] alpha: make pci_iommu respect the segment size limits

2007-10-24 Thread FUJITA Tomonori
This patch makes pci_iommu respect segment size limits when merging sg lists. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- arch/alpha/kernel/pci_iommu.c | 24 ++-- include/asm-alpha/pci.h |1 + 2 files changed, 19 insertions(+), 6 deletions(-)

[PATCH -mm 04/11] ppc: make iommu respect the segment size limits

2007-10-24 Thread FUJITA Tomonori
This patch makes iommu respect segment size limits when merging sg lists. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- arch/powerpc/kernel/dma_64.c |2 +- arch/powerpc/kernel/iommu.c |8 ++-- include/asm-powerpc/iommu.h |2 +- 3 files changed, 8 insertions

[PATCH -mm 03/11] x86: make pci-gart iommu respect the segment size limits

2007-10-24 Thread FUJITA Tomonori
This patch makes pci-gart iommu respect segment size limits when merging sg lists. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- arch/x86/kernel/pci-gart_64.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/arch/x86/kernel/pci-gart_64.c b/arch/x86/

[PATCH -mm 01/11] add device_dma_parameters structure

2007-10-24 Thread FUJITA Tomonori
parameters are added. So we can easily change where to place struct device_dma_parameters in the future. - dma_get_max_seg_size returns 64K if dma_parms in struct device isn't set up properly. 64K is the default max_segment_size in the block layer. Signed-off-by: FUJITA Tomonori <[EMAIL P

[PATCH -mm 02/11] PCI: add device_dma_parameters support

2007-10-24 Thread FUJITA Tomonori
This adds struct device_dma_parameters in struct pci_dev and properly sets up a pointer in struct device. The default max_segment_size is set to 64K, same to the block layer's default value. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/pci/pci.c |8

Re: [PATCH -mm 11/11] aacraid: use pci_set_dma_max_seg_size

2007-10-24 Thread FUJITA Tomonori
On Wed, 24 Oct 2007 07:31:30 -0400 Jeff Garzik <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > This sets the segment size limit properly via pci_set_dma_max_seg_size > > and remove blk_queue_max_segment_size because scsi-ml calls it. > > > > Signe

Re: [BUG] 2.6.23-git18 Kernel oops in sg helpers

2007-10-24 Thread FUJITA Tomonori
On Wed, 24 Oct 2007 12:54:36 +0100 Andy Whitcroft <[EMAIL PROTECTED]> wrote: > On Tue, Oct 23, 2007 at 08:44:20PM +0200, Jens Axboe wrote: > > On Tue, Oct 23 2007, Kamalesh Babulal wrote: > > > Hi, > > > > > > Kernel oops is triggered while running fsx-linux test, followed by cpu > > > softlock

Re: [PATCH -mm 02/11] PCI: add device_dma_parameters support

2007-10-24 Thread FUJITA Tomonori
On Wed, 24 Oct 2007 07:34:07 -0400 Jeff Garzik <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > index 463a5a9..54edea2 100644 > > --- a/drivers/pci/probe.c > > +++ b/drivers/pci/probe.c > > @@ -985,8 +985,11 @@ void pci_device_add(struct pci_d

Re: [PATCH -mm 09/11] call blk_queue_segment_boundary in __scsi_alloc_queue

2007-10-24 Thread FUJITA Tomonori
On Wed, 24 Oct 2007 07:39:16 -0400 Jeff Garzik <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > request_queue and device struct must have the same value of a segment > > size limit. This patch adds blk_queue_segment_boundary in > > __scsi_alloc_queue so L

Re: [PATCH -mm 0/11] fix iommu sg merging problem

2007-10-24 Thread FUJITA Tomonori
On Wed, 24 Oct 2007 07:40:50 -0400 Jeff Garzik <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > IOMMUs merges scatter/gather segments without considering a low level > > driver's restrictions. The problem is that IOMMUs can't access to the > > limitati

Re: [PATCH -mm 09/11] call blk_queue_segment_boundary in __scsi_alloc_queue

2007-10-24 Thread FUJITA Tomonori
On Wed, 24 Oct 2007 16:28:11 +0200 Jens Axboe <[EMAIL PROTECTED]> wrote: > On Wed, Oct 24 2007, FUJITA Tomonori wrote: > > On Wed, 24 Oct 2007 07:39:16 -0400 > > Jeff Garzik <[EMAIL PROTECTED]> wrote: > > > > > FUJITA Tomonori wrote: > > > &

RE: [PATCH -mm 11/11] aacraid: use pci_set_dma_max_seg_size

2007-10-24 Thread FUJITA Tomonori
On Wed, 24 Oct 2007 09:34:23 -0400 "Salyzyn, Mark" <[EMAIL PROTECTED]> wrote: > ACK Thanks. > Based on the presence of the call. 2.6.22, for instance, does not have > this capability... > > I did not test this change, just accepting on the principals. How much > testing of the change did you do

Re: [BUG] 2.6.23-git18 Kernel oops in sg helpers

2007-10-24 Thread FUJITA Tomonori
On Wed, 24 Oct 2007 21:38:30 +0530 Kamalesh Babulal <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > On Wed, 24 Oct 2007 12:54:36 +0100 > > Andy Whitcroft <[EMAIL PROTECTED]> wrote: > > > >> On Tue, Oct 23, 2007 at 08:44:20PM +0200, Jens Axboe

Re: 2.6.24-rc1 oops

2007-10-24 Thread FUJITA Tomonori
On Thu, 25 Oct 2007 01:40:00 +0100 Sid Boyce <[EMAIL PROTECTED]> wrote: > x86_64 dual, gcc version 4.2.2 (SUSE Linux). > [ cut here ] > kernel BUG at include/linux/scatterlist.h:50! > invalid opcode: [1] SMP > CPU 1 > Modules linked in: ub crc_itu_t hwmon cdrom soundco

Re: 2.6.24-rc1 oops

2007-10-24 Thread FUJITA Tomonori
Jens, I should have CC'ed to you. On Thu, 25 Oct 2007 10:22:02 +0900 FUJITA Tomonori <[EMAIL PROTECTED]> wrote: > On Thu, 25 Oct 2007 01:40:00 +0100 > Sid Boyce <[EMAIL PROTECTED]> wrote: > > > x86_64 dual, gcc version 4.2.2 (SUSE Linux). > > [

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 >

[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

[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] 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

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

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 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
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 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 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 20:37:58 +0900 FUJITA Tomonori <[EMAIL PROTECTED]> wrote: > 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: > &

Re: [bug] ata subsystem related crash with latest -git

2007-10-17 Thread FUJITA Tomonori
On Wed, 17 Oct 2007 14:11:34 -0700 (PDT) Linus Torvalds <[EMAIL PROTECTED]> wrote: > > > On Wed, 17 Oct 2007, Jens Axboe wrote: > > > > That would hurt... Care to commit your for_each_sg() uglification fixup > > for now then? Or disable the allocation debug config entry, so that the > > sg+1 der

[PATCH -mm 0/3] convert IOMMUs to use iova

2007-11-02 Thread FUJITA Tomonori
This patchset convert the PPC64 IOMMU to use the iova code for free area management. The IOMMUs ignores low level drivers' restrictions, the maximum segment size and segment boundary. I fixed the former: http://thread.gmane.org/gmane.linux.scsi/35602 The latter makes the free area management co

[PATCH 2/3] move iova cache code to iova.c

2007-11-02 Thread FUJITA Tomonori
This detaches iova cache code from intel-iommu.c to iova.c in order to enable IOMMUs to use iova code. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/pci/intel-iommu.c | 14 ++ include/linux/iova.h |6 +++--- lib/iova.c

[PATCH 1/3] move iova from drivers/pci/ to lib/

2007-11-02 Thread FUJITA Tomonori
iova could be used by several IOMMUs. This patch just moves iova from drivers/pci/ to lib/ and fixes the appropriate Makefile and Kconfig files. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- arch/x86/Kconfig.x86_64 |1 + drivers/pci/Makefile |2 +- drivers/pci

[PATCH 3/3] POWERPC: convert the IOMMU to use iova

2007-11-02 Thread FUJITA Tomonori
This converts the PPC64 IOMMU to use iova for free area management. TODO: we might need to modify iova to use the tricks like avoiding cacheline bouncing. Performance tests are necessary. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- arch/powerpc/kernel/iommu.c

Re: [PATCH -mm 0/3] convert IOMMUs to use iova

2007-11-02 Thread FUJITA Tomonori
On Fri, 2 Nov 2007 19:12:27 +0200 Muli Ben-Yehuda <[EMAIL PROTECTED]> wrote: > On Sat, Nov 03, 2007 at 02:05:39AM +0900, FUJITA Tomonori wrote: > > > This patchset convert the PPC64 IOMMU to use the iova code for free > > area management. > > > >

Re: [PATCH -mm 0/3] convert IOMMUs to use iova

2007-11-07 Thread FUJITA Tomonori
On Fri, 2 Nov 2007 19:12:27 +0200 Muli Ben-Yehuda <[EMAIL PROTECTED]> wrote: > On Sat, Nov 03, 2007 at 02:05:39AM +0900, FUJITA Tomonori wrote: > > > This patchset convert the PPC64 IOMMU to use the iova code for free > > area management. > > > >

Re: ibmvstgt/aio broken with 2.6.20-rc6 powerpc

2007-01-29 Thread FUJITA Tomonori
From: Bastian Blank <[EMAIL PROTECTED]> Subject: ibmvstgt/aio broken with 2.6.20-rc6 powerpc Date: Sun, 28 Jan 2007 19:52:58 +0100 > I'm not really sure if this is a ibmvstgt or a generic aio problem. You use 2.6.20-rc6 with the aio-epoll-wait patch, right? I think that this is due to the aio-ep

Re: Possible memory leak in block/ll_rw_blk.c

2006-12-13 Thread FUJITA Tomonori
; > [] blkdev_driver_ioctl > > [] blkdev_ioctl > > [] block_ioctl > > [] do_ioctl > > I can confirm that the memory leak disappeared with commit > 77d172ce2719b5ad2dc0637452c8871d9cba344c (by Fujita Tomonori). Thanks. And sorry for introducing that bug. - To

Re: [PATCH 00/33] SG table chaining support

2007-07-16 Thread FUJITA Tomonori
inks this patchset > from 70 to 33 patches. Great! --- From: FUJITA Tomonori <[EMAIL PROTECTED]> Subject: [PATCH] ips: sg chaining support ips properly uses scsi_for_each_sg for the normal I/O path, however, the breakup path doesn't. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECT

Re: [PATCH 00/33] SG table chaining support

2007-07-16 Thread FUJITA Tomonori
inks this patchset > from 70 to 33 patches. Great! It's against Jens' sglist branch though there are lots of changes to the data path in Mike's iscsi tree. --- From: FUJITA Tomonori <[EMAIL PROTECTED]> Subject: [PATCH] iscsi_tcp: sg chaining support Signed-off-by: FUJITA

Re: [PATCH 31/33] Fusion: sg chaining support

2007-07-16 Thread FUJITA Tomonori
From: Jens Axboe <[EMAIL PROTECTED]> Subject: [PATCH 31/33] Fusion: sg chaining support Date: Mon, 16 Jul 2007 11:47:45 +0200 > Cc: [EMAIL PROTECTED] > Signed-off-by: Jens Axboe <[EMAIL PROTECTED]> > --- > drivers/message/fusion/mptscsih.c |4 ++-- > 1 files changed, 2 insertions(+), 2 deleti

Re: [PATCH 29/33] infiniband: sg chaining support

2007-07-16 Thread FUJITA Tomonori
From: Jens Axboe <[EMAIL PROTECTED]> Subject: [PATCH 29/33] infiniband: sg chaining support Date: Mon, 16 Jul 2007 11:47:43 +0200 > @@ -226,7 +228,8 @@ static int iser_sg_to_page_vec(struct iser_data_buf *data, > struct iser_page_vec *page_vec, >

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

2007-07-16 Thread FUJITA Tomonori
From: Gabriel C <[EMAIL PROTECTED]> Subject: Re: Someone ( bsg merge ? ) broke {sd,hd}parm on current git Date: Tue, 17 Jul 2007 02:44:38 +0200 > Gabriel C wrote: > > Hello, > > > > sdparm and hdparm are broken for me on git ( > > abce891a10559343d8ac9f79b46d78afdba63a40 ) > > > > > > ~$ sudo hdpa

Re: block/bsg.c

2007-07-16 Thread FUJITA Tomonori
From: "Satyam Sharma" <[EMAIL PROTECTED]> Subject: Re: block/bsg.c Date: Tue, 17 Jul 2007 06:22:25 +0530 > On 7/17/07, Andrew Morton <[EMAIL PROTECTED]> wrote: > > > > CONFIG_BLK_DEV_BSG=y > CONFIG_SCSI=m > > block/built-in.o: In function `bsg_init': > block/bsg.c:1097: undefined reference to `s

Re: block/bsg.c

2007-07-16 Thread FUJITA Tomonori
From: Jens Axboe <[EMAIL PROTECTED]> Subject: Re: block/bsg.c Date: Tue, 17 Jul 2007 08:38:11 +0200 > On Mon, Jul 16 2007, Andrew Morton wrote: > > > > A belated review (I've never seen this before and there it is in mainline) > > > > > static char bsg_version[] = "block layer sg (bsg) 0.4"; > >

Re: block/bsg.c

2007-07-17 Thread FUJITA Tomonori
From: Jens Axboe <[EMAIL PROTECTED]> Subject: Re: block/bsg.c Date: Tue, 17 Jul 2007 08:59:40 +0200 > On Tue, Jul 17 2007, FUJITA Tomonori wrote: > > From: Jens Axboe <[EMAIL PROTECTED]> > > Subject: Re: block/bsg.c > > Date: Tue, 17 Jul 2007 08:38:11 +0200 >

Re: block/bsg.c

2007-07-17 Thread FUJITA Tomonori
From: Jens Axboe <[EMAIL PROTECTED]> Subject: Re: block/bsg.c Date: Tue, 17 Jul 2007 09:10:45 +0200 > On Tue, Jul 17 2007, FUJITA Tomonori wrote: > > From: Jens Axboe <[EMAIL PROTECTED]> > > Subject: Re: block/bsg.c > > Date: Tue, 17 Jul 2007 08:59:40 +0200 >

Re: block/bsg.c

2007-07-17 Thread FUJITA Tomonori
From: Jens Axboe <[EMAIL PROTECTED]> Subject: Re: block/bsg.c Date: Tue, 17 Jul 2007 08:38:11 +0200 > > As a reader of this code I'm wondering "hm, why is > > BSG_SUB_PROTOCOL_SCSI_CMD unprivileged, while other modes require > > CAP_SYS_RAWIO"?. > > > > This design/policy decision maybe was discus

Re: block/bsg.c

2007-07-17 Thread FUJITA Tomonori
From: Jens Axboe <[EMAIL PROTECTED]> Subject: Re: block/bsg.c Date: Tue, 17 Jul 2007 09:10:45 +0200 > On Tue, Jul 17 2007, FUJITA Tomonori wrote: > > From: Jens Axboe <[EMAIL PROTECTED]> > > Subject: Re: block/bsg.c > > Date: Tue, 17 Jul 2007 08:59:40 +0200 >

Re: block/bsg.c

2007-07-17 Thread FUJITA Tomonori
From: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Subject: Re: block/bsg.c Date: Tue, 17 Jul 2007 22:52:25 +0200 > /* > * TODO > * - Should this get merged, block/scsi_ioctl.c will be migrated into > *this file. To keep maintenance down, it's easier to have them > *seper

Re: block/bsg.c

2007-07-17 Thread FUJITA Tomonori
From: James Bottomley <[EMAIL PROTECTED]> Subject: Re: block/bsg.c Date: Tue, 17 Jul 2007 13:53:54 -0500 > On Tue, 2007-07-17 at 12:19 +0200, Jens Axboe wrote: > > > > Since Linus is happily snoring by now, could you test and see if the > > > > tree works for you? > > > > > > It works for me. I'l

Re: block/bsg.c

2007-07-18 Thread FUJITA Tomonori
From: James Bottomley <[EMAIL PROTECTED]> Subject: Re: block/bsg.c Date: Wed, 18 Jul 2007 09:23:44 -0500 > On Wed, 2007-07-18 at 08:54 -0500, James Bottomley wrote: > > You're welcome ... although there's still a problem for modular builds. > > This is what my /sys/class/bsg looks like: > > Sorry

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

2007-07-18 Thread FUJITA Tomonori
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: Someone ( bsg merge ? ) broke {sd,hd}parm o

Re: block/bsg.c

2007-07-18 Thread FUJITA Tomonori
From: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> Subject: Re: block/bsg.c Date: Wed, 18 Jul 2007 22:39:38 +0200 > > On Wednesday 18 July 2007, FUJITA Tomonori wrote: > > > They were fixed. Please check the latest code: > > > > git://git.kernel.org/pub/scm

Re: [PATCH] Chaining sg lists for big IO commands v5

2007-05-24 Thread FUJITA Tomonori
From: Jens Axboe <[EMAIL PROTECTED]> Subject: Re: [PATCH] Chaining sg lists for big IO commands v5 Date: Thu, 24 May 2007 11:34:41 +0200 > On Tue, May 22 2007, Badari Pulavarty wrote: > > > If iommu=off works, can you try a normal boot but with this applied on > > > top of the sglist patches? That

Re: [PATCH] Chaining sg lists for big IO commands v5

2007-05-24 Thread FUJITA Tomonori
(cc's added) From: Jens Axboe <[EMAIL PROTECTED]> Subject: Re: [PATCH] Chaining sg lists for big IO commands v5 Date: Thu, 24 May 2007 14:05:54 +0200 > On Thu, May 24 2007, Jens Axboe wrote: > > > http://git.kernel.org/?p=linux/kernel/git/tomo/linux-2.6-bidi.git;a=commitdiff;h=fb2747a9f2465ac4a1e

Re: [PATCH] Chaining sg lists for big IO commands v5

2007-05-24 Thread FUJITA Tomonori
From: James Bottomley <[EMAIL PROTECTED]> Subject: Re: [PATCH] Chaining sg lists for big IO commands v5 Date: Thu, 24 May 2007 10:39:44 -0500 > On Thu, 2007-05-24 at 21:44 +0900, FUJITA Tomonori wrote: > > (cc's added) > > > > From: Jens Axboe <[EMAIL

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: 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
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: [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: [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: 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 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: 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: 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 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 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

[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] 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 -mm] intel-iommu sg chaining support

2007-09-29 Thread FUJITA Tomonori
x86_64 defines ARCH_HAS_SG_CHAIN. So if IOMMU implementations don't support sg chaining, we will get data corruption. Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- drivers/pci/intel-iommu.c | 32 1 files changed, 16 insertions(+), 16 deletio

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: 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: 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

<    1   2