From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
To: "Li, Liang Z" <liang.z.li@intel.com>
Cc: Kevin Wolf <kwolf@redhat.com>,
"qemu-block@nongnu.org" <qemu-block@nongnu.org>,
"quintela@redhat.com" <quintela@redhat.com>,
"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
"Zhang, Yang Z" <yang.z.zhang@intel.com>,
"amit.shah@redhat.com" <amit.shah@redhat.com>
Subject: Re: [Qemu-devel] [PATCH] migration: flush the bdrv before stopping VM
Date: Thu, 19 Mar 2015 14:40:18 +0000 [thread overview]
Message-ID: <20150319144018.GG2409@work-vm> (raw)
In-Reply-To: <F2CBF3009FA73547804AE4C663CAB28E4D2F95@shsmsx102.ccr.corp.intel.com>
* Li, Liang Z (liang.z.li@intel.com) wrote:
> > * Li, Liang Z (liang.z.li@intel.com) wrote:
> > > > > > First explanation, why I think this don't fix the full problem.
> > > > > > Whith this patch, we fix the problem where we have a dirty block
> > > > > > layer but basically nothing dirtying the memory on the guest (we
> > > > > > are moving the 20 seconds from max_downtime for the blocklayer
> > > > > > flush), to 20 seconds until we have decided that the amount of
> > > > > > dirty memory is small enough to be transferred during
> > > > > > max_downtime. But it is still going to take 20 seconds to flush
> > > > > > the block layer, and during that 20 seconds, the amount of memory
> > that can be dirty is HUGE.
> > > > >
> > > > > It's true.
> > > >
> > > > What kind of cache is it actually that takes 20s to flush here?
> > > >
> > >
> > > I run a script in the guest which do a dd operation, like this:
> > >
> > > #!/bin/sh
> > > for i in {1..1000000}
> > > do
> > > time dd if=/dev/zero of=/time.bdf bs=4k count=200000
> > > rm /time.bdf
> > > done
> > >
> > > It's an extreme case.
> >
> > With what qemu options for the device, and what was your device backed by?
>
> Very simple:
> ./qemu-system-x86_64 -enable-kvm -smp 4 -m 4096 -net none rhel6u5.img -monitor stdio
>
> And it's a local migration. I will do the test between two physical machines later.
OK, but for shared storage you would have to add cache=none (or something like that),
so that would change the behaviour anyway.
Dave
>
>
> Liang
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
prev parent reply other threads:[~2015-03-19 14:40 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-17 8:53 [Qemu-devel] [PATCH] migration: flush the bdrv before stopping VM Liang Li
2015-03-17 12:12 ` Juan Quintela
2015-03-18 3:19 ` Li, Liang Z
2015-03-18 11:17 ` Kevin Wolf
2015-03-18 12:36 ` Juan Quintela
2015-03-18 12:59 ` Paolo Bonzini
2015-03-18 13:42 ` Kevin Wolf
2015-03-20 7:22 ` Li, Liang Z
2015-03-25 10:50 ` Juan Quintela
2015-03-25 10:53 ` Kevin Wolf
2015-03-26 1:13 ` Li, Liang Z
2015-06-24 11:08 ` Li, Liang Z
2015-06-25 12:34 ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2015-03-18 13:39 ` [Qemu-devel] " Li, Liang Z
2015-03-18 16:55 ` Dr. David Alan Gilbert
2015-03-19 14:06 ` Li, Liang Z
2015-03-19 14:40 ` Dr. David Alan Gilbert [this message]
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=20150319144018.GG2409@work-vm \
--to=dgilbert@redhat.com \
--cc=amit.shah@redhat.com \
--cc=kwolf@redhat.com \
--cc=liang.z.li@intel.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=quintela@redhat.com \
--cc=yang.z.zhang@intel.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.