Re: [RFC 3/6] Make nr_pagecache a per zone counter

From: Nick Piggin
Date: Sun Dec 11 2005 - 22:51:02 EST


Marcelo Tosatti wrote:
On Sun, Dec 11, 2005 at 08:48:40PM +0100, Andi Kleen wrote:

By the way, why does nr_pagecache needs to be an atomic variable on UP systems?

At least on X86 UP atomic doesn't use the LOCK prefix and is thus quite
cheap. I would expect other architectures who care about UP performance
(= not IA64) to be similar.


But in practice the variable does not need to be an atomic type for UP, but
simply a word, since stores are atomic on UP systems, no?

Several arches seem to use additional atomicity instructions on atomic functions:


Yeah, this is to protect from interrupts and is common to most
load store architectures. It is possible we could have
atomic_xxx_irq / atomic_xxx_irqsave functions for these, however
I think nobody has yet demostrated the improvements outweigh the
complexity that would be added.

--
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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/