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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 08 Aug 2007 11:58:14 -0500
Mike Christie <michaelc@xxxxxxxxxxx> wrote:

> FUJITA Tomonori wrote:
> > On Tue, 07 Aug 2007 12:13:41 -0500
> > Mike Christie <michaelc@xxxxxxxxxxx> wrote:
> > 
> >> FUJITA Tomonori wrote:
> >>> Allocating 64K contiguous memory is not good so the next thing to do
> >>> is converting sg to use the sg chaining support fully. Or it might be
> >> For LLDs like aic7xxx, I think we are stuck with a small 
> >> scsi_host_template->sg_tablesize, so to continue to get large requests 
> >> like before will we have to still allocate large segments?
> > 
> > No. sg.c has:
> > 
> > sizeof(struct scatterlist) * min(q->max_hw_segments, q->max_phys_segments)
> > 
> > If a lld has small max_hw_segments, it doesn't allocate big contiguous
> > memory.
> > 
> 
> Are we talking about the same thing? Are you saying that it does not 
> allocate big continuous memory for the scatterlist right? I was asking 
> about continuous memory for the buffer sg.c copies data to/from for the 
> IO operation. I was saying that currently for something like aic if we 
> want to continue to support 8 MB commands (or whatever it was) like 
> before, then because its sg_tablesize/max_hw_segments is so small we 
> have to continue allocating large IO buffers. That did not change right? 
> If so let me know because you save me :)

Oops, sorry. I think that nothing changes about large IO buffers. You
have to contiunue to allocate large IO buffers like before.
-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux