Re: [PATCH RFC] x86/smpboot: Set safer __max_logical_packages limit

From	Peter Zijlstra <peterz@infradead.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH RFC] x86/smpboot: Set safer __max_logical_packages limit
Date	2017-04-20 18:20 +0200
Message-ID	<tymYh-2Ig-5@gated-at.bofh.it> (permalink)
References	<tykjM-11j-19@gated-at.bofh.it> <tylSy-26m-21@gated-at.bofh.it> <tymvf-2j8-7@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Thu, Apr 20, 2017 at 05:40:37PM +0200, Vitaly Kuznetsov wrote:
> > This is getting ludicrous. Xen is plain broken, and instead of fixing
> > it, you propose to somehow deal with its obviously crack induced
> > behaviour :-(
> 
> Totally agree and I don't like the solution I propose (and that's why
> this is RFC)... The problem is that there are such Xen setups in the
> wild and with the recent changes some guests will BUG() :-(
> 
> Alternatively, we can just remove the BUG() and do something with CPUs
> which have their pkg >= __max_logical_packages, e.g. assign them to the
> last package. Far from ideal but will help to avoid the regression.

So currently none of the stuff that uses this should appear in Xen. Its
all drivers for hardware that isn't virtualized (afaik). So assigning to
the last package 'works'.

But the moment this ends up getting used that explodes, because we'll
need different object instances for each piece of hardware.

There just isn't a good solution; on the one hand the BIOS is prone to
providing crap numbers, on the other hand virt (esp. Xen as it turns
out) provides absolutely bonkers/inconsistent topology information.

Very frustrating :-/

Thread

[PATCH RFC] x86/smpboot: Set safer __max_logical_packages limit Vitaly Kuznetsov <vkuznets@redhat.com> - 2017-04-20 15:30 +0200
  Re: [PATCH RFC] x86/smpboot: Set safer __max_logical_packages limit Peter Zijlstra <peterz@infradead.org> - 2017-04-20 17:10 +0200
    Re: [PATCH RFC] x86/smpboot: Set safer __max_logical_packages limit Vitaly Kuznetsov <vkuznets@redhat.com> - 2017-04-20 17:50 +0200
      Re: [Xen-devel] [PATCH RFC] x86/smpboot: Set safer  __max_logical_packages limit Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2017-04-20 18:10 +0200
        Re: [Xen-devel] [PATCH RFC] x86/smpboot: Set safer __max_logical_packages limit Vitaly Kuznetsov <vkuznets@redhat.com> - 2017-04-20 18:30 +0200
      Re: [PATCH RFC] x86/smpboot: Set safer __max_logical_packages limit Peter Zijlstra <peterz@infradead.org> - 2017-04-20 18:20 +0200
        Re: [Xen-devel] [PATCH RFC] x86/smpboot: Set safer  __max_logical_packages limit Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2017-04-20 19:20 +0200
    Re: [Xen-devel] [PATCH RFC] x86/smpboot: Set safer  __max_logical_packages limit Andrew Cooper <andrew.cooper3@citrix.com> - 2017-04-20 19:10 +0200

csiph-web