Messages in this thread |  | | From | Daniel Phillips <> | Subject | Re: linux-2.4.10-pre5 | Date | Mon, 10 Sep 2001 04:40:21 +0200 |
| |
On September 10, 2001 04:20 am, Chris Mason wrote: > On Monday, September 10, 2001 04:22:25 AM +0200 Daniel Phillips > <phillips@bonn-fries.net> wrote: > > > On September 10, 2001 04:15 am, Daniel Phillips wrote: > >> On September 10, 2001 03:55 am, Andrea Arcangeli wrote: > >> > getblk should unconditionally alloc a new bh entity and only care to > >> > map it to the right cache backing store with a pagecache hash lookup. > >> > >> To feel anything like the original the new getblk has to be idempotent: > >> subsequent calls return the same block. > > > > Err, buffer_head > > How about subsequent calls for the same offset with the same blocksize need > to return the same buffer head?
Are we picking nits? Better add "the same dev" and "until the buffer head is freed" ;-)
<Attempting to add some content> I've always felt that passing the blocksize and hashing on it is just a little bizarre. This means for example that you can have two 2K buffers overlaid on a 4K buffer, and they will be entirely non-coherent. If we substitute "mapping" where "dev" goes we wouldn't need the size any more. We could still take it and just BUG if it doesn't match the mapping.
Bearing in mind Linus's comments about the liklihood that we might see mixed block sizes in a single mapping something in the future.
-- Daniel - 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/
|  |