Hey,

This is the third take of the patchset to separate out kernfs from
sysfs.

Changes between the second[S] and third take are

* cb26a311578e ("sysfs: drop kobj_ns_type handling") had a bug in
  symlink handling incorrectly enabling tag filtering in a directory
  which contains a symlink pointing to a tagged directory and got
  reverted.  0001-sysfs-drop-kobj_ns_type-handling-take-2.patch added
  to try it again.

* 0001 adds SYSFS_NS_FLAG and testing code in sysfs.
  0040-sysfs-kernfs-implement-kernfs_ns_enabled.patch added to
  encapsulate the flag access.

Changes between the first[F] and second[S] take are

* Patches applied to driver-core-next are dropped and the rest are
  refreshed.

* Mount portion of kernfs implemented.  kernfs now supports multiple
  sysfs_dirent hierarchies and sysfs is just one of them.

* Dummy interface for kernfs for !CONFIG_SYSFS updated to return
  -ENOSYS / ERR_PTR(-ENOSYS) where applicable.

* Several bug fixes.

This patchset contains 41 patches to separate out core sysfs features
into kernfs.  kernfs exclusively deals with sysfs_dirent, which will
be later renamed to kernfs_node, and kernfs_ops.  sysfs becomes a
wrapping layer over sysfs which interfaces kobject and
[bin_]attribute.

The goal of these changes is to allow other users to make use of the
core features of sysfs instead of rolling their own pseudo filesystem
implementation which usually fails to deal with issues with file
shutdowns, locking separation from vfs layer and so on.  This patchset
refactors sysfs and separates out core functionalities to kernfs.
After the patchset, kernfs is fully functional - multiple sysfs_dirent
hierarchies can be created and mounted and sysfs is just one user of
kernfs.

This patchset shouldn't introduce any behavior differences and
contains the following patches.

 0001-sysfs-drop-kobj_ns_type-handling-take-2.patch
 0002-sysfs-make-__sysfs_add_one-fail-if-the-parent-isn-t-.patch
 0003-sysfs-kernfs-add-skeletons-for-kernfs.patch
 0004-sysfs-kernfs-introduce-kernfs_remove-_by_name-_ns.patch
 0005-sysfs-kernfs-introduce-kernfs_create_link.patch
 0006-sysfs-kernfs-introduce-kernfs_rename-_ns.patch
 0007-sysfs-kernfs-introduce-kernfs_setattr.patch
 0008-sysfs-kernfs-replace-sysfs_dirent-s_dir.kobj-and-s_a.patch
 0009-sysfs-kernfs-introduce-kernfs_create_dir-_ns.patch
 0010-sysfs-kernfs-prepare-read-path-for-kernfs.patch
 0011-sysfs-kernfs-prepare-write-path-for-kernfs.patch
 0012-sysfs-kernfs-prepare-mmap-path-for-kernfs.patch
 0013-sysfs-kernfs-prepare-open-release-poll-paths-for-ker.patch
 0014-sysfs-kernfs-move-sysfs_open_file-to-include-linux-k.patch
 0015-sysfs-kernfs-introduce-kernfs_ops.patch
 0016-sysfs-kernfs-add-sysfs_dirent-s_attr.size.patch
 0017-sysfs-kernfs-remove-SYSFS_KOBJ_BIN_ATTR.patch
 0018-sysfs-kernfs-introduce-kernfs_create_file-_ns.patch
 0019-sysfs-kernfs-remove-sysfs_add_one.patch
 0020-sysfs-kernfs-add-kernfs_ops-seq_-start-next-stop.patch
 0021-sysfs-kernfs-introduce-kernfs_notify.patch
 0022-sysfs-kernfs-reorganize-SYSFS_-constants.patch
 0023-sysfs-kernfs-revamp-sysfs_dirent-active_ref-lockdep-.patch
 0024-sysfs-kernfs-introduce-kernfs-_find_and-_get-and-ker.patch
 0025-sysfs-kernfs-move-internal-decls-to-fs-kernfs-kernfs.patch
 0026-sysfs-kernfs-move-inode-code-to-fs-kernfs-inode.c.patch
 0027-sysfs-kernfs-move-dir-core-code-to-fs-kernfs-dir.c.patch
 0028-sysfs-kernfs-move-file-core-code-to-fs-kernfs-file.c.patch
 0029-sysfs-kernfs-move-symlink-core-code-to-fs-kernfs-sym.patch
 0030-sysfs-kernfs-drop-unused-params-from-sysfs_fill_supe.patch
 0031-sysfs-kernfs-make-sysfs_super_info-ns-const.patch
 0032-sysfs-kernfs-no-need-to-kern_mount-sysfs-from-sysfs_.patch
 0033-sysfs-kernfs-introduce-sysfs_root_sd.patch
 0034-sysfs-kernfs-implement-kernfs_create-destroy_root.patch
 0035-sysfs-kernfs-make-inode-number-ida-per-kernfs_root.patch
 0036-sysfs-kernfs-make-super_blocks-bind-to-different-ker.patch
 0037-sysfs-kernfs-prepare-mount-path-for-kernfs.patch
 0038-sysfs-kernfs-move-mount-core-code-to-fs-kernfs-mount.patch
 0039-sysfs-kernfs-make-sysfs_dirent-definition-public.patch
 0040-sysfs-kernfs-implement-kernfs_ns_enabled.patch
 0041-sysfs-kernfs-remove-cross-inclusions-of-internal-hea.patch

 0001 retries removing kobj_ns_type handling.

 0002 is prep patch.

 0003 preps fs/kernfs/ directory with skeleton files.

 0004-0024 refactor various code paths so that all externally visible
 interfaces are split to core kernfs interface which deals with
 sysfs_dirent and kernfs_ops and sysfs wrapping it to provide the
 existing interface.

 0025-0029 move kernfs part of the implementation under fs/kernfs.

 0030-0036 update mount logic so that it can support multiple
 sysfs_dirent hierarchies instead of assuming single sysfs hierarchy.

 0037-0038 refactor mount code path for kernfs and moves kernfs
 interface and implementation to fs/kernfs.

 0039-0041 cut the remaining entanglement between sysfs and kernfs and
 make the former proper user of the latter.

Multiple hierarchy support tested with the attached module (needs
adding EXPORT_SYMBOL to kernfs interface).  Works as expected.

This patchset is on top of

  v3.13-rc1 6ce4eac1f600 ("Linux 3.13-rc1")
+ [1] sysfs: handle duplicate removal attempts in sysfs_remove_group()
+ [2] sysfs: use a separate locking class for open files depending on mmap

The patches are also available in the following git branch.

 git://git.kernel.org/pub/scm/linux/kernel/git/tj/misc.git 
review-separate-out-kernfs

diffstat follows.  Thanks.

 fs/Makefile                 |    2 
 fs/kernfs/Makefile          |    5 
 fs/kernfs/dir.c             | 1014 +++++++++++++++++++++++++++++++++++++++++++
 fs/kernfs/file.c            |  814 ++++++++++++++++++++++++++++++++++
 fs/kernfs/inode.c           |  337 ++++++++++++++
 fs/kernfs/kernfs-internal.h |  115 ++++
 fs/kernfs/mount.c           |  165 +++++++
 fs/kernfs/symlink.c         |  149 ++++++
 fs/namespace.c              |    2 
 fs/sysfs/Makefile           |    2 
 fs/sysfs/dir.c              | 1030 --------------------------------------------
 fs/sysfs/file.c             |  931 +++++++--------------------------------
 fs/sysfs/group.c            |   59 +-
 fs/sysfs/inode.c            |  331 --------------
 fs/sysfs/mount.c            |  182 -------
 fs/sysfs/symlink.c          |  179 -------
 fs/sysfs/sysfs.h            |  228 ---------
 include/linux/kernfs.h      |  348 ++++++++++++++
 include/linux/sysfs.h       |   43 -
 lib/kobject.c               |   27 -
 20 files changed, 3252 insertions(+), 2711 deletions(-)

--
tejun

[F] https://lkml.kernel.org/g/1382629780-10006-1-git-send-email...@kernel.org
[S] https://lkml.kernel.org/g/1383472398-17812-1-git-send-email...@kernel.org
[1] https://lkml.kernel.org/g/20131123183508.ga28...@mtj.dyndns.org
[2] https://lkml.kernel.org/g/20131123183540.gb28...@mtj.dyndns.org
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to