Re: [PATCH 3/4] map multiple blocks for mpage_readpages()

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

 



Badari Pulavarty <[email protected]> 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
course..

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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux