Re: [PATCH 3/6] sysfs: Implement sysfs tagged directory support.

From: Tejun Heo
Date: Wed Mar 31 2010 - 04:14:45 EST


Hello, Eric.

On 03/31/2010 04:43 PM, Eric W. Biederman wrote:
>> Is it at all possible to implement it in properly layered manner?
>> ie. sysfs providing mechanisms for selective visibility, driver model
>> wraps it and exports it and namespace implements namespaces on top of
>> those mechanisms?
>
> I think that is roughly what I have.

Yeah, well, in a sense. It's all a matter of degree.

> As for the layering itself. Down in sysfs there are only two bits
> visible. A void * pointer that in addition to the name is what we use
> to define selective visibility. A context that we capture when we
> mount sysfs. Those bits are fundamental things sysfs needs to do.

Well, I guess all I wanna say is... is there *ANY* way to do it w/
less callbacks? It's very difficult to follow what's going on for
what.

If you think all those callbacks are absolute necessities, can you
please at least add boatload of comments around them explaning what
they're meant to do and how they're gonna be used? It's probably
because I don't have any experience with namespaces but I really can't
wrap my head around it as it currently stands.

Thanks.

--
tejun
--
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/