[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4BDA74D8.9020503@gmail.com>
Date: Fri, 30 Apr 2010 08:12:40 +0200
From: Tejun Heo <htejun@...il.com>
To: "Eric W. Biederman" <ebiederm@...ssion.com>
CC: Greg KH <gregkh@...e.de>, bcrl@...et.ca, benjamin.thery@...l.net,
cornelia.huck@...ibm.com, eric.dumazet@...il.com,
kay.sievers@...y.org, netdev@...r.kernel.org, serue@...ibm.com
Subject: Re: patch sysfs-implement-sysfs-tagged-directory-support.patch added
to gregkh-2.6 tree
Hello,
On 04/30/2010 07:37 AM, Tejun Heo wrote:
>> Tejun I think for the code to make any sense to you I would need to rip
>> out out and/or rewrite the kobject layer, and possible the device
>> model code as well.
>
> And yes, in the long run, please do that.
Let me add a little bit here just in case.
IIRC, the initial sysfs tag support wasn't too different from the
interface side but the implementation inside sysfs was very hacky, so
I complained on both accounts. You did a wonderful job of
restructuing sysfs so that it basically behaves as a distributed file
system and exposing different subsets is not too hacky and doesn't
violate layering. Your work there was admirable and much better than
what I had in mind.
I don't think the hooking part from NSes would be as major an overhaul
as you're suggesting above. I haven't tried it so I can't tell with
certainty (again so no nack) but I just can't imagine it being that
difficult or major after all the things you've done in the area and
was hoping that you repost something which is more digestible soonish.
I'm sorry that I couldn't be more specific but I'm almost sure you'll
be able to come up with something better than what I can think of.
So, yeah, please,
* Add comments to the current implementation. If for nothing else,
for cases where you get sick of it for some time and other people
have to work on it before you feel like coming back.
* While doing that, please add "FIXME: or TODO:" comments describing
what would be a better direction in the long run.
Thanks.
--
tejun
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists