Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756106AbaFYUQ0 (ORCPT ); Wed, 25 Jun 2014 16:16:26 -0400 Received: from mx1.redhat.com ([209.132.183.28]:37925 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755330AbaFYUQX (ORCPT ); Wed, 25 Jun 2014 16:16:23 -0400 Date: Wed, 25 Jun 2014 16:16:04 -0400 From: Rafael Aquini To: Motohiro Kosaki Cc: "linux-mm@kvack.org" , Andrew Morton , Rik van Riel , Mel Gorman , Johannes Weiner , Motohiro Kosaki JP , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH] mm: export NR_SHMEM via sysinfo(2) / si_meminfo() interfaces Message-ID: <20140625201603.GA1534@t510.redhat.com> References: <4b46c5b21263c446923caf3da3f0dca6febc7b55.1403709665.git.aquini@redhat.com> <6B2BA408B38BA1478B473C31C3D2074E341D585464@SV-EXCHANGE1.Corp.FC.LOCAL> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <6B2BA408B38BA1478B473C31C3D2074E341D585464@SV-EXCHANGE1.Corp.FC.LOCAL> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jun 25, 2014 at 12:41:17PM -0700, Motohiro Kosaki wrote: > > > > -----Original Message----- > > From: Rafael Aquini [mailto:aquini@redhat.com] > > Sent: Wednesday, June 25, 2014 2:40 PM > > To: linux-mm@kvack.org > > Cc: Andrew Morton; Rik van Riel; Mel Gorman; Johannes Weiner; Motohiro Kosaki JP; linux-kernel@vger.kernel.org > > Subject: [PATCH] mm: export NR_SHMEM via sysinfo(2) / si_meminfo() interfaces > > > > This patch leverages the addition of explicit accounting for pages used by shmem/tmpfs -- "4b02108 mm: oom analysis: add shmem > > vmstat" -- in order to make the users of sysinfo(2) and si_meminfo*() friends aware of that vmstat entry consistently across the > > interfaces. > > Why? Because we do not report consistently across the interfaces we declare exporting that data. Check sysinfo(2) manpage, for instance: [...] struct sysinfo { long uptime; /* Seconds since boot */ unsigned long loads[3]; /* 1, 5, and 15 minute load averages */ unsigned long totalram; /* Total usable main memory size */ unsigned long freeram; /* Available memory size */ unsigned long sharedram; /* Amount of shared memory */ <<<<< [...] userspace tools resorting to sysinfo() syscall will get a hardcoded 0 for shared memory which is reported differently from /proc/meminfo. Also, si_meminfo() & si_meminfo_node() are utilized within the kernel to gather statistics for /proc/meminfo & friends, and so we can leverage collecting sharedmem from those calls as well, just as we do for totalram, freeram & bufferram. Regards, -- Rafael > Traditionally sysinfo.sharedram was not used for shmem. It was totally strange semantics and completely outdated feature. > So, we may reuse it for another purpose. But I'm not sure its benefit. > > Why don't you use /proc/meminfo? > I'm afraid userland programs get a confusion. > > > > > > Signed-off-by: Rafael Aquini > > --- > > drivers/base/node.c | 2 +- > > fs/proc/meminfo.c | 2 +- > > mm/page_alloc.c | 3 ++- > > 3 files changed, 4 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/base/node.c b/drivers/base/node.c index 8f7ed99..c6d3ae0 100644 > > --- a/drivers/base/node.c > > +++ b/drivers/base/node.c > > @@ -126,7 +126,7 @@ static ssize_t node_read_meminfo(struct device *dev, > > nid, K(node_page_state(nid, NR_FILE_PAGES)), > > nid, K(node_page_state(nid, NR_FILE_MAPPED)), > > nid, K(node_page_state(nid, NR_ANON_PAGES)), > > - nid, K(node_page_state(nid, NR_SHMEM)), > > + nid, K(i.sharedram), > > nid, node_page_state(nid, NR_KERNEL_STACK) * > > THREAD_SIZE / 1024, > > nid, K(node_page_state(nid, NR_PAGETABLE)), diff --git a/fs/proc/meminfo.c b/fs/proc/meminfo.c index > > 7445af0..aa1eee0 100644 > > --- a/fs/proc/meminfo.c > > +++ b/fs/proc/meminfo.c > > @@ -168,7 +168,7 @@ static int meminfo_proc_show(struct seq_file *m, void *v) > > K(global_page_state(NR_WRITEBACK)), > > K(global_page_state(NR_ANON_PAGES)), > > K(global_page_state(NR_FILE_MAPPED)), > > - K(global_page_state(NR_SHMEM)), > > + K(i.sharedram), > > K(global_page_state(NR_SLAB_RECLAIMABLE) + > > global_page_state(NR_SLAB_UNRECLAIMABLE)), > > K(global_page_state(NR_SLAB_RECLAIMABLE)), > > diff --git a/mm/page_alloc.c b/mm/page_alloc.c index 20d17f8..f72ea38 100644 > > --- a/mm/page_alloc.c > > +++ b/mm/page_alloc.c > > @@ -3040,7 +3040,7 @@ static inline void show_node(struct zone *zone) void si_meminfo(struct sysinfo *val) { > > val->totalram = totalram_pages; > > - val->sharedram = 0; > > + val->sharedram = global_page_state(NR_SHMEM); > > val->freeram = global_page_state(NR_FREE_PAGES); > > val->bufferram = nr_blockdev_pages(); > > val->totalhigh = totalhigh_pages; > > @@ -3060,6 +3060,7 @@ void si_meminfo_node(struct sysinfo *val, int nid) > > for (zone_type = 0; zone_type < MAX_NR_ZONES; zone_type++) > > managed_pages += pgdat->node_zones[zone_type].managed_pages; > > val->totalram = managed_pages; > > + val->sharedram = node_page_state(nid, NR_SHMEM); > > val->freeram = node_page_state(nid, NR_FREE_PAGES); #ifdef CONFIG_HIGHMEM > > val->totalhigh = pgdat->node_zones[ZONE_HIGHMEM].managed_pages; > > -- > > 1.9.3 > -- 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/