On Wed, 2006-03-01 at 20:45 -0800, Andrew Morton wrote:
> 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.
Yep. Can be done. I will take a pass at it.
Thanks,
Badari
-
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]
