[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [PATCH] fix endless look at kvm migration
From: |
Glauber Costa |
Subject: |
Re: [Qemu-devel] Re: [PATCH] fix endless look at kvm migration |
Date: |
Mon, 27 Jul 2009 17:36:17 -0300 |
User-agent: |
Jack Bauer |
On Mon, Jul 27, 2009 at 05:22:51PM -0300, Luiz Capitulino wrote:
> On Mon, 27 Jul 2009 17:21:34 -0300
> Glauber Costa <address@hidden> wrote:
>
> > On Mon, Jul 27, 2009 at 08:44:59PM +0200, Jan Kiszka wrote:
> > > Glauber Costa wrote:
> > > > If we try to sync bitmaps withouth KVM_MEM_LOG_DIRTY_PAGES being set
> > > > on a slot, we will enter an endless loop. Remove the test, since it
> > > > is not strictly needed anyway
> > > >
> > > > Signed-off-by: Glauber Costa <address@hidden>
> > > > CC: Alexander Graf <address@hidden>
> > > > ---
> > > > kvm-all.c | 5 -----
> > > > 1 files changed, 0 insertions(+), 5 deletions(-)
> > > >
> > > > diff --git a/kvm-all.c b/kvm-all.c
> > > > index 824bb4c..d3bbd3c 100644
> > > > --- a/kvm-all.c
> > > > +++ b/kvm-all.c
> > > > @@ -331,11 +331,6 @@ int
> > > > kvm_physical_sync_dirty_bitmap(target_phys_addr_t start_addr,
> > > > break;
> > > > }
> > > >
> > > > - /* We didn't activate dirty logging? Don't care then. */
> > > > - if(!(mem->flags & KVM_MEM_LOG_DIRTY_PAGES)) {
> > > > - continue;
> > > > - }
> > > > -
> > > > size = ((mem->memory_size >> TARGET_PAGE_BITS) + 7) / 8;
> > > > if (!d.dirty_bitmap) {
> > > > d.dirty_bitmap = qemu_malloc(size);
> > >
> > > Please let us revert the whole bd83677.
> > I'm not opposed either. Avi can surely do that if he prefer.
>
> As you have submitted the if() removal fix, you could submit
> the revert as well, so that we keep this going.
Just that I don't see a point. Reverting a patch is a matter of doing
a "git revert <treeish>"
That's up to the maintainer to do. And btw, I totally forgot that we're
in qemu list here ;-) It's Anthony, not avi.