[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20090615141912.878168ca.akpm@linux-foundation.org>
Date: Mon, 15 Jun 2009 14:19:12 -0700
From: Andrew Morton <akpm@...ux-foundation.org>
To: Mel Gorman <mel@....ul.ie>
Cc: mel@....ul.ie, kosaki.motohiro@...fujitsu.com, riel@...hat.com,
cl@...ux-foundation.org, fengguang.wu@...el.com,
linuxram@...ibm.com, linux-mm@...ck.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 3/3] Count the number of times zone_reclaim() scans and
fails
On Thu, 11 Jun 2009 11:47:53 +0100
Mel Gorman <mel@....ul.ie> wrote:
> + PGSCAN_ZONERECLAIM_FAILED,
> @@ -2492,6 +2492,9 @@ int zone_reclaim(struct zone *zone, gfp_t gfp_mask, unsigned int order)
> + "zreclaim_failed",
So we have "zonereclaim", "zone_reclaim" and "zreclaim", which isn't
terribly developer-friendly.
This?
--- a/include/linux/vmstat.h~vmscan-count-the-number-of-times-zone_reclaim-scans-and-fails-fix
+++ a/include/linux/vmstat.h
@@ -37,7 +37,7 @@ enum vm_event_item { PGPGIN, PGPGOUT, PS
FOR_ALL_ZONES(PGSCAN_KSWAPD),
FOR_ALL_ZONES(PGSCAN_DIRECT),
#ifdef CONFIG_NUMA
- PGSCAN_ZONERECLAIM_FAILED,
+ PGSCAN_ZONE_RECLAIM_FAILED,
#endif
PGINODESTEAL, SLABS_SCANNED, KSWAPD_STEAL, KSWAPD_INODESTEAL,
PAGEOUTRUN, ALLOCSTALL, PGROTATED,
diff -puN mm/vmscan.c~vmscan-count-the-number-of-times-zone_reclaim-scans-and-fails-fix mm/vmscan.c
--- a/mm/vmscan.c~vmscan-count-the-number-of-times-zone_reclaim-scans-and-fails-fix
+++ a/mm/vmscan.c
@@ -2520,7 +2520,7 @@ int zone_reclaim(struct zone *zone, gfp_
zone_clear_flag(zone, ZONE_RECLAIM_LOCKED);
if (!ret)
- count_vm_event(PGSCAN_ZONERECLAIM_FAILED);
+ count_vm_event(PGSCAN_ZONE_RECLAIM_FAILED);
return ret;
}
diff -puN mm/vmstat.c~vmscan-count-the-number-of-times-zone_reclaim-scans-and-fails-fix mm/vmstat.c
--- a/mm/vmstat.c~vmscan-count-the-number-of-times-zone_reclaim-scans-and-fails-fix
+++ a/mm/vmstat.c
@@ -674,7 +674,7 @@ static const char * const vmstat_text[]
TEXTS_FOR_ZONES("pgscan_direct")
#ifdef CONFIG_NUMA
- "zreclaim_failed",
+ "zone_reclaim_failed",
#endif
"pginodesteal",
"slabs_scanned",
_
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists