Re: [PATCH 15/20] sysfs: Kill sysfs_addrm_start and sysfs_addrm_finish

Tejun Heo
2009-05-21T09:31:24Z

Eric W. Biederman wrote:
> From: Eric W. Biederman <ebiederm@xmission.com>
>
> With lazy inode updates and dentry operations bringing everything
> into sync on demand there is no longer any need to immediately
> update the vfs or grab i_mutex to protect those updates as we
> make changes to sysfs.
>
> So stop updating the vfs inodes and move what remains of
> sysfs_addrm_start and sysfs_addrm_finsih (just barely more than taking
> the sysfs_mutex) into sysfs_add_one and sysfs_remove_one.
>
> Signed-off-by: Eric W. Biederman <ebiederm@aristanetworks.com>

This looks so better than the original code. :-)

Acked-by: Tejun Heo <tj@kernel.org>

--
tejun