[lkml]   [2006]   [Mar]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 3/4] map multiple blocks for mpage_readpages()
    Badari Pulavarty <> wrote:
    > do_mpage_readpage(struct bio *bio, struct page *page, unsigned nr_pages,
    > - sector_t *last_block_in_bio, get_block_t get_block)
    > + sector_t *last_block_in_bio, struct buffer_head *map_bh,
    > + unsigned long *first_logical_block, int *map_valid,
    > + get_block_t get_block)

    I wonder if we really need that map_valid pointer there. The normal way of
    communicating the validity of a bh is via buffer_uptodate(). I _think_
    that's an appropriate thing to use here. With appropriate comments, of

    If those options are not a sufficiently good fit then we can easily create
    a new buffer-head.state bit for this purpose - there are lots to spare.
    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: 2006-03-02 05:49    [W:0.035 / U:16.020 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site