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

Re: [PATCH v2 2/7] x86: don't build with EFI support in shim-exclusive mode


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Date: Tue, 18 Aug 2020 15:00:28 +0200
  • Authentication-results: esa4.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, "Andrew Cooper" <andrew.cooper3@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
  • Delivery-date: Tue, 18 Aug 2020 13:00:48 +0000
  • Ironport-sdr: x3qodcmwxt+RT4kptlzkIXUvj4u2I6JhuB99+0+V/d2XSbTO8rjHiqlaW4ImqfPw4GqSm3eOGp AvGzwcDNW+yiDoLeeYy9aJ1481b2P1C5HV1gUluh8yHGwG9gRoGsbJdHBiTRPPBDLQMwbzgQmc BGwLpznVwQe4JtgsjQPQ2nW9X1JC207KlPePA7Q8aYgnuvj5pfPE9ec5QWZrHFBl3heVQuxyBl 6bT7Q0WUTWWtrIRLIECwLyrrnGbgdsBI62gjnTAFf8L6milYCSF7bpLs886z722FNOaajEJHzO rpo=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Fri, Aug 07, 2020 at 01:32:38PM +0200, Jan Beulich wrote:
> There's no need for xen.efi at all, and there's also no need for EFI
> support in xen.gz since the shim runs in PVH mode, i.e. without any
> firmware (and hence by implication also without EFI one).
> 
> The slightly odd looking use of $(space) is to ensure the new ifneq()
> evaluates consistently between "build" and "install" invocations of
> make.

I would likely add a comment to the code itself, as it's not obvious
without a hint IMO.

> 
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

Reviewed-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>

I wonder however if there would be a less tricky way to arrange all
this. Maybe the Kconfig work will remove some of this hacks?

Thanks, Roger.



 


Rackspace

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