lkml.org 
[lkml]   [2006]   [Apr]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [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/

\
 
 \ /
  Last update: 2006-04-06 02:27    [W:1.598 / U:0.028 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site