Re: [PATCH 2/2] x86/sgx: Maintain encl->refcount for each encl->mm_list entry

2021-02-07 Thread Jarkko Sakkinen
On Sun, Feb 07, 2021 at 11:29:49PM +0200, Jarkko Sakkinen wrote: > On Fri, Feb 05, 2021 at 11:36:57AM -0800, Dave Hansen wrote: > > On 2/5/21 10:28 AM, Jarkko Sakkinen wrote: > > > This has been shown in tests: > > > > > > [ +0.08] WARNING: CPU: 3 PID: 7620 at kernel/rcu/srcutree.c:374 > > >

Re: [PATCH 2/2] x86/sgx: Maintain encl->refcount for each encl->mm_list entry

2021-02-07 Thread Jarkko Sakkinen
On Fri, Feb 05, 2021 at 11:36:57AM -0800, Dave Hansen wrote: > On 2/5/21 10:28 AM, Jarkko Sakkinen wrote: > > This has been shown in tests: > > > > [ +0.08] WARNING: CPU: 3 PID: 7620 at kernel/rcu/srcutree.c:374 > > cleanup_srcu_struct+0xed/0x100 > > > > There are two functions that drain e

Re: [PATCH 2/2] x86/sgx: Maintain encl->refcount for each encl->mm_list entry

2021-02-05 Thread Dave Hansen
On 2/5/21 10:28 AM, Jarkko Sakkinen wrote: > This has been shown in tests: > > [ +0.08] WARNING: CPU: 3 PID: 7620 at kernel/rcu/srcutree.c:374 > cleanup_srcu_struct+0xed/0x100 > > There are two functions that drain encl->mm_list: > > - sgx_release() (i.e. VFS release) removes the remaining

[PATCH 2/2] x86/sgx: Maintain encl->refcount for each encl->mm_list entry

2021-02-05 Thread Jarkko Sakkinen
This has been shown in tests: [ +0.08] WARNING: CPU: 3 PID: 7620 at kernel/rcu/srcutree.c:374 cleanup_srcu_struct+0xed/0x100 There are two functions that drain encl->mm_list: - sgx_release() (i.e. VFS release) removes the remaining mm_list entries. - sgx_mmu_notifier_release() removes mm_l

[PATCH 2/2] x86/sgx: Maintain encl->refcount for each encl->mm_list entry

2021-02-05 Thread Jarkko Sakkinen
This has been shown in tests: [ +0.08] WARNING: CPU: 3 PID: 7620 at kernel/rcu/srcutree.c:374 cleanup_srcu_struct+0xed/0x100 There are two functions that drain encl->mm_list: - sgx_release() (i.e. VFS release) removes the remaining mm_list entries. - sgx_mmu_notifier_release() removes mm_l