All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Jan Beulich" <jbeulich@novell.com>
To: Guanqun Lu <guanqun.lu@intel.com>, Kevin Tian <kevin.tian@intel.com>
Cc: "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
	KeirFraser <keir.fraser@eu.citrix.com>
Subject: RE: [PATCH] small adjustment to asm	 constraintsforc/s19400
Date: Tue, 31 Mar 2009 12:30:02 +0100	[thread overview]
Message-ID: <49D21ADA.76E4.0078.0@novell.com> (raw)
In-Reply-To: <0A882F4D99BBF6449D58E61AAFD7EDD61018412B@pdsmsx502.ccr.corp.intel.com>

>>> "Tian, Kevin" <kevin.tian@intel.com> 31.03.09 13:09 >>>
>Thanks. After recovering those 3 SYSENTER MSRs, now 32-on-64
>S3 could succeeds several times which is a big step. However there
>seems to be some other instability issues to cause machine 
>occasionally reboot after more S3 actions. Both screen and serial
>port is dark and thus no more info is available. We'll continue to dig out 
>the exact reason. But this shouldn't prevent sending out this 
>reasonable fix anyway.

I think you need to be a little more careful here - subarch_percpu_traps_init()
writes them only on Intel CPUs, and I think you ought to follow this for
both saving and restoring them.

Jan

  reply	other threads:[~2009-03-31 11:30 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-27 16:35 [PATCH] small adjustment to asm constraints for c/s 19400 Jan Beulich
2009-03-30  4:45 ` Lu, Guanqun
2009-03-30  6:42   ` [PATCH] small adjustment to asm constraints for c/s19400 Jan Beulich
2009-03-30  8:58     ` Lu, Guanqun
2009-03-30  8:45   ` [PATCH] small adjustment to asm constraints for c/s 19400 Keir Fraser
2009-03-30  8:56     ` Lu, Guanqun
2009-03-30  9:13       ` [PATCH] small adjustment to asm constraints for c/s19400 Jan Beulich
2009-03-31  8:54         ` Lu, Guanqun
2009-03-31  9:05           ` Tian, Kevin
2009-03-31  9:24           ` [PATCH] small adjustment to asm constraints forc/s19400 Jan Beulich
2009-03-31  9:26             ` Tian, Kevin
2009-03-31  9:40               ` Jan Beulich
2009-03-31  9:47             ` Tian, Kevin
2009-03-31 10:15               ` Jan Beulich
2009-03-31 10:21                 ` Tian, Kevin
2009-03-31 10:54                   ` [PATCH] small adjustment to asm constraintsforc/s19400 Jan Beulich
2009-03-31 11:09                     ` Tian, Kevin
2009-03-31 11:30                       ` Jan Beulich [this message]
2009-03-31 11:35                         ` Tian, Kevin

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=49D21ADA.76E4.0078.0@novell.com \
    --to=jbeulich@novell.com \
    --cc=guanqun.lu@intel.com \
    --cc=keir.fraser@eu.citrix.com \
    --cc=kevin.tian@intel.com \
    --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.