[fedora-virt] [PATCH] memory taken by zero qemu processes should be 0

Justin M. Forbes jmforbes at linuxtx.org
Fri Sep 17 13:44:54 UTC 2010


On Fri, 2010-09-17 at 15:17 +0200, Dan Kenigsberg wrote:
> On Thu, Aug 26, 2010 at 11:43:01AM -0500, Justin M. Forbes wrote:
> > On Thu, 2010-08-26 at 11:23 +0300, Dan Kenigsberg wrote:
> > > Former code behaved badly when `pgrep qemu` was empty.
> > > 
> > > Thanks to lihuang of BZ#609016
> > > ---
> > >  ksmtuned |    4 ++--
> > >  1 files changed, 2 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/ksmtuned b/ksmtuned
> > > index 205ff84..d5d1494 100644
> > > --- a/ksmtuned
> > > +++ b/ksmtuned
> > > @@ -71,8 +71,8 @@ KSMCTL () {
> > >  committed_memory () {
> > >      # calculate how much memory is committed to running qemu processes
> > >      local progname
> > > -    progname=${1:-qemu}
> > > -    ps -o vsz `pgrep $progname` | awk '{ sum += $1 }; END { print sum }'
> > > +    progname=${1:-qemu-kvm}
> > > +    ps -C "$progname" -o rsz | awk '{ sum += $1 }; END { print sum }'
> > >  }
> > >  
> > >  free_memory () {
> > 
> > Applied upstream. Will make the next Fedora packages.
> 
> I hate to nag, but why isn't it here
> http://pkgs.fedoraproject.org/gitweb/?p=qemu.git;a=summary  yet?

It got overwritten when I did the base update.  I realized it as soon as
I did the commit, but knew there would be another update very soon (as
in the next day or two), so didn't see the need to make another quick
build for just that change.

Justin



More information about the virt mailing list