public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jeremy Fitzhardinge <jeremy@goop.org>
To: "H. Peter Anvin" <hpa@zytor.com>
Cc: Borislav Petkov <bp@amd64.org>,
	the arch/x86 maintainers <x86@kernel.org>,
	Tigran Aivazian <tigran@aivazian.fsnet.co.uk>,
	Xen Devel <xen-devel@lists.xensource.com>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>,
	Ingo Molnar <mingo@elte.hu>, Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [PATCH 0/3] x86/microcode: support for microcode update in Xen dom0
Date: Wed, 12 Oct 2011 12:18:22 -0700	[thread overview]
Message-ID: <4E95E7FE.6050302@goop.org> (raw)
In-Reply-To: <4E95D9E7.6090304@zytor.com>

On 10/12/2011 11:18 AM, H. Peter Anvin wrote:
> On 10/12/2011 03:16 AM, Borislav Petkov wrote:
>> Why would a xen microcode solution would even be contingent on how
>> upstream Linux solves it (and when)?
>>
> Agreed... especially since Xen is "special" when it comes to booting anyway.

"Special" only in the sense that you need a new line in grub.

But doing boot-time microcode is a lot more "special" since it would
require distros to move the microcode files to /boot.  That would either
mean having two sets of microcode packages, or also changing the way
that Linux does its microcode loading (if only by changing the paths,
but I have no idea what that implies since its all tied up with the rest
of the firmware loading stuff).

The whole thrust of the Xen upstreaming work has been to minimize the
amount of arbitrary "specialness" so that its as easy as possibly for
people and distros to deploy and use Xen.

With respect to microcode loading, this patch means that users and
distros just don't need to worry about microcode any more.  It packages
and operates exactly like it would for native Linux, which is exactly
how we like it.

While doing the whole boot time multiboot thing may offer some small
hypothetical technical advantages, it has the significant cost of just
complicating the whole deployment and use story.

If there were a general shift to "this is how we're going to do
microcode in the future", then Xen will happily go along for the ride. 
But for right now, this patch seems like the pragmatic solution.

I think the real question is where there's something objectionable about
the patch itself?

Thanks,
    J

  reply	other threads:[~2011-10-12 19:18 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-26 18:17 [PATCH 0/3] x86/microcode: support for microcode update in Xen dom0 Jeremy Fitzhardinge
2011-09-26 18:18 ` [PATCH 1/3] xen/acpi: Domain0 acpi parser related platform hypercall Jeremy Fitzhardinge
2011-09-26 18:18 ` [PATCH 2/3] xen: add dom0_op hypercall Jeremy Fitzhardinge
2011-09-26 18:18 ` [PATCH 3/3] xen: add CPU microcode update driver Jeremy Fitzhardinge
2011-10-12  0:40 ` [PATCH 0/3] x86/microcode: support for microcode update in Xen dom0 Jeremy Fitzhardinge
2011-10-12 10:16   ` Borislav Petkov
2011-10-12 15:39     ` Jeremy Fitzhardinge
2011-10-12 18:18     ` H. Peter Anvin
2011-10-12 19:18       ` Jeremy Fitzhardinge [this message]
2011-10-12 19:45         ` Borislav Petkov
2011-10-12 20:40           ` Konrad Rzeszutek Wilk
2011-10-12 21:31             ` H. Peter Anvin
2011-10-12 21:40               ` Borislav Petkov
2011-10-12 22:49                 ` H. Peter Anvin
2011-10-12 23:40                   ` Jeremy Fitzhardinge
2011-10-12 23:55                     ` H. Peter Anvin
2011-10-13  0:00                       ` Jeremy Fitzhardinge
2011-10-13  0:03                         ` H. Peter Anvin
2011-10-13  7:00                           ` Borislav Petkov
2012-01-04  0:49                             ` Sven Köhler
2011-10-13  7:33                   ` Borislav Petkov
2011-10-13  9:57                     ` Borislav Petkov
2011-10-13 16:44                       ` H. Peter Anvin
2011-10-17 17:08                       ` [Xen-devel] " Konrad Rzeszutek Wilk
2011-10-17 17:40                         ` H. Peter Anvin
2011-10-18  1:11                           ` Henrique de Moraes Holschuh
2011-10-18  8:56                           ` Christoph Egger
2011-10-18  9:35                             ` Borislav Petkov
2011-10-18  9:49                               ` Christoph Egger
     [not found]                             ` <m2n.s.1RGYF2-136675@chiark.greenend.org.uk>
2011-10-25 16:53                               ` Ian Jackson
2011-10-12 22:27           ` Jeremy Fitzhardinge
2011-10-12 22:29             ` H. Peter Anvin
2011-10-12 22:33               ` Jeremy Fitzhardinge
2011-10-12 22:34                 ` H. Peter Anvin
2011-10-12 22:38                 ` Borislav Petkov
2011-10-12 20:22         ` H. Peter Anvin
2011-10-12 22:34           ` Jeremy Fitzhardinge
2011-10-12 22:36             ` H. Peter Anvin
     [not found]         ` <m2n.s.1RE5NW-151309@chiark.greenend.org.uk>
2011-10-17 13:47           ` [Xen-devel] " Ian Jackson
     [not found] ` <CAOvdn6VfZ524ku2DBMfmTV+dHGXrSV8Ls_iwHdm=ESXO9kUjcg@mail.gmail.com>
2011-11-03 16:47   ` [Xen-devel] " 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=4E95E7FE.6050302@goop.org \
    --to=jeremy@goop.org \
    --cc=bp@amd64.org \
    --cc=hpa@zytor.com \
    --cc=jeremy.fitzhardinge@citrix.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=tglx@linutronix.de \
    --cc=tigran@aivazian.fsnet.co.uk \
    --cc=x86@kernel.org \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox