linux-kernel - RE: [PATCH 1/1] mm: Export a function to read vm_committed

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]

Message-ID: <c04bb062-bbce-4980-b2b3-fbbb18e64b66@default>
Date:	Mon, 12 Nov 2012 15:32:11 -0800 (PST)
From:	Dan Magenheimer <dan.magenheimer@...cle.com>
To:	KY Srinivasan <kys@...rosoft.com>,
	David Rientjes <rientjes@...gle.com>,
	Konrad Wilk <konrad.wilk@...cle.com>
Cc:	gregkh@...uxfoundation.org, linux-kernel@...r.kernel.org,
	devel@...uxdriverproject.org, olaf@...fle.de, apw@...onical.com,
	andi@...stfloor.org, akpm@...ux-foundation.org, linux-mm@...ck.org,
	kamezawa.hiroyuki@...il.com, mhocko@...e.cz, hannes@...xchg.org,
	yinghan@...gle.com
Subject: RE: [PATCH 1/1] mm: Export a function to read vm_committed_as

> From: KY Srinivasan [mailto:kys@...rosoft.com]
> Sent: Monday, November 12, 2012 3:58 PM
> To: David Rientjes
> Cc: gregkh@...uxfoundation.org; linux-kernel@...r.kernel.org; devel@...uxdriverproject.org;
> olaf@...fle.de; apw@...onical.com; andi@...stfloor.org; akpm@...ux-foundation.org; linux-mm@...ck.org;
> kamezawa.hiroyuki@...il.com; mhocko@...e.cz; hannes@...xchg.org; yinghan@...gle.com; Dan Magenheimer
> Subject: RE: [PATCH 1/1] mm: Export a function to read vm_committed_as
> 
> > -----Original Message-----
> > From: David Rientjes [mailto:rientjes@...gle.com]
> > Sent: Monday, November 12, 2012 4:54 PM
> > To: KY Srinivasan
> > Cc: gregkh@...uxfoundation.org; linux-kernel@...r.kernel.org;
> > devel@...uxdriverproject.org; olaf@...fle.de; apw@...onical.com;
> > andi@...stfloor.org; akpm@...ux-foundation.org; linux-mm@...ck.org;
> > kamezawa.hiroyuki@...il.com; mhocko@...e.cz; hannes@...xchg.org;
> > yinghan@...gle.com
> > Subject: RE: [PATCH 1/1] mm: Export a function to read vm_committed_as
> >
> > On Sun, 11 Nov 2012, KY Srinivasan wrote:
> >
> > > Thanks for the prompt response. For the Linux balloon driver for Hyper-V, I
> > need access
> > > to the metric that reflects the system wide memory commitment made by the
> > guest kernel.
> > > In the Hyper-V case, this information is one of the many metrics used to drive
> > the policy engine
> > > on the host. Granted, the interface name I have chosen here could be more
> > generic; how about
> > > read_mem_commit_info(void). I am open to suggestions here.
> > >
> >
> > I would suggest vm_memory_committed() and there shouldn't be a comment
> > describing that this is just a wrapper for modules to read
> > vm_committed_as, that's apparent from the implementation: it should be
> > describing exactly what this value represents and why it is a useful
> > metric (at least in the case that you're concerned about).
> 
> Will do; thanks.
> >
> > > With regards to making changes to the Xen self ballooning code, I would like to
> > separate that patch
> > > from the patch that implements the exported mechanism to access the
> > memory commitment information.
> >
> > Why?  Is xen using it for a different inference?
> 
> I think it is good to separate these patches. Dan (copied here) wrote the code for the
> Xen self balloon driver. If it is ok with him I can submit the patch for Xen as well.

Hi KY --

If I understand correctly, this would be only a cosmetic (function renaming) change
to the Xen selfballooning code.  If so, then I will be happy to Ack when I
see the patch.  However, Konrad (konrad.wilk@...cle.com) is the maintainer
for all Xen code so you should ask him... and (from previous painful experience)
it can be difficult to sync even very simple interdependent changes going through
different maintainers without breaking linux-next.  So I can't offer any
help with that process, only commiseration. :-(

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