On Tue, 23 Oct 2007 13:47:20 +0530
"Thayumanavar Sachithanantham" <[EMAIL PROTECTED]> wrote:
> Hi All,
>
> Does the recent target mode support added for tgt support target mode
> for qla chipset (qla24xx series)?
We've been trying:
http://marc.info/?t=11885798674&r=1&w=2
But I heard that t
On Tue, 23 Oct 2007 07:48:30 -0700
Seokmann Ju <[EMAIL PROTECTED]> wrote:
> FUJITA Tomonori wrote:
> > On Tue, 23 Oct 2007 13:47:20 +0530
> > "Thayumanavar Sachithanantham" <[EMAIL PROTECTED]> wrote:
> >
> >> Hi All,
> >>
> >>
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
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/
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
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(
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
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
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(-)
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(-)
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
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/
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
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
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
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
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
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
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:
> > > &
Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]>
---
drivers/scsi/scsi_tgt_lib.c |4 +---
1 files changed, 1 insertions(+), 3 deletions(-)
diff --git a/drivers/scsi/scsi_tgt_lib.c b/drivers/scsi/scsi_tgt_lib.c
index deea3cd..7abef1a 100644
--- a/drivers/scsi/scsi_tgt_lib.c
+++ b/d
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
This converts tgt to use the data buffer accessors to reduce the size
of the scsi_data_buffer patch for tgt, which will be merged later.
-
Subject: [PATCH] tgt: convert to use the data buffer accessors
- convert to use the new accessors for the sg lists and the
parameters.
Signed-off-by: FUJITA
On Wed, 24 Oct 2007 21:25:24 -0600
Matthew Wilcox <[EMAIL PROTECTED]> wrote:
> On Wed, Oct 24, 2007 at 07:57:55PM -0700, Arjan van de Ven wrote:
> > On Thu, 25 Oct 2007 01:21:29 +0900
> > FUJITA Tomonori <[EMAIL PROTECTED]> wrote:
> > > - scsi_tgt_cmd_cac
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
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
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
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
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.
> >
> >
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.
> >
> >
On Tue, 06 Nov 2007 20:16:19 +0200
Boaz Harrosh <[EMAIL PROTECTED]> wrote:
>
> - If we export scsi_init_io()/scsi_release_buffers() instead of
> scsi_{alloc,free}_sgtable() from scsi_lib than tgt code is
> much more insulated from scsi_lib changes. As a bonus it will
> also gain bid
On Tue, 06 Nov 2007 20:38:33 +0200
Boaz Harrosh <[EMAIL PROTECTED]> wrote:
> On Tue, Nov 06 2007 at 20:25 +0200, Mike Christie <[EMAIL PROTECTED]> wrote:
> > Boaz Harrosh wrote:
> >> [1]
> >> I propose a small change to scsi_tgt_lib.c that will make
> >> tgt completely neutral to the scsi_data_buf
de (req_data_dir() != READ && != WRITE)
>
> - tgt: convert libsrp to use scsi_data_buffer
> - isd200: This driver still bangs on scsi_cmnd IO members,
> so need changing
>
> Signed-off-by: Boaz Harrosh <[EMAIL PROTECTED]>
> Signed-off-by: FUJITA Tomonori <
On Thu, 08 Nov 2007 11:24:36 +0200
Boaz Harrosh <[EMAIL PROTECTED]> wrote:
> >> diff --git a/drivers/scsi/sd.c b/drivers/scsi/sd.c
> >> index 18343a6..28cf6fe 100644
> >> --- a/drivers/scsi/sd.c
> >> +++ b/drivers/scsi/sd.c
> >> @@ -448,9 +448,6 @@ static int sd_prep_fn(struct request_queue *q, st
On Thu, 08 Nov 2007 10:32:56 +0200
Benny Halevy <[EMAIL PROTECTED]> wrote:
> On Nov. 08, 2007, 5:13 +0200, FUJITA Tomonori <[EMAIL PROTECTED]> wrote:
> > On Tue, 06 Nov 2007 20:16:19 +0200
> > Boaz Harrosh <[EMAIL PROTECTED]> wrote:
> >
> >> -
On Thu, 08 Nov 2007 16:01:53 +0200
Boaz Harrosh <[EMAIL PROTECTED]> wrote:
> On Thu, Nov 08 2007 at 15:04 +0200, FUJITA Tomonori <[EMAIL PROTECTED]> wrote:
> > On Thu, 08 Nov 2007 10:32:56 +0200
> > Benny Halevy <[EMAIL PROTECTED]> wrote:
> >
> >&g
On Mon, 12 Nov 2007 22:06:52 -0800
Andrew Morton <[EMAIL PROTECTED]> wrote:
> On Thu, 08 Nov 2007 18:59:30 +0200 Boaz Harrosh <[EMAIL PROTECTED]> wrote:
>
> > In preparation for bidi we abstract all IO members of scsi_cmnd,
> > that will need to duplicate, into a substructure.
> >
> > - Gr
On Mon, 12 Nov 2007 23:07:03 -0800
Andrew Morton <[EMAIL PROTECTED]> wrote:
> On Tue, 13 Nov 2007 15:40:42 +0900 FUJITA Tomonori <[EMAIL PROTECTED]> wrote:
>
> > On Mon, 12 Nov 2007 22:06:52 -0800
> > Andrew Morton <[EMAIL PROTECTED]> wrote:
> >
>
On Thu, 15 Nov 2007 11:53:25 +0200
Boaz Harrosh <[EMAIL PROTECTED]> wrote:
> From: Randy Dunlap <[EMAIL PROTECTED]>
>
> Fix scsi_tgt_lib build when dprintk is defined:
> Also fix accessors problem when dprintk is defined
>
> drivers/scsi/scsi_tgt_lib.c: In function 'scsi_tgt_cmd_destroy':
> driv
This adds PCI's accessor for segment_boundary_mask in
device_dma_parameters.
The default segment_boundary is set to 0x, same to the block
layer's default value (and the scsi mid layer uses the same value).
Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]>
---
drivers/pc
t set up properly. 0x is the default value used in
the block layer and the scsi mid layer.
Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]>
---
include/linux/dma-mapping.h | 15 +++
1 files changed, 15 insertions(+), 0 deletions(-)
diff --git a/include/linux/dma-
I'll send
a patch to do that if it is acceptable. James and Jeff?
Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]>
---
drivers/scsi/scsi_lib.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/scsi/scsi_lib.c b/drivers/scsi/scsi_lib.c
index 733176
This patch makes swiotlb not allocate a memory area spanning LLD's
segment boundary.
is_span_boundary() judges whether a memory area spans LLD's segment
boundary. If map_single finds such a area, map_single tries to find
the next available memory area.
Signed-off-by: FUJITA Tomono
This is the latter half of my iommu work to make the IOMMUs respect
LLDs restrictions.
IOMMUs allocate memory areas without considering a low level driver's
segment boundary limits. So we have some workarounds: splitting sg
segments again in LLDs; reserving all I/O space spanning 4GB boundary
in I
On Tue, 27 Nov 2007 15:40:05 -0800
"Love, Robert W" <[EMAIL PROTECTED]> wrote:
> Hello SCSI mailing list,
>
> I'd just like to introduce ourselves a bit before we get
> started. My name is Robert Love and I'm joined by a team of engineers
> including Vasu Dev, Chris Leech and Yi Zou. We are
patches from 2.6.22 to 2.6.22-rc1
> -one 7 Kb set of patches from 2.6.23-rc2 to 2.6.23-rc3
> -one 162 Kb set of patches from 2.6.23-rc9 to 2.6.23-rc10.
>
> When applying the 2.6.23-rc1-based driver to "my" 2.6.31.1 kernel,
> the "zero blocks"-symptom show
On Thu, 29 Nov 2007 17:45:57 +0100
Anders Henke <[EMAIL PROTECTED]> wrote:
> On Nov 29 2007, FUJITA Tomonori wrote:
> > On Thu, 29 Nov 2007 14:03:19 +0100
> > Jan Kara <[EMAIL PROTECTED]> wrote:
> >
> > >
t; -one 2 Kb small set of patches from 2.6.22 to 2.6.22-rc1
> > -one 7 Kb set of patches from 2.6.23-rc2 to 2.6.23-rc3
> > -one 162 Kb set of patches from 2.6.23-rc9 to 2.6.23-rc10.
> >
> > When applying the 2.6.23-rc1-based driver to "my" 2.6.31.1 kernel,
> &
On Tue, 4 Dec 2007 17:11:55 -0800
Andrew Morton <[EMAIL PROTECTED]> wrote:
> On Wed, 05 Dec 2007 10:04:03 +0900
> FUJITA Tomonori <[EMAIL PROTECTED]> wrote:
>
> > On Tue, 4 Dec 2007 16:57:38 -0800
> > Andrew Morton <[EMAIL PROTECTED]> wrote:
> &
On Wed, 5 Dec 2007 11:14:41 +0100
Anders Henke <[EMAIL PROTECTED]> wrote:
> On Tue, 4 Dec 2007 Andrew Morton wrote:
> > On Wed, 05 Dec 2007 10:30:54 +0900 FUJITA Tomonori <[EMAIL PROTECTED]>
> > wrote:
> >
> > > On Tue, 4 Dec 2007 17:11:55 -0800
>
oundary of devices are set to 4GB by
default. So we can remove 4GB boundary protection now.
Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]>
---
arch/powerpc/kernel/iommu.c | 21 +
1 files changed, 1 insertions(+), 20 deletions(-)
diff --git a/arch/powerpc/kerne
This patch converts PPC's IOMMU to use the IOMMU helper functions. The
IOMMU doesn't allocate a memory area spanning LLD's segment boundary
anymore.
iseries_hv_alloc and iseries_hv_map don't have proper device
struct. 4GB boundary is used for them.
Signed-off-by: FUJ
This patch converts gart IOMMU to use the IOMMU helper functions. The
IOMMU doesn't allocate a memory area spanning LLD's segment boundary
anymore.
Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]>
---
arch/x86/Kconfig |2 +-
arch/x86/kernel/pci-
This adds IOMMU helper functions for the free area management. These
functions take care of LLD's segment boundary limit for IOMMUs. They
would be useful for IOMMUs that use bitmap for the free area
management.
Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]>
---
include/linux/iom
This patchset is a sequel to my patchset to fix iommu segment boundary
problems:
http://www.mail-archive.com/linux-scsi@vger.kernel.org/msg11919.html
This adds new IOMMU helper functions for the free area
management. These functions take care of LLD's segment boundary limit
for IOMMUs. They are u
This kills unused __clear_bit_string and find_next_zero_string (they
were used by only gart and calgary IOMMUs).
Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]>
---
arch/x86/lib/Makefile_64|2 +-
arch/x86/lib/bitstr_64.c| 28
include/a
This patch converts calgary IOMMU to use the IOMMU helper
functions. The IOMMU doesn't allocate a memory area spanning LLD's
segment boundary anymore.
Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]>
---
arch/x86/Kconfig |3 +++
arch/x86/kernel/pci-cal
bsg SG_IO needs to return -EFAULT for the response field in struct
sg_io_v4.
Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]>
---
block/bsg.c |5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/block/bsg.c b/block/bsg.c
index 8e181ab..7781e63 100644
--- a/block
On Fri, 14 Dec 2007 18:05:56 -0500
"John Stoffel" <[EMAIL PROTECTED]> wrote:
>
> Hi,
>
> Just fired up 2.6.24-rc5-mm1 on a Dual CPU PIII 550mhz system with 2gb
> of RAM. Got the following error. Let me know if you need more
> details or want me to run tests or make changes. Looks like somethi
On Sun, 16 Dec 2007 20:05:51 -0500
"John Stoffel" <[EMAIL PROTECTED]> wrote:
> [ 215.007701] sym1: SCSI parity error detected: SCR1=1 DBC=1128 SBCL=ae
> [ 215.008145] sym1: SCSI parity error detected: SCR1=1 DBC=1128 SBCL=ae
> [ 215.008678] sym1: SCSI parity error detected: SCR1=1 DBC=1
Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]>
---
drivers/message/fusion/mptsas.c|2 ++
drivers/scsi/libsas/sas_expander.c |5 +
2 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/drivers/message/fusion/mptsas.c b/drivers/message/fusion/mptsas.c
index e4c94f9.
On Thu, 20 Dec 2007 14:13:15 +1100
"Julian Calaby" <[EMAIL PROTECTED]> wrote:
> On Dec 20, 2007 1:44 PM, FUJITA Tomonori <[EMAIL PROTECTED]> wrote:
> > --- a/drivers/message/fusion/mptsas.c
> > +++ b/drivers/message/fusion/mptsas.c
> > @@ -1343,6 +134
On Thu, 20 Dec 2007 16:49:30 +1100
Rusty Russell <[EMAIL PROTECTED]> wrote:
> Obvious counterpart to dma_map_sg. Note that this is arch-independent
> code; sg_rings are backwards compatible with simple sg arrays.
>
> Signed-off-by: Rusty Russell <[EMAIL PROTECTED]>
> ---
> drivers/base/dma-mapp
On Thu, 20 Dec 2007 16:45:18 +1100
Rusty Russell <[EMAIL PROTECTED]> wrote:
> OK, some fixes since last time, as I wade through more SCSI drivers. Some
> drivers use "use_sg" as a flag to know whether the request_buffer is a
> scatterlist: I don't need the counter, but I still need the flag, so
On Thu, 20 Dec 2007 18:22:54 +1100
Rusty Russell <[EMAIL PROTECTED]> wrote:
> Hi Alan,
>
> Was looking through initio.c to convert it to sg_ring, and noticed this
> code:
>
> initio_build_scb() around 2616:
> sg = &cblk->sglist[0];
> scsi_for_each_sg(cmnd, sgli
On Fri, 21 Dec 2007 09:58:56 +1100
Rusty Russell <[EMAIL PROTECTED]> wrote:
> On Thursday 20 December 2007 18:42:44 David Miller wrote:
> > From: FUJITA Tomonori <[EMAIL PROTECTED]>
> > Date: Thu, 20 Dec 2007 16:06:31 +0900
> >
> > > On Thu, 20 Dec 200
On Thu, 20 Dec 2007 16:40:00 -0800 (PST)
David Miller <[EMAIL PROTECTED]> wrote:
> From: Rusty Russell <[EMAIL PROTECTED]>
> Date: Fri, 21 Dec 2007 11:35:12 +1100
>
> > On Friday 21 December 2007 11:00:27 FUJITA Tomonori wrote:
> > > We need to pass the whol
On Fri, 21 Dec 2007 10:13:38 +1100
Rusty Russell <[EMAIL PROTECTED]> wrote:
> On Thursday 20 December 2007 18:58:07 David Miller wrote:
> > From: Rusty Russell <[EMAIL PROTECTED]>
> > Date: Thu, 20 Dec 2007 18:53:48 +1100
> >
> > > Manipulating the magic chains is horrible; it looks simple to the
On Fri, 21 Dec 2007 14:26:47 +1100
Rusty Russell <[EMAIL PROTECTED]> wrote:
> On Friday 21 December 2007 13:28:34 FUJITA Tomonori wrote:
> > I'm not sure about chaining the headers (as your sg_ring and
> > scsi_sgtable do) would simplify LLDs. Have you looked at ips or
On Wed, 26 Dec 2007 11:27:40 +1100
Rusty Russell <[EMAIL PROTECTED]> wrote:
> On Friday 21 December 2007 15:37:46 FUJITA Tomonori wrote:
> > Some scsi drivers like ips access to sglist in a tricky way.
>
> Indeed. I fail to see how this code works, in fact:
>
> d
On Mon, 24 Dec 2007 10:18:50 +0530
Balbir Singh <[EMAIL PROTECTED]> wrote:
> Hi,
>
> I've just seen this on my dmesg, this is new, never seen this before on
> this box and it happens only with this version of the kernel.
>
> In this configuration, the page size is set to 64K and I've enabled fak
On Thu, 27 Dec 2007 10:08:25 +0530
Balbir Singh <[EMAIL PROTECTED]> wrote:
> FUJITA Tomonori wrote:
> > On Mon, 24 Dec 2007 10:18:50 +0530
> > Balbir Singh <[EMAIL PROTECTED]> wrote:
> >
> [snip]
>
> >
> > I might break the IOMMU code. Can you
On Thu, 27 Dec 2007 11:15:37 +0530
Balbir Singh <[EMAIL PROTECTED]> wrote:
> FUJITA Tomonori wrote:
> > On Thu, 27 Dec 2007 10:08:25 +0530
> > Balbir Singh <[EMAIL PROTECTED]> wrote:
> >
> >> FUJITA Tomonori wrote:
> >>> On Mon, 24 Dec 20
On Fri, 28 Dec 2007 13:05:39 +0530
Balbir Singh <[EMAIL PROTECTED]> wrote:
> FUJITA Tomonori wrote:
> [snip]
> > Thanks,
> >
> > Can you try this?
> >
> >
> > diff --git a/lib/iommu-helper.c b/lib/iommu-helper.c
> > index e7d8544..495575a 1
From: "Love, Robert W" <[EMAIL PROTECTED]>
Subject: Open-FCoE on linux-scsi
Date: Tue, 27 Nov 2007 15:40:05 -0800
> Hello SCSI mailing list,
>
> I'd just like to introduce ourselves a bit before we get
> started. My name is Robert Love and I'm joined by a team of engineers
> including Vasu
From: James Bottomley <[EMAIL PROTECTED]>
Subject: [PATCH] libsas: add host SMP processing
Date: Fri, 28 Dec 2007 17:36:38 -0600
> This patch allows the sas host device to accept SMP commands. This
> brings the libsas attached hosts on to a par with the firmware based
> ones like mptsas.
>
> Jam
On Sat, 29 Dec 2007 09:44:32 -0600
James Bottomley <[EMAIL PROTECTED]> wrote:
> On Sat, 2007-12-29 at 14:24 +0900, FUJITA Tomonori wrote:
> > From: James Bottomley <[EMAIL PROTECTED]>
> > > --- a/drivers/scsi/libsas/sas_expander.c
> > > +++ b/drivers/scsi/
From: James Bottomley <[EMAIL PROTECTED]>
Subject: Re: [PATCH] libsas: add host SMP processing
Date: Sat, 29 Dec 2007 10:59:53 -0600
>
> On Sun, 2007-12-30 at 01:06 +0900, FUJITA Tomonori wrote:
> > On Sat, 29 Dec 2007 09:44:32 -0600
> > James Bottomley <[EMAIL PROTE
n);
> + if (ret > 0) {
> + /* positive number is the untransferred residual */
> + rsp->data_len = ret;
> + ret = 0;
> + }
Would be better to update dout_resid too (on sucess, we can set
req->data_len to zero, I think)?
Here&
On Mon, 31 Dec 2007 17:57:35 +0530
Balbir Singh <[EMAIL PROTECTED]> wrote:
> FUJITA Tomonori wrote:
> >
> > Oops, it's for -mm.
> >
>
> Hi,
>
> I just tested this patch and it works fine for me so far.
>
> Tested-by: Balbir Singh <
ere is a small possibility that a host need more sense buffers than
can_queue. The failure of the buffer allocation works just like the
failure of scsi_cmnd allocation. So everything should work as before.
Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]>
---
drivers/scsi/hosts.c
This patchset is to solve the sense buffer DMA issue:
http://www.mail-archive.com/linux-scsi@vger.kernel.org/msg12632.html
It's not good to fatten up scsi_cmnd so instead this allocates
sense_buffer dynamically with GFP_DMA.
We don't always need sense_buffer per scsi_cmnd but some LLDs assume
th
To remove sense_buffer array in scsi_cmnd structure, this replaces
sizeof sense_buffer with SCSI_SENSE_BUFFERSIZE in several LLDs.
Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]>
---
drivers/ata/libata-scsi.c |4 ++--
drivers/message/fusion/mptscsih.c |2 +-
d
On Mon, 31 Dec 2007 15:56:08 -0600
James Bottomley <[EMAIL PROTECTED]> wrote:
> ATA requires that all DMA transfers begin and end on word boundaries.
> Because of this, a large amount of machinery grew up in ide to adjust
> scatterlists on this basis. However, as of 2.5, the block layer has a
> d
From: "Love, Robert W" <[EMAIL PROTECTED]>
Subject: RE: Open-FCoE on linux-scsi
Date: Mon, 31 Dec 2007 08:34:38 -0800
> >> Hello SCSI mailing list,
> >>
> >>I'd just like to introduce ourselves a bit before we get
> >> started. My name is Robert Love and I'm joined by a team of engineers
> >>
On Thu, 3 Jan 2008 13:56:37 +0900
FUJITA Tomonori <[EMAIL PROTECTED]> wrote:
> This removes static array sense_buffer in scsi_cmnd and uses
> dynamically allocated sense_buffer (with GFP_DMA).
>
> scsi_add_host allocates as many buffers as
> scsi_host->can_queue. __s
On Thu, 03 Jan 2008 17:01:11 +0200
Boaz Harrosh <[EMAIL PROTECTED]> wrote:
> On Thu, Jan 03 2008 at 6:56 +0200, FUJITA Tomonori <[EMAIL PROTECTED]> wrote:
> > To remove sense_buffer array in scsi_cmnd structure, this replaces
> > sizeof sense_buffer with SCSI_SENSE
and
> removing them will cause corruption and an oops when their creator tries
> to remove them.
>
> Signed-off-by: David Dillow <[EMAIL PROTECTED]>
> Acked-by: FUJITA Tomonori <[EMAIL PROTECTED]>
> ---
Thanks!
James, please put this patch into scsi-rc-fixes.
> On
On Thu, 3 Jan 2008 13:58:29 -0800
"Love, Robert W" <[EMAIL PROTECTED]> wrote:
> >Talking about stability is a bit premature, I think. The first thing
> >to do is finding a design that can be accepted into mainline.
>
> How can we get this started? We've provided our current solution, but
> need f
On Fri, 04 Jan 2008 12:45:45 +0100
Stefan Richter <[EMAIL PROTECTED]> wrote:
> On 1/3/2008 10:58 PM, Love, Robert W wrote:
> [FUJITA Tomonori wrote]
> >>I would add one TODO item, better integration with scsi_transport_fc.
> >>If we have HW FCoE HBAs in the future,
On Thu, 3 Jan 2008 11:10:04 -0500
"Salyzyn, Mark" <[EMAIL PROTECTED]> wrote:
> ACK on aacraid/ips/dpt_i2o bits. Inspected others, this patch IS inert.
Thanks!
> NitMeBeingStupidAndAddingARiderToTheBill: I know it was a grep/replace.
> If you need to respin because of Boaz and do not mind, do no
From: Deepak Colluru <[EMAIL PROTECTED]>
Subject: [PATCH] bsg : Add support for io vectors in bsg
Date: Fri, 4 Jan 2008 21:47:34 +0530 (IST)
> From: Deepak Colluru <[EMAIL PROTECTED]>
>
> Add support for io vectors in bsg.
>
> Signed-off-by: Deepak Colluru <[EMAIL PROTECTED]>
> ---
> bsg.c |
On Sat, 05 Jan 2008 21:33:48 +0300
Vladislav Bolkhovitin <[EMAIL PROTECTED]> wrote:
> Thus, I believe, that partial user space, partial kernel space approach
> for building SCSI targets is the move in the wrong direction, because it
> brings practically nothing, but costs a lot.
We have not dis
On Fri, 4 Jan 2008 14:07:28 -0800
"Dev, Vasu" <[EMAIL PROTECTED]> wrote:
>
> >>
> >> _If_ there will indeed be dedicated FCoE HBAs in the future, the
> >> following stack could exist in addition to the one above:
> >>
> >> - SCSI core,
> >> scsi_transport_fc
> >> - FCoE HBA driver(s)
> >
On Sat, 05 Jan 2008 00:41:05 +0100
Stefan Richter <[EMAIL PROTECTED]> wrote:
> Dev, Vasu wrote:
> [FUJITA Tomonori wrote:]
> >> Agreed. My FCoE initiator design would be something like:
> >>
> >> scsi-ml
> >> fcoe initiator driver
> >>
On Thu, 3 Jan 2008 13:56:37 +0900
FUJITA Tomonori <[EMAIL PROTECTED]> wrote:
> This removes static array sense_buffer in scsi_cmnd and uses
> dynamically allocated sense_buffer (with GFP_DMA).
>
> scsi_add_host allocates as many buffers as
> scsi_host->can_queue. __s
On Sun, 23 Dec 2007 13:09:05 +0200
Boaz Harrosh <[EMAIL PROTECTED]> wrote:
> On Fri, Dec 21 2007 at 4:30 +0200, Benjamin Herrenschmidt <[EMAIL PROTECTED]>
> wrote:
> > The sense buffer ins scsi_cmnd can nowadays be DMA'ed into directly
> > by some low level drivers (that typically happens with US
Looks like that host_cmd_pool_mutex are necessary here.
Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]>
---
drivers/scsi/scsi.c |5 ++---
1 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/scsi/scsi.c b/drivers/scsi/scsi.c
index ebc0193..54ff611 100644
--- a/d
On Mon, 07 Jan 2008 15:25:36 +0200
Boaz Harrosh <[EMAIL PROTECTED]> wrote:
> On Mon, Jan 07 2008 at 8:53 +0200, FUJITA Tomonori <[EMAIL PROTECTED]> wrote:
> > On Sun, 23 Dec 2007 13:09:05 +0200
> > Boaz Harrosh <[EMAIL PROTECTED]> wrote:
> >
> >
CC'ed Jes,
On Tue, 8 Jan 2008 14:15:53 +0300
Evgeniy Dushistov <[EMAIL PROTECTED]> wrote:
> On Sun, Nov 25, 2007 at 01:24:25PM +0200, James Bottomley wrote:
> > This is a bit of a rash of bug fixes. The qla1280 is actually a bug fix
> > (in spite of the title---it's actually correcting an existi
On Tue, 8 Jan 2008 17:09:18 -0500
Pete Wyckoff <[EMAIL PROTECTED]> wrote:
> [EMAIL PROTECTED] wrote on Sat, 05 Jan 2008 14:01 +0900:
> > From: Deepak Colluru <[EMAIL PROTECTED]>
> > Subject: [PATCH] bsg : Add support for io vectors in bsg
> > Date: Fri, 4 Jan 2008 21:47:34 +0530 (IST)
> >
> > > F
1 - 100 of 569 matches
Mail list logo