On 6/8/07, Serge E. Hallyn <[email protected]> wrote:
I do fear that that could become a maintenance nightmare. For instance right now there's the call to fsnotify_mkdir(). Other such hooks might be placed at vfs_mkdir, which we'd then likely want to have placed in our container_mkdir() and container_clone() fns. And of course may_create() is static inline in fs/namei.c. It's trivial, but still if it changes we'd want to change the version in kernel/container.c as well.
Do we need to actually need to respect may_create() in container_clone()? I guess it would provide a way for root to control which processes could unshare namespaces.
What would be the main advantage of doing it this way? Do you consider the extra subys->auto_setup() hook to be avoidable bloat?
I was thinking that it would be nice to be able to atomically set up the resources in the new container at the point when it's created rather than later. But I guess this way can work too. Can we call it something like "clone()" rather than "auto_setup()"? Paul - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
- Follow-Ups:
- Re: [ckrm-tech] [PATCH 00/10] Containers(V10): Generic Process Containers
- From: "Serge E. Hallyn" <[email protected]>
- Re: [ckrm-tech] [PATCH 00/10] Containers(V10): Generic Process Containers
- References:
- Re: [PATCH 00/10] Containers(V10): Generic Process Containers
- From: "Serge E. Hallyn" <[email protected]>
- Re: [ckrm-tech] [PATCH 00/10] Containers(V10): Generic Process Containers
- From: "Serge E. Hallyn" <[email protected]>
- Re: [ckrm-tech] [PATCH 00/10] Containers(V10): Generic Process Containers
- From: Paul Jackson <[email protected]>
- Re: [ckrm-tech] [PATCH 00/10] Containers(V10): Generic Process Containers
- From: "Serge E. Hallyn" <[email protected]>
- Re: [ckrm-tech] [PATCH 00/10] Containers(V10): Generic Process Containers
- From: Paul Jackson <[email protected]>
- Re: [ckrm-tech] [PATCH 00/10] Containers(V10): Generic Process Containers
- From: "Serge E. Hallyn" <[email protected]>
- Re: [ckrm-tech] [PATCH 00/10] Containers(V10): Generic Process Containers
- From: "Paul Menage" <[email protected]>
- Re: [ckrm-tech] [PATCH 00/10] Containers(V10): Generic Process Containers
- From: "Serge E. Hallyn" <[email protected]>
- Re: [ckrm-tech] [PATCH 00/10] Containers(V10): Generic Process Containers
- From: "Paul Menage" <[email protected]>
- Re: [ckrm-tech] [PATCH 00/10] Containers(V10): Generic Process Containers
- From: "Serge E. Hallyn" <[email protected]>
- Re: [PATCH 00/10] Containers(V10): Generic Process Containers
- Prev by Date: Re: 2.6.22-rc4-mm1 -- x86_64 ACPI panic
- Next by Date: [0/1] [patch -mm] Add containerstats (v3)
- Previous by thread: Re: [ckrm-tech] [PATCH 00/10] Containers(V10): Generic Process Containers
- Next by thread: Re: [ckrm-tech] [PATCH 00/10] Containers(V10): Generic Process Containers
- Index(es):