Re: + vmscan-rename-functions.patch added to -mm tree
From: Nick Piggin
Date: Tue Feb 14 2006 - 22:17:04 EST
Coywolf Qi Hunt wrote:
2006/2/12, akpm@xxxxxxxx <akpm@xxxxxxxx>:
The patch titled
vmscan: rename functions
has been added to the -mm tree. Its filename is
vmscan-rename-functions.patch
See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find
out what to do about this
From: Andrew Morton <akpm@xxxxxxxx>
We have:
try_to_free_pages
->shrink_caches(struct zone **zones, ..)
->shrink_zone(struct zone *, ...)
->shrink_cache(struct zone *, ...)
->shrink_list(struct list_head *, ...)
which is fairly irrational.
Rename things so that we have
try_to_free_pages
->shrink_zones(struct zone **zones, ..)
->shrink_zone(struct zone *, ...)
->do_shrink_zone(struct zone *, ...)
->shrink_page_list(struct list_head *, ...)
Every time I read this part it annoys me. Thanks.
I don't much care, but if there is renaming afoot, I'd vote for
->shrink_zones(struct zone **zones, ..)
->shrink_zone(struct zone *, ...)
->shrink_inactive_list(struct zone *, ...)
->shrink_page_list(struct list_head *, ...)
->shrink_active_list (alternatively, leave as refill_inactive_list)
shrink_zone and do_shrink_zone don't really say any more to me than
shrink_zone and shrink_cache.
--
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/