Hi, > This patch need to be broken into multiple sub patches for the easy to > review and keeping the code bisectable. The first thing we need to do is > introduce the new CCWorkArea concept and update the existing Sev support to > use the new CCWorkArea.
Fully agree. Also moving around code without functional change (i.e. move the SEV bits to a new asm file) should go to a separate patch(es). That way the patches actually adding new functionality are smaller and easier to review. thanks, Gerd -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#79349): https://edk2.groups.io/g/devel/message/79349 Mute This Topic: https://groups.io/mt/84631106/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-