From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from NAM11-BN8-obe.outbound.protection.outlook.com (NAM11-BN8-obe.outbound.protection.outlook.com [40.107.236.100]) by mx.groups.io with SMTP id smtpd.web11.17424.1684833850670964261 for ; Tue, 23 May 2023 02:24:10 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="no key for verify" header.i=@amperemail.onmicrosoft.com header.s=selector1-amperemail-onmicrosoft-com header.b=n3jL6UiN; spf=pass (domain: os.amperecomputing.com, ip: 40.107.236.100, mailfrom: nhi@os.amperecomputing.com) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=WUA7bGsqQj/cbiV+1iZUQkg6PlubSNY+5zNfYe6i+15lVAKk8lUZQvRNh0oO192YAs5mxRy8wFQyJnRq2iD4IOg/ASEQd4EM1fY0vf2Xht32fUdAKpUOeuh5pC8MHEp8FdLpsIubyvu3U4EOMS9LYEskfCjB5SnYn0TnsbHnVLleZhjvd+WeokrQwkw+KuIOK5QpThM29xLCxIp9wrmg8ea4/R84Hnu8jBvmXrW9HDsl3cgQZb/Gd8hYzVUnHRCElakA8hmh0x4xXVn6PmOv82LNOS/lHIMbmJneL9NRa8g1RBM2kD0BOTMn0fKjY3vPVsbhXy9S1PfHrXtHbngTuA== 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=L6YDNruzToLyJjHvwKtlnA0OuzO3TZC/b5YLS6UoEfg=; b=KpxxeAtlwqvD4QtVNh55Bu2D/kC+XK7hEtgOQfTUPmHSvyb/pTLBxFU2CeUQvUoc4WEQcBAnVAaPRg3gduXP9RPBUF/xlANtf0IeyHi351bD+3Mqy7UKC1avALT1qJEvUPC4GXYN5R4rLpQUa52BSjQn0xb4BlqHYDmroheutvxMUJ1D5+oKa+r4nNAcnBbiIbL9NSH4ez+mIKUaARKnDMK4hDkX/NrXuKAYiyXaLJkRTa1v24XZFlDIio3IhLVhhy8njnK2QmDXx7hEx5pHDYE+j6nxw5i8/ebdww7c5Tb9qvs9Ef6dl1V1ZIFUw6seTf8+vhiR5T2VEEq0eW0ATg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=os.amperecomputing.com; dmarc=pass action=none header.from=amperemail.onmicrosoft.com; dkim=pass header.d=amperemail.onmicrosoft.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amperemail.onmicrosoft.com; s=selector1-amperemail-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=L6YDNruzToLyJjHvwKtlnA0OuzO3TZC/b5YLS6UoEfg=; b=n3jL6UiN0elstqveKWvqK0NSSvSmRe7BMMAgdsqT9WoZMYscVvLwu8wmuwTTahlioTlf56XcJOWaVh7zgEHURnzvIPSBr6YfEnTmXdzmN/uJwIk8VWjuIIWpfMJXFbHhFhIxfAptrnGwO0WIGs9o7kqXy2LB+C5d0HLgwvIqRSQ= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=amperemail.onmicrosoft.com; Received: from PH0PR01MB7287.prod.exchangelabs.com (2603:10b6:510:10a::21) by PH7PR01MB8513.prod.exchangelabs.com (2603:10b6:510:2ee::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6411.28; Tue, 23 May 2023 09:24:08 +0000 Received: from PH0PR01MB7287.prod.exchangelabs.com ([fe80::eff9:3711:1275:7fb4]) by PH0PR01MB7287.prod.exchangelabs.com ([fe80::eff9:3711:1275:7fb4%4]) with mapi id 15.20.6411.028; Tue, 23 May 2023 09:24:07 +0000 Message-ID: <1b974228-6444-ca2b-86ba-fe6e50a104a5@amperemail.onmicrosoft.com> Date: Tue, 23 May 2023 16:23:57 +0700 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.11.0 Subject: Re: [PATCH 0/2] Fix an issue in SmbiosMiscDxe module To: Minh Nguyen , devel@edk2.groups.io, ardb+tianocore@kernel.org, quic_llindhol@quicinc.com, sami.mujawar@arm.com Cc: khanh.pham@os.amperecomputing.com, nhi@os.amperecomputing.com, tinhnguyen@os.amperecomputing.com References: <20230516070812.3606476-1-minhnguyen1@os.amperecomputing.com> From: "Nhi Pham" In-Reply-To: <20230516070812.3606476-1-minhnguyen1@os.amperecomputing.com> X-ClientProxiedBy: SI1PR02CA0021.apcprd02.prod.outlook.com (2603:1096:4:1f4::14) To PH0PR01MB7287.prod.exchangelabs.com (2603:10b6:510:10a::21) Return-Path: nhi@os.amperecomputing.com MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: PH0PR01MB7287:EE_|PH7PR01MB8513:EE_ X-MS-Office365-Filtering-Correlation-Id: 66aa7e85-f7f6-49a5-544d-08db5b6f7532 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: ncmB3J0Fa0a3qp3rfWogvUhyWse7WFdRC4Ks8ORms0YKrc9tGQT+59jZKDw4mtdqxEOVrnb8XAHdzXhdzXmo5F4Z+dfwdymkdiEF9vY3laCgZh9VS3AvUuiLIP6enWs3GK4KX3OvI8PYSLWIDsogKA45JU5PZq7TItOpHXdG0C089p4jN7I/uIpbB0RafAiJETMGiXbrLCh3ao5thywNOasSya+JVH8Vr5RK1yyy+MAv41uJlJWLulpUdB84wIb8BSK2ruQN2M2pB02BJxl/9JQ0ICwL4Vr0VdmVhlilH80Y7kDnuo7mJKWPspHlTyIj3Gv9TTqqeqYZ8kJw3bCgp+pkbnMBZLdAI4ms7Gt1Sgs6H1+ujvEn8XwrpV8W21tKhTSvzMnxIB1OO8dgdY3gGUfiwNnFpKhPa6ZnNcQi/ckzvbnt9oR5KkD0ui1sTMHzeP5M1Nzvb6QP5m10McviD4xGtMdrQvshdVt9EDnzxRTHuCV3tJmiLfNirIPYcse4FgBHHZdRRbHW9jASMOKTmgv05Um7+EoQ3guGTn7lx7OunEkFAjp11jmgU0w82iaIK5phZbrF/hfpMQeQqOJNOUs9p+XcbxnbyzUfSch5r0j9CXTvEvjvrWo9QWbzUkVJ X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PH0PR01MB7287.prod.exchangelabs.com;PTR:;CAT:NONE;SFS:(13230028)(4636009)(346002)(366004)(376002)(136003)(39850400004)(396003)(451199021)(83170400001)(478600001)(31696002)(66946007)(66556008)(66476007)(38100700002)(31686004)(83380400001)(42882007)(2906002)(6486002)(4744005)(2616005)(6666004)(316002)(4326008)(8936002)(8676002)(53546011)(107886003)(26005)(186003)(5660300002)(6506007)(6512007)(41300700001)(43740500002);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?cEdidVYxZFh1V3NRZDRqK1BoeHNSR3hxby9CRWEvemlhK0pIeXl2a3c3U0Z4?= =?utf-8?B?V2U5b2hVbVdzYlRUa2JDaE4zRUtLQkx2Sk9HYm40ZjhjQ1RhYWRNSkdjeU5k?= =?utf-8?B?bDZ3ZHJramNvSHNwakZSMW51aU5FY1FLK1JERVp0QXFwYXNKRlAvNWttb3ZE?= =?utf-8?B?Y1VSRTV5K1ByTjVLWXJsZEpxNW44QzNNSkZ0SWkzOG5jQTBZTlVsc01lSHg1?= =?utf-8?B?MFgxQk4zSitRQ2FoNWsvZWw4c0Y4TEFlUDNtdy9Kd1lMZTRQVHM3UWFaT1Ra?= =?utf-8?B?eE9Ka1JuczAwOUM2ckxPOEpBRzNrNEhDM3FkMFRLeXlKM21wZGEvNTdST2RK?= =?utf-8?B?OXpzUGlyQTBjR0txK0VqUTNSL2Q4amtuRzEwOVlwUG9uSzY3azlZSDVjM3lU?= =?utf-8?B?L0dyWFlyUHlQZ0hBUzcwUFZ1M2VzRUNoWE9XZ1dJcTdtYzdOUHdIay82b2Rq?= =?utf-8?B?OE1ydWhXbGZWK2RmeUJXa1A2ck1KcktIWWNLRGtORlVXb0llb2Q1RzZaOVlS?= =?utf-8?B?dkMzVUdac3FuMTc1MTZQOVRxb0kvRlZ5MDE2VmE5VXMxZ3NSL2J1a1U0bnZY?= =?utf-8?B?M2UvdTVxM2ZiU2piQXFXMWFLUk5sMFZnL3k0eVF0aXg3OGlxWmNyL0ZQT3FS?= =?utf-8?B?K3M2aFA4c1h2enZ6NjlTaG8wekRRa1o5UW5PVlBvK0JGUi92RUxBN1R0dDlq?= =?utf-8?B?anlwSERGVWJHcFlJc0NGZUtvY3M1cExQUEx6cWxQQmRadzNnQU16YnhHek12?= =?utf-8?B?MC9xZDQvak14cTdPUFM3Smo0allNZHI5Vm9oWWsrUTkyQ0NRd3BmUHY4bnZv?= =?utf-8?B?WnVsOW9PV3oxUWljckNzaEduMk9qWDBvNTFSZUZ4cnc4VjlmRGVWLzJ2RDNw?= =?utf-8?B?QWo5eE96WlJxVGQ5RHNCcWVsbllieXpRZ2hmMzRWVW1PaWtFVjN4TkVmd2hP?= =?utf-8?B?TnhCa1ZTc21WWUpjZ01BWVRaZ3J3akNFTkgrNFpuVmFNWjR0ay9vMGhqcTk0?= =?utf-8?B?b2VnODRBQWpCSjBLaWZ6V25tbHk1MTVZRkpQYWVvVm55Q2VaS3h0aHhmbDMx?= =?utf-8?B?azlhNEZaQ1cwNERIOG9ObWhFWHovcWw0R1FIaSt6QkNZSk1wMEZ3Wkk3ZUhM?= =?utf-8?B?THhEeExtZ0pvREFCS0JRWmQ2VnFrWEZrRTBFb1c1Y09JeDZUVVI0RUc5Rm9j?= =?utf-8?B?VloxOEUvMkJ6QzJGS1ljKzlnamRkZ1g1czVlV0xJc3Z0MjJFL1EweEFUdE5T?= =?utf-8?B?N2ZmT2F4YkRuWWppT1NOQ2xkbm9iU2xuK3FRWTY5S3NQUVhzSGtDSnVTZEM5?= =?utf-8?B?K0xTNDRFcEYyUVgxNXFDbW9KSjlwZXdTM01WeXNDRUlwWVVjK21tRHdXQW1m?= =?utf-8?B?Ujg0QkU1TFVTM1JvNG9RQyt3TFFJeURicFYwNUNzZVJxWjczczFYTWlETlRp?= =?utf-8?B?c2xtdFl4SGhha0I2aWkrRlpjdDRtOEszYjFXTnRUTEZONHpFTFpUendFWFFB?= =?utf-8?B?UkRkT3RobWZzd1B3R3o0MnVQVkhLU3JDWXl6bWtXK1pIVFhHOUpqZkNMSXdy?= =?utf-8?B?RkpRVmg1VEdLRVFLb3pLK29iSklyZjhxMTk5MFVVZFRMd0RzTjRKeWxMOWU1?= =?utf-8?B?RUQzZ3NqTXhaSE5kZmQ2TnQ0ZXMzaWMxUXV1RlZ5cWRIQVEwdjZhOUNZZkxl?= =?utf-8?B?ODJnRmVPWGozMExEb0pHWnlpMUVVM3BTNUlzeGhaZytuczUvaTM4ZERZTXNC?= =?utf-8?B?dDVja0Z3UlFweVZxRCtkOGhwcE9hVFNMWnNSbjdxZ2FFYjV1MGw5aFgrUHc2?= =?utf-8?B?YnRIYTB4SkdweVBrYzJuS0x2SzhOWGkrM1J5US9pSDEwVEhveEdKMTZ1ekk4?= =?utf-8?B?MEd6eWFTNVhTa1daUXJ6VkQ2VlVwYVBFbjMwMnpYTXA0d3Y2RENBb09hd0cv?= =?utf-8?B?TWJWNUErMnZvd0l0TTRyVDNmZXBydUFqUDVsVk0yTC9tWnUrVk5LQ3RtM1pp?= =?utf-8?B?dEsreExCdWdYNlRXVkpFSHNmVmg2Sitza0k2eEZLdkhTc3lWamJkRm41U0JZ?= =?utf-8?B?OEhUeHVmQ2lEYXk1MUJxODJ0S3ltaFlKL3hiVG84c0p1KzJ0Tk9RNGhraVlT?= =?utf-8?B?WTVjUWVYSVNBNkNlTjBEMXlNbE0ranJ4WWp6em5OblRDdTNlVXhRR3NPUUdw?= =?utf-8?Q?WS7GhWVBhMOGNgpO9eGEP0s=3D?= X-OriginatorOrg: amperemail.onmicrosoft.com X-MS-Exchange-CrossTenant-Network-Message-Id: 66aa7e85-f7f6-49a5-544d-08db5b6f7532 X-MS-Exchange-CrossTenant-AuthSource: PH0PR01MB7287.prod.exchangelabs.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 May 2023 09:24:07.3403 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3bc2b170-fd94-476d-b0ce-4229bdc904a7 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: TlkwkZTLBynS1XVvT+Zmkfwz78jJZsws09Ay3HBl9t9XslqzW5mli8zH2jGGa0aYEh5eRNqhKJE5HuZayWT0wD+ey3j1tKGAS+J0d+N5d60= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR01MB8513 Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit This patch series looks good to me. Reviewed-by: Nhi Pham Thanks, Nhi On 5/16/2023 2:08 PM, Minh Nguyen wrote: > These patches help to fix an issue in SmbiosMiscDxe and remove redundant asign for SmbiosHandle. > > Minh Nguyen (2): > ArmPkg/SmbiosMiscDxe: Fix procedure to get handle of SMBIOS records > ArmPkg/SmbiosMiscDxe: Remove redundant assign for SmbiosHandle > > ArmPkg/Universal/Smbios/SmbiosMiscDxe/SmbiosMiscEntryPoint.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) >