Re: [PATCH 11/11] sysfs: user namespaces: fix bug with clone(CLONE_NEWUSER) with fairsched

From: Eric W. Biederman
Date: Wed Jun 25 2008 - 17:21:37 EST


"Serge E. Hallyn" <serue@xxxxxxxxxx> writes:

> Except the sysfs mount holds no refcount on the userns. So as long as we
> do the ida tagging as you suggested in your response to patch 6, there
> should be no reference to the user_ns left in sysfs code.
>
> The extra reference in patch #9 is for a light ref on the network
> namespace. I'm still not sure that needs to be there, since if the
> network namespace goes away, it will properly unregister its sysfs
> mounts. Eric, Benjamin, I really don't see any use for the hold_net()
> from sysfs. What is it doing?

Mostly just being a sanity check. We can remove that if it easier.

Eric
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/