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

Re: [PATCH v2 1/7] x86/EFI: sanitize build logic


  • To: Jan Beulich <jbeulich@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • Date: Mon, 10 Aug 2020 15:38:26 +0100
  • Authentication-results: esa2.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none
  • Cc: Wei Liu <wl@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Delivery-date: Mon, 10 Aug 2020 14:38:41 +0000
  • Ironport-sdr: yVArj69cwGdbAc4XJMplrUGOO5p8oFN0LYe3fIvfu5trejZ/7t3u0AkmIrBo42c1bDwzodo8jG 6iRfa82htnOV74g/teD02lpdANupbD07AK6gWIUvXaj72SRckfbJZkn23SSCXaC3zNHhFcstPT 4lHCeOOBZ3NirmPQIGcXfXzHqGQmtqwnJUCGyo4vlGfvJfsBAaiDrQd6aOGqnT6Bf3ZP4iEyrF 6XO3x4cQYPLID9lTCmrTgzMNPvipD94yy8dQvlpnJHbuxHIwpfRoLSlT7lfDjGHQI3B49GgWGc y2E=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 07/08/2020 17:33, Andrew Cooper wrote:
> On 07/08/2020 12:32, Jan Beulich wrote:
>> With changes done over time and as far as linking goes, the only special
>> thing about building with EFI support enabled is the need for the dummy
>> relocations object for xen.gz uniformly in all build stages. All other
>> efi/*.o can be consumed from the built_in*.o files.
>>
>> In efi/Makefile, besides moving relocs-dummy.o to "extra", also properly
>> split between obj-y and obj-bin-y.
>>
>> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
>
> I'd prefer to see this all in Kconfig, but this is a clear improvement
> in its own right.

Actually, it breaks the build with LIVEPATCH enabled.

make[2]: *** No rule to make target 'efi/buildid.o', needed by
'/local/security/xen.git/xen/xen.efi'.  Stop.
make[2]: *** Waiting for unfinished jobs....
Makefile:355: recipe for target '/local/security/xen.git/xen/xen' failed




 


Rackspace

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