All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chris Lalancette <clalance@redhat.com>
To: Dan Magenheimer <dan.magenheimer@oracle.com>
Cc: Jeremy Fitzhardinge <jeremy@goop.org>,
	"Xen-Devel (E-mail)" <xen-devel@lists.xensource.com>,
	Keir Fraser <keir.fraser@eu.citrix.com>
Subject: Re: write_tsc in a PV domain?
Date: Thu, 27 Aug 2009 15:17:22 +0200	[thread overview]
Message-ID: <4A968762.2090804@redhat.com> (raw)
In-Reply-To: <5637cad0-4cd7-448e-8f72-23fb1b60d69a@default>

Dan Magenheimer wrote:
>> Dan Magenheimer wrote:
>>> In that case, are you saying it is an illegal instruction for a PV
>>> guest to execute?  If so, we should not ignore it, we should fail
>>> the guest.  But that would be unfortunate for the RHEL5-64bit
>>> PV guests that actually DO use it.
>> Wait, what?  Could you point out where this is in RHEL-5 
>> 64-bit PV?  The only
>> case of write_tsc() I see in the code is in 
>> arch/i386/kernel/smpboot.c, which is
>> not used by the Xen PV implementation in RHEL-5.  Where else in the PV
>> implementation does a write_tsc?
> 
> Hi Chris --
> 
> I was surprised also, and digging deeper it looks like I was mistaken.
> 
> I instrumented a hypervisor so that Xen would printk a console
> message if it was ignoring a wrmsr and was getting output
> when I launched a RHEL-5 PV guest.  But I refined the
> printk and it is NOT wrmsr(0x10) so you're right, it is
> NOT a write_tsc.
> 
> Thanks for pointing out my error.

OK, cool, no problem.  I just wanted to make sure I wasn't missing something.

Thanks,
-- 
Chris Lalancette

  reply	other threads:[~2009-08-27 13:17 UTC|newest]

Thread overview: 61+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-25 21:54 write_tsc in a PV domain? Dan Magenheimer
2009-08-25 22:28 ` Jeremy Fitzhardinge
2009-08-25 23:09   ` Dan Magenheimer
2009-08-26  6:23     ` Keir Fraser
2009-08-26 15:42       ` Dan Magenheimer
2009-08-26 15:58         ` Keir Fraser
2009-08-26 19:45         ` Jeremy Fitzhardinge
2009-08-26 20:23           ` Dan Magenheimer
2009-08-26 22:30             ` Jeremy Fitzhardinge
2009-08-26 23:10               ` Dan Magenheimer
2009-08-27  8:39                 ` Chris Lalancette
2009-08-27 13:00                   ` Dan Magenheimer
2009-08-27 13:17                     ` Chris Lalancette [this message]
2009-08-27  8:48               ` Alan Cox
2009-08-27 19:10                 ` Jeremy Fitzhardinge
2009-08-28  3:29                   ` Dan Magenheimer
2009-08-28  9:49                     ` Alan Cox
2009-08-28 15:16                       ` Dan Magenheimer
2009-08-28 15:30                         ` Alan Cox
2009-08-28 17:49                           ` rdtsc: correctness vs performance on Xen (and KVM?) Dan Magenheimer
2009-08-31 23:52                             ` Dan Magenheimer
2009-09-01  0:22                               ` Jeremy Fitzhardinge
2009-09-01 13:54                                 ` Dan Magenheimer
2009-09-01 14:34                                   ` Keir Fraser
2009-09-01 14:53                                     ` Dan Magenheimer
2009-09-01 15:08                                       ` Keir Fraser
2009-09-01 15:26                                         ` Dan Magenheimer
2009-09-01 15:32                                           ` Jan Beulich
2009-09-01 15:56                                             ` Dan Magenheimer
2009-09-01 16:04                                               ` Jan Beulich
2009-09-01 16:41                                                 ` Dan Magenheimer
2009-09-02  7:05                                                   ` Jan Beulich
2009-09-01 21:25                                                 ` Keir Fraser
2009-09-01 22:08                                                   ` Dan Magenheimer
2009-09-01 22:21                                                     ` Jeremy Fitzhardinge
2009-09-01 22:41                                                       ` Dan Magenheimer
2009-09-01 23:26                                                         ` Jeremy Fitzhardinge
2009-09-02  7:20                                                           ` Keir Fraser
2009-09-02 21:44                                                             ` Jeremy Fitzhardinge
2009-09-02 21:50                                                               ` Keir Fraser
2009-09-02 22:05                                                                 ` Jeremy Fitzhardinge
2009-09-03  8:23                                                                   ` Jan Beulich
2009-09-03 17:29                                                                     ` Jeremy Fitzhardinge
2009-09-04  7:19                                                                       ` Jan Beulich
2009-09-04 15:44                                                                         ` Jeremy Fitzhardinge
2009-09-03 14:22                                                                   ` Dan Magenheimer
2009-09-02  7:16                                                     ` Jan Beulich
2009-09-02  7:01                                                   ` Jan Beulich
2009-09-01 16:06                                               ` Keir Fraser
2009-09-01 16:55                                                 ` Dan Magenheimer
2009-09-01 15:43                                           ` Keir Fraser
2009-08-28 17:49                           ` write_tsc in a PV domain? Dan Magenheimer
2009-08-28 17:02                     ` Jeremy Fitzhardinge
2009-08-28 17:49                       ` Dan Magenheimer
2009-08-28 23:01                         ` Jeremy Fitzhardinge
2009-08-29 17:51                           ` Dan Magenheimer
2009-08-31 18:11                             ` Dan Magenheimer
2009-08-31 19:06                               ` Keir Fraser
2009-08-31 21:06                                 ` Dan Magenheimer
2009-09-01  7:16                                   ` Keir Fraser
2009-08-31 19:18                               ` Jeremy Fitzhardinge

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=4A968762.2090804@redhat.com \
    --to=clalance@redhat.com \
    --cc=dan.magenheimer@oracle.com \
    --cc=jeremy@goop.org \
    --cc=keir.fraser@eu.citrix.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.