lkml.org 
[lkml]   [2006]   [Feb]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH 0/3] map multiple blocks in get_block() and mpage_readpages()
    From
    Date
    On Mon, 2006-02-20 at 18:41 -0800, Jeremy Higdon wrote:
    > On Tue, Feb 21, 2006 at 08:59:53AM +1100, Nathan Scott wrote:
    > > On Mon, Feb 20, 2006 at 01:21:27PM -0800, Badari Pulavarty wrote:
    > >
    > > I've been running these patches in my development tree for awhile
    > > and have not seen any problems. My one (possibly minor) concern
    > > is that we pass get_block a size in units of bytes, e.g....
    > >
    > > bh->b_size = 1 << inode->i_blkbits;
    > > err = get_block(inode, block, bh, 1);
    > >
    > > And b_size is a u32. We have had the situation in the past where
    > > people (I'm looking at you, Jeremy ;) have been issuing multiple-
    > > gigabyte direct reads/writes through XFS. The syscall interface
    > > takes an (s)size_t in bytes, which on 64 bit platforms is a 64 bit
    > > byte count.
    > >
    > > I wonder if this change will end up ruining things for the lunatic
    > > fringe issuing these kinds of IOs? Maybe the get_block call could
    >
    > Hey! Lunatic fringe indeed. Harumph! :-)
    >
    > Yes, there are a few people out there who will need to issue really
    > large I/O reads or writes to get maximum I/O bandwidth on large
    > stripes. The largest I've done so far is 4GiB, but I expect that
    > number will likely increase this year, and more likely next year,
    > if not.

    Thinking more about it, currently (without my patches) only DIO
    code can request for large chunks of mapping through get_blocks().
    Since b_size is only u32, you can only map 4GB max. Isn't it ?
    With my patches, now mpage_readpages() also can request large
    chunks. (through readahead). So, my patches are not adding any
    extra limitation. Its carrying the same existing limitation.

    In order to handle larger chunks of disk mapping, changing b_size
    to u64 is required and we should request for it irrespective of my
    patches.

    Thanks,
    Badari

    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2006-02-21 17:05    [W:0.028 / U:0.432 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site