linux-kernel - Re: [PATCH] node meminfo Active & Inactive pages to Kbytes

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 3 Jul 2008 15:42:30 -0700
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	John Blackwood <john.blackwood@...r.com>
Cc:	linux-kernel@...r.kernel.org, andi-suse@...stfloor.org,
	todd.allen@...r.com, stable@...nel.org
Subject: Re: [PATCH] node meminfo Active & Inactive pages to Kbytes

On Thu, 3 Jul 2008 10:35:57 -0400
John Blackwood <john.blackwood@...r.com> wrote:

> Hi Andrew and Andi,
> 
> We found a minor bug in the output of
> /sys/devices/system/node/node[n]/meminfo
> where the Active and Inactive values are in pages instead of Kbytes.
> 
> Looks like this occurred back in 2.6.20 when the code was changed
> over to use node_page_state().
> 
> Thanks.
> 
> --- linux.2.6.26-rc8/drivers/base/node.c	2008-07-03 09:26:01.000000000 -0400
> +++ new/drivers/base/node.c	2008-07-03 09:25:45.000000000 -0400
> @@ -84,8 +84,8 @@ static ssize_t node_read_meminfo(struct
>   		       nid, K(i.totalram),
>   		       nid, K(i.freeram),
>   		       nid, K(i.totalram - i.freeram),
> -		       nid, node_page_state(nid, NR_ACTIVE),
> -		       nid, node_page_state(nid, NR_INACTIVE),
> +		       nid, K(node_page_state(nid, NR_ACTIVE)),
> +		       nid, K(node_page_state(nid, NR_INACTIVE)),
>   #ifdef CONFIG_HIGHMEM
>   		       nid, K(i.totalhigh),
>   		       nid, K(i.freehigh),

ug.  After all that time there might be applications out there which
use the existing numbers and work around the kernel bug.  Which we'll
break if we fix it.

otoh, who the heck cares about the per-node active and inactive page
counts?

I guess we just fix this, backport the fix as far as we can and
apologise for any fallout.  Not that there's likely to be any.


Please do remember to add a Signed-off-by: for kernel patches.

--
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/