[lkml]   [2003]   [May]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] Only use MSDOS-Partitions by default on X86
    On Wed, May 07 2003, David Woodhouse wrote:
    > On Wed, 2003-05-07 at 08:22, Jens Axboe wrote:
    > > > That's a little short of what I was intending. Ideally we stick 'struct
    > > > request', 'struct buffer_head' and 'struct bio' inside #ifdef
    > > > CONFIG_BLK_DEV, then kill all the dead code which uses them.
    > >
    > > struct request can be a goner with my patch, the others not really.
    > > request is really a block private property, so it's easy to kill off.
    > > You are going for the really minimal approach, basically ruling out lots
    > > of filesystems and requiring major surgery all around. While I can see
    > > that make sense for an embedded kernel, I'm having a hard time
    > > envisioning this as something mergable :-)
    > Last time I looked, it wasn't that bad until I got mired in VM code.
    > I haven't looked at that since we got CONFIG_SWAP, but I'm fairly sure
    > the VM bits will be a lot nicer now too.

    I'll let you deal with that, my head is spinning from just thinking
    about it :)

    > > > mtdblock.c cleanup noted with interest -- I'll play with that shortly;
    > > > thanks. Note that you don't actually need flash hardware, you can load
    > > > the 'mtdram' device which fakes it with vmalloc-backed storage instead.
    > > > Not too useful for powerfail-testing but for mounting something like
    > > > ext2 on mtdblock on mtdram it's fine.
    > >
    > > I'm attaching an updated version, I don't think it's safe to use atomic
    > > kmaps across the do_cached_read/write.
    > It's not -- the flash read/write/erase functions may sleep.

    Thought so.

    > > Also, I want bio_endio() to increment the sector position of the bio as
    > > well. Makes for a nicer api, and the sector var in mtdblock would then
    > > be killable.
    > OK. Let me know when you're done and I'll fix up FTL and NFTL
    > accordingly too.

    The only change to the patch attached in the previous mail is just to
    drop the sector_t sector and use bio->bi_sector throughout the function.
    So nothing major.

    Jens Axboe

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:35    [W:0.020 / U:41.236 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site