Re: [PATCHSET 3/4] sysfs: divorce sysfs from kobject and driver model

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Tejun Heo
Date: Saturday, September 29, 2007 - 3:06 pm

Hello, Eric.

Eric W. Biederman wrote:

Hmmm...  I think most current non-driver-model sysfs users are deep in
kernel anyway, but I think not exporting sysfs interface at all might be
a bit too restrictive.  I think we need to examine the current
non-driver-model sysfs users thoroughly to determine what to do about
this.  But, yes, I do agree that we need to put restrictions one way or
the other.


I think it would be better if namespace comes after interface update and
other new features, especially symlink renaming, but, under the current
circumstance, it might delay namespace unnecessarily and I have no
problem with your patches going in first.  My concerns are...

* Do you think you can use new rename implementation contained in this
series?  It borrows basic ideas from the implementation you used for
namespace but is more generic.  It would be great if you can use it
without too much modification.

* I'm still against using callbacks to determine namespace tags because
callbacks need to be coupled with sysfs internals more tightly and are
more difficult to grasp interface-wise.


Ah... Coming few months will be fun, won't they?  :-)

-- 
tejun
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 05/22] sysfs: implement sysfs_find_child(), Tejun Heo, (Thu Sep 20, 1:05 am)
[PATCH 06/22] sysfs: restructure addrm helpers, Tejun Heo, (Thu Sep 20, 1:05 am)
[PATCH 01/22] sysfs: make sysfs_root a pointer, Tejun Heo, (Thu Sep 20, 1:05 am)
[PATCH 04/22] sysfs: make SYSFS_COPY_NAME a flag, Tejun Heo, (Thu Sep 20, 1:05 am)
[PATCH 14/22] sysfs: s/symlink/link/g, Tejun Heo, (Thu Sep 20, 1:05 am)
[PATCH 21/22] sysfs: kill sysfs_hash_and_remove(), Tejun Heo, (Thu Sep 20, 1:05 am)
Re: [PATCHSET 3/4] sysfs: divorce sysfs from kobject and d ..., Eric W. Biederman, (Thu Sep 27, 12:25 pm)
Re: [PATCHSET 3/4] sysfs: divorce sysfs from kobject and d ..., Tejun Heo, (Sat Sep 29, 3:06 pm)
Re: [PATCHSET 3/4] sysfs: divorce sysfs from kobject and d ..., Eric W. Biederman, (Wed Oct 10, 6:16 am)
Re: [PATCHSET 3/4] sysfs: divorce sysfs from kobject and d ..., Eric W. Biederman, (Wed Oct 10, 2:16 pm)
Re: [PATCHSET 3/4] sysfs: divorce sysfs from kobject and d ..., Eric W. Biederman, (Tue Oct 16, 4:54 pm)