lkml.org 
[lkml]   [2003]   [Jul]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH] 1/5 VM changes: zone-pressure.patch
Andrew Morton writes:
> Nikita Danilov <Nikita@Namesys.COM> wrote:
> >
> > Add zone->pressure field. It estimates scanning intensity for this zone and
> > is calculated as exponentially decaying average of the scanning priority
> > required to free enough pages in this zone (mm/vmscan.c:zone_adj_pressure()).
> >
> > zone->pressure is supposed to be used in stead of scanning priority by
> > vmscan.c. This is used by later patches in a series.
> >
>
> OK, fixes a bug.

What bug?

>
> >
> > diff -puN include/linux/mmzone.h~zone-pressure include/linux/mmzone.h
> > --- i386/include/linux/mmzone.h~zone-pressure Wed Jul 9 12:24:50 2003
> > +++ i386-god/include/linux/mmzone.h Wed Jul 9 12:24:50 2003
> > @@ -84,11 +84,23 @@ struct zone {
> > atomic_t refill_counter;
> > unsigned long nr_active;
> > unsigned long nr_inactive;
> > - int all_unreclaimable; /* All pages pinned */
> > + int all_unreclaimable:1; /* All pages pinned */
>
> I'll revert this change. Once we start adding bitfields in there they all
> need common locking and it gets messy. integers are simpler.

OK.

>

Nikita.
-
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: 2005-03-22 13:46    [W:0.057 / U:0.204 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site