From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by mx.groups.io with SMTP id smtpd.web08.19560.1629083774427764855 for ; Sun, 15 Aug 2021 20:16:15 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=fOT5FsNC; spf=pass (domain: intel.com, ip: 134.134.136.126, mailfrom: ray.ni@intel.com) X-IronPort-AV: E=McAfee;i="6200,9189,10077"; a="202944585" X-IronPort-AV: E=Sophos;i="5.84,324,1620716400"; d="scan'208,217";a="202944585" Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Aug 2021 20:16:12 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.84,324,1620716400"; d="scan'208,217";a="509483325" Received: from fmsmsx604.amr.corp.intel.com ([10.18.126.84]) by fmsmga004.fm.intel.com with ESMTP; 15 Aug 2021 20:16:12 -0700 Received: from fmsmsx610.amr.corp.intel.com (10.18.126.90) by fmsmsx604.amr.corp.intel.com (10.18.126.84) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.10; Sun, 15 Aug 2021 20:16:12 -0700 Received: from fmsmsx604.amr.corp.intel.com (10.18.126.84) by fmsmsx610.amr.corp.intel.com (10.18.126.90) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.10; Sun, 15 Aug 2021 20:16:11 -0700 Received: from fmsedg602.ED.cps.intel.com (10.1.192.136) by fmsmsx604.amr.corp.intel.com (10.18.126.84) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.10 via Frontend Transport; Sun, 15 Aug 2021 20:16:11 -0700 Received: from NAM04-BN8-obe.outbound.protection.outlook.com (104.47.74.41) by edgegateway.intel.com (192.55.55.71) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2242.10; Sun, 15 Aug 2021 20:16:11 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=MBmcz9f0cht+StjyaxWn40s2P1wrWhSDoNNvdZcII0xNyDYJWQrOvQnb+JKXznezF5I9AR0dxzb6kJGV89n67TC9VibfVpm+m0U+aQ7zd/dLdB+q6Z3QVRs3IqGaARIyUm+tOLAB8dcgnuKtnJLWhqRPkzqbgagk00Y1OjaumjUr3bs604tH2bbKkgiVInGoZTb19sHr4QBxucjEfHBUc+DmbRg/YTgYikqB9SiqaSkB8LR6Jvw1MYXnXs6NDeEk+mXK0Ww2bNN4keCM22JqcuZsXOZFqcSkhXYyH3vIYjXUiY5sy3A6MFyDBE7EXjiTzXyBx/9gW4SMa3ibDD6z2Q== 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=knlbfeaOZnMEWQIZOCfIB50/BkNvAtluvDbkeEdEv2Y=; b=jjiuF5cjGz1q1Ge/wT3pm+KMtreABRb4p3ohXJQl6OEqrZ6C6wOc7rcgVxRI/xpY6y7ZsHWNuNv+Et1M9yxuoHpZHu3LXn3pKOCKRCICXBGrb0Scbq+xv5BflyDuV0MGioVDIXECkKCEnrJdlUEdboxDZ4MtSf9rPmawiH44Hosbv9QZQTb87nzsY/Eh8jLQdqBfvvYXnyBG7Yoz0T5LzYuth8IkPrr37kkeIT/v2P91zr+V8n+ie0iHIv9C7Gz2JBoY+lT03UZfcSzGj6Z9l3cGEGKpIyrBXV+k5aiI9FBy4zdZRXrH8mm3FxyA7GNY9mY627vNRFuj8ZGDBK1QMA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=knlbfeaOZnMEWQIZOCfIB50/BkNvAtluvDbkeEdEv2Y=; b=fOT5FsNCNod+DZYK88cCRolD49J9EpM1nNK8Dcp9FbRVsEqxeuWpX6to9YfXrRflKej6GkVdBQ7/6s7/fjllZYydM+x29WtYVVsPByjWFTnti4elcfqWZdxcU2YcOZJhWDOgx/73qM0sz2a+ATfWA2nEHAEsXi5m+wGSedqtI1k= Received: from CO1PR11MB4930.namprd11.prod.outlook.com (2603:10b6:303:9b::11) by MW3PR11MB4570.namprd11.prod.outlook.com (2603:10b6:303:5f::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4415.16; Mon, 16 Aug 2021 03:16:10 +0000 Received: from CO1PR11MB4930.namprd11.prod.outlook.com ([fe80::c98d:a1f9:48a8:8865]) by CO1PR11MB4930.namprd11.prod.outlook.com ([fe80::c98d:a1f9:48a8:8865%6]) with mapi id 15.20.4415.023; Mon, 16 Aug 2021 03:16:10 +0000 From: "Ni, Ray" To: "Zhang, Qi1" CC: "Yao, Jiewen" , "devel@edk2.groups.io" Subject: Why SetVariableCheckHandlerMor() is implemented in VariableSmm and TcgMorLockSmm? Thread-Topic: Why SetVariableCheckHandlerMor() is implemented in VariableSmm and TcgMorLockSmm? Thread-Index: AdeSTMF6h1SuufF1QU+skt8s2GeRww== Date: Mon, 16 Aug 2021 03:16:10 +0000 Message-ID: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=none action=none header.from=intel.com; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: a81a0e8a-57dd-46a8-6ec2-08d96064321e x-ms-traffictypediagnostic: MW3PR11MB4570: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:8273; x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: znBgxs8mMS5g7Ahxvje5Z/VayxBMVRSlEOAvC2Lsw6nRcXx7v+0JUEGbXSmcI23Pix5k/gS6w+KqtY5m0OrzT+1njA6EeEgroVtGrQtzJmDXUgnXZglS61nnq1Z1T/YoDKef5O/VxGbRx59Pgsfoup+3Zn7vPhrIMmVPhzzBChDcVGMA2wCRIC6MtDObo/I34JHMfGVvI07/5zIYOnO/D8fsmBq0lAQS0/YTBrT6CzF8apTe2sdXcgEbDHBiMOYttCMo7JVJzsscQp9OTPp1LIxE9lvlosfp4Y+5Ut12rg3Y0tH1OUTOzw5T1pp3ofVse6gXdd1Nb8JfvrY7aZ8J2KZ1pUqWP2bltousvyexa0zYCXJ+ns9RX4Xjvoe7bo9nEB6SjRRhA8INMiait67vLUugmsWkW2iUerJ6QImnZP2ULKWzf+NXil+N8vsXVVinLa+RMEhG/qHSG+DNufyh9K5zGM2W/0Fq9MfziB21gMNzF008hHYZmVp7pCxHuvp7e5clFahNjP/OkNF94X23df/L/tCo0jcRba+oUCuuNsdodBYnO9JXrzDRLMZgvTk/+Uau6E4KvOzCVywhiZl6FIblWEXsPvY3Y6n3D72jqHRhlSuVRzhSIYEP2wGABN9V3z5PmaJA/ny6RXPa2vV/6sVQZt4Wv+c3mJ+BhTO19c7EUzkvhuZ7EiSMZsZlbrHlAXaENJs/dc8QSp1Yy8kWBTNt22shI8ecQQxneZYK/hQ= x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:CO1PR11MB4930.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(366004)(66476007)(558084003)(33656002)(86362001)(64756008)(66556008)(66446008)(38100700002)(6862004)(83380400001)(26005)(2906002)(7696005)(6506007)(38070700005)(66946007)(122000001)(5660300002)(186003)(76116006)(8936002)(71200400001)(6636002)(4326008)(52536014)(55016002)(8676002)(9686003)(508600001)(54906003)(316002)(213903007);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?Y+ORQDBciWHrQBlSxZ1ief8miq0iI9H0XDfODS+EY1ZY8eW6Q8ZJU/MasrZN?= =?us-ascii?Q?SwrzhLQDJ3FGqo4EJQeRwGuLziru8vDdWgL59+nKv57V/OZODIngNx6gu8f/?= =?us-ascii?Q?Q1TgHoXhdn5s7G8gcf29pw2a+oBvoiSQjqELil8rUZTQaRtvBVidtvpBW0XA?= =?us-ascii?Q?R81QnEf2zeEodm2Q3IYL2KFVVj12fqKyzt4Q4WNPV8YW606UXAznlaaGBHsU?= =?us-ascii?Q?kNyKrXCqePTXyIoN/c0+GGG0xMkOP1zihALoMuKrQBT32F1EehEHixdad4ho?= =?us-ascii?Q?SjEcQggBDfd5nLz/Emrda5qB1d828bCngV2EqSZ4mhcNC6oSYdzF3X0jzYIR?= =?us-ascii?Q?U7Ioy12Ho5uBgGphr12Cpm2ssUDxkfhgDnrpPAkCl7IeS3UJx4P+oZ2eTBwT?= =?us-ascii?Q?ML90xpvEHIEQA7CrXhYSOQINNttyOyB+WV2UEJea9yVeMn0LX8YrrtDyAA2D?= =?us-ascii?Q?8H/j3qUS9gQ+LkkHJCWRDMFGXGWJS8hf+OV0Q1ZLFzqsL18efChVU0kJ57A2?= =?us-ascii?Q?85+IKjUdZgEksDMtpsu8TL53q0jx0Mt50ASqqioeDqc4w/gL01on9RheOP82?= =?us-ascii?Q?ODicthLyPqc+iojMsRTFqdVXflWK58uAVVWuP9lMKlNWIvp1QPQm8AnSKK2b?= =?us-ascii?Q?rgrOWyM6kvyrASaLMvnc5AjjjxGP2MvxX+hP9/kbUVdR2FXflQTMhgaxuT19?= =?us-ascii?Q?VR9cpDXZbLrk8RH8woY0+UTWe45EGKahMhTjr2AH6puHo9vH7QKYCqmNaJOG?= =?us-ascii?Q?VyD3CFMwk8RJmWJeA7wWC13e9Y2+JwTbLS6sD/yY/LgJ5ZN+YkJci/zY1s6w?= =?us-ascii?Q?tn+g23L8WUJdTctf/450I+e3SQ4RtfX1M4saP6LxVUF/mN6jvdgF91KEkB11?= =?us-ascii?Q?aRCpQZgfAesCXVRPlP6zeIzjlzwzC5AzHv/DUWw3nSDJf9/+jfVlOGHbslSp?= =?us-ascii?Q?EGUvXCPPj4j/NKj3mdU8kg6zQMz9KP7Ji610YtVjxVVp+j9nnGkQuErzPcGL?= =?us-ascii?Q?wiHUL/Yreyjb7kg9ShrsqJ8hrLWBDYrd7oqyGrCA/kEzD6JLhYV3w9dX3p+1?= =?us-ascii?Q?QfCeV14as0bxTZts0ZA59c1VGG6rHXU3NWpq4jFwD+MR65gfM6ZEKZA5ZHe5?= =?us-ascii?Q?xkddve4oai8z0QYZgj+WOfjJoA3caCjyHyS6ks4yFoTA+AcaEgHllgmAIKTj?= =?us-ascii?Q?MBptlkBPfg10aq1IXRctyUC/pQx8Eyhr9GdP1BmRqd+QKC5IIoi9VNLteh5v?= =?us-ascii?Q?5N/jjrccu43AQZoDprU528cNFYIeISFJlIpUu/4PmP74czSmsSSt1/kO1/Ba?= =?us-ascii?Q?pE/3BY1j6Qaw7dxaKEn73Zip?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: CO1PR11MB4930.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: a81a0e8a-57dd-46a8-6ec2-08d96064321e X-MS-Exchange-CrossTenant-originalarrivaltime: 16 Aug 2021 03:16:10.2700 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: MeFRCZZPa9i0T5WR0lxmZrm5nfs+zK7iOALOhzpembZASoD8b+dau/Alnx62lSOlSsryVYBERn2IJv4tc8ngIQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW3PR11MB4570 Return-Path: ray.ni@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: multipart/alternative; boundary="_000_CO1PR11MB4930C057C7407841E3FAEB6B8CFD9CO1PR11MB4930namp_" --_000_CO1PR11MB4930C057C7407841E3FAEB6B8CFD9CO1PR11MB4930namp_ Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Qi, I happened to find that there are duplicated implementations in VariableSmm= driver and TcgMorLockSmm driver. Do you know why the check needs to be done in two places? Can we remove the one in VariableSmm driver? Thanks, Ray --_000_CO1PR11MB4930C057C7407841E3FAEB6B8CFD9CO1PR11MB4930namp_ Content-Type: text/html; charset="us-ascii" Content-Transfer-Encoding: quoted-printable

Qi,

I happened to find that there are duplicated impleme= ntations in VariableSmm driver and TcgMorLockSmm driver.

Do you know why the check needs to be done in two pl= aces?

 

Can we remove the one in VariableSmm driver?

 

Thanks,

Ray

--_000_CO1PR11MB4930C057C7407841E3FAEB6B8CFD9CO1PR11MB4930namp_--