Eric, On 04/22/20 11:01, Dong, Eric wrote: > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2683
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 call them {v1, v2, v3, v4}; you called them {v1, v2, v2, v2}. I can only work off of the dates/times at which you sent the paches. To make the confusion worse: - your third (v3?) posting says "0/2" in the cover letter, but I can only see one patch below it. - your last (v4?) posting (that I can see) doesn't even have a cover letter. (a) [edk2-devel] [PATCH 0/2] UefiCpuPkg/MpInitLib: Fix ASSERT in AP procedure. https://edk2.groups.io/g/devel/message/57772 Date: Wed, 22 Apr 2020 09:34:54 +0800 (b) [edk2-devel] [PATCH v2 0/2] UefiCpuPkg/MpInitLib: Fix ASSERT in AP procedure https://edk2.groups.io/g/devel/message/57786 Date: Wed, 22 Apr 2020 16:45:56 +0800 (~7 hours after (a)) (c) [edk2-devel] [PATCH v2 0/2] UefiCpuPkg/MpInitLib: Fix ASSERT in AP procedure https://edk2.groups.io/g/devel/message/57791 Date: Wed, 22 Apr 2020 16:52:51 +0800 (~7 minutes after (b)) (d) [edk2-devel] [PATCH v2 1/2] UefiCpuPkg/MpInitLib: Restore IDT context for APs. https://edk2.groups.io/g/devel/message/57794 Date: Wed, 22 Apr 2020 17:01:10 +0800 (~8 minutes after (c)) And your latest comment on the bugzilla ticket, at <https://bugzilla.tianocore.org/show_bug.cgi?id=2683#c2>, 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. I think I won't review any of these patches now. (I don't even know under which I should send this response.) Thanks Laszlo -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#57881): https://edk2.groups.io/g/devel/message/57881 Mute This Topic: https://groups.io/mt/73191434/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-