All of lore.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <gregkh@linuxfoundation.org>
To: Olaf Hering <olaf@aepfle.de>
Cc: linux-kernel@vger.kernel.org, stable@kernel.org,
	KY Srinivasan <kys@microsoft.com>
Subject: Re: [PATCH] x86: Hyper-V: register clocksource only if its advertised
Date: Fri, 25 Jan 2013 08:10:16 -0800	[thread overview]
Message-ID: <20130125161016.GA9906@kroah.com> (raw)
In-Reply-To: <1359121036-16309-1-git-send-email-olaf@aepfle.de>

On Fri, Jan 25, 2013 at 02:37:16PM +0100, Olaf Hering wrote:
> Enable hyperv_clocksource only if its advertised as a feature.
> XenServer 6 returns the signature which is checked in
> ms_hyperv_platform(), but it does not offer all features. Currently the
> clocksource is enabled unconditionally in ms_hyperv_init_platform(), and
> the result is a hanging guest.
> 
> Hyper-V spec Bit 1 indicates the availability of Partition Reference
> Counter.  Register the clocksource only if this bit is set.
> 
> The guest in question prints this in dmesg:
>  [    0.000000] Hypervisor detected: Microsoft HyperV
>  [    0.000000] HyperV: features 0x70, hints 0x0
> 
> This bug can be reproduced easily be setting 'viridian=1' in a HVM domU
> .cfg file. A workaround without this patch is to boot the HVM guest with
> 'clocksource=jiffies'.
> 
> Signed-off-by: Olaf Hering <olaf@aepfle.de>
> Cc: stable@kernel.org
> Cc: KY Srinivasan <kys@microsoft.com>
> Cc: Greg KH <gregkh@linuxfoundation.org>
> ---
>  arch/x86/kernel/cpu/mshyperv.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

This needs to go through the x86 maintainers, not me, sorry.

greg k-h

  reply	other threads:[~2013-01-25 16:10 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-25 13:37 [PATCH] x86: Hyper-V: register clocksource only if its advertised Olaf Hering
2013-01-25 16:10 ` Greg KH [this message]
2013-01-25 16:43 ` KY Srinivasan
2013-01-25 16:54   ` Olaf Hering
2013-01-25 17:04     ` KY Srinivasan
2013-01-25 17:19       ` Olaf Hering
2013-01-25 17:39         ` KY Srinivasan
2013-01-25 20:00           ` Olaf Hering
2013-01-25 20:03             ` KY Srinivasan
2013-01-26 16:57             ` KY Srinivasan
2013-01-26 18:21               ` Olaf Hering
2013-01-28 17:44           ` Stefano Stabellini
2013-01-29  8:35             ` Jan Beulich
2013-01-29  9:26               ` Olaf Hering
2013-01-29 14:32                 ` KY Srinivasan
2013-01-29 15:30                   ` Stefano Stabellini
2013-01-29 15:58                     ` KY Srinivasan
2013-01-29 16:14                       ` Stefano Stabellini

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=20130125161016.GA9906@kroah.com \
    --to=gregkh@linuxfoundation.org \
    --cc=kys@microsoft.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=olaf@aepfle.de \
    --cc=stable@kernel.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.