Re: [PATCH #2] OOM kill trigger

[Rik van Riel]

On Fri, 6 Jul 2001, Jeff Garzik wrote:
> Rik van Riel wrote:
> > +       cache_mem = atomic_read(&page_cache_size);
> > +       cache_mem += atomic_read(&buffermem_pages);
> > +       cache_mem -= swapper_space.nrpages;
> > +       limit = (page_cache.min_percent + buffer_mem.min_percent);
>
> Don't you need extra protection around swapper_space.nrpages?
> A barrier right above it?

No. It's just a heuristic.

Besides, all that could change is the VALUE of
swapper_space.nrpages and it won't change by all
that much...

Rik
--
Executive summary of a recent Microsoft press release:
   "we are concerned about the GNU General Public License (GPL)"

		http://www.surriel.com/
http://www.conectiva.com/	http://distro.conectiva.com/