OpenVZ Forum


Home » Mailing lists » Devel » nxproxy patches
nxproxy patches [message #17420] Thu, 01 February 2007 01:25 Go to next message
Andrew Morton is currently offline  Andrew Morton
Messages: 127
Registered: December 2005
Senior Member
Sorry, I've run out of confidence on these patches: heaps of rejects and
build errors and me making changes in an ameteurish fashion to try to fix
them.  Plus I have a feeling that there's still some disagreement about
the actual implementation details in there.

So I'm going to drop the lot.  These, in this order:

introduce-and-use-get_task_mnt_ns.patch
nsproxy-externalizes-exit_task_namespaces.patch
user-namespace-add-the-framework.patch
user-ns-add-user_namespace-ptr-to-vfsmount.patch
user-ns-hook-permission.patch
user-ns-prepare-copy_tree-copy_mnt-and-their-callers-to-handle-errs.patch
user-ns-implement-shared-mounts.patch
user_ns-handle-file-sigio.patch
user-ns-implement-user-ns-unshare.patch
#
rename-attach_pid-to-find_attach_pid.patch
attach_pid-with-struct-pid-parameter.patch
remove-find_attach_pid.patch
statically-initialize-struct-pid-for-swapper.patch
explicitly-set-pgid-sid-of-init.patch
#


are at http://userweb.kernel.org/~akpm/nsproxy/

I seem to have folded two inappropriate patches into
uts-namespace-remove-config_uts_ns.patch so perhaps it should be split up
again, or just renamed.

Could someone please fix them up, retest, re-changelog and resubmit them
all for re-review?

Thanks.
_______________________________________________
Containers mailing list
Containers@lists.osdl.org
https://lists.osdl.org/mailman/listinfo/containers
Re: nxproxy patches [message #17426 is a reply to message #17420] Thu, 01 February 2007 09:55 Go to previous message
Cedric Le Goater is currently offline  Cedric Le Goater
Messages: 443
Registered: February 2006
Senior Member
Andrew Morton wrote:
> Sorry, I've run out of confidence on these patches: heaps of rejects and
> build errors and me making changes in an ameteurish fashion to try to fix
> them.  Plus I have a feeling that there's still some disagreement about
> the actual implementation details in there.

well, here's what i recall from the last exchanges we had :  

* eric said he wouldn't nak them if the status was marked experimental 
  (mostly because uid checks haven't been done yet). that's a micro
  patch i have not sent yet.
* herbert said he could use them.
* so did the openvz team 

Agree ? 

if not, are the uid checks not being done the only argument to 
declare the user namespace not mergeable yet (in 2.6.21) ?

> So I'm going to drop the lot.  These, in this order:
> 
> introduce-and-use-get_task_mnt_ns.patch
> nsproxy-externalizes-exit_task_namespaces.patch
> user-namespace-add-the-framework.patch
> user-ns-add-user_namespace-ptr-to-vfsmount.patch
> user-ns-hook-permission.patch
> user-ns-prepare-copy_tree-copy_mnt-and-their-callers-to-handle-errs.patch
> user-ns-implement-shared-mounts.patch
> user_ns-handle-file-sigio.patch
> user-ns-implement-user-ns-unshare.patch
> #
> rename-attach_pid-to-find_attach_pid.patch
> attach_pid-with-struct-pid-parameter.patch
> remove-find_attach_pid.patch
> statically-initialize-struct-pid-for-swapper.patch
> explicitly-set-pgid-sid-of-init.patch
> #
> 
> 
> are at http://userweb.kernel.org/~akpm/nsproxy/

ok. got them.

> I seem to have folded two inappropriate patches into
> uts-namespace-remove-config_uts_ns.patch so perhaps it should be split up
> again, or just renamed.
> 
> Could someone please fix them up, retest, re-changelog and resubmit them
> all for re-review?

We can definitely improve the testing ... I'll take them back in the -lxc 
patchset and we will give them a good test shake in our labs before 
resending.

thanks,

C.
_______________________________________________
Containers mailing list
Containers@lists.osdl.org
https://lists.osdl.org/mailman/listinfo/containers
Previous Topic: Re: Please confirm your request to join NESTJOBS
Next Topic: [RFC] Allow access to /proc/$PID/fd after setuid()
Goto Forum:
  


Current Time: Sun Aug 10 16:13:44 GMT 2025

Total time taken to generate the page: 0.07278 seconds