[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Re [PATCH] x86/CET: Fix build following c/s 43b98e7190


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • Date: Tue, 2 Jun 2020 18:15:20 +0100
  • Authentication-results: esa1.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Tue, 02 Jun 2020 17:15:47 +0000
  • Ironport-sdr: 66j4nB9tX4MKCNNEx3k4bJt96of05gtniKtRPYJL+KSKT+5uUKDFbF2VJUyb4jQ1PHlR9x35Ci KuiAykyO9bxDXsjH9/otQn6p2wVnBq/cinzjJ4HY6ILxMbstXoiblqgMOUTfDhkH/3uDv6S+PA bJQG6cXMLJSq2mlyfJKvfP3oZcD+AwJXT8PK8I619db2BU1wg+qt4+S5o4SDj5u27WeO3wO0qf vQP7ImvLPJjxMvdPRsKuup5Zh1vlQRatzAVzJB0CQs/1GW7ECIqpAIl6GqQlElGUGZocJz6mUX 4nk=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 02/06/2020 15:21, Jan Beulich wrote:
>> OSSTest reports:
>>
>>   x86_64.S: Assembler messages:
>>   x86_64.S:57: Error: no such instruction: `setssbsy'
>>   /home/osstest/build.150510.build-amd64/xen/xen/Rules.mk:183: recipe for 
>> target 'head.o' failed
>>   make[4]: Leaving directory 
>> '/home/osstest/build.150510.build-amd64/xen/xen/arch/x86/boot'
>>   make[4]: *** [head.o] Error 1
>>
>> All use of CET instructions, even those inside alternative blocks, needs to 
>> be
>> behind CONFIG_XEN_SHSTK, as it indicates suitable toolchain support.
>>
>> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> That's quite a bit of #ifdef-ary here. Simple (operand-less) insns
> like SETSSBSY could easily be made available via .byte directives.
> Would you be amenable to to ack-ing a patch to replace some of the
> #ifdef-s (at least the ones at the lstar, cstar, and sysenter
> entry points), after 4.14?

Yeah - that was a bit of a mess in the end.  (But given the
circumstances, and that I've got past form typo'ing the SETSSBSY opcode,
it probably was the right move even in hindsight).

Reducing it to .byte should be fine so long as some form of /* setssbsy
*/ comment appears.

One other option would be to introduce a SETSSBSY macro, but that hides
the alternative so is something I'd prefer to avoid.

~Andrew



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.