OpenVZ Forum


Home » Mailing lists » Devel » [PATCH 1/9] namespaces: add nsproxy
Re: [PATCH 1/9] namespaces: add nsproxy [message #3301 is a reply to message #3299] Sun, 21 May 2006 23:38 Go to previous messageGo to previous message
ebiederm is currently offline  ebiederm
Messages: 1354
Registered: February 2006
Senior Member
Sam Vilain <sam@vilain.net> writes:

> Serge E. Hallyn wrote:
>
>>@@ -1585,7 +1591,15 @@ asmlinkage long sys_unshare(unsigned lon
>>
>> if (new_fs || new_ns || new_sigh || new_mm || new_fd || new_ulist) {
>>
>>+ old_nsproxy = current->nsproxy;
>>+ new_nsproxy = dup_namespaces(old_nsproxy);
>>+ if (!new_nsproxy) {
>>+ err = -ENOMEM;
>>+ goto bad_unshare_cleanup_semundo;
>>+ }
>>+
>> task_lock(current);
>>
>>
>
> We'll get lots of duplicate nsproxy structures before we move all of the
> pointers for those subsystems into it. Do we need to dup namespaces on
> all of those conditions?

Ugh. Good catch. The new nsproxy needs to be just for the fs and the uts
namespace.

I guess that means that test should be moved up a few lines.

Eric
 
Read Message
Read Message
Read Message
Read Message
Previous Topic: [Fwd: [PATCH 1/1] scsi : megaraid_{mm, mbox}: a fix on 64-bit DMA capability check]
Next Topic: Re: [PATCH 0/9] namespaces: Introduction
Goto Forum:
  


Current Time: Fri Jul 25 11:38:35 GMT 2025

Total time taken to generate the page: 0.42905 seconds