Tested-by: Nataliia Bondarevska <bond...@google.com>
On Fri, May 2, 2025 at 1:56 AM Jarkko Sakkinen <jar...@kernel.org> wrote: > > On Fri, 2025-05-02 at 07:22 +0000, Reshetova, Elena wrote: > > > > > > > > On Wed, Apr 30, 2025 at 06:53:32AM +0000, Reshetova, Elena wrote: > > > > 2. Switch to Sean's approach to execute EUPDATESVN during the > > > sgx_open(). > > > > Btw, Sean do you agree that we don't gain much doing it second > > > > time during > > > > release() given the microcode flow? > > > > I would rather leave only one invocation of eupdatesvn during > > > sgx_inc_usage_count(). > > > > > > > > Proc: No new uABI. More predictable on svn change compared to > > > > option 1. > > > > > > > Cons: Two explicit paths to hook: sgx_open() and sgx_vepc_open(). > > > > > > Why this is a con? > > > > Well, just from the pov of not having a single path to enable. > > Are you ok with option 2? > > > > Yep, as SGX is anyway very much run-time managed feature and these > hooks fit better on how it is used. > > > Best Regards, > > Elena. > > BR, Jarkko >
