From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
To: John Snow <jsnow@redhat.com>
Cc: pbonzini@redhat.com, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 2/2] atapi migration: Throw recoverable error to avoid recovery
Date: Wed, 10 Dec 2014 20:09:13 +0000 [thread overview]
Message-ID: <20141210200912.GN2311@work-vm> (raw)
In-Reply-To: <5487E4C8.4050102@redhat.com>
* John Snow (jsnow@redhat.com) wrote:
> What makes me the most curious is how this plays out in Windows if this case
> is triggered. Throw a trace around the fake error and see if you can't
> observe it getting called during a pingpong test while Windows reads a CD.
I've not managed to trigger it on Windows (2012).
I've run InfraRecorder's track test with heavy migration and it
doesn't seem to trigger this case, or cause any extra errors.
(Curiously it does complain about not being able to read extended
error information and say our CDROM is giving about 35% C2 errors -
even without migration; but I think this is just bogus junk it's
picking up, although feel free to polish the scratches out of
our virtual CD).
Dave
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
next prev parent reply other threads:[~2014-12-10 20:09 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-09 18:15 [Qemu-devel] [PATCH 0/2] ATAPI migration fix/hack Dr. David Alan Gilbert (git)
2014-12-09 18:15 ` [Qemu-devel] [PATCH 1/2] Restore atapi_dma flag across migration Dr. David Alan Gilbert (git)
2014-12-10 5:04 ` John Snow
2014-12-09 18:15 ` [Qemu-devel] [PATCH 2/2] atapi migration: Throw recoverable error to avoid recovery Dr. David Alan Gilbert (git)
2014-12-10 6:14 ` John Snow
2014-12-10 12:05 ` Dr. David Alan Gilbert
2014-12-10 20:09 ` Dr. David Alan Gilbert [this message]
2014-12-10 22:04 ` Paolo Bonzini
2014-12-11 19:45 ` Dr. David Alan Gilbert
2014-12-18 19:39 ` Dr. David Alan Gilbert
2014-12-18 23:42 ` John Snow
2015-01-16 17:28 ` John Snow
2015-02-02 12:11 ` Kevin Wolf
2015-01-07 16:26 ` [Qemu-devel] [PATCH 0/2] ATAPI migration fix/hack Dr. David Alan Gilbert
2015-01-30 16:07 ` Dr. David Alan Gilbert
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=20141210200912.GN2311@work-vm \
--to=dgilbert@redhat.com \
--cc=jsnow@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).