On Sun, Feb 20, 2022 at 09:09:27PM +0100, Jarkko Sakkinen wrote:
On Sun, Feb 20, 2022 at 10:58:00AM -0800, Dave Hansen wrote:
On 2/20/22 10:52, Jarkko Sakkinen wrote:
Am I just all twisted around from looking at this code too much? Could you please take another look and send a new version of the patch?
Yeah, I will. It took me two weeks to make full remote attestation implementation for SGX in Rust but at least I can confirm that all the kernel bits work on that (and this will also help me to finish the man page because it is pretty nasty to document it if you haven't gone to the rabbit hole yourself).
I'll prioritize now to get SGX2 patches tested with Enarx implementation but this 2nd thing in my kernel queue.
I really prioritize bug fixes over new features. New features can's really get merged until this gets fixed up.
Fair enough. In this case I was not exactly sure whether it is categorized as bug or enhancement. Should I also put a fixes tag to this to the original reclaimer code (since it has been like this since the epoch)?
I'll keep the fixes tag, and send a new version of the patch, as soon as I have tested it (working on it right now).
BR, Jarkko