[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <b98f7b78-1834-4fa0-b79c-d5ac562e4809@amd.com>
Date: Fri, 11 Apr 2025 11:37:26 +0530
From: Shivank Garg <shivankg@....com>
To: Paul Moore <paul@...l-moore.com>
Cc: seanjc@...gle.com, david@...hat.com, vbabka@...e.cz, willy@...radead.org,
akpm@...ux-foundation.org, shuah@...nel.org, pbonzini@...hat.com,
ackerleytng@...gle.com, jmorris@...ei.org, serge@...lyn.com, pvorel@...e.cz,
bfoster@...hat.com, tabba@...gle.com, vannapurve@...gle.com,
chao.gao@...el.com, bharata@....com, nikunj@....com, michael.day@....com,
yan.y.zhao@...el.com, Neeraj.Upadhyay@....com, thomas.lendacky@....com,
michael.roth@....com, aik@....com, jgg@...dia.com, kalyazin@...zon.com,
peterx@...hat.com, linux-fsdevel@...r.kernel.org, linux-mm@...ck.org,
linux-kernel@...r.kernel.org, linux-security-module@...r.kernel.org,
kvm@...r.kernel.org, linux-kselftest@...r.kernel.org,
linux-coco@...ts.linux.dev
Subject: Re: [PATCH RFC v7 3/8] security: Export
security_inode_init_security_anon for KVM guest_memfd
Hi Paul,
On 4/10/2025 1:49 AM, Paul Moore wrote:
> On Tue, Apr 8, 2025 at 7:25 AM Shivank Garg <shivankg@....com> wrote:
>>
>> KVM guest_memfd is implementing its own inodes to store metadata for
>> backing memory using a custom filesystem. This requires the ability to
>> initialize anonymous inode using security_inode_init_security_anon().
>>
>> As guest_memfd currently resides in the KVM module, we need to export this
>> symbol for use outside the core kernel. In the future, guest_memfd might be
>> moved to core-mm, at which point the symbols no longer would have to be
>> exported. When/if that happens is still unclear.
>
> Can you help me understand the timing just a bit more ... do you
> expect the move to the core MM code to happen during the lifetime of
> this patchset, or is it just some hand-wavy "future date"? No worries
> either way, just trying to understand things a bit better.
I am not sure about it, any ideas David?
Thanks,
Shivank
>
>> Signed-off-by: Shivank Garg <shivankg@....com>
>> ---
>> security/security.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/security/security.c b/security/security.c
>> index fb57e8fddd91..097283bb06a5 100644
>> --- a/security/security.c
>> +++ b/security/security.c
>> @@ -1877,6 +1877,7 @@ int security_inode_init_security_anon(struct inode *inode,
>> return call_int_hook(inode_init_security_anon, inode, name,
>> context_inode);
>> }
>> +EXPORT_SYMBOL(security_inode_init_security_anon);
>>
>> #ifdef CONFIG_SECURITY_PATH
>> /**
>> --
>> 2.34.1
>
Powered by blists - more mailing lists