On Sun, 17 Sep 2000, Linus Torvalds wrote:
> On Sun, 17 Sep 2000, Alexander Viro wrote:
> >
> > Looks sane. But I really wonder if we could just do it in
> > create_page_buffers() if page is up-to-date. OTOH it would require attempt
> > to map them all. Comments?
>
> That would certainly simplify a lot.
>
> And as we've seen, simplifying this area would not necessarily be a bad
> thing ;)
>
> Right now I'll just do the minimal fix, though, I think.
Fine with me. I'll do the full analysis tonight, anyway, and try to write
down the rules for that stuff. One thing that makes me seriously uneasy
is the fact that VM knows about ->buffers - I would be much happier if all
this stuff would be contained in fs/buffer.c. IOW, I'm not sure that
block_flushpage()/try_to_free_buffers() is a happy camper.
I'm not proposing it for immediate inclusion, but I don't feel good about
all this code - current rules are too complex and rely on details of
VM/buffer interaction too much for my taste. It may be correct, but it's
not obviously correct...
[mingo, Daniel and Chris added to cc for obvious reasons]
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Sat Sep 23 2000 - 21:00:16 EST