From nobody Mon Feb 9 18:44:34 2026 Delivered-To: importer@patchew.org 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+65562+1787277+3901457@groups.io; arc=fail (BodyHash is different from the expected one) Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 1600948173011862.0856548001148; Thu, 24 Sep 2020 04:49:33 -0700 (PDT) Return-Path: X-Received: by 127.0.0.2 with SMTP id a3k5YY1788612xnp82lPWpsY; Thu, 24 Sep 2020 04:49:32 -0700 X-Received: from EUR05-VI1-obe.outbound.protection.outlook.com (EUR05-VI1-obe.outbound.protection.outlook.com [40.107.21.40]) by mx.groups.io with SMTP id smtpd.web10.14832.1600948171094595083 for ; Thu, 24 Sep 2020 04:49:32 -0700 X-Received: from AM6P194CA0080.EURP194.PROD.OUTLOOK.COM (2603:10a6:209:8f::21) by HE1PR0802MB2457.eurprd08.prod.outlook.com (2603:10a6:3:df::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3391.19; Thu, 24 Sep 2020 11:49:27 +0000 X-Received: from VE1EUR03FT010.eop-EUR03.prod.protection.outlook.com (2603:10a6:209:8f:cafe::e0) by AM6P194CA0080.outlook.office365.com (2603:10a6:209:8f::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3412.20 via Frontend Transport; Thu, 24 Sep 2020 11:49:27 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 63.35.35.123) smtp.mailfrom=arm.com; edk2.groups.io; dkim=pass (signature was verified) header.d=armh.onmicrosoft.com;edk2.groups.io; dmarc=bestguesspass action=none header.from=arm.com; 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+65562+1787277+3901457@groups.io; helo=web01.groups.io; Received-SPF: Pass (protection.outlook.com: domain of arm.com designates 63.35.35.123 as permitted sender) receiver=protection.outlook.com; client-ip=63.35.35.123; helo=64aa7808-outbound-1.mta.getcheckrecipient.com; X-Received: from 64aa7808-outbound-1.mta.getcheckrecipient.com (63.35.35.123) by VE1EUR03FT010.mail.protection.outlook.com (10.152.18.113) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3412.21 via Frontend Transport; Thu, 24 Sep 2020 11:49:27 +0000 X-Received: ("Tessian outbound 7a6fb63c1e64:v64"); Thu, 24 Sep 2020 11:49:27 +0000 X-CheckRecipientChecked: true X-CR-MTA-CID: d7fca4c1aa85fb19 X-CR-MTA-TID: 64aa7808 X-Received: from 756ab1743ef6.1 by 64aa7808-outbound-1.mta.getcheckrecipient.com id 4CBD435D-8409-46C7-BD6B-69EECBF28733.1; Thu, 24 Sep 2020 11:49:19 +0000 X-Received: from EUR05-VI1-obe.outbound.protection.outlook.com by 64aa7808-outbound-1.mta.getcheckrecipient.com with ESMTPS id 756ab1743ef6.1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384); Thu, 24 Sep 2020 11:49:19 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=OK5RdqDwtCee0pcvwAWcEMLMdzu+39KUqiuh2tjX/cQpmaEPIoXfX9ZESt+y8VjzxvR8SkpWkWiyyk1gvVGj+u1HLUaIvZLxapnnb/Rrzas2JmsiC6p/ybjf+EboMXhruw2EQv8bWhBiZa0FDU2v/I9JruAg//ofVvJSeWi+rTNhfo2o8kUKv0UmAw07zmKD1nSlMN0erBwgLZhfcF3NG+2xDGoEClRSDFmPaCcaY/1kraEX2YZESkL6z5TNbrrKmubWGG5oSk+cBOYoSjCrG0gkqn9xfS1/Ld5P3gxT0xN0iKiXA7RZ7Oj/qeNo7CbHMSsqZQGR8yQzwPqxZoa6vA== 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=Sm1bYvV4ijKNbmxg3n7rJDBl7ZCon1KmrlFnBSOWtPA=; b=kMdyZUa3wL8VhrA3qGOPqqH44T2OImSxxLq+UkvRPO+j2dTvAHG4AkkB0p50ZmrANOc0V9qE+oCXHQQh4hJImvrRgl90YZivWPUycUPaZ7If87qVm3pDQ5ceGdzoCpVOZT3aktlUVOwkGDpNYDpU7WTugRujujBMqBB8WpcTxC6cCCJXn5XTUeA5pG1fP5rryXGaWrZ1KBTOGe1+gAzxI+HUiG3ST/jo2csuBJMYeakrcYMAP09Hw8TPJ5xlG1vy/Gea6WcG81FsVjw3Ht9paB7sObFsbQNqKP8qrFAym4gvemkzn9t2Neuqo2MxZLXV/8Ym5Zfb9NxeahfLhpByhw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 40.67.248.234) smtp.rcpttodomain=edk2.groups.io smtp.mailfrom=arm.com; dmarc=bestguesspass action=none header.from=arm.com; dkim=none (message not signed); arc=none X-Received: from MR2P264CA0044.FRAP264.PROD.OUTLOOK.COM (2603:10a6:500::32) by DB8PR08MB4570.eurprd08.prod.outlook.com (2603:10a6:10:ae::29) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3412.22; Thu, 24 Sep 2020 11:49:17 +0000 X-Received: from VE1EUR03FT025.eop-EUR03.prod.protection.outlook.com (2603:10a6:500:0:cafe::4d) by MR2P264CA0044.outlook.office365.com (2603:10a6:500::32) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3412.20 via Frontend Transport; Thu, 24 Sep 2020 11:49:17 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 40.67.248.234) smtp.mailfrom=arm.com; edk2.groups.io; dkim=none (message not signed) header.d=none;edk2.groups.io; dmarc=bestguesspass action=none header.from=arm.com; Received-SPF: Pass (protection.outlook.com: domain of arm.com designates 40.67.248.234 as permitted sender) receiver=protection.outlook.com; client-ip=40.67.248.234; helo=nebula.arm.com; X-Received: from nebula.arm.com (40.67.248.234) by VE1EUR03FT025.mail.protection.outlook.com (10.152.18.74) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.3412.21 via Frontend Transport; Thu, 24 Sep 2020 11:49:17 +0000 X-Received: from AZ-NEU-EX01.Emea.Arm.com (10.251.26.4) by AZ-NEU-EX03.Arm.com (10.251.24.31) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.1.2044.4; Thu, 24 Sep 2020 11:49:16 +0000 X-Received: from AZ-NEU-EX03.Arm.com (10.251.24.31) by AZ-NEU-EX01.Emea.Arm.com (10.251.26.4) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1779.2; Thu, 24 Sep 2020 11:49:16 +0000 X-Received: from E107187.Arm.com (10.57.53.17) by mail.arm.com (10.251.24.31) with Microsoft SMTP Server id 15.1.2044.4 via Frontend Transport; Thu, 24 Sep 2020 11:49:15 +0000 From: "Sami Mujawar" To: CC: Sami Mujawar , , , , , Subject: [edk2-devel] [PATCH v2 2/2] ShellPkg/AcpiView: Fix field validator invocation Date: Thu, 24 Sep 2020 12:49:13 +0100 Message-ID: <20200924114913.109188-3-sami.mujawar@arm.com> In-Reply-To: <20200924114913.109188-1-sami.mujawar@arm.com> References: <20200924114913.109188-1-sami.mujawar@arm.com> MIME-Version: 1.0 X-EOPAttributedMessage: 1 X-MS-Office365-Filtering-HT: Tenant X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 38f2e3b3-6eac-4e4a-5466-08d8607fe3cf X-MS-TrafficTypeDiagnostic: DB8PR08MB4570:|HE1PR0802MB2457: X-Microsoft-Antispam-PRVS: x-checkrecipientrouted: true NoDisclaimer: true X-MS-Oob-TLC-OOBClassifiers: OLM:8273;OLM:8273; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam-Untrusted: BCL:0; X-Microsoft-Antispam-Message-Info-Original: kEDRi1pnG9D3gkv/WoEar82acGT0jem54Qm5YSx/zLBgxXLQDVSdq1YQtqbrsImAuFct61xwotx8VwnN7c54hvoIRyPoQJnX+QYtBny9i638sAMXax7R4EtdAC7eKlHu0071E6JoLARvmi4aknQ7qrlXeJ7jjvs3JT2gU9o4hCANakhncZt70ujE1Eeq0jmSRYHUULJJDSLTol1LO81e/2DNcQ4R8xLyQCU1axUTNHipwgLptvp3SLGd2/N+XZsxJDBDhAZmGmyUK0vUAD2HxHm3XLOh7IoEzi3+0+U+sZtOmtO+fPPqOrFKas45nqRMQL3uAAvCTutfmP2WHASQOls1uy/A9qm4kkXJeiX0W1bHudwVXy0CfdeXEAWJMAxn2dsx0tCQ2dd8E6iXMX/z2IHVVpL9Ke9JyvHikScWX7aUd7vQ3yZHg0f7gYe4Jk1n X-Forefront-Antispam-Report-Untrusted: CIP:40.67.248.234;CTRY:IE;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:nebula.arm.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(4636009)(396003)(346002)(136003)(39860400002)(376002)(46966005)(478600001)(54906003)(6916009)(4326008)(316002)(44832011)(86362001)(2906002)(8676002)(8936002)(5660300002)(36756003)(1076003)(82310400003)(83380400001)(70206006)(70586007)(7696005)(2616005)(47076004)(81166007)(82740400003)(336012)(26005)(186003)(426003)(356005);DIR:OUT;SFP:1101; X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB8PR08MB4570 X-MS-Exchange-Transport-CrossTenantHeadersStripped: VE1EUR03FT010.eop-EUR03.prod.protection.outlook.com X-MS-Office365-Filtering-Correlation-Id-Prvs: cf3ab5f1-0741-4084-fd5d-08d8607fdded X-Microsoft-Antispam-Message-Info: Plrlk9jJjBfu3CRW0HQ/HwYZQ0YuF/3xHDVAaTytXTC0Tm13LljM5SLdz1G/BZNimL3SEQQSxjcP8s/oPGh3DH6UbEj1BkWY6RIsFbVgx7JO93d+7cklXMClzftUHpMFhdi4rlolZYodWO92HoChU5E9FwsYCTPQZbdpU/KTcSnWtU6rrGkxm2oultry+8j45/Vb/i53RqpWEpG7iw6SKlkDvUb5UXWhKJme/Ngcuqg3ltH8LeD+KT8D61kXCrthph8R2pu0dnJH5fOyKRAM/T7+UzyTFEo2OnDX0ltjMSlLa/QYuJnChmqIfAWueV2ecaWfZih9bGGyNH87qMfr8MrS5DfLuqOXC/g7HdJauXaq5QSd1t2VBHyRThC8YUQK3mq1E3fYOoHQGsY1zVwCRQ== X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Sep 2020 11:49:27.0944 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 38f2e3b3-6eac-4e4a-5466-08d8607fe3cf X-MS-Exchange-CrossTenant-Id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=f34e5979-57d9-4aaa-ad4d-b122a662184d;Ip=[63.35.35.123];Helo=[64aa7808-outbound-1.mta.getcheckrecipient.com] X-MS-Exchange-CrossTenant-AuthSource: VE1EUR03FT010.eop-EUR03.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0802MB2457 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,sami.mujawar@arm.com X-Gm-Message-State: VjVQQjsbsp1E2QES99IhA3P5x1787277AA= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1600948172; bh=Ob3h4KJ8nI8JUYMBl+49/fyDQuIgdx8W+O6BFxML7xk=; h=CC:Content-Type:Date:From:Reply-To:Subject:To; b=vBLz/yqYHO/ogX3AGyArKNjOIlSWeYk4u3ObL1P2vzuNFUaFjIxSUgFVtUcyCWqGxR9 5p6ca8A5mpqeC9Tapbmirb+rl+azAvRpd8iP7FFL5hVG8z5/XbfnCKHFsHiLZJ4cweSXh kvd/kDChBmgVQVwS+/YdOJR7h05pnwBqO6w= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" The field validator function provides means to validate fields in the ACPI table structures. To print complex field types a print formatter function is provided. The field validator was being invoked for simple data fields for which the default print format is used. However, the field validator function was not invoked if a print formatter function was provided. This problem is noticed when a Generic Address Structure (GAS) is printed using DumpGas() and a field validator is present to validate the GAS structure. To fix this move the invocation of the field validator after the field is printed such that the validation function is called even when a print formatter function is present. Signed-off-by: Sami Mujawar --- Notes: v2: - This is a new patch introduced in v2 series. [SAMI] ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.c | 15 +++++++----= ---- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.c b/Sh= ellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.c index 02f6d771c7e1a9b8faa577b38e51c9de350d24fd..01ac9a9bafeb2ca12c1ba19f406= d626b108f5fe2 100644 --- a/ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.c +++ b/ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.c @@ -1,7 +1,7 @@ /** @file ACPI parser =20 - Copyright (c) 2016 - 2020, ARM Limited. All rights reserved. + Copyright (c) 2016 - 2020, Arm Limited. All rights reserved. SPDX-License-Identifier: BSD-2-Clause-Patent **/ =20 @@ -596,13 +596,12 @@ ParseAcpi ( Parser[Index].Length ); } // switch - - // Validating only makes sense if we are tracing - // the parsed table entries, to report by table name. - if (GetConsistencyChecking () && - (Parser[Index].FieldValidator !=3D NULL)) { - Parser[Index].FieldValidator (Ptr, Parser[Index].Context); - } + } + // Validating only makes sense if we are tracing + // the parsed table entries, to report by table name. + if (GetConsistencyChecking () && + (Parser[Index].FieldValidator !=3D NULL)) { + Parser[Index].FieldValidator (Ptr, Parser[Index].Context); } Print (L"\n"); } // if (Trace) --=20 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)' -=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 (#65562): https://edk2.groups.io/g/devel/message/65562 Mute This Topic: https://groups.io/mt/77055382/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-