From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lj1-f181.google.com (mail-lj1-f181.google.com [209.85.208.181]) by mx.groups.io with SMTP id smtpd.web08.7140.1628348761438455847 for ; Sat, 07 Aug 2021 08:06:01 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@google.com header.s=20161025 header.b=OS9tB49t; spf=pass (domain: google.com, ip: 209.85.208.181, mailfrom: chengchieh@google.com) Received: by mail-lj1-f181.google.com with SMTP id u13so16547475lje.5 for ; Sat, 07 Aug 2021 08:06:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=7eb23SXyNj2TyUZu2/pEGjbHBekS+7G6i+2RNOOnCm0=; b=OS9tB49t/Wl7xfALmLznLlhU/cPomNcKnpxu+kLUaz9/+uUNOpitstfKNp5mDsIS1r i2yE/ZrG0gjJsRH809n7KpxH9t//JCvj8JpXUIYIT2U1CAKf6ou9nuAU8UfG0GcQ0SmI OK4UjSK81Nj7pMsNCfVYUELy+BexZgE0lB7mAlZwYWpVYUtwkGvGHd9s3BxESBNRkzr2 TO/4/iuQ07LTn1H4VFE9akw2CCBgOJfwYhtbkEHYQXkIlEMi8jeOOW3LfghSo9lenAy6 VKn2VHHRZiePOJUFkvIUApUXu3kt0YI9gdX00Qx8LYLmZczZawObQxUl62HtAJ1ksuuA RShA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=7eb23SXyNj2TyUZu2/pEGjbHBekS+7G6i+2RNOOnCm0=; b=BHiaAo/AS0LUxt2BKNRiEza3R/qcXszYm5++aK55OmRzfIJCjvn/Ke4+QKALumZGc2 nhUHtxCQBcFfa/7V2G/1s1fKEFC9pgZ0F5yVmWpCdIx1ZNid86Nc5pt/QII+L99mI5yF NCMMY6zhsRfRJsPPVTYMr0b6yFYK/QKbabiDXnb9NHwSnrqeN4gaMCjMKrrov9HLNov/ KOAClWdcxRvyMuslkSyhdZJbWO+I8J1oShmpsYF0plp4osxrEcdcalD80pzya/6CKMs0 FjLJrFh8IPiS99Yr1qB1tHW/DloU8me1Y2gyulcggAttbI1CwMfhX/xwtdLDhdVRh63B QToA== X-Gm-Message-State: AOAM533hEEqQuVi7lLnaMibtT8ugvoFkQLcq4dGNjb8WCFkYt6jQDEne d3dTwNZJkkXufAHISjZ3x7agB8M9A5Yqmg7S2K0TCfzo1S7PQw== X-Google-Smtp-Source: ABdhPJyyeTvwpMOzPF8PFOHEgUO/G1W/WEkRGinEGfCn6/37Aq3wv6BHn7xxtRbW9hM0JZzHtiZSxQrWBpd05Z0TG2c= X-Received: by 2002:a2e:8713:: with SMTP id m19mr518680lji.146.1628348759097; Sat, 07 Aug 2021 08:05:59 -0700 (PDT) MIME-Version: 1.0 References: <20210807145106.2236803-1-chengchieh@google.com> In-Reply-To: <20210807145106.2236803-1-chengchieh@google.com> From: Cheng-Chieh Huang Date: Sat, 7 Aug 2021 23:05:47 +0800 Message-ID: Subject: Re: [PATCH v2 0/4] UefiPayloadPkg: LinuxBoot Support in UefiPayload To: devel@edk2.groups.io Cc: Daniel Schaefer , Trammell Hudson , Maurice Ma , Guo Dong , Benjamin You Content-Type: multipart/alternative; boundary="0000000000005c1f9605c8f97d5f" --0000000000005c1f9605c8f97d5f Content-Type: text/plain; charset="UTF-8" Hi Guo, I saw you left messages in https://github.com/tianocore/edk2/pull/1820. If it's also possible to upstreaming using github PR. I can just update my old branch. -- Cheng-chieh On Sat, Aug 7, 2021 at 10:51 PM Cheng-Chieh Huang wrote: > These are necessary patches to Support LinuxBoot in UefiPayload. > With these paches, we can boot to ESXi and Windows from a linux in QEMU. > > This is second parse. In addition to fixing reviwer's suggestions, > I removed the following CLs. > * Add DISABLE_MMX_SSE to avoid generating floating points operation > -> will send a seperate patch to add these flags to BaseTools > > * LinuxBoot: use a text format for the configuration block. > -> will work with Trammell Hudson to cover this patch to EDK2 style. > > LinuxBoot README: > https://github.com/linuxboot/edk2/blob/uefipayload/UefiPayloadPkg/README.md > > v2 PR to tianocore: > https://github.com/tianocore/edk2/pull/1873 > > Cheng-Chieh Huang (4): > UefiPayloadPkg: Add LINUXBOOT payload target > UefiPayloadPkg: Use legacy timer in Linuxboot payload > UefiPayloadPkg: Update maximum logic processor to 256 > UefiPayloadPkg: Reserve Payload config in runtime services data > > UefiPayloadPkg/UefiPayloadPkg.dsc | 24 ++- > UefiPayloadPkg/UefiPayloadPkg.fdf | 5 + > UefiPayloadPkg/Library/LbParseLib/LbParseLib.inf | 39 +++++ > UefiPayloadPkg/Library/LbParseLib/Linuxboot.h | 47 +++++ > UefiPayloadPkg/Library/LbParseLib/LbParseLib.c | 182 > ++++++++++++++++++++ > UefiPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeSupport.c | 6 +- > UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c | 4 + > 7 files changed, 299 insertions(+), 8 deletions(-) > create mode 100644 UefiPayloadPkg/Library/LbParseLib/LbParseLib.inf > create mode 100644 UefiPayloadPkg/Library/LbParseLib/Linuxboot.h > create mode 100644 UefiPayloadPkg/Library/LbParseLib/LbParseLib.c > > Cc: Cheng-Chieh Huang > Cc: Daniel Schaefer > Cc: Trammell Hudson > Cc: Maurice Ma > Cc: Guo Dong > Cc: Benjamin You > > > -- > 2.32.0.605.g8dce9f2422-goog > > --0000000000005c1f9605c8f97d5f Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Hi Guo,

I saw you left messages in=C2= =A0https://github.c= om/tianocore/edk2/pull/1820. If it's also possible to upstreaming u= sing github PR. I can just update my old branch.

-= -
Cheng-chieh

On Sat, Aug 7, 2021 at 10:51 PM Cheng-Chieh H= uang <chengchieh@google.com= > wrote:
Thes= e are necessary patches to Support LinuxBoot in UefiPayload.
With these paches, we can boot to ESXi and Windows from a linux in QEMU.
This is second parse. In addition to fixing reviwer's suggestions,
I removed the following CLs.
* Add DISABLE_MMX_SSE to avoid generating floating points operation
-> will send a seperate patch to add these flags to BaseTools

* LinuxBoot: use a text format for the configuration block.
-> will work with Trammell Hudson to cover this patch to EDK2 style.

LinuxBoot README:
https://github.com/linuxb= oot/edk2/blob/uefipayload/UefiPayloadPkg/README.md

v2 PR to tianocore:
https://github.com/tianocore/edk2/pull/1873

Cheng-Chieh Huang (4):
=C2=A0 UefiPayloadPkg: Add LINUXBOOT payload target
=C2=A0 UefiPayloadPkg: Use legacy timer in Linuxboot payload
=C2=A0 UefiPayloadPkg: Update maximum logic processor to 256
=C2=A0 UefiPayloadPkg: Reserve Payload config in runtime services data

=C2=A0UefiPayloadPkg/UefiPayloadPkg.dsc=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 |=C2= =A0 24 ++-
=C2=A0UefiPayloadPkg/UefiPayloadPkg.fdf=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 |=C2= =A0 =C2=A05 +
=C2=A0UefiPayloadPkg/Library/LbParseLib/LbParseLib.inf=C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0|=C2=A0 39 +++++
=C2=A0UefiPayloadPkg/Library/LbParseLib/Linuxboot.h=C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 |=C2=A0 47 +++++
=C2=A0UefiPayloadPkg/Library/LbParseLib/LbParseLib.c=C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| 182 ++++++++++++++++++++
=C2=A0UefiPayloadPkg/Library/PciHostBridgeLib/PciHostBridgeSupport.c |=C2= =A0 =C2=A06 +-
=C2=A0UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c=C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0|=C2=A0 =C2=A04 +
=C2=A07 files changed, 299 insertions(+), 8 deletions(-)
=C2=A0create mode 100644 UefiPayloadPkg/Library/LbParseLib/LbParseLib.inf =C2=A0create mode 100644 UefiPayloadPkg/Library/LbParseLib/Linuxboot.h
=C2=A0create mode 100644 UefiPayloadPkg/Library/LbParseLib/LbParseLib.c

Cc: Cheng-Chieh Huang <chengchieh@google.com>
Cc: Daniel Schaefer <daniel.schaefer@hpe.com>
Cc: Trammell Hudson <hudson@trmm.net>
Cc: Maurice Ma <maurice.ma@intel.com>
Cc: Guo Dong <gu= o.dong@intel.com>
Cc: Benjamin You <benjamin.you@intel.com>


--
2.32.0.605.g8dce9f2422-goog

--0000000000005c1f9605c8f97d5f--