From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by mx.groups.io with SMTP id smtpd.web12.670.1612550305894858903 for ; Fri, 05 Feb 2021 10:38:25 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=VL7Vx3/P; spf=pass (domain: intel.com, ip: 192.55.52.120, mailfrom: ray.ni@intel.com) IronPort-SDR: BEcYEpTXcINvmzabwTWcKJUxZq7IyIJxw0AuVY/6Irw0aVcZwidEyvL2lllFzn4bCps5JNx4Qh Nxh/eKkAcugg== X-IronPort-AV: E=McAfee;i="6000,8403,9886"; a="178912641" X-IronPort-AV: E=Sophos;i="5.81,155,1610438400"; d="scan'208,217";a="178912641" Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 05 Feb 2021 10:38:25 -0800 IronPort-SDR: 90u/mzZTRwItdfgoMQOlM6+ncUNU8GFzG8wfDizxKaldGrGPfsySiN7zFjfguh9yQV0CzJjB0k 9psguPCEow6Q== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.81,155,1610438400"; d="scan'208,217";a="416341391" Received: from fmsmsx605.amr.corp.intel.com ([10.18.126.85]) by fmsmga002.fm.intel.com with ESMTP; 05 Feb 2021 10:38:25 -0800 Received: from fmsmsx609.amr.corp.intel.com (10.18.126.89) by fmsmsx605.amr.corp.intel.com (10.18.126.85) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2106.2; Fri, 5 Feb 2021 10:38:24 -0800 Received: from fmsmsx609.amr.corp.intel.com (10.18.126.89) by fmsmsx609.amr.corp.intel.com (10.18.126.89) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2106.2; Fri, 5 Feb 2021 10:38:24 -0800 Received: from fmsedg602.ED.cps.intel.com (10.1.192.136) by fmsmsx609.amr.corp.intel.com (10.18.126.89) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2106.2 via Frontend Transport; Fri, 5 Feb 2021 10:38:24 -0800 Received: from NAM04-BN8-obe.outbound.protection.outlook.com (104.47.74.44) 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.1713.5; Fri, 5 Feb 2021 10:38:24 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=jDPQF4DhYdijZEYYoUDHG2xzG21osYhjOAg3NiillVvT6PUhVUA6ljhEtI52qkZU2deOBmz0gDF1aaXn3odS56ufV0/QuJhul7v0/9hk3bIA+Da2shjv4SZlHXsYjky4+7xD7poXSI2C3006ukEZjKG/XKjIQkRGKgwFlAIIgf/5DWZx8sbehnbqYTcuhWhhWFfud45Ci9UOAUf0iNOWJxAK/vrj6P5Dd3oaIdAEgfOLQKr7hHrFG23DUbaAOMlk3ZrqN0YLTfFdMo4mARVUMbNBkXwpc0T7Z0dgEnv/rKpCk3bIIs2XWbbSbO4bYLOqxg3d2XO/GMMPdl2abIDYhg== 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=ZcQPFZl3qe6wMwoI5Q0VUZgnRUM/oCUDUoegkpN/a54=; b=es2aUZPlPO/Oyeh+vIqmN+it1OQRu859Ox0zi6jRfNvNYvTPdutoo3PogjDjX7+31Lj+QugzRnR0ZgXwoGEJqBjz0vzOq2gIiFsdCXXaYv1Bz37+mPUAGkUZpD7HVLlIbYJslOP0r59GSHBKm5XkAl9DR7jM5V8LraaAZ2NEMkkz7VV6OEKOcbE0JRQXEtSo7/QhGix7Th0UmW2KJ07kPoOMrzdNXoWPXeRUw9SQRdDRw8lHUFjxUhq4zyRb3MVPlz0ZffkecBlYtYVGHq3iIuq668EoU+7C5Vjs6Rj2V9XAb9ZddE4a2IfZjOepjvkJibae+scBAgzBjvi61L+Wrw== 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=ZcQPFZl3qe6wMwoI5Q0VUZgnRUM/oCUDUoegkpN/a54=; b=VL7Vx3/PpPVxxFm3oAUdmC9THgFiANsZIx1YMDEX4QECMinLjGSY9Z8PX71YExT7coU/asnKZ3Su2XIUWR0JfkosO5qD10DheXgGClSP9QzEOmddpijKQEgeEP/cCl4baG7AAyFzJqIQtILlOSjJ+MtzHLheg/qR/SprqlWFpgU= Received: from CO1PR11MB4930.namprd11.prod.outlook.com (2603:10b6:303:9b::11) by MWHPR1101MB2222.namprd11.prod.outlook.com (2603:10b6:301:57::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3825.19; Fri, 5 Feb 2021 18:38:21 +0000 Received: from CO1PR11MB4930.namprd11.prod.outlook.com ([fe80::8d64:91ed:c259:e95]) by CO1PR11MB4930.namprd11.prod.outlook.com ([fe80::8d64:91ed:c259:e95%7]) with mapi id 15.20.3825.025; Fri, 5 Feb 2021 18:38:21 +0000 From: "Ni, Ray" To: "Kinney, Michael D" , "devel@edk2.groups.io" Subject: Re: [edk2-devel] [PATCH v2 0/3] Use XADD to avoid lock acquire/release Thread-Topic: [edk2-devel] [PATCH v2 0/3] Use XADD to avoid lock acquire/release Thread-Index: AQHW+5S28cUnsp5iuUGOYa61LJJ2+qpJypdQgAAZONY= Date: Fri, 5 Feb 2021 18:38:21 +0000 Message-ID: References: <20210205075810.981-1-ray.ni@intel.com>, In-Reply-To: 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-originating-ip: [124.77.195.36] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 586764fd-9c9b-4af4-d4d8-08d8ca0536e1 x-ms-traffictypediagnostic: MWHPR1101MB2222: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:6790; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 2/Sld2yjVkRox1DsNIuZxz5Nf9KgcAaNFXhiJbrLZqjMoT5l8R9EpVSBhXATU4O37wa1hrGIvumz0OxwBnuYzVBDhCMFHh7qGtWTFWSHrPwnwUQZr0AYNHpR59nvaeC/UFz1j2QXPPTtj2hpDhepBjRRMIgi4FCwY/xb23Qh2k2/g9GLhhS9+ac9fIiDnWciRPbLa9hNwS5w0rI5hRN8Gz5VwYCcYeig+tlcVp9T+Tt8a95Lkd17ANlqoTfs9p0EdKXZI1FCoCXBOAeyKy5GCHON5VvAE78RYa/1gzov1O/F6nkot9mlwZ6p0/bkCMMz4EjK0bE4tISlfPFze86q/pxZhV5xhz2kFAB5VKU9czAuBHacBb6xH0fs94F+QARbcsOFsx/YtNxkvt+rHdZXxHkpsLynkajQSvm/sIoRK1vTRI5fIREPJbHRWDsCUuZ+hpSQIFgVEJI61bscIDLlAjVkqmLcEjZyXY90rQzAY+JsE1pU0ejmuSWXdtleMg9P4JEewLmzSg4ynxtvFwrd/P5E+GC8Q6NI/WQlhregdVrtG6vLdHtglBEDIBb8VvYxxqJoZGR8eRq8j0/JbIjjnkbe66Lg02VHyqxL6xsKDqQ= 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:(396003)(136003)(376002)(366004)(346002)(39860400002)(83380400001)(9686003)(110136005)(2906002)(71200400001)(8936002)(26005)(33656002)(52536014)(5660300002)(7696005)(55016002)(186003)(6506007)(66556008)(966005)(8676002)(76116006)(91956017)(66476007)(64756008)(53546011)(66446008)(66946007)(166002)(478600001)(86362001)(316002)(19627235002);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: =?gb2312?B?dHVuNW5KVHlmWXN2ZUE2dW1rVjNiMENtYmd4bWhTcWhTRlcrUlRZODJpUEoz?= =?gb2312?B?L0YvVkdDUmZjZC96MWhhbnpkUTVOei9VbjRTRytmS2ExVWk4RVQ5WE51cXRI?= =?gb2312?B?dVVTbkVqaFNSZXRqSm1sSVBZRnpxRkowVmp2M1hTczFXTFZCU3I4d2tEdXli?= =?gb2312?B?allPd1l4WG5JUVZ3VS9IK1Y1aVdhaFBjbGdiNHFsODJML1YyOTBSWjFsMWtt?= =?gb2312?B?V1J4eDJMY3k5QzlXSnFQcHZLNjBtRTRGTU5ocGNGSTVBaWF5SUUrZ2NvbmtG?= =?gb2312?B?Um5CWFRTaXZtbjVQakVZblFvS0xrYTFHZ2QxVHJFZHdMUjBHRm1RQmRyMkRI?= =?gb2312?B?T3k3c004L3ZwN2IwUGZUcHFSeXBxYzVtYUY0OGgrSTh5RFpkdjdEeDgwRXdu?= =?gb2312?B?azVRcmEreTZMTXpLYjFxMHk0WFI3dHBCaHV2WUlEUHBJclpSelA1WVJLdTNm?= =?gb2312?B?bWtKQXduREdRVXF6cXlHdVBPa25KcjJmbFBJQzBkeG4xV3d5eGI0QTFqVmFX?= =?gb2312?B?V1c3S1o1ZjgycW5ML3BUQWkzVEoybkJhYXp0TC9IK1JOV3puQjdoMFBhY2RG?= =?gb2312?B?a3lHeHh6VFZERitQNHlBNjBaY25HU1ExVTZYSU03OWFPSjV0U3FQWmo4bVVC?= =?gb2312?B?YlBTbnNRN2xWSDA3Nm01cGNYTGtOdGpxdG9tck95dXA2OVN4SGNmb3kzenZK?= =?gb2312?B?Vm0zZ3NBZlAvKzVBbFRTTTB3a3NSY1kzdWwvb1NxcFFTdTlTK0VEYnNKN2Ry?= =?gb2312?B?NXROclArMzQ4UlRta29iR01HSFZxMmJWeFJSODcxT1lIY0VtcjBBQW9tUHpn?= =?gb2312?B?LzcwSkFDRDZMVUp4bHd5VFAxS2NibXkzTWhOSHpGeVVPM0M4RzkwVXNmNlJ5?= =?gb2312?B?WVBxclRsaGhBcGNpbytBQXFFaXlIdXZ6cWlkVW1KR2lMczN0Y3pnUnhudFJI?= =?gb2312?B?bEMvL2JFM3VZNEwxS3pFZ2hxbWJLV29neUlWb1p6SjFLNGIwdDhrelAxTHo5?= =?gb2312?B?aEQyNzdzSGlNYzhGNng4MW1kQVcyZW8wYldhcmx5RTQ4c01YTFU4Y1dhSy9T?= =?gb2312?B?QWZKRms0dk5Ca1p5b3paSkUrQ3hOdkltSVJ0cmFzNWVvTG43ZkhiemVOTktt?= =?gb2312?B?QUFHdDlqRzljbHZVQWkzRkJwV3VsSE9NOWlUOFh1eVdxY2puMnNqRUd6UytT?= =?gb2312?B?dkpMb2x2bFlHUlpnaFBZQThCSnN3eWkwbHhxSTBLbm44bm8yZG05SU5NQWdS?= =?gb2312?B?TVZYR0dJbEdjMHlwMHRsenFUZVZBeXl1MUNzKzZBWVdvU1dEVEswOUpHREZY?= =?gb2312?B?b1FuUkhqV3pyaEovR0NBQlI3eUhSdzBIUzk5OWIvRkFXVkR3ODZKVHlBSVFG?= =?gb2312?B?Ylo3b09sdERkSUFDemZuRm5ySnVRY2xDdVlsbjZEaHFLdkhlSnQwZHNrd0RI?= =?gb2312?B?WkR4OTViMktvRWtrSHMxWlplakcvWFIzT25kZWxWVnpZc3dpWEx0QytlV2Zt?= =?gb2312?B?ampGeDZMc0UyMmpRZ2dnNGkzUllaQnFCV1VqWnZlZVpxS3VHblRScUlFZEFo?= =?gb2312?B?djdOSmprRksyTXRKbFpwZjBkNDNmaHROTWVER3NvS3Fxa0R3Vm52dkZLWk95?= =?gb2312?B?ZFNpdEVmWVVzV3RHWkRDNlg5UERuWk9xT1lScmpnRGJGT2hJUXErcGRuckwv?= =?gb2312?B?R01jVWRqNDZrVTh2UFVQaFNEdFk4N1lmVlBzWDYweXE4RzRVOWxKOXdPOC9W?= =?gb2312?B?bE95ektSTEVLTk9RWWxFNXRNT25PRVFMN0RoVXBBd3kzRTlCNDd6bzlGS2VC?= =?gb2312?B?b2ZvQysxSksvK3d6cERBUT09?= 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: 586764fd-9c9b-4af4-d4d8-08d8ca0536e1 X-MS-Exchange-CrossTenant-originalarrivaltime: 05 Feb 2021 18:38:21.6475 (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: f+I+xxKHw2+qH4CxHk2ajTgWed2P08+h+VUTdznI5UFJBor/8axDT3sUPLtHg16zJJvbKBEDdDeI3dPQThWFqw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR1101MB2222 Return-Path: ray.ni@intel.com X-OriginatorOrg: intel.com Content-Language: zh-CN Content-Type: multipart/alternative; boundary="_000_CO1PR11MB49307B227ECD58A88161CF0D8CB29CO1PR11MB4930namp_" --_000_CO1PR11MB49307B227ECD58A88161CF0D8CB29CO1PR11MB4930namp_ Content-Type: text/plain; charset="gb2312" Content-Transfer-Encoding: base64 TWlrZSwNCg0KVGhlIGNsZWFuIHVwIGRvZXNuoa90IGNhdXNlIGFueSBmaW5hbCBpbnN0cnVjdGlv biBjaGFuZ2UgYW5kIEkgaGF2ZSB2ZXJpZmllZCB0aGF0Lg0KVGhlIHJlYXNvbiBJIHB1dCB0aGUg Zml4IGluIGxhc3QgYmVjYXVzZSB0aGUgTG9jayBmaWVsZCBpcyBub3QgbmVlZGVkIHdpdGggdGhl IGZpeCBidXQgcmVtb3ZpbmcgdGhlIExvY2sgcmVxdWlyZXMgdG8gYWRqdXN0IGFsbCB0aGUgaGFy ZCBjb2RlIG9mZnNldHMuDQoNCldoYXQgcG90ZW50aWFsIGlzc3VlIGRvIHlvdSBzZWU/DQoNClRo YW5rcywNClJheQ0KDQp0aGFua3MsDQpyYXkNCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fDQq3orz+yMs6IEtpbm5leSwgTWljaGFlbCBEIDxtaWNoYWVsLmQua2lubmV5QGludGVsLmNv bT4NCreiy83KsbzkOiBTYXR1cmRheSwgRmVicnVhcnkgNiwgMjAyMSAxOjExOjE5IEFNDQrK1bz+ yMs6IGRldmVsQGVkazIuZ3JvdXBzLmlvIDxkZXZlbEBlZGsyLmdyb3Vwcy5pbz47IE5pLCBSYXkg PHJheS5uaUBpbnRlbC5jb20+OyBLaW5uZXksIE1pY2hhZWwgRCA8bWljaGFlbC5kLmtpbm5leUBp bnRlbC5jb20+DQrW98ziOiBSRTogW2VkazItZGV2ZWxdIFtQQVRDSCB2MiAwLzNdIFVzZSBYQURE IHRvIGF2b2lkIGxvY2sgYWNxdWlyZS9yZWxlYXNlDQoNCkhpIFJheSwNCg0KSSByZWFsbHkgbGlr ZSB0aGUgY2xlYW51cCB0byByZW1vdmUgaGFyZCBjb2RlZCBvZmZzZXRzLCBidXQgSSB0aGluayB0 aGF0IGNoYW5nZSBzaG91bGQgYmUgaXRzIG93biBwYXRjaCBzZXJpZXMuDQoNCkNhbiB3ZSBtYWtl IHRoZSBmdW5jdGlvbmFsIGNoYW5nZSB0byB1c2UgWEFERCBhcyBpdHMgb3duIHBhdGNoIHNlcmll cyBiZWZvcmUgdGhlIGNoYW5nZSB0byByZW1vdmUgaGFyZCBjb2RlZCBvZmZzZXRzIGFuZCB1c2Ug c3RydWN0Pw0KDQpUaGVuIGhhdmUgYSAybmQgcGF0Y2ggc2VyaWVzIHRoYXQgaXMgYSBub24tZnVu Y3Rpb25hbCBjaGFuZ2UgdG8gcmVtb3ZlIGhhcmQgY29kZWQgb2Zmc2V0cyBhbmQgdXNlIHN0cnVj dCBhbmQgcmVtb3ZlIHRoZSB1bnVzZWQgTG9jayBmaWVsZD8NCg0KVGhhbmtzLA0KDQpNaWtlDQoN Cj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gRnJvbTogZGV2ZWxAZWRrMi5ncm91cHMu aW8gPGRldmVsQGVkazIuZ3JvdXBzLmlvPiBPbiBCZWhhbGYgT2YgTmksIFJheQ0KPiBTZW50OiBU aHVyc2RheSwgRmVicnVhcnkgNCwgMjAyMSAxMTo1OCBQTQ0KPiBUbzogZGV2ZWxAZWRrMi5ncm91 cHMuaW8NCj4gU3ViamVjdDogW2VkazItZGV2ZWxdIFtQQVRDSCB2MiAwLzNdIFVzZSBYQUREIHRv IGF2b2lkIGxvY2sgYWNxdWlyZS9yZWxlYXNlDQo+DQo+IFBhdGNoICMxIGZvbGxvd3MgTGFzemxv J3Mgc3VnZ2VzdGlvbiB0byBhZGQgZ2xvYmFsIE5BU00gbWFjcm9zDQo+DQo+ICAgZm9yIE5BU00g c3RydWMgdXNhZ2UuDQo+DQo+IFBhdGNoICMyIGNoYW5nZXMgYWxsIGhhcmRjb2RlIG9mZnNldCB0 byB1c2Ugc3RydWMuDQo+DQo+IFBhdGNoICMzIGRvZXNuJ3QgaGF2ZSBhbnkgY2hhbmdlIGNvbXBh cmluZyB0byBWMSBleGNlcHQNCj4NCj4gICAxKS4gZHdvcmQvcXdvcmQgcHJlZml4IGlzIGFkZGVk Lg0KPg0KPiAgIDIpLiB0aGUgY29tbWVudHMgInByb2dyYW0gQVAgc3RhY2siIGlzIHJlbW92ZWQu DQo+DQo+IFJheSBOaSAoMyk6DQo+ICAgTWRlUGtnL05hc20uaW5jOiBhZGQgbWFjcm9zIGZvciBD IHR5cGVzIHVzZWQgaW4gc3RydWN0dXJlIGRlZmluaXRpb24NCj4gICBVZWZpQ3B1UGtnL01wSW5p dExpYjogVXNlIE5BU00gc3RydWMgdG8gYXZvaWQgaGFyZGNvZGUgb2Zmc2V0DQo+ICAgVWVmaUNw dVBrZy9NcEluaXRMaWI6IFVzZSBYQUREIHRvIGF2b2lkIGxvY2sgYWNxdWlyZS9yZWxlYXNlDQo+ DQo+ICBNZGVQa2cvSW5jbHVkZS9JYTMyL05hc20uaW5jICAgICAgICAgICAgICAgICAgfCAgMzgg KysrKysrDQo+ICBNZGVQa2cvSW5jbHVkZS9YNjQvTmFzbS5pbmMgICAgICAgICAgICAgICAgICAg fCAgMzggKysrKysrDQo+ICBVZWZpQ3B1UGtnL0xpYnJhcnkvTXBJbml0TGliL0R4ZU1wSW5pdExp Yi5pbmYgfCAgIDUgKy0NCj4gIFVlZmlDcHVQa2cvTGlicmFyeS9NcEluaXRMaWIvSWEzMi9NcEVx dS5pbmMgICB8ICA0MyAtLS0tLS0tDQo+ICAuLi4vTGlicmFyeS9NcEluaXRMaWIvSWEzMi9NcEZ1 bmNzLm5hc20gICAgICAgfCAgOTggKysrKysrKy0tLS0tLS0tLQ0KPiAgVWVmaUNwdVBrZy9MaWJy YXJ5L01wSW5pdExpYi9NcEVxdS5pbmMgICAgICAgIHwgIDk5ICsrKysrKysrKysrKysrKysNCj4g IFVlZmlDcHVQa2cvTGlicmFyeS9NcEluaXRMaWIvTXBMaWIuYyAgICAgICAgICB8ICAgMSAtDQo+ ICBVZWZpQ3B1UGtnL0xpYnJhcnkvTXBJbml0TGliL01wTGliLmggICAgICAgICAgfCAgIDMgKy0N Cj4gIFVlZmlDcHVQa2cvTGlicmFyeS9NcEluaXRMaWIvUGVpTXBJbml0TGliLmluZiB8ICAgNSAr LQ0KPiAgVWVmaUNwdVBrZy9MaWJyYXJ5L01wSW5pdExpYi9YNjQvTXBFcXUuaW5jICAgIHwgIDQ1 IC0tLS0tLS0tDQo+ICBVZWZpQ3B1UGtnL0xpYnJhcnkvTXBJbml0TGliL1g2NC9NcEZ1bmNzLm5h c20gfCAxMDggKysrKysrKystLS0tLS0tLS0tDQo+ICAxMSBmaWxlcyBjaGFuZ2VkLCAyNzIgaW5z ZXJ0aW9ucygrKSwgMjExIGRlbGV0aW9ucygtKQ0KPiAgZGVsZXRlIG1vZGUgMTAwNjQ0IFVlZmlD cHVQa2cvTGlicmFyeS9NcEluaXRMaWIvSWEzMi9NcEVxdS5pbmMNCj4gIGNyZWF0ZSBtb2RlIDEw MDY0NCBVZWZpQ3B1UGtnL0xpYnJhcnkvTXBJbml0TGliL01wRXF1LmluYw0KPiAgZGVsZXRlIG1v ZGUgMTAwNjQ0IFVlZmlDcHVQa2cvTGlicmFyeS9NcEluaXRMaWIvWDY0L01wRXF1LmluYw0KPg0K PiAtLQ0KPiAyLjI3LjAud2luZG93cy4xDQo+DQo+DQo+DQo+IC09LT0tPS09LT0tPQ0KPiBHcm91 cHMuaW8gTGlua3M6IFlvdSByZWNlaXZlIGFsbCBtZXNzYWdlcyBzZW50IHRvIHRoaXMgZ3JvdXAu DQo+IFZpZXcvUmVwbHkgT25saW5lICgjNzEzNDQpOiBodHRwczovL2VkazIuZ3JvdXBzLmlvL2cv ZGV2ZWwvbWVzc2FnZS83MTM0NA0KPiBNdXRlIFRoaXMgVG9waWM6IGh0dHBzOi8vZ3JvdXBzLmlv L210LzgwNDAxMjkwLzE2NDM0OTYNCj4gR3JvdXAgT3duZXI6IGRldmVsK293bmVyQGVkazIuZ3Jv dXBzLmlvDQo+IFVuc3Vic2NyaWJlOiBodHRwczovL2VkazIuZ3JvdXBzLmlvL2cvZGV2ZWwvdW5z dWIgW21pY2hhZWwuZC5raW5uZXlAaW50ZWwuY29tXQ0KPiAtPS09LT0tPS09LT0NCj4NCg0K --_000_CO1PR11MB49307B227ECD58A88161CF0D8CB29CO1PR11MB4930namp_ Content-Type: text/html; charset="gb2312" Content-Transfer-Encoding: quoted-printable
Mike,

The clean up doesn=A1=AFt cause any final instruction change and I have ver= ified that.
The reason I put the fix in last because the Lock field is not needed with = the fix but removing the Lock requires to adjust all the hard code offsets.=

What potential issue do you see?

Thanks,
Ray

thanks,
ray
=B7=A2=BC=FE=C8=CB: Kinney,= Michael D <michael.d.kinney@intel.com>
=B7=A2=CB=CD=CA=B1=BC=E4: Saturday, February 6, 2021 1:11:19 AM
=CA=D5=BC=FE=C8=CB: devel@edk2.groups.io <devel@edk2.groups.io>= ;; Ni, Ray <ray.ni@intel.com>; Kinney, Michael D <michael.d.kinney= @intel.com>
=D6=F7=CC=E2: RE: [edk2-devel] [PATCH v2 0/3] Use XADD to avoid lock= acquire/release
 
Hi Ray,

I really like the cleanup to remove hard coded offsets, but I think that ch= ange should be its own patch series.

Can we make the functional change to use XADD as its own patch series befor= e the change to remove hard coded offsets and use struct?

Then have a 2nd patch series that is a non-functional change to remove hard= coded offsets and use struct and remove the unused Lock field?

Thanks,

Mike

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of N= i, Ray
> Sent: Thursday, February 4, 2021 11:58 PM
> To: devel@edk2.groups.io
> Subject: [edk2-devel] [PATCH v2 0/3] Use XADD to avoid lock acquire/re= lease
>
> Patch #1 follows Laszlo's suggestion to add global NASM macros
>
>   for NASM struc usage.
>
> Patch #2 changes all hardcode offset to use struc.
>
> Patch #3 doesn't have any change comparing to V1 except
>
>   1). dword/qword prefix is added.
>
>   2). the comments "program AP stack" is removed.<= br> >
> Ray Ni (3):
>   MdePkg/Nasm.inc: add macros for C types used in structure = definition
>   UefiCpuPkg/MpInitLib: Use NASM struc to avoid hardcode off= set
>   UefiCpuPkg/MpInitLib: Use XADD to avoid lock acquire/relea= se
>
>  MdePkg/Include/Ia32/Nasm.inc      =             |  = 38 ++++++
>  MdePkg/Include/X64/Nasm.inc      &= nbsp;            |&n= bsp; 38 ++++++
>  UefiCpuPkg/Library/MpInitLib/DxeMpInitLib.inf |   5 +-=
>  UefiCpuPkg/Library/MpInitLib/Ia32/MpEqu.inc   |  = 43 -------
>  .../Library/MpInitLib/Ia32/MpFuncs.nasm    &= nbsp;  |  98 +++++++---------
>  UefiCpuPkg/Library/MpInitLib/MpEqu.inc    &n= bsp;   |  99 ++++++++++++++++
>  UefiCpuPkg/Library/MpInitLib/MpLib.c    &nbs= p;     |   1 -
>  UefiCpuPkg/Library/MpInitLib/MpLib.h    &nbs= p;     |   3 +-
>  UefiCpuPkg/Library/MpInitLib/PeiMpInitLib.inf |   5 +-=
>  UefiCpuPkg/Library/MpInitLib/X64/MpEqu.inc    |&n= bsp; 45 --------
>  UefiCpuPkg/Library/MpInitLib/X64/MpFuncs.nasm | 108 ++++++++----= ------
>  11 files changed, 272 insertions(+), 211 deletions(-)
>  delete mode 100644 UefiCpuPkg/Library/MpInitLib/Ia32/MpEqu.inc >  create mode 100644 UefiCpuPkg/Library/MpInitLib/MpEqu.inc
>  delete mode 100644 UefiCpuPkg/Library/MpInitLib/X64/MpEqu.inc >
> --
> 2.27.0.windows.1
>
>
>
> -=3D-=3D-=3D-=3D-=3D-=3D
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#71344): https://edk2.groups.io/g/devel/message/71344
> Mute This Topic: htt= ps://groups.io/mt/80401290/1643496
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://= edk2.groups.io/g/devel/unsub [michael.d.kinney@intel.com]
> -=3D-=3D-=3D-=3D-=3D-=3D
>

--_000_CO1PR11MB49307B227ECD58A88161CF0D8CB29CO1PR11MB4930namp_--