Messages in this thread | | | Date | Wed, 05 Apr 2006 10:27:36 +1000 | From | Nick Piggin <> | Subject | Re: [patch 2/3] mm: speculative get_page |
| |
Christoph Lameter wrote: > On Tue, 4 Apr 2006, Nick Piggin wrote: > > >>+ /* >>+ * PageNoNewRefs is set in order to prevent new references to the >>+ * page (eg. before it gets removed from pagecache). Wait until it >>+ * becomes clear (and checks below will ensure we still have the >>+ * correct one). >>+ */ >>+ while (unlikely(PageNoNewRefs(page))) >>+ cpu_relax(); > > > That part looks suspiciously like we need some sort of lock here. >
It's very light-weight now. A lock of course would only be page local, so it wouldn't really harm scalability, however it would slow down the single threaded case. At the moment, single threaded performance of find_get_page is anywhere from about 15-100% faster than before the lockless patches.
I don't see why you think there needs to be a lock? Before the write side clears PageNoNewRefs, they will have moved 'page' out of pagecache, so when this loop breaks, the subsequent test will fail and this function will be repeated.
-- 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 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/
| |