[Devel] [PATCH VZ10 1/8] ve/mntns: add ve_owner to struct mnt_namespace

Pavel Tikhomirov ptikhomirov at virtuozzo.com
Fri May 15 13:55:19 MSK 2026



On 5/14/26 16:39, Konstantin Khorenko wrote:
> On 4/29/26 15:41, Pavel Tikhomirov wrote:
>> Introduce a ve_owner field on struct mnt_namespace pointing at the ve
>> that owns the namespace.
>>
>> This will be used to derive correct ownership of newly created mounts in
>> case of simultaneous creation of new ve namespace and mount namespace,
>> were we would like new mounts to have the new ve as an owner.
> 
> s/were/where

Yeh, thanks for fixing it everywhere!

> 
>>
>> There is no change in behaviour yet: alloc_mnt_ns() still resolves the
>> owner from current ve via get_exec_env().
>>
>> To ensure the pointer is valid, alloc_mnt_ns takes and free_mnt_ns()
>> drops the reference.
>>
>> https://virtuozzo.atlassian.net/browse/VSTOR-129744
>> Signed-off-by: Pavel Tikhomirov <ptikhomirov at virtuozzo.com>
>> Feature: ve: ve generic structures
>> ---
>>   fs/mount.h     | 6 ++++++
>>   fs/namespace.c | 8 ++++++++
>>   2 files changed, 14 insertions(+)
>>
>> diff --git a/fs/mount.h b/fs/mount.h
>> index 8e8fa47f54fd..dd1ef7e40639 100644
>> --- a/fs/mount.h
>> +++ b/fs/mount.h
>> @@ -4,6 +4,9 @@
>>   #include <linux/poll.h>
>>   #include <linux/ns_common.h>
>>   #include <linux/fs_pin.h>
>> +#ifdef CONFIG_VE
>> +#include <linux/ve.h>
>> +#endif
> 
> i will remove ifdef CONFIG_VE here.
> We have many places of includes without ifdef, the !CONFIG_VE case should be handled inside ve.h
> 
> 

Agreed. Thanks!


-- 
Best regards, Pavel Tikhomirov
Senior Software Developer, Virtuozzo.



More information about the Devel mailing list