From nobody Thu May 2 09:27:23 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+79728+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+79728+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1629734067; cv=none; d=zohomail.com; s=zohoarc; b=EM55WvWlv57tWmXyfLamapmh2g9BnaHu5N1M36pMM/osZd4mc3K0h5f0xW8r/9bSvYIkRQb10J5iRp65WIeItW23uuKWirkCpnkyy1Q/LnWdVy4P8GzWGVdZNOYNQWVroixpk6KtJJXW1vYCGwPxiVUdr/qxFRdvqPoJ6LeJBmo= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1629734067; h=Content-Type:Date:From:List-Subscribe:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:Sender:Subject:To; bh=z3MwVZfMapaBudpEv7RfiE20Eo+fDtPp7voVPK5RaxE=; b=CxzfytMTsQ5PMbpWLv9iWdkZH98Lsh3LGv6QE0iXlsBF9du2h0Q4sUTvtW8BgacwFxKfXOwclEl9dKf1VoLhKSsm6AZgoLStCcAH+WMMU98cvzC1yv3l0/ACrDHePx3/4/I3qptwosiKaRVAkqXtzba9i1FhQxBxYGMGJZ9Xv5k= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+79728+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 162973406725813.077992004449584; Mon, 23 Aug 2021 08:54:27 -0700 (PDT) Return-Path: X-Received: by 127.0.0.2 with SMTP id euiJYY1788612xHI8P1Dr4SJ; Mon, 23 Aug 2021 08:54:26 -0700 X-Received: from mail-io1-f50.google.com (mail-io1-f50.google.com [209.85.166.50]) by mx.groups.io with SMTP id smtpd.web10.26079.1629734066248806233 for ; Mon, 23 Aug 2021 08:54:26 -0700 X-Received: by mail-io1-f50.google.com with SMTP id a21so22408536ioq.6 for ; Mon, 23 Aug 2021 08:54:26 -0700 (PDT) X-Gm-Message-State: 7AltHqL8x1ZE85y5t8QiYh32x1787277AA= X-Google-Smtp-Source: ABdhPJxfXkNGLWLL4KUBYhOuRuPMmVTGzweBE1cRrbmTrm5BgJPw5WxhOL5/K/0O26IxFvG3z+F8dwgsnMbbP4TUobo= X-Received: by 2002:a05:6602:d9:: with SMTP id z25mr27406867ioe.154.1629734065588; Mon, 23 Aug 2021 08:54:25 -0700 (PDT) MIME-Version: 1.0 From: "King Sumo" Date: Mon, 23 Aug 2021 12:54:14 -0300 Message-ID: Subject: [edk2-devel] UefiPayloadPkg build error To: devel@edk2.groups.io Precedence: Bulk List-Unsubscribe: List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,kingsumos@gmail.com Content-Type: multipart/alternative; boundary="0000000000000ecda705ca3c0876" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1629734066; bh=HQL7WFlxjByMHv3q6QKWbcNhUySo+/wg7PqLprarBUA=; h=Content-Type:Date:From:Reply-To:Subject:To; b=hPi6ywgYL9eUg8eRjwbQq1Ie05lTd3F4yLysyvy4ahJCZVxqQop2t4MA4+Pgf2oftDx hHObi8aTXcVFscJNmRfcEpFLml/fjQIN1Z+cZwVRna3/ScFlIvwTQtbEE3FV7hOSBcPfM WranyzJ08qCt5zTXeEa+iNt8KYdAW/SomP0= X-ZohoMail-DKIM: pass (identity @groups.io) X-ZM-MESSAGEID: 1629734068044100002 Content-Transfer-Encoding: quoted-printable --0000000000000ecda705ca3c0876 MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 SGksCgpUaGUgZm9sbG93aW5nIGNvbW1pdCBicm9rZSB0aGUgVWVmaVBheWxvYWRQa2cgYnVpbGQ6 Cgpjb21taXQgZDYzNTk1YzNjOTE2MjRmMjU4ZjI5MWFkZWUzMjk3MjRlZGVhYzEyZSAoSEVBRCkK QXV0aG9yOiBaaGlndWFuZyBMaXUgPHpoaWd1YW5nLmxpdUBpbnRlbC5jb20+CkRhdGU6ICAgU3Vu IEFwciAyNSAxNTo1MDo0NiAyMDIxICswODAwCgoKICAgIFVlZmlQYXlsb2FkUGtnOiBVcGRhdGUg dGhlIGZ1bmN0aW9uIGRlZmluaXRpb24gb2YgSG9iQ29uc3RydWN0b3IKCgpFbnZpcm9ubWVudDog R0NDNSAvIG9wZW5TVVNFIDE1LngKIGJ1aWxkIC1hIElBMzIgLWEgWDY0IC1wIFVlZmlQYXlsb2Fk UGtnL1VlZmlQYXlsb2FkUGtnLmRzYyAtYiBERUJVRyAtdCBHQ0M1Ci1EIEJPT1RMT0FERVI9M0RD T1JFQk9PVAoKQnVpbGQgbG9nOgovaG9tZS9seHVzZXIvZWRrMi9lZGsyLW9yaWcvVWVmaVBheWxv YWRQa2cvTGlicmFyeS9QYXlsb2FkRW50cnlIb2JMaWIvSG9iLmM9Cjo4MDozMDoKZXJyb3I6IGNh c3QgZnJvbSBwb2ludGVyIHRvIGludGVnZXIgb2YgZGlmZmVyZW50IHNpemUKWy1XZXJyb3I9M0Rw b2ludGVyLXRvLWludC1jYXN0XQogICBIb2ItPkVmaU1lbW9yeVRvcCAgICAgICAgPTNEIChFRklf UEhZU0lDQUxfQUREUkVTUykgRWZpTWVtb3J5VG9wOwogICAgICAgICAgICAgICAgICAgICAgICAg ICAgICBeCi9ob21lL2x4dXNlci9lZGsyL2VkazItb3JpZy9VZWZpUGF5bG9hZFBrZy9MaWJyYXJ5 L1BheWxvYWRFbnRyeUhvYkxpYi9Ib2IuYz0KOjgxOjMwOgplcnJvcjogY2FzdCBmcm9tIHBvaW50 ZXIgdG8gaW50ZWdlciBvZiBkaWZmZXJlbnQgc2l6ZQpbLVdlcnJvcj0zRHBvaW50ZXItdG8taW50 LWNhc3RdCiAgIEhvYi0+RWZpTWVtb3J5Qm90dG9tICAgICA9M0QgKEVGSV9QSFlTSUNBTF9BRERS RVNTKSBFZmlNZW1vcnlCb3R0b207CiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIF4KL2hv bWUvbHh1c2VyL2VkazIvZWRrMi1vcmlnL1VlZmlQYXlsb2FkUGtnL0xpYnJhcnkvUGF5bG9hZEVu dHJ5SG9iTGliL0hvYi5jPQo6ODI6MzA6CmVycm9yOiBjYXN0IGZyb20gcG9pbnRlciB0byBpbnRl Z2VyIG9mIGRpZmZlcmVudCBzaXplClstV2Vycm9yPTNEcG9pbnRlci10by1pbnQtY2FzdF0KICAg SG9iLT5FZmlGcmVlTWVtb3J5VG9wICAgID0zRCAoRUZJX1BIWVNJQ0FMX0FERFJFU1MpIEVmaUZy ZWVNZW1vcnlUb3A7CiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIF4KCgoKVGhlIGJlbG93 IHBhdGNoIGZpeGVzIHRoZSBidWlsZCwgaG93ZXZlciBJJ20gbm90IHN1cmUgaWYgdGhpcyBpcyB0 aGUgYmVzdAp3YXkgdG8gc29sdmUgdGhlIGlzc3VlIC0gb3IgaWYgdGhlcmUgYXJlIGFueSBzaWRl IGVmZmVjdHM6CgpkaWZmIC0tZ2l0IGEvVWVmaVBheWxvYWRQa2cvTGlicmFyeS9QYXlsb2FkRW50 cnlIb2JMaWIvSG9iLmMKYi9VZWZpUGF5bG9hZFBrZy9MaWJyYXJ5L1BheWxvYWRFbnRyeUhvYkxp Yi9Ib2IuYwppbmRleCA3NjhjM2RiNzcwLi41M2VlMTQ0MGY1IDEwMDY0NApSZXZpZXdlZC1ieTog R3VvIERvbmcgPGd1by5kb25nQGludGVsLmNvbTxtYWlsdG86Z3VvLmRvbmdAaW50ZWwuY29tPj4K LS0tIGEvVWVmaVBheWxvYWRQa2cvTGlicmFyeS9QYXlsb2FkRW50cnlIb2JMaWIvSG9iLmMKKysr IGIvVWVmaVBheWxvYWRQa2cvTGlicmFyeS9QYXlsb2FkRW50cnlIb2JMaWIvSG9iLmMKQEAgLTc3 LDkgKzc3LDkgQEAgSG9iQ29uc3RydWN0b3IgKAogICBIb2ItPlZlcnNpb24gICAgICAgICAgICAg PTNEIEVGSV9IT0JfSEFORE9GRl9UQUJMRV9WRVJTSU9OOwogICBIb2ItPkJvb3RNb2RlICAgICAg ICAgICAgPTNEIEJPT1RfV0lUSF9GVUxMX0NPTkZJR1VSQVRJT047CgotICBIb2ItPkVmaU1lbW9y eVRvcCAgICAgICAgPTNEIChFRklfUEhZU0lDQUxfQUREUkVTUykgRWZpTWVtb3J5VG9wOwotICBI b2ItPkVmaU1lbW9yeUJvdHRvbSAgICAgPTNEIChFRklfUEhZU0lDQUxfQUREUkVTUykgRWZpTWVt b3J5Qm90dG9tOwotICBIb2ItPkVmaUZyZWVNZW1vcnlUb3AgICAgPTNEIChFRklfUEhZU0lDQUxf QUREUkVTUykgRWZpRnJlZU1lbW9yeVRvcDsKKyAgSG9iLT5FZmlNZW1vcnlUb3AgICAgICAgID0z RCAoRUZJX1BIWVNJQ0FMX0FERFJFU1MpIChVSU5UTilFZmlNZW1vcnlUb3A7CisgIEhvYi0+RWZp TWVtb3J5Qm90dG9tICAgICA9M0QgKEVGSV9QSFlTSUNBTF9BRERSRVNTKSAoVUlOVE4pRWZpTWVt b3J5Qm90dD0Kb207CisgIEhvYi0+RWZpRnJlZU1lbW9yeVRvcCAgICA9M0QgKEVGSV9QSFlTSUNB TF9BRERSRVNTKQooVUlOVE4pRWZpRnJlZU1lbW9yeVRvcDsKICAgSG9iLT5FZmlGcmVlTWVtb3J5 Qm90dG9tID0zRCAoRUZJX1BIWVNJQ0FMX0FERFJFU1MpIChVSU5UTikgKEhvYkVuZCsxKTsKICAg SG9iLT5FZmlFbmRPZkhvYkxpc3QgICAgID0zRCAoRUZJX1BIWVNJQ0FMX0FERFJFU1MpIChVSU5U TikgSG9iRW5kOwoKVGhhbmtzLApTdW1vCgoKLT0zRC09M0QtPTNELT0zRC09M0QtPTNELT0zRC09 M0QtPTNELT0zRC09M0QtCkdyb3Vwcy5pbyBMaW5rczogWW91IHJlY2VpdmUgYWxsIG1lc3NhZ2Vz IHNlbnQgdG8gdGhpcyBncm91cC4KVmlldy9SZXBseSBPbmxpbmUgKCM3OTcyOCk6IGh0dHBzOi8v ZWRrMi5ncm91cHMuaW8vZy9kZXZlbC9tZXNzYWdlLzc5NzI4Ck11dGUgVGhpcyBUb3BpYzogaHR0 cHM6Ly9ncm91cHMuaW8vbXQvODUwODg2NzEvMTc4NzI3NwpHcm91cCBPd25lcjogZGV2ZWwrb3du ZXJAZWRrMi5ncm91cHMuaW8KVW5zdWJzY3JpYmU6IGh0dHBzOi8vZWRrMi5ncm91cHMuaW8vZy9k ZXZlbC91bnN1YiBbaW1wb3J0ZXJAcGF0Y2hldy5vcmddCi09M0QtPTNELT0zRC09M0QtPTNELT0z RC09M0QtPTNELT0zRC09M0QtPTNELQoK --0000000000000ecda705ca3c0876 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Hi,

The following commit broke the Uefi= PayloadPkg build:
commit d63595c3c91624f258f291adee329724edeac12e (HEAD)
Author: Zhiguang Liu <z= higuang.liu@intel.com>
Date: =C2=A0 Sun Apr 25 15:50:46 20= 21 +0800

=C2=A0 =C2=A0 UefiPayloadPkg: Update th= e function definition of HobConstructor

Environment: GCC5 / openSUSE 15.x
=C2=A0build -a IA32 -a X64 -= p UefiPayloadPkg/UefiPayloadPkg.dsc -b DEBUG -t GCC5 -D BOOTLOADER=3DCOREBO= OT

Build log:
/home/lxuser/edk2/edk2-orig/UefiPayloadP= kg/Library/PayloadEntryHobLib/Hob.c:80:30: error: cast from pointer to inte= ger of different size [-Werror=3Dpointer-to-int-cast]
=C2=A0 =C2=A0Hob-&= gt;EfiMemoryTop =C2=A0 =C2=A0 =C2=A0 =C2=A0=3D (EFI_PHYSICAL_ADDRESS) EfiMe= moryTop;
=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 ^
/home/lxuser/edk2/edk2-orig/= UefiPayloadPkg/Library/PayloadEntryHobLib/Hob.c:81:30: error: cast from poi= nter to integer of different size [-Werror=3Dpointer-to-int-cast]
=C2=A0= =C2=A0Hob->EfiMemoryBottom =C2=A0 =C2=A0 =3D (EFI_PHYSICAL_ADDRESS) Efi= MemoryBottom;
=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 ^
/home/lxuser/edk2/edk= 2-orig/UefiPayloadPkg/Library/PayloadEntryHobLib/Hob.c:82:30: error: cast f= rom pointer to integer of different size [-Werror=3Dpointer-to-int-cast]=C2=A0 =C2=A0Hob->EfiFreeMemoryTop =C2=A0 =C2=A0=3D (EFI_PHYSICAL_ADDRE= SS) EfiFreeMemoryTop;
=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 ^


=

The below patch fixes the build, however I'm not su= re if this is the best way to solve the issue - or if there are any side ef= fects:

diff --git a/UefiPayloadPkg/Library/Payload= EntryHobLib/Hob.c b/UefiPayloadPkg/Library/PayloadEntryHobLib/Hob.c
inde= x 768c3db770..53ee1440f5 100644
--- a/UefiPayloadPkg/Library/PayloadEntr= yHobLib/Hob.c
+++ b/UefiPayloadPkg/Library/PayloadEntryHobLib/Hob.c
@= @ -77,9 +77,9 @@ HobConstructor (
=C2=A0 =C2=A0Hob->Version =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =3D EFI_HOB_HANDOFF_TABLE_VERSION;
= =C2=A0 =C2=A0Hob->BootMode =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0=3D = BOOT_WITH_FULL_CONFIGURATION;

- =C2=A0Hob->EfiMemoryTop =C2=A0 = =C2=A0 =C2=A0 =C2=A0=3D (EFI_PHYSICAL_ADDRESS) EfiMemoryTop;
- =C2=A0Hob= ->EfiMemoryBottom =C2=A0 =C2=A0 =3D (EFI_PHYSICAL_ADDRESS) EfiMemoryBott= om;
- =C2=A0Hob->EfiFreeMemoryTop =C2=A0 =C2=A0=3D (EFI_PHYSICAL_ADDR= ESS) EfiFreeMemoryTop;
+ =C2=A0Hob->EfiMemoryTop =C2=A0 =C2=A0 =C2=A0= =C2=A0=3D (EFI_PHYSICAL_ADDRESS) (UINTN)EfiMemoryTop;
+ =C2=A0Hob->E= fiMemoryBottom =C2=A0 =C2=A0 =3D (EFI_PHYSICAL_ADDRESS) (UINTN)EfiMemoryBot= tom;
+ =C2=A0Hob->EfiFreeMemoryTop =C2=A0 =C2=A0=3D (EFI_PHYSICAL_ADD= RESS) (UINTN)EfiFreeMemoryTop;
=C2=A0 =C2=A0Hob->EfiFreeMemoryBottom = =3D (EFI_PHYSICAL_ADDRESS) (UINTN) (HobEnd+1);
=C2=A0 =C2=A0Hob->EfiE= ndOfHobList =C2=A0 =C2=A0 =3D (EFI_PHYSICAL_ADDRESS) (UINTN) HobEnd;

Thanks,
Sumo
_._,_._,_
G= roups.io Links:

You receive all messages sent to this group.

Vi= ew/Reply Online (#79728) | | Mute This Topic | New Topic
Your Subscription | Contact Group Owner | Unsubscribe [importer@patchew.org]
_._,_._,_=20 --0000000000000ecda705ca3c0876--