From: Sarah Newman <srn@prgmr.com>
To: Jan Beulich <JBeulich@suse.com>
Cc: xen-devel <xen-devel@lists.xenproject.org>,
Luke Crawford <lsc@prgmr.com>
Subject: Re: [PATCH] x86/fpu: CR0.TS should be set before trap into PV guest's #NM exception handle
Date: Fri, 29 Nov 2013 16:16:10 -0800 [thread overview]
Message-ID: <52992E4A.3050109@prgmr.com> (raw)
In-Reply-To: <52986F90020000780010814A@nat28.tlf.novell.com>
On 11/29/2013 01:42 AM, Jan Beulich wrote:
> I'm personally not eager to see something like this go in, most
> importantly because I think bugs should be fixed where they
> got introduced, not worked around elsewhere, and also
> because I think that a generally available workaround would
> lower the chances of the bug getting fixed properly.
If the proper bug fix was released first would that help?
>
> Apart from that this workaround of yours would have very
> ugly behavior: If a guest later got updated to a fixed kernel,
> you'd have to alter its configuration along with booting into
> the new kernel (i.e. a simple reboot of the VM won't do), or
> else your new kernel would crash due to not clearing CR0.TS
> before trying to restore FPU context.
Assuming it predictably crashed, that would be easier to deal with than occasional silent memory
corruption.
I think something could be done with the on_reboot action, maybe a reload-restart?
next prev parent reply other threads:[~2013-11-30 0:16 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <52967C45.3030506@prgmr.com>
2013-11-27 23:35 ` [PATCH] x86/fpu: CR0.TS should be set before trap into PV guest's #NM exception handle Sarah Newman
2013-11-29 9:42 ` Jan Beulich
2013-11-30 0:16 ` Sarah Newman [this message]
2013-12-02 8:34 ` Jan Beulich
2013-12-02 9:58 ` Luke S. Crawford
2013-12-02 10:04 ` Jan Beulich
2013-12-02 10:25 ` Luke S. Crawford
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=52992E4A.3050109@prgmr.com \
--to=srn@prgmr.com \
--cc=JBeulich@suse.com \
--cc=lsc@prgmr.com \
--cc=xen-devel@lists.xenproject.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 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.