From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f171.google.com (mail-pl1-f171.google.com [209.85.214.171]) by mx.groups.io with SMTP id smtpd.web10.32598.1641234735160449051 for ; Mon, 03 Jan 2022 10:32:15 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@gmail.com header.s=20210112 header.b=U+0dLtVp; spf=pass (domain: gmail.com, ip: 209.85.214.171, mailfrom: kuqin12@gmail.com) Received: by mail-pl1-f171.google.com with SMTP id h6so21562378plf.6 for ; Mon, 03 Jan 2022 10:32:15 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=message-id:date:mime-version:user-agent:subject:content-language:to :cc:references:from:in-reply-to:content-transfer-encoding; bh=XC3UMuAFPF6hKSc8bByrGHgjdoVwi3Q6Bswqd3LIvbI=; b=U+0dLtVpV4EUsdRO2xrzK5iSYesD3uTfL1zmxwwp5WN940FsqP/70GBNwGTE9k5oos ofz01ymrg0LG6nHaPlCBLsI2ikF0wB6Swcu0A4+Iw5HmhjjZANOjkRXo6dkyAe3tnJfo e/jcH/ARUlmbWU4RaocKM7TUa/4p4w9khmwcN/AEtDxVfHMJ7pILYLSfbbWAd8HNEXzN jT/TCEM+hbZ9c6CyAa1UNVfWvjrY8Op1vkoEb+3OZiA1UA0WH+nCosM86gSJzkQ9UEeJ qCggo9lS7yHokYHj67TeWZKwwnpnqjDHg0B1zJFJXKy0XBo8V9dg+gjcTL/P4jJP+haJ k9lA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=XC3UMuAFPF6hKSc8bByrGHgjdoVwi3Q6Bswqd3LIvbI=; b=7AUmcnYsPQr3JoyYIpXo/9XicsIZB8EQ51o64ntTu9188mWCtA+unqwGcuvdchZDd/ 5oXR8mJdSKECqB7ULhpXZ57LnDH78LfvR9OexzLvuksvuop52Y1m15IvtmnpO9RMlPHB U0/UkUX6oVIa6Kmm1wOCGuaWsOoAl4Sb75G5O6n8hj83M16t6Eht2qKVvBmxltCZLVaw Uk5eZXlXyI+bQIfA7/cX3Z1Yax+Dvbw5yH1eSnY7+iPjd0xEKUHJOvdJlhCwlIeaxDzL jR1dgikvdDysH1AIA63WsfL1rqiEhGAtWMduc39wyFT25Q0IQqRzb3p3Ib0Ge3/IZQFJ DooQ== X-Gm-Message-State: AOAM530G+CJU3wB2/UnMiSGAsg37HozIUXrl3EjdHQD9n6CVO6gmlN2E TUUk1OGOphBTUzaoa3AaildK3/Ah2wg= X-Google-Smtp-Source: ABdhPJyTuZ8JNC0/d+Z/gwzpqN8IPiYyxCKJl9uMKnH/T1IlhofP3RhS8zg1zTNpjN5RqABVEn0Z3Q== X-Received: by 2002:a17:90b:1c07:: with SMTP id oc7mr56060849pjb.127.1641234734542; Mon, 03 Jan 2022 10:32:14 -0800 (PST) Return-Path: Received: from [192.168.1.18] ([50.35.74.198]) by smtp.gmail.com with ESMTPSA id f20sm41348972pfe.166.2022.01.03.10.32.13 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 03 Jan 2022 10:32:13 -0800 (PST) Message-ID: <6892acc9-0ce2-9970-76e2-2ab2d56497d5@gmail.com> Date: Mon, 3 Jan 2022 10:32:13 -0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.4.1 Subject: Re: [edk2-devel] [PATCH v2 0/6] MM communicate functionality in variable policy To: devel@edk2.groups.io, Jian J Wang , Liming Gao , Hao A Wu , Michael D Kinney , Zhiguang Liu , Leif Lindholm , Ard Biesheuvel Cc: Bret Barkelew , Michael Kubacki References: <16C2A13241682C7F.28461@groups.io> From: "Kun Qin" In-Reply-To: <16C2A13241682C7F.28461@groups.io> Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Hi MdePkg, MdeModulePkg and ArmPkg maintainers, Happy new year! It has been a while since this v2 patch series has been sent out for review. Could you please take a look and provide feedback? Any input is appreciated. Regards, Kun On 12/20/2021 17:33, Kun Qin via groups.io wrote: > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3709 > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3751 > > This patch series is a follow up of previous submission: > https://edk2.groups.io/g/devel/message/84140 > > v2 patches mainly focus on feedback for commits submitted in v1 patches: > a. Splitted the original ArmPkg patch into 4 separate patches; > b. Updated patches according to Uncrustify scanning results; > > Patch v2 branch: https://github.com/kuqin12/edk2/tree/mm_communicate_check_v2 > > Cc: Jian J Wang > Cc: Liming Gao > Cc: Hao A Wu > Cc: Michael D Kinney > Cc: Zhiguang Liu > Cc: Leif Lindholm > Cc: Ard Biesheuvel > Cc: Bret Barkelew > Cc: Michael Kubacki > > Kun Qin (6): > MdeModulePkg: VariableSmmRuntimeDxe: Fix Variable Policy Message > Length > MdePkg: MmCommunication2: Update MM communicate2 function description > ArmPkg: MmCommunicationDxe: MM communicate function argument > attributes > ArmPkg: MmCommunicationDxe: Update MM communicate `CommBuffer**` > checks > ArmPkg: MmCommunicationDxe: Update MM communicate `CommSize` check > ArmPkg: MmCommunicationDxe: Update MM communicate `MessageLength` > check > > ArmPkg/Drivers/MmCommunicationDxe/MmCommunication.c | 46 ++++++++++++-------- > MdeModulePkg/Universal/Variable/RuntimeDxe/VariablePolicySmmDxe.c | 10 ++--- > MdePkg/Include/Protocol/MmCommunication2.h | 13 +++--- > 3 files changed, 41 insertions(+), 28 deletions(-) >