From nobody Thu May 2 08:02:07 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+55038+1787277+3901457@groups.io; helo=web01.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+55038+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1582839558; cv=none; d=zohomail.com; s=zohoarc; b=Zhr2CkasG/pMpgQMocmAoIXbBYhxAQOPUf2lUuSaGhGh7HrBmmLPR77s+towoWcsFzfTjA1PPpv/WeTzSGs3dzIve0dLFAinBrxPbYEBQX91nlJUOaCqFYtFLIHyLa8wWGsQi6F2+CbQuXZtUxTbHNBFFNYpxEVExOwqBsBbdZQ= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1582839558; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=ftItV+wg6vpSR+M6CFLoqGploNfAPl1K2ONgGClRddc=; b=aWGsnzyAerBuLIIywLOCVWr/bP2CuC6DGj+O35r7CuDxzeXI2XDjvho4z1rZCfHroyoDy3/mAC5AVXVC1BJVR02GI7Ro1VzJlyKROCEza2y1hzpHjotNTuPkPiX+AJh2RTUZL7Y0T0WJTMWApYnC2P1iQk+Iqwwt7nEmdeaIPCA= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+55038+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 1582839558985688.5211328011296; Thu, 27 Feb 2020 13:39:18 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id B4FoYY1788612xBLlRAPHwcF; Thu, 27 Feb 2020 13:39:17 -0800 X-Received: from us-smtp-1.mimecast.com (us-smtp-1.mimecast.com [207.211.31.120]) by mx.groups.io with SMTP id smtpd.web09.1012.1582839556194400888 for ; Thu, 27 Feb 2020 13:39:16 -0800 X-Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-151-GndyQWUXOBScDDcKVQMUbQ-1; Thu, 27 Feb 2020 16:39:09 -0500 X-MC-Unique: GndyQWUXOBScDDcKVQMUbQ-1 X-Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 22740DB20; Thu, 27 Feb 2020 21:39:08 +0000 (UTC) X-Received: from lacos-laptop-7.usersys.redhat.com (ovpn-116-46.ams2.redhat.com [10.36.116.46]) by smtp.corp.redhat.com (Postfix) with ESMTP id BFD49165DA; Thu, 27 Feb 2020 21:39:06 +0000 (UTC) From: "Laszlo Ersek" To: edk2-devel-groups-io Cc: Ard Biesheuvel , Leif Lindholm , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= Subject: [edk2-devel] [PATCH 1/3] ArmPkg: convert LFs to CRLF, expand hard TABs Date: Thu, 27 Feb 2020 22:39:01 +0100 Message-Id: <20200227213903.13884-2-lersek@redhat.com> In-Reply-To: <20200227213903.13884-1-lersek@redhat.com> References: <20200227213903.13884-1-lersek@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Precedence: Bulk List-Unsubscribe: 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,lersek@redhat.com X-Gm-Message-State: jMx7WcDSX91dtSkDzoJVVLIOx1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1582839557; bh=ftItV+wg6vpSR+M6CFLoqGploNfAPl1K2ONgGClRddc=; h=Cc:Content-Type:Date:From:Reply-To:Subject:To; b=tngCFZcWN5ghP+sKFPB1Xxp5mfH8mX14IqD+z43pNDehhUM7fR6lBn5bPDqcd44Uk+8 DrZxSoje/nQdCiNZXo/Q+AeIqEyHSzjWxrFKgg0cgD7m9wZjuW3blv0D4PEXSajH9R4pA owDHVnmeQ6owwvfclCOqC+YhQmlmHLqp/c4= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" We're going to switch the internal line terminators globally to LF at some point, but until then, let's use CRLF consistently. Convert source files with LFs in them to CRLF, using "unix2dos". "git show -b" prints no code changes for this patch. (I collected all the file name suffixes in this package, with: $ git ls-files -- $PACKAGE | rev | cut -f 1 -d . | sort -u | rev I eliminated those suffixes that didn't stand for text files, then blanket-converted the rest with unix2dos. Finally, picked up the actual changes with git-add.) At the same time, the following three files had to undergo TAB expansion: ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c ArmPkg/Library/GccLto/liblto-aarch64.s ArmPkg/Library/GccLto/liblto-arm.s I used "expand -t 2", in order to stay close to the edk2 coding style (which uses two spaces for indentation.) Both the CRLF conversion and the TAB expansion are motivated by "PatchCheck.py". "PatchCheck.py" is also the reason why CRLF conversion and TAB expansion have to happen in the same patch. Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Philippe Mathieu-Daud=C3=A9 Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3D1659 Signed-off-by: Laszlo Ersek Acked-by: Ard Biesheuvel Reviewed-by: Philippe Mathieu-Daude --- ArmPkg/Library/ArmSmcLibNull/ArmSmcLibNull.c | 32 +- ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c | 566 ++++++++++---------- ArmPkg/Library/CompilerIntrinsicsLib/memset.c | 110 ++-- ArmPkg/Library/GccLto/liblto-aarch64.s | 42 +- ArmPkg/Library/GccLto/liblto-arm.s | 110 ++-- 5 files changed, 430 insertions(+), 430 deletions(-) diff --git a/ArmPkg/Library/ArmSmcLibNull/ArmSmcLibNull.c b/ArmPkg/Library/= ArmSmcLibNull/ArmSmcLibNull.c index 2cd292aabf88..2d79aadaf1fa 100644 --- a/ArmPkg/Library/ArmSmcLibNull/ArmSmcLibNull.c +++ b/ArmPkg/Library/ArmSmcLibNull/ArmSmcLibNull.c @@ -1,16 +1,16 @@ -// -// Copyright (c) 2016, Linaro Limited. All rights reserved. -// -// SPDX-License-Identifier: BSD-2-Clause-Patent -// -// - -#include -#include - -VOID -ArmCallSmc ( - IN OUT ARM_SMC_ARGS *Args - ) -{ -} +// +// Copyright (c) 2016, Linaro Limited. All rights reserved. +// +// SPDX-License-Identifier: BSD-2-Clause-Patent +// +// + +#include +#include + +VOID +ArmCallSmc ( + IN OUT ARM_SMC_ARGS *Args + ) +{ +} diff --git a/ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c b/ArmPkg/Libr= ary/ArmSoftFloatLib/ArmSoftFloatLib.c index 42bed7700c99..77e2473678a2 100644 --- a/ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c +++ b/ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c @@ -1,283 +1,283 @@ -/* - * Copyright (c) 2015 - 2019, Linaro Limited - * - * SPDX-License-Identifier: BSD-2-Clause-Patent - */ - -#include "platform.h" -#include - -/* - * On ARM32 EABI defines both a soft-float ABI and a hard-float ABI, - * hard-float is basically a super set of soft-float. Hard-float requires - * all the support routines provided for soft-float, but the compiler may - * choose to optimize to not use some of them. - * - * The AEABI functions uses soft-float calling convention even if the - * functions are compiled for hard-float. So where float and double would - * have been expected we use aeabi_float_t and aeabi_double_t respectively - * instead. - */ -typedef uint32_t aeabi_float_t; -typedef uint64_t aeabi_double_t; - -/* - * Helpers to convert between float32 and aeabi_float_t, and float64 and - * aeabi_double_t used by the AEABI functions below. - */ -static aeabi_float_t f32_to_f(float32_t val) -{ - return val.v; -} - -static float32_t f32_from_f(aeabi_float_t val) -{ - float32_t res; - - res.v =3D val; - - return res; -} - -static aeabi_double_t f64_to_d(float64_t val) -{ - return val.v; -} - -static float64_t f64_from_d(aeabi_double_t val) -{ - float64_t res; - - res.v =3D val; - - return res; -} - -/* - * From ARM Run-time ABI for ARM Architecture - * ARM IHI 0043D, current through ABI release 2.09 - * - * 4.1.2 The floating-point helper functions - */ - -/* - * Table 2, Standard aeabi_double_t precision floating-point arithmetic he= lper - * functions - */ - -aeabi_double_t __aeabi_dadd(aeabi_double_t a, aeabi_double_t b) -{ - return f64_to_d(f64_add(f64_from_d(a), f64_from_d(b))); -} - -aeabi_double_t __aeabi_ddiv(aeabi_double_t a, aeabi_double_t b) -{ - return f64_to_d(f64_div(f64_from_d(a), f64_from_d(b))); -} - -aeabi_double_t __aeabi_dmul(aeabi_double_t a, aeabi_double_t b) -{ - return f64_to_d(f64_mul(f64_from_d(a), f64_from_d(b))); -} - - -aeabi_double_t __aeabi_drsub(aeabi_double_t a, aeabi_double_t b) -{ - return f64_to_d(f64_sub(f64_from_d(b), f64_from_d(a))); -} - -aeabi_double_t __aeabi_dsub(aeabi_double_t a, aeabi_double_t b) -{ - return f64_to_d(f64_sub(f64_from_d(a), f64_from_d(b))); -} - -/* - * Table 3, double precision floating-point comparison helper functions - */ - -int __aeabi_dcmpeq(aeabi_double_t a, aeabi_double_t b) -{ - return f64_eq(f64_from_d(a), f64_from_d(b)); -} - -int __aeabi_dcmplt(aeabi_double_t a, aeabi_double_t b) -{ - return f64_lt(f64_from_d(a), f64_from_d(b)); -} - -int __aeabi_dcmple(aeabi_double_t a, aeabi_double_t b) -{ - return f64_le(f64_from_d(a), f64_from_d(b)); -} - -int __aeabi_dcmpge(aeabi_double_t a, aeabi_double_t b) -{ - return f64_le(f64_from_d(b), f64_from_d(a)); -} - -int __aeabi_dcmpgt(aeabi_double_t a, aeabi_double_t b) -{ - return f64_lt(f64_from_d(b), f64_from_d(a)); -} - -/* - * Table 4, Standard single precision floating-point arithmetic helper - * functions - */ - -aeabi_float_t __aeabi_fadd(aeabi_float_t a, aeabi_float_t b) -{ - return f32_to_f(f32_add(f32_from_f(a), f32_from_f(b))); -} - -aeabi_float_t __aeabi_fdiv(aeabi_float_t a, aeabi_float_t b) -{ - return f32_to_f(f32_div(f32_from_f(a), f32_from_f(b))); -} - -aeabi_float_t __aeabi_fmul(aeabi_float_t a, aeabi_float_t b) -{ - return f32_to_f(f32_mul(f32_from_f(a), f32_from_f(b))); -} - -aeabi_float_t __aeabi_frsub(aeabi_float_t a, aeabi_float_t b) -{ - return f32_to_f(f32_sub(f32_from_f(b), f32_from_f(a))); -} - -aeabi_float_t __aeabi_fsub(aeabi_float_t a, aeabi_float_t b) -{ - return f32_to_f(f32_sub(f32_from_f(a), f32_from_f(b))); -} - -/* - * Table 5, Standard single precision floating-point comparison helper - * functions - */ - -int __aeabi_fcmpeq(aeabi_float_t a, aeabi_float_t b) -{ - return f32_eq(f32_from_f(a), f32_from_f(b)); -} - -int __aeabi_fcmplt(aeabi_float_t a, aeabi_float_t b) -{ - return f32_lt(f32_from_f(a), f32_from_f(b)); -} - -int __aeabi_fcmple(aeabi_float_t a, aeabi_float_t b) -{ - return f32_le(f32_from_f(a), f32_from_f(b)); -} - -int __aeabi_fcmpge(aeabi_float_t a, aeabi_float_t b) -{ - return f32_le(f32_from_f(b), f32_from_f(a)); -} - -int __aeabi_fcmpgt(aeabi_float_t a, aeabi_float_t b) -{ - return f32_lt(f32_from_f(b), f32_from_f(a)); -} - -/* - * Table 6, Standard floating-point to integer conversions - */ - -int __aeabi_d2iz(aeabi_double_t a) -{ - return f64_to_i32_r_minMag(f64_from_d(a), false); -} - -unsigned __aeabi_d2uiz(aeabi_double_t a) -{ - return f64_to_ui32_r_minMag(f64_from_d(a), false); -} - -long long __aeabi_d2lz(aeabi_double_t a) -{ - return f64_to_i64_r_minMag(f64_from_d(a), false); -} - -unsigned long long __aeabi_d2ulz(aeabi_double_t a) -{ - return f64_to_ui64_r_minMag(f64_from_d(a), false); -} - -int __aeabi_f2iz(aeabi_float_t a) -{ - return f32_to_i32_r_minMag(f32_from_f(a), false); -} - -unsigned __aeabi_f2uiz(aeabi_float_t a) -{ - return f32_to_ui32_r_minMag(f32_from_f(a), false); -} - -long long __aeabi_f2lz(aeabi_float_t a) -{ - return f32_to_i64_r_minMag(f32_from_f(a), false); -} - -unsigned long long __aeabi_f2ulz(aeabi_float_t a) -{ - return f32_to_ui64_r_minMag(f32_from_f(a), false); -} - -/* - * Table 7, Standard conversions between floating types - */ - -aeabi_float_t __aeabi_d2f(aeabi_double_t a) -{ - return f32_to_f(f64_to_f32(f64_from_d(a))); -} - -aeabi_double_t __aeabi_f2d(aeabi_float_t a) -{ - return f64_to_d(f32_to_f64(f32_from_f(a))); -} - -/* - * Table 8, Standard integer to floating-point conversions - */ - -aeabi_double_t __aeabi_i2d(int a) -{ - return f64_to_d(i32_to_f64(a)); -} - -aeabi_double_t __aeabi_ui2d(unsigned a) -{ - return f64_to_d(ui32_to_f64(a)); -} - -aeabi_double_t __aeabi_l2d(long long a) -{ - return f64_to_d(i64_to_f64(a)); -} - -aeabi_double_t __aeabi_ul2d(unsigned long long a) -{ - return f64_to_d(ui64_to_f64(a)); -} - -aeabi_float_t __aeabi_i2f(int a) -{ - return f32_to_f(i32_to_f32(a)); -} - -aeabi_float_t __aeabi_ui2f(unsigned a) -{ - return f32_to_f(ui32_to_f32(a)); -} - -aeabi_float_t __aeabi_l2f(long long a) -{ - return f32_to_f(i64_to_f32(a)); -} - -aeabi_float_t __aeabi_ul2f(unsigned long long a) -{ - return f32_to_f(ui64_to_f32(a)); -} +/* + * Copyright (c) 2015 - 2019, Linaro Limited + * + * SPDX-License-Identifier: BSD-2-Clause-Patent + */ + +#include "platform.h" +#include + +/* + * On ARM32 EABI defines both a soft-float ABI and a hard-float ABI, + * hard-float is basically a super set of soft-float. Hard-float requires + * all the support routines provided for soft-float, but the compiler may + * choose to optimize to not use some of them. + * + * The AEABI functions uses soft-float calling convention even if the + * functions are compiled for hard-float. So where float and double would + * have been expected we use aeabi_float_t and aeabi_double_t respectively + * instead. + */ +typedef uint32_t aeabi_float_t; +typedef uint64_t aeabi_double_t; + +/* + * Helpers to convert between float32 and aeabi_float_t, and float64 and + * aeabi_double_t used by the AEABI functions below. + */ +static aeabi_float_t f32_to_f(float32_t val) +{ + return val.v; +} + +static float32_t f32_from_f(aeabi_float_t val) +{ + float32_t res; + + res.v =3D val; + + return res; +} + +static aeabi_double_t f64_to_d(float64_t val) +{ + return val.v; +} + +static float64_t f64_from_d(aeabi_double_t val) +{ + float64_t res; + + res.v =3D val; + + return res; +} + +/* + * From ARM Run-time ABI for ARM Architecture + * ARM IHI 0043D, current through ABI release 2.09 + * + * 4.1.2 The floating-point helper functions + */ + +/* + * Table 2, Standard aeabi_double_t precision floating-point arithmetic he= lper + * functions + */ + +aeabi_double_t __aeabi_dadd(aeabi_double_t a, aeabi_double_t b) +{ + return f64_to_d(f64_add(f64_from_d(a), f64_from_d(b))); +} + +aeabi_double_t __aeabi_ddiv(aeabi_double_t a, aeabi_double_t b) +{ + return f64_to_d(f64_div(f64_from_d(a), f64_from_d(b))); +} + +aeabi_double_t __aeabi_dmul(aeabi_double_t a, aeabi_double_t b) +{ + return f64_to_d(f64_mul(f64_from_d(a), f64_from_d(b))); +} + + +aeabi_double_t __aeabi_drsub(aeabi_double_t a, aeabi_double_t b) +{ + return f64_to_d(f64_sub(f64_from_d(b), f64_from_d(a))); +} + +aeabi_double_t __aeabi_dsub(aeabi_double_t a, aeabi_double_t b) +{ + return f64_to_d(f64_sub(f64_from_d(a), f64_from_d(b))); +} + +/* + * Table 3, double precision floating-point comparison helper functions + */ + +int __aeabi_dcmpeq(aeabi_double_t a, aeabi_double_t b) +{ + return f64_eq(f64_from_d(a), f64_from_d(b)); +} + +int __aeabi_dcmplt(aeabi_double_t a, aeabi_double_t b) +{ + return f64_lt(f64_from_d(a), f64_from_d(b)); +} + +int __aeabi_dcmple(aeabi_double_t a, aeabi_double_t b) +{ + return f64_le(f64_from_d(a), f64_from_d(b)); +} + +int __aeabi_dcmpge(aeabi_double_t a, aeabi_double_t b) +{ + return f64_le(f64_from_d(b), f64_from_d(a)); +} + +int __aeabi_dcmpgt(aeabi_double_t a, aeabi_double_t b) +{ + return f64_lt(f64_from_d(b), f64_from_d(a)); +} + +/* + * Table 4, Standard single precision floating-point arithmetic helper + * functions + */ + +aeabi_float_t __aeabi_fadd(aeabi_float_t a, aeabi_float_t b) +{ + return f32_to_f(f32_add(f32_from_f(a), f32_from_f(b))); +} + +aeabi_float_t __aeabi_fdiv(aeabi_float_t a, aeabi_float_t b) +{ + return f32_to_f(f32_div(f32_from_f(a), f32_from_f(b))); +} + +aeabi_float_t __aeabi_fmul(aeabi_float_t a, aeabi_float_t b) +{ + return f32_to_f(f32_mul(f32_from_f(a), f32_from_f(b))); +} + +aeabi_float_t __aeabi_frsub(aeabi_float_t a, aeabi_float_t b) +{ + return f32_to_f(f32_sub(f32_from_f(b), f32_from_f(a))); +} + +aeabi_float_t __aeabi_fsub(aeabi_float_t a, aeabi_float_t b) +{ + return f32_to_f(f32_sub(f32_from_f(a), f32_from_f(b))); +} + +/* + * Table 5, Standard single precision floating-point comparison helper + * functions + */ + +int __aeabi_fcmpeq(aeabi_float_t a, aeabi_float_t b) +{ + return f32_eq(f32_from_f(a), f32_from_f(b)); +} + +int __aeabi_fcmplt(aeabi_float_t a, aeabi_float_t b) +{ + return f32_lt(f32_from_f(a), f32_from_f(b)); +} + +int __aeabi_fcmple(aeabi_float_t a, aeabi_float_t b) +{ + return f32_le(f32_from_f(a), f32_from_f(b)); +} + +int __aeabi_fcmpge(aeabi_float_t a, aeabi_float_t b) +{ + return f32_le(f32_from_f(b), f32_from_f(a)); +} + +int __aeabi_fcmpgt(aeabi_float_t a, aeabi_float_t b) +{ + return f32_lt(f32_from_f(b), f32_from_f(a)); +} + +/* + * Table 6, Standard floating-point to integer conversions + */ + +int __aeabi_d2iz(aeabi_double_t a) +{ + return f64_to_i32_r_minMag(f64_from_d(a), false); +} + +unsigned __aeabi_d2uiz(aeabi_double_t a) +{ + return f64_to_ui32_r_minMag(f64_from_d(a), false); +} + +long long __aeabi_d2lz(aeabi_double_t a) +{ + return f64_to_i64_r_minMag(f64_from_d(a), false); +} + +unsigned long long __aeabi_d2ulz(aeabi_double_t a) +{ + return f64_to_ui64_r_minMag(f64_from_d(a), false); +} + +int __aeabi_f2iz(aeabi_float_t a) +{ + return f32_to_i32_r_minMag(f32_from_f(a), false); +} + +unsigned __aeabi_f2uiz(aeabi_float_t a) +{ + return f32_to_ui32_r_minMag(f32_from_f(a), false); +} + +long long __aeabi_f2lz(aeabi_float_t a) +{ + return f32_to_i64_r_minMag(f32_from_f(a), false); +} + +unsigned long long __aeabi_f2ulz(aeabi_float_t a) +{ + return f32_to_ui64_r_minMag(f32_from_f(a), false); +} + +/* + * Table 7, Standard conversions between floating types + */ + +aeabi_float_t __aeabi_d2f(aeabi_double_t a) +{ + return f32_to_f(f64_to_f32(f64_from_d(a))); +} + +aeabi_double_t __aeabi_f2d(aeabi_float_t a) +{ + return f64_to_d(f32_to_f64(f32_from_f(a))); +} + +/* + * Table 8, Standard integer to floating-point conversions + */ + +aeabi_double_t __aeabi_i2d(int a) +{ + return f64_to_d(i32_to_f64(a)); +} + +aeabi_double_t __aeabi_ui2d(unsigned a) +{ + return f64_to_d(ui32_to_f64(a)); +} + +aeabi_double_t __aeabi_l2d(long long a) +{ + return f64_to_d(i64_to_f64(a)); +} + +aeabi_double_t __aeabi_ul2d(unsigned long long a) +{ + return f64_to_d(ui64_to_f64(a)); +} + +aeabi_float_t __aeabi_i2f(int a) +{ + return f32_to_f(i32_to_f32(a)); +} + +aeabi_float_t __aeabi_ui2f(unsigned a) +{ + return f32_to_f(ui32_to_f32(a)); +} + +aeabi_float_t __aeabi_l2f(long long a) +{ + return f32_to_f(i64_to_f32(a)); +} + +aeabi_float_t __aeabi_ul2f(unsigned long long a) +{ + return f32_to_f(ui64_to_f32(a)); +} diff --git a/ArmPkg/Library/CompilerIntrinsicsLib/memset.c b/ArmPkg/Library= /CompilerIntrinsicsLib/memset.c index c5ae32e5ee71..24398d591f79 100644 --- a/ArmPkg/Library/CompilerIntrinsicsLib/memset.c +++ b/ArmPkg/Library/CompilerIntrinsicsLib/memset.c @@ -1,55 +1,55 @@ -//------------------------------------------------------------------------= ------ -// -// Copyright (c) 2016, Linaro Ltd. All rights reserved.
-// -// SPDX-License-Identifier: BSD-2-Clause-Patent -// -//------------------------------------------------------------------------= ------ - -typedef __SIZE_TYPE__ size_t; - -static __attribute__((__used__)) -void *__memset(void *s, int c, size_t n) -{ - unsigned char *d =3D s; - - while (n--) - *d++ =3D c; - - return s; -} - -// -// Other modules (such as CryptoPkg/IntrinsicLib) may provide another -// implementation of memset(), which may conflict with this one if this -// object was pulled into the link due to the definitions below. So make -// our memset() 'weak' to let the other implementation take precedence. -// -__attribute__((__weak__, __alias__("__memset"))) -void *memset(void *dest, int c, size_t n); - -#ifdef __arm__ - -void __aeabi_memset(void *dest, size_t n, int c) -{ - __memset(dest, c, n); -} - -__attribute__((__alias__("__aeabi_memset"))) -void __aeabi_memset4(void *dest, size_t n, int c); - -__attribute__((__alias__("__aeabi_memset"))) -void __aeabi_memset8(void *dest, size_t n, int c); - -void __aeabi_memclr(void *dest, size_t n) -{ - __memset(dest, 0, n); -} - -__attribute__((__alias__("__aeabi_memclr"))) -void __aeabi_memclr4(void *dest, size_t n); - -__attribute__((__alias__("__aeabi_memclr"))) -void __aeabi_memclr8(void *dest, size_t n); - -#endif +//------------------------------------------------------------------------= ------ +// +// Copyright (c) 2016, Linaro Ltd. All rights reserved.
+// +// SPDX-License-Identifier: BSD-2-Clause-Patent +// +//------------------------------------------------------------------------= ------ + +typedef __SIZE_TYPE__ size_t; + +static __attribute__((__used__)) +void *__memset(void *s, int c, size_t n) +{ + unsigned char *d =3D s; + + while (n--) + *d++ =3D c; + + return s; +} + +// +// Other modules (such as CryptoPkg/IntrinsicLib) may provide another +// implementation of memset(), which may conflict with this one if this +// object was pulled into the link due to the definitions below. So make +// our memset() 'weak' to let the other implementation take precedence. +// +__attribute__((__weak__, __alias__("__memset"))) +void *memset(void *dest, int c, size_t n); + +#ifdef __arm__ + +void __aeabi_memset(void *dest, size_t n, int c) +{ + __memset(dest, c, n); +} + +__attribute__((__alias__("__aeabi_memset"))) +void __aeabi_memset4(void *dest, size_t n, int c); + +__attribute__((__alias__("__aeabi_memset"))) +void __aeabi_memset8(void *dest, size_t n, int c); + +void __aeabi_memclr(void *dest, size_t n) +{ + __memset(dest, 0, n); +} + +__attribute__((__alias__("__aeabi_memclr"))) +void __aeabi_memclr4(void *dest, size_t n); + +__attribute__((__alias__("__aeabi_memclr"))) +void __aeabi_memclr8(void *dest, size_t n); + +#endif diff --git a/ArmPkg/Library/GccLto/liblto-aarch64.s b/ArmPkg/Library/GccLto= /liblto-aarch64.s index 66819534a045..02a55ef4456c 100644 --- a/ArmPkg/Library/GccLto/liblto-aarch64.s +++ b/ArmPkg/Library/GccLto/liblto-aarch64.s @@ -1,21 +1,21 @@ -// -// Copyright (c) 2016, Linaro Ltd. All rights reserved.
-// -// SPDX-License-Identifier: BSD-2-Clause-Patent -// - -// -// GCC in LTO mode interoperates poorly with non-standard libraries that -// provide implementations of compiler intrinsics such as memcpy/memset -// or the stack protector entry points. -// -// By referencing these functions from a non-LTO object that can be passed -// to the linker via the -plugin-opt=3D-pass-through=3D-lxxx options, the -// intrinsics are included in the link in a way that allows them to be -// pruned again if no other references to them exist. -// - - .long memcpy - . - .long memset - . - .long __stack_chk_fail - . - .long __stack_chk_guard - . +// +// Copyright (c) 2016, Linaro Ltd. All rights reserved.
+// +// SPDX-License-Identifier: BSD-2-Clause-Patent +// + +// +// GCC in LTO mode interoperates poorly with non-standard libraries that +// provide implementations of compiler intrinsics such as memcpy/memset +// or the stack protector entry points. +// +// By referencing these functions from a non-LTO object that can be passed +// to the linker via the -plugin-opt=3D-pass-through=3D-lxxx options, the +// intrinsics are included in the link in a way that allows them to be +// pruned again if no other references to them exist. +// + + .long memcpy - . + .long memset - . + .long __stack_chk_fail - . + .long __stack_chk_guard - . diff --git a/ArmPkg/Library/GccLto/liblto-arm.s b/ArmPkg/Library/GccLto/lib= lto-arm.s index 4b26d4320d52..f19fb455519d 100644 --- a/ArmPkg/Library/GccLto/liblto-arm.s +++ b/ArmPkg/Library/GccLto/liblto-arm.s @@ -1,55 +1,55 @@ -// -// Copyright (c) 2016, Linaro Ltd. All rights reserved.
-// -// SPDX-License-Identifier: BSD-2-Clause-Patent -// - -// -// GCC in LTO mode interoperates poorly with non-standard libraries that -// provide implementations of compiler intrinsics such as memcpy/memset -// or the stack protector entry points. -// -// By referencing these functions from a non-LTO object that can be passed -// to the linker via the -plugin-opt=3D-pass-through=3D-lxxx options, the -// intrinsics are included in the link in a way that allows them to be -// pruned again if no other references to them exist. -// - - .long memcpy - . - .long memset - . - .long __stack_chk_fail - . - .long __stack_chk_guard - . - .long __ashrdi3 - . - .long __ashldi3 - . - .long __aeabi_idiv - . - .long __aeabi_idivmod - . - .long __aeabi_uidiv - . - .long __aeabi_uidivmod - . - .long __divdi3 - . - .long __divsi3 - . - .long __lshrdi3 - . - .long __aeabi_memcpy - . - .long __aeabi_memset - . - .long memmove - . - .long __modsi3 - . - .long __moddi3 - . - .long __muldi3 - . - .long __aeabi_lmul - . - .long __ARM_ll_mullu - . - .long __udivsi3 - . - .long __umodsi3 - . - .long __udivdi3 - . - .long __umoddi3 - . - .long __udivmoddi4 - . - .long __clzsi2 - . - .long __ctzsi2 - . - .long __ucmpdi2 - . - .long __switch8 - . - .long __switchu8 - . - .long __switch16 - . - .long __switch32 - . - .long __aeabi_ulcmp - . - .long __aeabi_uldivmod - . - .long __aeabi_ldivmod - . - .long __aeabi_llsr - . - .long __aeabi_llsl - . +// +// Copyright (c) 2016, Linaro Ltd. All rights reserved.
+// +// SPDX-License-Identifier: BSD-2-Clause-Patent +// + +// +// GCC in LTO mode interoperates poorly with non-standard libraries that +// provide implementations of compiler intrinsics such as memcpy/memset +// or the stack protector entry points. +// +// By referencing these functions from a non-LTO object that can be passed +// to the linker via the -plugin-opt=3D-pass-through=3D-lxxx options, the +// intrinsics are included in the link in a way that allows them to be +// pruned again if no other references to them exist. +// + + .long memcpy - . + .long memset - . + .long __stack_chk_fail - . + .long __stack_chk_guard - . + .long __ashrdi3 - . + .long __ashldi3 - . + .long __aeabi_idiv - . + .long __aeabi_idivmod - . + .long __aeabi_uidiv - . + .long __aeabi_uidivmod - . + .long __divdi3 - . + .long __divsi3 - . + .long __lshrdi3 - . + .long __aeabi_memcpy - . + .long __aeabi_memset - . + .long memmove - . + .long __modsi3 - . + .long __moddi3 - . + .long __muldi3 - . + .long __aeabi_lmul - . + .long __ARM_ll_mullu - . + .long __udivsi3 - . + .long __umodsi3 - . + .long __udivdi3 - . + .long __umoddi3 - . + .long __udivmoddi4 - . + .long __clzsi2 - . + .long __ctzsi2 - . + .long __ucmpdi2 - . + .long __switch8 - . + .long __switchu8 - . + .long __switch16 - . + .long __switch32 - . + .long __aeabi_ulcmp - . + .long __aeabi_uldivmod - . + .long __aeabi_ldivmod - . + .long __aeabi_llsr - . + .long __aeabi_llsl - . --=20 2.19.1.3.g30247aa5d201 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#55038): https://edk2.groups.io/g/devel/message/55038 Mute This Topic: https://groups.io/mt/71596309/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- From nobody Thu May 2 08:02:07 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+55040+1787277+3901457@groups.io; helo=web01.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+55040+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1582839559; cv=none; d=zohomail.com; s=zohoarc; b=GTv+gsxe/b+9Z6oD8PNpb5HQZ+Zqz9Dr+pquOHA/laY/it41krvjZQyG9gtMq8C1X3BCYY0s3E0lSzE7GlgJr+SGr8c1FelMYD0J/N+gqtTnrXW3LeN3lk6cutM19e2eMmWZtj47iSyjSck7bFD/CAtSuUILjpP7j4WEa3l6RBc= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1582839559; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=85TQfNHcKC6QFXzjR+t+/FJbGnam4DLXlmprox/vxdg=; b=QnD1/KnmpxD2RGVlOoxy6kFLfkwo5zy+QVxG97f4if6D8HiVBVXXmgHe/vGkbzJ1d9jsYDU1HMAreUIYviJAR8cazRCd/b2DrrvaTwK/ZsWPRtCDcfnipSZHyNHBiccd0KHO/6w1sBjh7tKlf2p8oJyv8xrr56FiCSW3DtmYXoA= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+55040+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 1582839559012611.8793491701557; Thu, 27 Feb 2020 13:39:19 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id 3aqrYY1788612xHYvaTY6auX; Thu, 27 Feb 2020 13:39:17 -0800 X-Received: from us-smtp-1.mimecast.com (us-smtp-1.mimecast.com [207.211.31.120]) by mx.groups.io with SMTP id smtpd.web11.1020.1582839557351459380 for ; Thu, 27 Feb 2020 13:39:17 -0800 X-Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-59-209WVClgMOua0Sm3kb_eIA-1; Thu, 27 Feb 2020 16:39:10 -0500 X-MC-Unique: 209WVClgMOua0Sm3kb_eIA-1 X-Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 9BC29189F760; Thu, 27 Feb 2020 21:39:09 +0000 (UTC) X-Received: from lacos-laptop-7.usersys.redhat.com (ovpn-116-46.ams2.redhat.com [10.36.116.46]) by smtp.corp.redhat.com (Postfix) with ESMTP id 7899D90BC; Thu, 27 Feb 2020 21:39:08 +0000 (UTC) From: "Laszlo Ersek" To: edk2-devel-groups-io Cc: Ard Biesheuvel , Leif Lindholm , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= Subject: [edk2-devel] [PATCH 2/3] ArmVirtPkg: convert LFs to CRLF Date: Thu, 27 Feb 2020 22:39:02 +0100 Message-Id: <20200227213903.13884-3-lersek@redhat.com> In-Reply-To: <20200227213903.13884-1-lersek@redhat.com> References: <20200227213903.13884-1-lersek@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Precedence: Bulk List-Unsubscribe: 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,lersek@redhat.com X-Gm-Message-State: eGUojBLb4KlgiRa9MGlDpIkcx1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1582839557; bh=85TQfNHcKC6QFXzjR+t+/FJbGnam4DLXlmprox/vxdg=; h=Cc:Content-Type:Date:From:Reply-To:Subject:To; b=qv7mtPdwVR3JXi4EEA+hlGkuuy0+UiPYfF8JvM0dQvI6cHd+O17pQACzfP8tE7QIBM9 eUgq2xXuy4J01U1N1X7euTVhCxoMosEzT506Z1EI4mgD6w9fyox4Sm7t6m+PotCkvRPMV Y+jqOa5ejk3g32bWhxDv7b06fCEA09m+P/o= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" We're going to switch the internal line terminators globally to LF at some point, but until then, let's use CRLF consistently. Convert source files with LFs in them to CRLF, using "unix2dos". "git show -b" prints no code changes for this patch. (I collected all the file name suffixes in this package, with: $ git ls-files -- $PACKAGE | rev | cut -f 1 -d . | sort -u | rev I eliminated those suffixes that didn't stand for text files, then blanket-converted the rest with unix2dos. Finally, picked up the actual changes with git-add.) The CRLF conversion is motivated by "PatchCheck.py". Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Philippe Mathieu-Daud=C3=A9 Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3D1659 Signed-off-by: Laszlo Ersek Acked-by: Ard Biesheuvel Reviewed-by: Philippe Mathieu-Daude --- ArmVirtPkg/Include/Platform/Hidden.h | 44 ++++++++++---------- 1 file changed, 22 insertions(+), 22 deletions(-) diff --git a/ArmVirtPkg/Include/Platform/Hidden.h b/ArmVirtPkg/Include/Plat= form/Hidden.h index c69cc662fa62..7a7bdb42b8bd 100644 --- a/ArmVirtPkg/Include/Platform/Hidden.h +++ b/ArmVirtPkg/Include/Platform/Hidden.h @@ -1,22 +1,22 @@ -/** @file - - Copyright (c) 2018, Linaro Limited. All rights reserved. - - SPDX-License-Identifier: BSD-2-Clause-Patent - -**/ - -#ifndef __PLATFORM_HIDDEN_H -#define __PLATFORM_HIDDEN_H - -// -// Setting the GCC -fvisibility=3Dhidden command line option is not quite = the same -// as setting the pragma below: the former only affects definitions, where= as the -// pragma affects extern declarations as well. So if we want to ensure tha= t no -// GOT indirected symbol references are emitted, we need to use the pragma= , or -// GOT based cross object references could be emitted, e.g., in libraries,= and -// these cannot be relaxed to ordinary symbol references at link time. -// -#pragma GCC visibility push (hidden) - -#endif +/** @file + + Copyright (c) 2018, Linaro Limited. All rights reserved. + + SPDX-License-Identifier: BSD-2-Clause-Patent + +**/ + +#ifndef __PLATFORM_HIDDEN_H +#define __PLATFORM_HIDDEN_H + +// +// Setting the GCC -fvisibility=3Dhidden command line option is not quite = the same +// as setting the pragma below: the former only affects definitions, where= as the +// pragma affects extern declarations as well. So if we want to ensure tha= t no +// GOT indirected symbol references are emitted, we need to use the pragma= , or +// GOT based cross object references could be emitted, e.g., in libraries,= and +// these cannot be relaxed to ordinary symbol references at link time. +// +#pragma GCC visibility push (hidden) + +#endif --=20 2.19.1.3.g30247aa5d201 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#55040): https://edk2.groups.io/g/devel/message/55040 Mute This Topic: https://groups.io/mt/71596311/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- From nobody Thu May 2 08:02:07 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+55041+1787277+3901457@groups.io; helo=web01.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+55041+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1582839561; cv=none; d=zohomail.com; s=zohoarc; b=m7v+8BLm3wwkmUug/7WmaacABcfo5ug9oMmMZ0go18W+s3YND2o5vO3HUa+Cn1dqFv7CHQKwwTeTpz+Tlzr1b7haSvQ0FB0YHwhvsOVjwPEBHjOC5hPxs68+NLez0azaJhLl9T/oWdInONhfp+0xu/NFBeu+ap5LXyFYpsArJ/A= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1582839561; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=HJ3M0y6bJiF8uDl0mb/rODJQdVS77WgJXX9AzzOAzjA=; b=kZf4s9WZ9N3cM5XSLH7XUVl535j+jpDASWIDqQ9Da7CWbwZnEBlyBlojGIQAfY0i5xbE4UU8uqobb99v4pSuF12hiu3ue5nZTGJnbyc9HQARS/1XoQhv3NoLopMrzIrOrQTiS8KzLkwxtWFNEUm/BG6dWn005brAj6+K9JqSuGI= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+55041+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 1582839561426403.97015846984687; Thu, 27 Feb 2020 13:39:21 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id rFcJYY1788612xCSTfazecgR; Thu, 27 Feb 2020 13:39:20 -0800 X-Received: from us-smtp-delivery-1.mimecast.com (us-smtp-delivery-1.mimecast.com [205.139.110.61]) by mx.groups.io with SMTP id smtpd.web12.1012.1582839559312709699 for ; Thu, 27 Feb 2020 13:39:19 -0800 X-Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-74-14hZ2ZopPhKXZ5sZ3sIFwg-1; Thu, 27 Feb 2020 16:39:12 -0500 X-MC-Unique: 14hZ2ZopPhKXZ5sZ3sIFwg-1 X-Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 4CA79107ACC5; Thu, 27 Feb 2020 21:39:11 +0000 (UTC) X-Received: from lacos-laptop-7.usersys.redhat.com (ovpn-116-46.ams2.redhat.com [10.36.116.46]) by smtp.corp.redhat.com (Postfix) with ESMTP id F16E690BC; Thu, 27 Feb 2020 21:39:09 +0000 (UTC) From: "Laszlo Ersek" To: edk2-devel-groups-io Cc: Ard Biesheuvel , Leif Lindholm , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= Subject: [edk2-devel] [PATCH 3/3] ArmPlatformPkg: convert LFs to CRLF, expand hard TABs Date: Thu, 27 Feb 2020 22:39:03 +0100 Message-Id: <20200227213903.13884-4-lersek@redhat.com> In-Reply-To: <20200227213903.13884-1-lersek@redhat.com> References: <20200227213903.13884-1-lersek@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Precedence: Bulk List-Unsubscribe: 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,lersek@redhat.com X-Gm-Message-State: lVXf28GdWMNdv6847kZ5SMcYx1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1582839560; bh=HJ3M0y6bJiF8uDl0mb/rODJQdVS77WgJXX9AzzOAzjA=; h=Cc:Content-Type:Date:From:Reply-To:Subject:To; b=UvFtTUG3Y9Up1sPfqDLi0DB3XOcDHxD8kD14hYgjSNsLPf4PNpVTvyH1Qer/qkrcZxg wGVlYDTfvXPUplBLsmTEhLBB27ObYap90q5g3AyzEeMjk2dXR1V7C9h7D7Lle0uoyTiAo +qFt4WTJf9X1JXnKhhGtUk02H/sEmEpKYes= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" We're going to switch the internal line terminators globally to LF at some point, but until then, let's use CRLF consistently. Convert source files with LFs in them to CRLF, using "unix2dos". "git show -b" prints no code changes for this patch. (I collected all the file name suffixes in this package, with: $ git ls-files -- $PACKAGE | rev | cut -f 1 -d . | sort -u | rev I eliminated those suffixes that didn't stand for text files, then blanket-converted the rest with unix2dos. Finally, picked up the actual changes with git-add.) At the same time, the following file had to undergo TAB expansion: ArmPlatformPkg/Scripts/Ds5/profile.py I used "expand -t 4", conforming to the Indentation section of PEP-8 . Both the CRLF conversion and the TAB expansion are motivated by "PatchCheck.py". "PatchCheck.py" is also the reason why CRLF conversion and TAB expansion have to happen in the same patch. Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Philippe Mathieu-Daud=C3=A9 Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3D1659 Signed-off-by: Laszlo Ersek Acked-by: Ard Biesheuvel Reviewed-by: Philippe Mathieu-Daude --- ArmPlatformPkg/Scripts/Ds5/profile.py | 656 ++++++++++---------- 1 file changed, 328 insertions(+), 328 deletions(-) diff --git a/ArmPlatformPkg/Scripts/Ds5/profile.py b/ArmPlatformPkg/Scripts= /Ds5/profile.py index f87dee24695c..979c6ea2bdc2 100644 --- a/ArmPlatformPkg/Scripts/Ds5/profile.py +++ b/ArmPlatformPkg/Scripts/Ds5/profile.py @@ -1,328 +1,328 @@ -#!/usr/bin/python - -# -# Copyright (c) 2014, ARM Limited. All rights reserved. -# -# SPDX-License-Identifier: BSD-2-Clause-Patent -# - -import getopt -import operator -import os -import pickle -import sys -from sys import argv -from cStringIO import StringIO - -modules =3D {} -functions =3D {} -functions_addr =3D {} - -def usage(): - print "-t,--trace: Location of the Trace file" - print "-s,--symbols: Location of the symbols and modules" - -def get_address_from_string(address): - return int(address.strip("S:").strip("N:").strip("EL2:").strip("EL1:"), 1= 6) - -def get_module_from_addr(modules, addr): - for key,value in modules.items(): - if (value['start'] <=3D addr) and (addr <=3D value['end']): - return key - return None - -def add_cycles_to_function(functions, func_name, addr, cycles): - if func_name !=3D "": - # Check if we are still in the previous function - if add_cycles_to_function.prev_func_name =3D=3D func_name: - add_cycles_to_function.prev_entry['cycles'] +=3D cycles - return (add_cycles_to_function.prev_func_name, add_cycles_to_function.p= rev_module_name) - - if func_name in functions.keys(): - for module_name, module_value in functions[func_name].iteritems(): - if (module_value['start'] <=3D addr) and (addr < module_value['end']): - module_value['cycles'] +=3D cycles - - add_cycles_to_function.prev_func_name =3D func_name - add_cycles_to_function.prev_module_name =3D module_name - add_cycles_to_function.prev_entry =3D module_value - return (func_name, module_name) - elif (module_value['end'] =3D=3D 0): - module_value['cycles'] +=3D cycles - - add_cycles_to_function.prev_func_name =3D func_name - add_cycles_to_function.prev_module_name =3D module_name - add_cycles_to_function.prev_entry =3D module_value - return (func_name, module_name) - - # Workaround to fix the 'info func' limitation that does not expose the = 'static' function - module_name =3D get_module_from_addr(modules, addr) - functions[func_name] =3D {} - functions[func_name][module_name] =3D {} - functions[func_name][module_name]['start'] =3D 0 - functions[func_name][module_name]['end'] =3D 0 - functions[func_name][module_name]['cycles'] =3D cycles - functions[func_name][module_name]['count'] =3D 0 - - add_cycles_to_function.prev_func_name =3D func_name - add_cycles_to_function.prev_module_name =3D module_name - add_cycles_to_function.prev_entry =3D functions[func_name][module_= name] - return (func_name, module_name) - else: - # Check if we are still in the previous function - if (add_cycles_to_function.prev_entry is not None) and (add_cycles_to_fu= nction.prev_entry['start'] <=3D addr) and (addr < add_cycles_to_function.pr= ev_entry['end']): - add_cycles_to_function.prev_entry['cycles'] +=3D cycles - return (add_cycles_to_function.prev_func_name, add_cycles_to_function.p= rev_module_name) - - # Generate the key for the given address - key =3D addr & ~0x0FFF - - if key not in functions_addr.keys(): - if 'Unknown' not in functions.keys(): - functions['Unknown'] =3D {} - if 'Unknown' not in functions['Unknown'].keys(): - functions['Unknown']['Unknown'] =3D {} - functions['Unknown']['Unknown']['cycles'] =3D 0 - functions['Unknown']['Unknown']['count'] =3D 0 - functions['Unknown']['Unknown']['cycles'] +=3D cycles - - add_cycles_to_function.prev_func_name =3D None - return None - - for func_key, module in functions_addr[key].iteritems(): - for module_key, module_value in module.iteritems(): - if (module_value['start'] <=3D addr) and (addr < module_value['end']): - module_value['cycles'] +=3D cycles - - # In case o we prefer to fallback on the direct search - add_cycles_to_function.prev_func_name =3D func_key - add_cycles_to_function.prev_module_name =3D module_key - add_cycles_to_function.prev_entry =3D module_value - return (func_key, module_key) - - print "Warning: Function %s @ 0x%x not found" % (func_name, addr) - - add_cycles_to_function.prev_func_name =3D None - return None - -# Static variables for the previous function -add_cycles_to_function.prev_func_name =3D None -add_cycles_to_function.prev_entry =3D None - -def trace_read(): - global trace_process - line =3D trace.readline() - trace_process +=3D len(line) - return line - -# -# Parse arguments -# -trace_name =3D None -symbols_file =3D None - -opts,args =3D getopt.getopt(sys.argv[1:], "ht:vs:v", ["help","trace=3D","s= ymbols=3D"]) -if (opts is None) or (not opts): - usage() - sys.exit() - -for o,a in opts: - if o in ("-h","--help"): - usage() - sys.exit() - elif o in ("-t","--trace"): - trace_name =3D a - elif o in ("-s","--symbols"): - symbols_file =3D a - else: - assert False, "Unhandled option (%s)" % o - -# -# We try first to see if we run the script from DS-5 -# -try: - from arm_ds.debugger_v1 import Debugger - from arm_ds.debugger_v1 import DebugException - - # Debugger object for accessing the debugger - debugger =3D Debugger() - - # Initialisation commands - ec =3D debugger.getExecutionContext(0) - ec.getExecutionService().stop() - ec.getExecutionService().waitForStop() - # in case the execution context reference is out of date - ec =3D debugger.getExecutionContext(0) - - # - # Get the module name and their memory range - # - info_file =3D ec.executeDSCommand("info file") - info_file_str =3D StringIO(info_file) - - line =3D info_file_str.readline().strip('\n') - while line !=3D '': - if ("Symbols from" in line): - # Get the module name from the line 'Symbols from "/home/...."' - module_name =3D line.split("\"")[1].split("/")[-1] - modules[module_name] =3D {} - - # Look for the text section - line =3D info_file_str.readline().strip('\n') - while (line !=3D '') and ("Symbols from" not in line): - if ("ER_RO" in line): - modules[module_name]['start'] =3D get_address_from_string(line.split(= )[0]) - modules[module_name]['end'] =3D get_address_from_string(line.split(= )[2]) - line =3D info_file_str.readline().strip('\n') - break; - if (".text" in line): - modules[module_name]['start'] =3D get_address_from_string(line.split(= )[0]) - modules[module_name]['end'] =3D get_address_from_string(line.split(= )[2]) - line =3D info_file_str.readline().strip('\n') - break; - line =3D info_file_str.readline().strip('\n') - line =3D info_file_str.readline().strip('\n') - - # - # Get the function name and their memory range - # - info_func =3D ec.executeDSCommand("info func") - info_func_str =3D StringIO(info_func) - - # Skip the first line 'Low-level symbols ...' - line =3D info_func_str.readline().strip('\n') - func_prev =3D None - while line !=3D '': - # We ignore all the functions after 'Functions in' - if ("Functions in " in line): - line =3D info_func_str.readline().strip('\n') - while line !=3D '': - line =3D info_func_str.readline().strip('\n') - line =3D info_func_str.readline().strip('\n') - continue - - if ("Low-level symbols" in line): - # We need to fixup the last function of the module - if func_prev is not None: - func_prev['end'] =3D modules[module_name]['end'] - func_prev =3D None - - line =3D info_func_str.readline().strip('\n') - continue - - func_name =3D line.split()[1] - func_start =3D get_address_from_string(line.split()[0]) - module_name =3D get_module_from_addr(modules, func_start) - - if func_name not in functions.keys(): - functions[func_name] =3D {} - functions[func_name][module_name] =3D {} - functions[func_name][module_name]['start'] =3D func_start - functions[func_name][module_name]['cycles'] =3D 0 - functions[func_name][module_name]['count'] =3D 0 - - # Set the end address of the previous function - if func_prev is not None: - func_prev['end'] =3D func_start - func_prev =3D functions[func_name][module_name] - - line =3D info_func_str.readline().strip('\n') - - # Fixup the last function - func_prev['end'] =3D modules[module_name]['end'] - - if symbols_file is not None: - pickle.dump((modules, functions), open(symbols_file, "w")) -except: - if symbols_file is None: - print "Error: Symbols file is required when run out of ARM DS-5" - sys.exit() - - (modules, functions) =3D pickle.load(open(symbols_file, "r")) - -# -# Build optimized table for the functions -# -functions_addr =3D {} -for func_key, module in functions.iteritems(): - for module_key, module_value in module.iteritems(): - key =3D module_value['start'] & ~0x0FFF - if key not in functions_addr.keys(): - functions_addr[key] =3D {} - if func_key not in functions_addr[key].keys(): - functions_addr[key][func_key] =3D {} - functions_addr[key][func_key][module_key] =3D module_value - -# -# Process the trace file -# -if trace_name is None: - sys.exit() - -trace =3D open(trace_name, "r") -trace_size =3D os.path.getsize(trace_name) -trace_process =3D 0 - -# Get the column names from the first line -columns =3D trace_read().split() -column_addr =3D columns.index('Address') -column_cycles =3D columns.index('Cycles') -column_function =3D columns.index('Function') - -line =3D trace_read() -i =3D 0 -prev_callee =3D None -while line: - try: - func_name =3D line.split('\t')[column_function].strip() - address =3D get_address_from_string(line.split('\t')[column_addr]) - cycles =3D int(line.split('\t')[column_cycles]) - callee =3D add_cycles_to_function(functions, func_name, address, cycles) - if (prev_callee !=3D None) and (prev_callee !=3D callee): - functions[prev_callee[0]][prev_callee[1]]['count'] +=3D 1 - prev_callee =3D callee - except ValueError: - pass - line =3D trace_read() - if ((i % 1000000) =3D=3D 0) and (i !=3D 0): - percent =3D (trace_process * 100.00) / trace_size - print "Processing file ... (%.2f %%)" % (percent) - i =3D i + 1 - -# Fixup the last callee -functions[prev_callee[0]][prev_callee[1]]['count'] +=3D 1 - -# -# Process results -# -functions_cycles =3D {} -all_functions_cycles =3D {} -total_cycles =3D 0 - -for func_key, module in functions.iteritems(): - for module_key, module_value in module.iteritems(): - key =3D "%s/%s" % (module_key, func_key) - functions_cycles[key] =3D (module_value['cycles'], module_value['count']) - total_cycles +=3D module_value['cycles'] - - if func_key not in all_functions_cycles.keys(): - all_functions_cycles[func_key] =3D (module_value['cycles'], module_valu= e['count']) - else: - all_functions_cycles[func_key] =3D tuple(map(sum, zip(all_functions_cyc= les[func_key], (module_value['cycles'], module_value['count'])))) - -sorted_functions_cycles =3D sorted(functions_cycles.iteritems(), key= =3Doperator.itemgetter(1), reverse =3D True) -sorted_all_functions_cycles =3D sorted(all_functions_cycles.items(), key= =3Doperator.itemgetter(1), reverse =3D True) - -print -print "----" -for (key,value) in sorted_functions_cycles[:20]: - if value[0] !=3D 0: - print "%s (cycles: %d - %d%%, count: %d)" % (key, value[0], (value[0] * = 100) / total_cycles, value[1]) - else: - break; -print "----" -for (key,value) in sorted_all_functions_cycles[:20]: - if value[0] !=3D 0: - print "%s (cycles: %d - %d%%, count: %d)" % (key, value[0], (value[0] * = 100) / total_cycles, value[1]) - else: - break; +#!/usr/bin/python + +# +# Copyright (c) 2014, ARM Limited. All rights reserved. +# +# SPDX-License-Identifier: BSD-2-Clause-Patent +# + +import getopt +import operator +import os +import pickle +import sys +from sys import argv +from cStringIO import StringIO + +modules =3D {} +functions =3D {} +functions_addr =3D {} + +def usage(): + print "-t,--trace: Location of the Trace file" + print "-s,--symbols: Location of the symbols and modules" + +def get_address_from_string(address): + return int(address.strip("S:").strip("N:").strip("EL2:").strip("EL1:")= , 16) + +def get_module_from_addr(modules, addr): + for key,value in modules.items(): + if (value['start'] <=3D addr) and (addr <=3D value['end']): + return key + return None + +def add_cycles_to_function(functions, func_name, addr, cycles): + if func_name !=3D "": + # Check if we are still in the previous function + if add_cycles_to_function.prev_func_name =3D=3D func_name: + add_cycles_to_function.prev_entry['cycles'] +=3D cycles + return (add_cycles_to_function.prev_func_name, add_cycles_to_f= unction.prev_module_name) + + if func_name in functions.keys(): + for module_name, module_value in functions[func_name].iteritem= s(): + if (module_value['start'] <=3D addr) and (addr < module_va= lue['end']): + module_value['cycles'] +=3D cycles + + add_cycles_to_function.prev_func_name =3D func_name + add_cycles_to_function.prev_module_name =3D module_name + add_cycles_to_function.prev_entry =3D module_val= ue + return (func_name, module_name) + elif (module_value['end'] =3D=3D 0): + module_value['cycles'] +=3D cycles + + add_cycles_to_function.prev_func_name =3D func_name + add_cycles_to_function.prev_module_name =3D module_name + add_cycles_to_function.prev_entry =3D module_val= ue + return (func_name, module_name) + + # Workaround to fix the 'info func' limitation that does not expos= e the 'static' function + module_name =3D get_module_from_addr(modules, addr) + functions[func_name] =3D {} + functions[func_name][module_name] =3D {} + functions[func_name][module_name]['start'] =3D 0 + functions[func_name][module_name]['end'] =3D 0 + functions[func_name][module_name]['cycles'] =3D cycles + functions[func_name][module_name]['count'] =3D 0 + + add_cycles_to_function.prev_func_name =3D func_name + add_cycles_to_function.prev_module_name =3D module_name + add_cycles_to_function.prev_entry =3D functions[func_name][m= odule_name] + return (func_name, module_name) + else: + # Check if we are still in the previous function + if (add_cycles_to_function.prev_entry is not None) and (add_cycles= _to_function.prev_entry['start'] <=3D addr) and (addr < add_cycles_to_funct= ion.prev_entry['end']): + add_cycles_to_function.prev_entry['cycles'] +=3D cycles + return (add_cycles_to_function.prev_func_name, add_cycles_to_f= unction.prev_module_name) + + # Generate the key for the given address + key =3D addr & ~0x0FFF + + if key not in functions_addr.keys(): + if 'Unknown' not in functions.keys(): + functions['Unknown'] =3D {} + if 'Unknown' not in functions['Unknown'].keys(): + functions['Unknown']['Unknown'] =3D {} + functions['Unknown']['Unknown']['cycles'] =3D 0 + functions['Unknown']['Unknown']['count'] =3D 0 + functions['Unknown']['Unknown']['cycles'] +=3D cycles + + add_cycles_to_function.prev_func_name =3D None + return None + + for func_key, module in functions_addr[key].iteritems(): + for module_key, module_value in module.iteritems(): + if (module_value['start'] <=3D addr) and (addr < module_va= lue['end']): + module_value['cycles'] +=3D cycles + + # In case o we prefer to fallback on the dir= ect search + add_cycles_to_function.prev_func_name =3D func_key + add_cycles_to_function.prev_module_name =3D module_key + add_cycles_to_function.prev_entry =3D module_val= ue + return (func_key, module_key) + + print "Warning: Function %s @ 0x%x not found" % (func_name, addr) + + add_cycles_to_function.prev_func_name =3D None + return None + +# Static variables for the previous function +add_cycles_to_function.prev_func_name =3D None +add_cycles_to_function.prev_entry =3D None + +def trace_read(): + global trace_process + line =3D trace.readline() + trace_process +=3D len(line) + return line + +# +# Parse arguments +# +trace_name =3D None +symbols_file =3D None + +opts,args =3D getopt.getopt(sys.argv[1:], "ht:vs:v", ["help","trace=3D","s= ymbols=3D"]) +if (opts is None) or (not opts): + usage() + sys.exit() + +for o,a in opts: + if o in ("-h","--help"): + usage() + sys.exit() + elif o in ("-t","--trace"): + trace_name =3D a + elif o in ("-s","--symbols"): + symbols_file =3D a + else: + assert False, "Unhandled option (%s)" % o + +# +# We try first to see if we run the script from DS-5 +# +try: + from arm_ds.debugger_v1 import Debugger + from arm_ds.debugger_v1 import DebugException + + # Debugger object for accessing the debugger + debugger =3D Debugger() + + # Initialisation commands + ec =3D debugger.getExecutionContext(0) + ec.getExecutionService().stop() + ec.getExecutionService().waitForStop() + # in case the execution context reference is out of date + ec =3D debugger.getExecutionContext(0) + + # + # Get the module name and their memory range + # + info_file =3D ec.executeDSCommand("info file") + info_file_str =3D StringIO(info_file) + + line =3D info_file_str.readline().strip('\n') + while line !=3D '': + if ("Symbols from" in line): + # Get the module name from the line 'Symbols from "/home/...."' + module_name =3D line.split("\"")[1].split("/")[-1] + modules[module_name] =3D {} + + # Look for the text section + line =3D info_file_str.readline().strip('\n') + while (line !=3D '') and ("Symbols from" not in line): + if ("ER_RO" in line): + modules[module_name]['start'] =3D get_address_from_str= ing(line.split()[0]) + modules[module_name]['end'] =3D get_address_from_str= ing(line.split()[2]) + line =3D info_file_str.readline().strip('\n') + break; + if (".text" in line): + modules[module_name]['start'] =3D get_address_from_str= ing(line.split()[0]) + modules[module_name]['end'] =3D get_address_from_str= ing(line.split()[2]) + line =3D info_file_str.readline().strip('\n') + break; + line =3D info_file_str.readline().strip('\n') + line =3D info_file_str.readline().strip('\n') + + # + # Get the function name and their memory range + # + info_func =3D ec.executeDSCommand("info func") + info_func_str =3D StringIO(info_func) + + # Skip the first line 'Low-level symbols ...' + line =3D info_func_str.readline().strip('\n') + func_prev =3D None + while line !=3D '': + # We ignore all the functions after 'Functions in' + if ("Functions in " in line): + line =3D info_func_str.readline().strip('\n') + while line !=3D '': + line =3D info_func_str.readline().strip('\n') + line =3D info_func_str.readline().strip('\n') + continue + + if ("Low-level symbols" in line): + # We need to fixup the last function of the module + if func_prev is not None: + func_prev['end'] =3D modules[module_name]['end'] + func_prev =3D None + + line =3D info_func_str.readline().strip('\n') + continue + + func_name =3D line.split()[1] + func_start =3D get_address_from_string(line.split()[0]) + module_name =3D get_module_from_addr(modules, func_start) + + if func_name not in functions.keys(): + functions[func_name] =3D {} + functions[func_name][module_name] =3D {} + functions[func_name][module_name]['start'] =3D func_start + functions[func_name][module_name]['cycles'] =3D 0 + functions[func_name][module_name]['count'] =3D 0 + + # Set the end address of the previous function + if func_prev is not None: + func_prev['end'] =3D func_start + func_prev =3D functions[func_name][module_name] + + line =3D info_func_str.readline().strip('\n') + + # Fixup the last function + func_prev['end'] =3D modules[module_name]['end'] + + if symbols_file is not None: + pickle.dump((modules, functions), open(symbols_file, "w")) +except: + if symbols_file is None: + print "Error: Symbols file is required when run out of ARM DS-5" + sys.exit() + + (modules, functions) =3D pickle.load(open(symbols_file, "r")) + +# +# Build optimized table for the functions +# +functions_addr =3D {} +for func_key, module in functions.iteritems(): + for module_key, module_value in module.iteritems(): + key =3D module_value['start'] & ~0x0FFF + if key not in functions_addr.keys(): + functions_addr[key] =3D {} + if func_key not in functions_addr[key].keys(): + functions_addr[key][func_key] =3D {} + functions_addr[key][func_key][module_key] =3D module_value + +# +# Process the trace file +# +if trace_name is None: + sys.exit() + +trace =3D open(trace_name, "r") +trace_size =3D os.path.getsize(trace_name) +trace_process =3D 0 + +# Get the column names from the first line +columns =3D trace_read().split() +column_addr =3D columns.index('Address') +column_cycles =3D columns.index('Cycles') +column_function =3D columns.index('Function') + +line =3D trace_read() +i =3D 0 +prev_callee =3D None +while line: + try: + func_name =3D line.split('\t')[column_function].strip() + address =3D get_address_from_string(line.split('\t')[column_addr= ]) + cycles =3D int(line.split('\t')[column_cycles]) + callee =3D add_cycles_to_function(functions, func_name, address, c= ycles) + if (prev_callee !=3D None) and (prev_callee !=3D callee): + functions[prev_callee[0]][prev_callee[1]]['count'] +=3D 1 + prev_callee =3D callee + except ValueError: + pass + line =3D trace_read() + if ((i % 1000000) =3D=3D 0) and (i !=3D 0): + percent =3D (trace_process * 100.00) / trace_size + print "Processing file ... (%.2f %%)" % (percent) + i =3D i + 1 + +# Fixup the last callee +functions[prev_callee[0]][prev_callee[1]]['count'] +=3D 1 + +# +# Process results +# +functions_cycles =3D {} +all_functions_cycles =3D {} +total_cycles =3D 0 + +for func_key, module in functions.iteritems(): + for module_key, module_value in module.iteritems(): + key =3D "%s/%s" % (module_key, func_key) + functions_cycles[key] =3D (module_value['cycles'], module_value['c= ount']) + total_cycles +=3D module_value['cycles'] + + if func_key not in all_functions_cycles.keys(): + all_functions_cycles[func_key] =3D (module_value['cycles'], mo= dule_value['count']) + else: + all_functions_cycles[func_key] =3D tuple(map(sum, zip(all_func= tions_cycles[func_key], (module_value['cycles'], module_value['count'])))) + +sorted_functions_cycles =3D sorted(functions_cycles.iteritems(), key= =3Doperator.itemgetter(1), reverse =3D True) +sorted_all_functions_cycles =3D sorted(all_functions_cycles.items(), key= =3Doperator.itemgetter(1), reverse =3D True) + +print +print "----" +for (key,value) in sorted_functions_cycles[:20]: + if value[0] !=3D 0: + print "%s (cycles: %d - %d%%, count: %d)" % (key, value[0], (value= [0] * 100) / total_cycles, value[1]) + else: + break; +print "----" +for (key,value) in sorted_all_functions_cycles[:20]: + if value[0] !=3D 0: + print "%s (cycles: %d - %d%%, count: %d)" % (key, value[0], (value= [0] * 100) / total_cycles, value[1]) + else: + break; --=20 2.19.1.3.g30247aa5d201 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#55041): https://edk2.groups.io/g/devel/message/55041 Mute This Topic: https://groups.io/mt/71596312/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-