> -----Original Message----- > From: David Rientjes [mailto:[email protected]] > Sent: Monday, November 12, 2012 4:54 PM > To: KY Srinivasan > Cc: [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected] > 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. Regards, K. Y > _______________________________________________ devel mailing list [email protected] http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
