lkml.org 
[lkml]   [2002]   [Apr]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [patch-2.5.8-pre] swapinfo accounting
On Thu, 11 Apr 2002, Hugh Dickins wrote:

| On Wed, 10 Apr 2002, Andrew Morton wrote:
| > "Randy.Dunlap" wrote:
| > >
| > > It looks to me like mm/swapfile.c::si_swapinfo()
| > > shouldn't be adding nr_to_be_unused to total_swap_pages
| > > or nr_swap_pages for return in val->freeswap and
| > > val->totalswap.
|
| Good observation, thanks Randy, but wrong fix.

Thanks for the patch. I confirm that it is giving me
numbers that look sane.

BTW, the patch was for discussion, so it worked.
"a la akpm" :)

~Randy

| > whee, an si_swapinfo() maintainer.
|
| That might be me? I rewrote that code for 2.4.10
| (when you called attention to si_swapinfo slowness).
|
...
|
| It is the right fix; but since that condition was misunderstood,
| and some other flag might be added in future, the safer patch
| would be this more explicit one (which I'll now send to Linus
| with a briefer description).
|
| But I hope nobody backports this to 2.4, where it would be
| wrong: 2.5.4 confusingly changed the nature of SWP_WRITEOK.
|
| Hugh
|
| --- 2.5.8-pre3/mm/swapfile.c Mon Mar 11 12:30:56 2002
| +++ linux/mm/swapfile.c Thu Apr 11 15:26:51 2002
| @@ -1095,7 +1095,8 @@
| swap_list_lock();
| for (i = 0; i < nr_swapfiles; i++) {
| unsigned int j;
| - if (!(swap_info[i].flags & SWP_USED))
| + if (!(swap_info[i].flags & SWP_USED) ||
| + (swap_info[i].flags & SWP_WRITEOK))
| continue;
| for (j = 0; j < swap_info[i].max; ++j) {
| switch (swap_info[i].swap_map[j]) {

-
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:25    [W:0.049 / U:0.352 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site