All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Daniel P. Berrange" <berrange@redhat.com>
To: Keir Fraser <Keir.Fraser@cl.cam.ac.uk>
Cc: xen-devel <xen-devel@lists.xensource.com>
Subject: Re: Xend transaction reduction breaks migration
Date: Thu, 15 Nov 2007 18:39:20 +0000	[thread overview]
Message-ID: <20071115183920.GC16499@redhat.com> (raw)
In-Reply-To: <C36138EA.187A2%Keir.Fraser@cl.cam.ac.uk>

On Wed, Nov 14, 2007 at 11:44:42PM +0000, Keir Fraser wrote:
> Hey Daniel,
> 
> It turns out that your changeset 16260 in xen-unstable breaks our localhost
> non-live migration tests. What happens is that randomly (but fairly often)
> xend will throw an assertion at line 'assert console_port' in
> XendCheckpoint.py. This is in the domain_restore_fd context, so result is
> that the restorer dies and then of course the saver fails soon after since
> the migration socket goes away.
> 
> Any idea how your patch could cause this rather bizarre behaviour? I haven't
> worked it out myself, though I'm 99% sure it *is* that changeset that has
> caused this regression (based on tests with the changeset reverted).

Basically, instead of doing many small transactions, my change made some stuff
all happen in one large transaction. Oddly though, I tried to be careful so
that I only changed the codepath for the 'xm list' operation (well the SEXPR
call to the list operation). Other operations like create, save, restore
ought to be left using fine grained transactions as before.

Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=| 

  reply	other threads:[~2007-11-15 18:39 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-14 23:44 Xend transaction reduction breaks migration Keir Fraser
2007-11-15 18:39 ` Daniel P. Berrange [this message]
2007-11-15 18:46   ` Keir Fraser
2007-11-15 22:11     ` Keir Fraser
2007-11-16 14:00       ` Daniel P. Berrange

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20071115183920.GC16499@redhat.com \
    --to=berrange@redhat.com \
    --cc=Keir.Fraser@cl.cam.ac.uk \
    --cc=xen-devel@lists.xensource.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.