On Thu, Jan 18, 2024 at 2:21 AM Liu, Zhiguang <zhiguang....@intel.com> wrote: > > Thanks Laszlo for the comment, I will send a new version of patch to fix this. > > Also include Pedro to see if Pedro have more comments.
The patch's subject really doesn't describe the fix (describe what you did in the patch/commit, don't describe what you're fixing) I also liked your IsFlushTlbNeeded suggestion (it would help clarify what you actually want to keep track of). -- Pedro -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#113998): https://edk2.groups.io/g/devel/message/113998 Mute This Topic: https://groups.io/mt/103781942/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-