Messages in this thread | | | Date | Sun, 9 May 2004 11:31:52 +0100 (BST) | From | Hugh Dickins <> | Subject | Re: [PATCH] rmap 24 pte_young first |
| |
On Sat, 8 May 2004, Hugh Dickins wrote: > On Sat, 8 May 2004, Russell King wrote: > > On Sat, May 08, 2004 at 11:39:32PM +0100, Hugh Dickins wrote: > > > On Sat, 8 May 2004, Christoph Hellwig wrote: > > > > > > > > stupid question - shouldn't the pte_young check simply move to > > > > the beginning of ptep_test_and_clear_young? > > > > > > I don't think that would be a good idea. > > > > EAGAIN.
I think that should be -EEXIST.
> Hah! Delightful refutation of my little lecture. Thanks a lot for > turning that up. Hmm. Well, I guess I need to research that one
Okay, the arch implementations of ptep_test_and_clear_young say Christoph is right: the preliminary pte_young test should be buried in there (in the SMP cases as in the UP cases), not out in rmap.c.
(Just in case anyone is in doubt, I'm absolutely not proposing a preliminary pte_dirty test in the SMP ptep_test_and_clear_dirty.)
I'll leave it for now: it's much better dealt with when we get to doing the (much more important) TLB flush after clearing young.
Andrew, please just ignore rmap 25 (billed as 24) "pte_young first".
Hugh
- 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/
| |