From: Benjamin Thery <benjamin.thery@bull.net>
To: Greg KH <greg@kroah.com>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>,
linux-kernel@vger.kernel.org,
"Serge E. Hallyn" <serue@us.ibm.com>,
Al Viro <viro@ftp.linux.org.uk>
Subject: Re: sysfs: tagged directories not merged completely yet
Date: Tue, 23 Sep 2008 16:24:30 +0200 [thread overview]
Message-ID: <48D8FC1E.6000601@bull.net> (raw)
In-Reply-To: <20080922153455.GA6238@kroah.com>
Greg KH wrote:
> On Mon, Sep 22, 2008 at 04:31:32PM +0200, Benjamin Thery wrote:
>> Hi Greg,
>>
>> Someone made me notice that Eric's "sysfs tagged directories" patchset
>> you merged recently isn't complete in your "patches" tree. The last two
>> patches that actually uses this new feature aren't in your tree yet.
>>
>> * [PATCH 7/8] netns: Enable tagging for net_class directories in sysfs
>> * [PATCH 8/8] sysfs: user namespaces: fix bug with clone(CLONE_NEWUSER)
>> with fairsched
>>
>> I didn't noticed it sooner because I'm still using my own version of the
>> patchset on top of linux-2.6.
>>
>> I'm wondering why you left these two patches out? Is it on purpose? Do
>> you expect other maintainers to merged them in their respective tree?
>> Dave Miller acked the patch for network namespace and said "You can send
>> it all through Greg or whoever, don't feel obligated to push it through
>> me since it depends upon the earlier bits in this series".
>
> I only got so far through the patch set and then got interrupted (was at
> different conferences for 2 weeks), sorry. I'll get the rest of the
> patchset added later today.
>
> Note, I did talk to Al Viro about these patches last week at the kernel
> summit, and he advised holding off on sending them to mainline before he
> could review them. So I'll keep them in my tree for now, but not push
> them to Linus until he acks them as he raised a few issues in our
> conversation.
Oh.
It's a pity Al couldn't re-review them before. We've already lost a lot
of time with this patchset and it's blocking easier testing of network
namespaces (right now, with a mainline kernel, we have to disable sysfs
to build network namespaces).
Eric sent the first version at the end of last year, then I've sent it 6
times between April and July to keep it alive, and finally Eric reworked
the patchset in July and sent a couple of versions to address the issues
Tejun Heo had formulated. It would have been great to receive all the
feedback at that time.
But, I know, we all are busy men and we all have our priorities, and
it takes time to get the things done right and then merged.
I have to learn patience. :)
Thanks anyway. It's a good step to have it in a tree.
Benjamin
> thanks,
>
> greg k-h
>
>
--
B e n j a m i n T h e r y - BULL/DT/Open Software R&D
http://www.bull.com
next prev parent reply other threads:[~2008-09-23 14:24 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-09-22 14:31 sysfs: tagged directories not merged completely yet Benjamin Thery
2008-09-22 15:34 ` Greg KH
2008-09-22 20:24 ` Eric W. Biederman
2008-09-23 14:24 ` Benjamin Thery [this message]
2008-09-23 18:23 ` Eric W. Biederman
2008-10-03 10:13 ` Al Viro
2008-10-05 5:32 ` Greg KH
2008-10-07 8:27 ` Eric W. Biederman
2008-10-07 10:47 ` [PATCH 0/3] minor sysfs tagged directory fixes Eric W. Biederman
2008-10-07 10:49 ` [PATCH 1/3] sysfs: Remove lock ordering violation in sysfs_chmod_file Eric W. Biederman
2008-10-07 10:51 ` [PATCH 2/3] sysfs: Fix and sysfs_mv_dir by using lock_rename Eric W. Biederman
2008-10-07 10:52 ` [PATCH 3/3] sysfs: Take sysfs_mutex when fetching the root inode Eric W. Biederman
2008-10-07 21:21 ` [PATCH 2/3] sysfs: Fix and sysfs_mv_dir by using lock_rename Dave Hansen
2008-10-07 21:19 ` [PATCH 1/3] sysfs: Remove lock ordering violation in sysfs_chmod_file Dave Hansen
2008-10-07 22:31 ` Eric W. Biederman
2008-10-07 22:27 ` sysfs: tagged directories not merged completely yet Greg KH
2008-10-07 22:54 ` Serge E. Hallyn
2008-10-07 23:39 ` Greg KH
2008-10-08 0:12 ` Serge E. Hallyn
2008-10-08 0:38 ` Greg KH
2008-10-08 14:18 ` Serge E. Hallyn
2008-10-07 23:34 ` Tejun Heo
2008-10-14 1:11 ` Eric W. Biederman
2008-10-14 7:55 ` Tejun Heo
2008-10-14 12:19 ` Eric W. Biederman
2008-10-15 11:04 ` Tejun Heo
2008-10-16 21:58 ` Eric W. Biederman
2008-10-14 18:53 ` Serge E. Hallyn
2008-10-15 0:48 ` Eric W. Biederman
2008-10-15 13:42 ` Serge E. Hallyn
2008-10-15 13:54 ` Benjamin Thery
2008-10-08 0:39 ` Eric W. Biederman
2008-10-08 1:29 ` Eric W. Biederman
2008-10-07 8:08 ` Eric W. Biederman
2008-10-07 9:01 ` Daniel Lezcano
2008-10-07 9:12 ` Tejun Heo
2008-10-07 11:56 ` Eric W. Biederman
2008-10-07 12:19 ` Tejun Heo
2008-10-07 23:17 ` Tejun Heo
2008-10-08 0:04 ` Eric W. Biederman
2008-10-08 0:20 ` Tejun Heo
2008-10-08 0:58 ` Eric W. Biederman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=48D8FC1E.6000601@bull.net \
--to=benjamin.thery@bull.net \
--cc=ebiederm@xmission.com \
--cc=greg@kroah.com \
--cc=linux-kernel@vger.kernel.org \
--cc=serue@us.ibm.com \
--cc=viro@ftp.linux.org.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.