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

[PATCH for-4.15 0/3] firmware: fix build on Alpine


  • To: <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Roger Pau Monne <roger.pau@xxxxxxxxxx>
  • Date: Fri, 26 Feb 2021 09:59:05 +0100
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=wPerxV7xu/IWz1CufDmUXDCXn9MszM2GBog3PpAeQX4=; b=OBv/jErmO4L3ak5qIBFMKBv1GOPSft6e4Q6ktlf1mDEgRH5yHnBLA1DHssNgcsEy7Fs4t5HfLPtDUmEa3KzfaDLxGuQejK3l26fkS2mEDUYyoDjrgW8d4FrlCkUnG9Dh244wnPXNGDsVBTW2dxBfBYcmzAtBynEY7mDwh8MUjmdCyGt0ICEswFHM/6I4Jk+bWbkvhLSJ9WdA8XJLSlfw31l+1UhXyjbWji318kHxYJaXk8NwmNM54cSa3tqoa4EmftM6bEtJpkdTo8BYdZuM3NpGg9rXz2AqJx5xVGDcKZFN0M3u1PvT2w5XUtcWIYVqsSU/ZivVQbwCxfts7FQ/FQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=L+3dau2enC6Vrd0BGIn8ZEFi6dL9QixsHT/yuKMsN9htHzqr1iEVy0M9/XWaeP47sqA6YrTfL1sEZNvn474r5SMnSp0XC3FAzmQo5XhXZy4mtXZkaS8q+eo7ivhD6bHipprYTagjLoeuGzVoc17AkXI/Pp2s0i5bqT++LVUvo9cgieLFxdf9MjtBv9dmIWmPdywe5xcKKzqSt+DAqRRYapQJJYp0v/8HhwsF9XfwSQEJYyC/PHB429dYacD+XeJtUrMiw/hSnzEBSHHqTudJ7bIRLJwPVR6zRNonDA6foPhRmIN6KTLgrZL421n8Q+hQ+977BGlK9I3gtuvIMsg66Q==
  • Authentication-results: esa5.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: Roger Pau Monne <roger.pau@xxxxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Doug Goldstein <cardoe@xxxxxxxxxx>
  • Delivery-date: Fri, 26 Feb 2021 09:00:41 +0000
  • Ironport-sdr: 0odw4RDyCKB1wbwD/7amzGcfHSxrbGua3dtSPSvZVsq6wO0PX4IVajTsXPHOmhaVHvi2Ts4/VZ awREbBGfIZlKgOIUFPM1BJbH0k/7oUbVNLKEUv84WX4ZZhBBjUqUSwE5o5i9aCGljbT2Lw1Gmh gu+VS6bWsh4+tT6a85encNUPdFrp4L2j2cuCpLFkNtT/Eit0KaU/YSghBRYip8JkXoGgI+p5Uo leeFPGgjapD3LYpIYnWJH5pyRdVwTs7v5gSa3Ssv02TRIqV4Rv5D6eDyx5LN8gt9AxmD2qEN6s olk=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

Hello,

While the series started as a build fix for Alpine I think they are
interesting on their own for other OSes/distros, since it allows to
remove the i386 libc as a build dependency.

THis is done by proving a suitable set of standalone headers that are
suitable for the needs of the firmware build. Patch 2 contains the full
description on why it's done this way.

The main risk for patches 1 and 2 is breaking the build in some obscure
distro/OS and toolchain combination. We aim to have this mostly covered
by gitlab CI. Patch 3 main risk is breaking the Alpine containers in
gitlab CI, but they are already failing.

Wanted to send this yesterday but was waiting on gitlab CI output, it's now
all green:

https://gitlab.com/xen-project/people/royger/xen/-/pipelines/261928726

Thanks, Roger.

Roger Pau Monne (3):
  hvmloader: do not include inttypes.h
  firmware: provide a stand alone set of headers
  automation: enable rombios build on Alpine

 README                                        |  3 --
 automation/scripts/build                      |  5 +--
 tools/firmware/Rules.mk                       | 11 ++++++
 tools/firmware/hvmloader/32bitbios_support.c  |  2 +-
 tools/firmware/include/stdarg.h               | 10 +++++
 tools/firmware/include/stdbool.h              |  9 +++++
 tools/firmware/include/stddef.h               | 10 +++++
 tools/firmware/include/stdint.h               | 39 +++++++++++++++++++
 tools/firmware/rombios/32bit/rombios_compat.h |  4 +-
 tools/firmware/rombios/rombios.c              |  5 +--
 10 files changed, 85 insertions(+), 13 deletions(-)
 create mode 100644 tools/firmware/include/stdarg.h
 create mode 100644 tools/firmware/include/stdbool.h
 create mode 100644 tools/firmware/include/stddef.h
 create mode 100644 tools/firmware/include/stdint.h

-- 
2.30.1




 


Rackspace

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