[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] augment info migrate with page status
From: |
Glauber Costa |
Subject: |
Re: [Qemu-devel] [PATCH] augment info migrate with page status |
Date: |
Thu, 21 May 2009 13:07:50 -0300 |
User-agent: |
Mutt/1.5.18 (2008-05-17) |
On Thu, May 21, 2009 at 04:04:57PM +0100, Daniel P. Berrange wrote:
> On Thu, May 21, 2009 at 05:58:10PM +0300, Avi Kivity wrote:
> > Daniel P. Berrange wrote:
> > >On Wed, May 20, 2009 at 07:20:05PM -0400, Glauber Costa wrote:
> > >
> > >>This patch augments info migrate output with status about:
> > >>* pages remaining
> > >>* pages transferred
> > >>
> > >
> > >Could you add 'pages total' and/or possibly 'page size', so apps
> > >using this can more easily do calculations, without having to worry
> > >about hardcoding page sizes for each architecture they deal with.
> > >
> > >Oh, what happens if the guest is backed by huge pages ? Does the
> > >'pages transferred' always count in terms of 4k pages, even if
> > >2 M & 1 GB pages are in use ?
> > >
> >
> > We should just use bytes. I don't see how backing page size matters.
>
> That works for me, because ultimately bytes is what I'd want at libvirt
> layer.
agreed.