On Saturday 10 February 2007 07:50, Con Kolivas wrote:
> On Saturday 10 February 2007 07:30, Andrew Morton wrote:
> > On Fri, 09 Feb 2007 14:13:03 +0100
> >
> > jos poortvliet <[email protected]> wrote:
> > > Nick's comment, replying to me some time ago:
> >
> > I think I was thinking of this:
> >
> > http://lkml.org/lkml/2006/2/6/509
>
> Fortunately that predates a lot of changes where I did address all those.
> These will seem out of context without looking at that original email so I
> apologise in advance.
>
> buffered_rmqueue and prefetching x86 specific (not into DMA) were dropped
>
> It is NUMA aware
>
> Global cacheline bouncing in page allocation and page reclaim paths I have
> no answer for as I have to tell swap prefetch that the vm is busy somehow
> and I do that by setting precisely one bit in a lockless manner.
>
> The trylocks were dropped.
>
> The other ideas were to :
> -extend the prefetching. That's extra features
> -knowing for sure when a system is really idle. I've tried hard to do that
> as cheaply as possible.
> -putting pages on the lru? well it puts them on the tail
> -papering over an issue? As I said, no matter how good the vm is, there
> will always be loads that swap.
Perhaps I haven't made this clear enough.
Nick has been kind enough to review pretty much all of swap prefetch at every
turn. He is understandably suspicious about any code that I generate since
I'm a doctor and not a computer programmer. I have addressed every concern he
has made along the way and even joked at the end that he "threatened to
review it over and over and over" which he did not take in jest.
Swap prefetch has actually had far more review than a lot of code so I'm
surprised that this is a remaning concern. It has been reviewed, and I have
addressed the concerns. It is possible to hold back code forever at the
suggestion that more review is always required, and basically that's what I
feel this has become.
If there is one valid concern with swap prefetch, there is a numa=64 scenario
that Rohit Seth brought to my attention and I gave him a patch to test 2
months ago. I've pinged him since, but I understand he's busy so has not had
a chance to test the patch.
--
-ck
-
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]
