Re: 2.6.18-rc1

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

 



Con Kolivas wrote:
I see the merge window closed and swap prefetch got bypassed again. I'd like to believe it was an oversight but far more likely that Andrew remains undecided about whether it should go in or not.

No bug reports have come from it in 6 months, the code has remained unchanged for 3 months, it is as unobtrusive as a driver that is not compiled in when !CONFIGed and there are numerous reports from satisfied users (even ones that made it to the scary grounds of lkml). The only thing that happens is Nick keeps threatening to review it over and over and over and....

I was going to review it again, but I noticed it has still has comments
(from Hugh and I, I believe) which still haven't been implemented. Like
duplicating most of read_swap_cache_async. I thought you might have some
improvements on the way, so I hadn't bothered yet.

But... excuse me? I *threaten* to review it? I volunteered to review it a
couple of times and found several problems. But OK if you take that as a
threat, then I won't review it.

And I haven't seen any numbers to show it even works in ideal conditions
after I told you how to fix the watermark code, let alone the real world
situations in which it is supposed to help (not that that seems to be a
showstopper to merging stuff like this, though)

I personally won't advocate it, but I wouldn't be upset if it goes
in... it isn't entirely unobtrusive: it is pretty close to the core mm,
and will have to be maintained as such. Mainly in Hugh's area, so he
would have a final veto there.


I'm not sure what else needs to happen?


Probably if nothing happens, it sounds like Andrew will merge it
eventually.

--
SUSE Labs, Novell Inc.

Send instant messages to your online friends http://au.messenger.yahoo.com -
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