| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Tue, 5 Feb 2013 14:26:40 -0500 From: Johannes Weiner <hannes@...xchg.org> To: Zhang Yanfei <zhangyanfei.yes@...il.com> Cc: akpm@...ux-foundation.org, Linux MM <linux-mm@...ck.org>, mgorman@...e.de, minchan@...nel.org, kamezawa.hiroyu@...fujitsu.com, m.szyprowski@...sung.com, linux-kernel@...r.kernel.org Subject: Re: [PATCH 0/3] mm: rename confusing function names On Wed, Feb 06, 2013 at 01:09:55AM +0800, Zhang Yanfei wrote: > Function nr_free_zone_pages, nr_free_buffer_pages and nr_free_pagecache_pages > are horribly badly named, they count present_pages - pages_high within zones > instead of free pages, so why not rename them to reasonable names, not cofusing > people. > > patch2 and patch3 are based on patch1. So please apply patch1 first. > > Zhang Yanfei (3): > mm: rename nr_free_zone_pages to nr_free_zone_high_pages > mm: rename nr_free_buffer_pages to nr_free_buffer_high_pages > mm: rename nr_free_pagecache_pages to nr_free_pagecache_high_pages I don't feel that this is an improvement. As you said, the "free" is already misleading, because those pages might all be allocated. "High" makes me think not just of highmem, but drug abuse in general. nr_available_*_pages? I don't know, but if we go through with all that churn, it had better improve something. -- 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