From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by mx.groups.io with SMTP id smtpd.web11.6954.1587603520214467596 for ; Wed, 22 Apr 2020 17:58:40 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.100, mailfrom: eric.dong@intel.com) IronPort-SDR: OgDJQejN15lpGUJi4j+HS8HfRp/efNYsrDqw5YfJ5HSgZzRzNhjgS3smlub00SUyzQv9hmVALf UlG425AzE3wQ== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 22 Apr 2020 17:58:39 -0700 IronPort-SDR: pJoCaPP+Vv/I2Y2lXIZlnDuMDElr9PpXGzxj7Rvcpq5DO/JhYUNNiksmNBxfBd4fa0sVoJB0xh Pe0Dcyz1Xomw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.73,305,1583222400"; d="scan'208";a="402741562" Received: from fmsmsx108.amr.corp.intel.com ([10.18.124.206]) by orsmga004.jf.intel.com with ESMTP; 22 Apr 2020 17:58:39 -0700 Received: from fmsmsx120.amr.corp.intel.com (10.18.124.208) by FMSMSX108.amr.corp.intel.com (10.18.124.206) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 22 Apr 2020 17:58:24 -0700 Received: from shsmsx154.ccr.corp.intel.com (10.239.6.54) by fmsmsx120.amr.corp.intel.com (10.18.124.208) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 22 Apr 2020 17:58:23 -0700 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.138]) by SHSMSX154.ccr.corp.intel.com ([169.254.7.214]) with mapi id 14.03.0439.000; Thu, 23 Apr 2020 08:58:21 +0800 From: "Dong, Eric" To: "devel@edk2.groups.io" , "lersek@redhat.com" CC: "Ni, Ray" , "Kumar, Chandana C" Subject: Re: [edk2-devel] [PATCH v2 1/2] UefiCpuPkg/MpInitLib: Restore IDT context for APs. Thread-Topic: [edk2-devel] [PATCH v2 1/2] UefiCpuPkg/MpInitLib: Restore IDT context for APs. Thread-Index: AQHWGO7HdoUT7CfqL0CR5N9UYQ3yx6iF4BnQ Date: Thu, 23 Apr 2020 00:58:20 +0000 Message-ID: References: <20200422090111.423-1-eric.dong@intel.com> <3c028212-e539-53bf-c883-bdbc518ff5fd@redhat.com> In-Reply-To: <3c028212-e539-53bf-c883-bdbc518ff5fd@redhat.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: eric.dong@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable > -----Original Message----- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Laszlo Ersek > Sent: Thursday, April 23, 2020 5:41 AM > To: devel@edk2.groups.io; Dong, Eric > Cc: Ni, Ray ; Kumar, Chandana C > > Subject: Re: [edk2-devel] [PATCH v2 1/2] UefiCpuPkg/MpInitLib: Restore I= DT > context for APs. >=20 > Eric, >=20 > On 04/22/20 11:01, Dong, Eric wrote: > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D2683 >=20 > I don't know what patches I'm supposed to review now. You seem to have > sent 4 (four) versions of this patch set to the list, but you didn't cal= l them {v1, > v2, v3, v4}; you called them {v1, v2, v2, v2}. I can only work off of th= e > dates/times at which you sent the paches. Laszlo, Sorry, It's my mistake to send out error patches. I have send out patch th= ree times: 1) Send v1 patch set. 2) Try to send V2 patch set, but I missed the "V2" tag for the 1/2 and 2/2= patches, just add it to 0/2. 3) only send 1/2 and 2/2 of V2 patch and add "V2" tag on these two patches= .=20 So please just review the patches which has V2 tag and ignore the other er= ror patches.=20 Thanks, Eric >=20 > To make the confusion worse: >=20 > - your third (v3?) posting says "0/2" in the cover letter, but I can onl= y see one > patch below it. >=20 > - your last (v4?) posting (that I can see) doesn't even have a cover let= ter. >=20 > (a) [edk2-devel] [PATCH 0/2] UefiCpuPkg/MpInitLib: Fix ASSERT in AP > procedure. > https://edk2.groups.io/g/devel/message/57772 >=20 > Date: Wed, 22 Apr 2020 09:34:54 +0800 >=20 > (b) [edk2-devel] [PATCH v2 0/2] UefiCpuPkg/MpInitLib: Fix ASSERT in AP > procedure > https://edk2.groups.io/g/devel/message/57786 >=20 > Date: Wed, 22 Apr 2020 16:45:56 +0800 > (~7 hours after (a)) >=20 > (c) [edk2-devel] [PATCH v2 0/2] UefiCpuPkg/MpInitLib: Fix ASSERT in AP > procedure > https://edk2.groups.io/g/devel/message/57791 >=20 > Date: Wed, 22 Apr 2020 16:52:51 +0800 > (~7 minutes after (b)) >=20 > (d) [edk2-devel] [PATCH v2 1/2] UefiCpuPkg/MpInitLib: Restore IDT contex= t > for APs. > https://edk2.groups.io/g/devel/message/57794 >=20 > Date: Wed, 22 Apr 2020 17:01:10 +0800 > (~8 minutes after (c)) >=20 > And your latest comment on the bugzilla ticket, at > , seems to > indicate that (b) is what you consider the most up-to-date version. But = you > send two versions ((c) and (d)) after that. >=20 > I think I won't review any of these patches now. >=20 > (I don't even know under which I should send this response.) >=20 > Thanks > Laszlo >=20 >=20 >=20