Messages in this thread | | | Date | Mon, 22 Mar 2004 00:52:07 +0100 | From | Andrea Arcangeli <> | Subject | Re: [PATCH] anobjrmap 1/6 objrmap |
| |
On Sun, Mar 21, 2004 at 08:30:34AM -0800, Martin J. Bligh wrote: > >> Mmmm, if you have a broken out patch, it'd be preferable. If I were to > >> apply the whole of -mjb, I'll get a damned sight better results than > >> any of them, but that's not really a fair comparison ;-) I'll can at > >> least check it's stable for me that way though. > >> > >> I did find your broken-out anon-vma patch, but it's against something > >> else, maybe half-way up your tree or something, and I didn't bother > >> trying to fix it ;-) > > > > this one is against mainline, but you must use my objrmap patch too > > which is fixed so it doesn't crash in 2.6.5-rc1. > > > > http://www.us.kernel.org/pub/linux/kernel/people/andrea/kernels/v2.6/2.6.5-rc1-aa2/00100_objrmap-core-1.gz > > http://www.us.kernel.org/pub/linux/kernel/people/andrea/kernels/v2.6/2.6.5-rc1-aa2/00101_anon_vma-2.gz > > > > just backout your objrmap and apply the above two, it should apply > > pretty well. > > I tried the aa3 equiv of the above, just on top of virgin 2.6.5-rc1, but > it doesn't work cleanly. Your whole aa3 tree runs nicely, but I'd prefer > to have the broken out patch before publishing comparisons, as otherwise > it's a bit unfair ;-) I'm not sure if the results come from your anon_vma > approach, or other patches in your tree ... > > I'm presuming you shifted the cost of find_get_page into find_trylock_page > and pgd_ctor into pgd_alloc from the profiles below ...
I cannot see how can find_trylock_page be affected by my anon_vma changes. The only difference I can see is taht Andrew's -mm writeback code is adding the _irq to the spinlocks there and I don't see other obvious changes in that function. I included all -mm writeback changes primarly to avoid me to maintain two slightly different versions of anon_vma and secondly to nuke the page->list. Other trees I'm dealing with daily have those applied already. At the very least that additional cost that you measured cannot be associated in any way with the allocation and maintainace of the anon_vma, since that find_trylock_page cost is a per-page pagecache thing absolutely unrelated to the anon_vmas costs.
It's probably best that I port my version of objrmap (basically the same as yours but with the shm swapout fixes) + anon_vma to your tree, it's not a big effort to do the porting once, I applied Andrew's patches primarly to avoid porting back and forth all the time.
Just tell me which is exactly the codebase I should port against and I'll send you a patch shortly.
Thanks! - 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/
| |