xfsprogs: libxfs update to current kernel code
Hi folks,
I just pushed a libxfs-3.19-update branch to
git://git.kernel.org/pub/scm/fs/xfs/xfsprogs-dev.git
That contains an update of the libxfs code to match the current kernel libxfs code. It's in pretty rough shape right now, but it seems to work. It will get rebased as I refine the patchset.
The stages of the update are:
- rework some libxfs definitions
- update to 3.16 code
- introduce libxfs error negation patches
- restructure libxfs to match kernel code
- update to the current for-next branch
In doing this update, there are a few things that need to be separated out to the head of the series and made explicit:
- support for anything pre- dir V2 goes away, so there's a bunch of new "use and older xfsprogs" conditions that need to be added and a significant amount of "handle old bugs" code that needs to go away
- the equivalent kernel code only supports v2 inodes - it unconditionally sets the NLINK feature bit and converts v1 inodes to v2 inodes. All the userspace tools need to be updated to match - they'll read v1 inodes, but they'll unconditionally set NLINK and write only v2 inodes.
- the SHARED superblock stuff goes away, so killing that needs to be a standalone patch
Given these changes in supported functionality, I expect the first release with this update to be a 3.3.0 release, not a 3.2.x release. This probably makes it a good time to update mkfs default behaviour as well (i.e. to default to CRC enabled filesystems).
There is also more cleanup work to be done after the 3.19 update:
- the include file structure is somewhat cludgy. I simply made the new libxfs structure link into the existing include/xfs structure in a slightly different way, but this needs substantial rework to clean it up.
- there is a convoluted mess of include files between include/ and libxfs/ that determine what code gets compiled and how it is seen externally. The first commit in the series sort of points out how messy this is, but this really needs to be better sorted to separate internal and externally visible libxfs functions, as well as separate libxfs structures from userspace structures like struct xfs_mount....
I don't really expect significant *code* review of the libxfs part of the update - it's pretty large at at +7000/-6500 lines. I'm more interested in getting feedback on the changes of supported functionality and the infrastructure changes. If anyone wants to kick the tyres they are more than welcome to - at this stage any feedback is better than none. :)
Cheers,
Dave.
Signature · 2 lines
Dave Chinner [email protected]
_______________________________________________
xfs mailing list
[email protected]
http://oss.sgi.com/mailman/listinfo/xfs
Quoted 9 lines · Dave Chinner
Hi folks,
I just pushed a libxfs-3.19-update branch to
git://git.kernel.org/pub/scm/fs/xfs/xfsprogs-dev.gitThat contains an update of the libxfs code to match the current kernel libxfs code. It's in pretty rough shape right now, but it seems to work. It will get rebased as I refine the patchset.
I just rebased this branch with a more recent version. Top commit is:
13ea6a1 libxfs: split out xfs->libxfs mappings
Quoted 8 lines
The stages of the update are:
- rework some libxfs definitions
- update to 3.16 code
- introduce libxfs error negation patches
- restructure libxfs to match kernel code
- update to the current for-next branch
This part of the branch is unchanged.
Quoted 14 lines
In doing this update, there are a few things that need to be separated out to the head of the series and made explicit:
- support for anything pre- dir V2 goes away, so there's a bunch of new "use and older xfsprogs" conditions that need to be added and a significant amount of "handle old bugs" code that needs to go away
- the equivalent kernel code only supports v2 inodes - it unconditionally sets the NLINK feature bit and converts v1 inodes to v2 inodes. All the userspace tools need to be updated to match - they'll read v1 inodes, but they'll unconditionally set NLINK and write only v2 inodes.
- the SHARED superblock stuff goes away, so killing that needs to be a standalone patch
i.e. none of this has been done yet.
Quoted 8 lines
Given these changes in supported functionality, I expect the first release with this update to be a 3.3.0 release, not a 3.2.x release. This probably makes it a good time to update mkfs default behaviour as well (i.e. to default to CRC enabled filesystems).
There is also more cleanup work to be done after the 3.19 update:
Quoted 12 lines
- the include file structure is somewhat cludgy. I simply made the new libxfs structure link into the existing
include/xfsstructure in a slightly different way, but this needs substantial rework to clean it up.
- there is a convoluted mess of include files betweeninclude/and libxfs/ that determine what code gets compiled and how it is seen externally. The first commit in the series sort of points out how messy this is, but this really needs to be better sorted to separate internal and externally visible libxfs functions, as well as separate libxfs structures from userspace structures like structxfs_mount....
But this is starting to take shape. What I've changed in terms of structure is that libxfs/ now has a libxfs/support subdirectory, into which I've moved all the userspace specific libxfs code. i.e:
libxfs/*[ch]
- matches kernel size fs/xfs/libxfs/*[ch]
libxfs/support/*
- contains all userspace libxfs support code
I've also changed some of the include files - there are some "kernel equivalent" header files in libxfs/support now, e.g. xfs_mount.h and xfs_inode.h which contain the userspace definitions of the structures (e.g. struct xfs_mount) used by the libxfs code. This means that th elibxfs code can mostly use the same include files and so reduce the differences between the two code bases.
I've renamed libxfs/xfs.h to libxfs/support/libxfs_priv.h to remove the ambiguity with include/xfs.h.
I've also separated out all the xfs->libxfs mapping macros into a libxfs/support/libxfs_api_defs.h file. This is included into both libxfs/support/libxfs_priv.h and include/libxfs.h, allowing include/libxfs.h to get rid of all the libxfs_* function prototypes as everything is correctly mapped just by including the libxfs/xfs*h header files after the mapping macros.
This has greatly simplified include/libxfs.h, and shown exactly where we still have mismatches between userspace and kernel space. There's still more work to do here, but the majority of the userspace libxfs restructuring is present in the branch.
FYI, ignoring file movement, the diffstat so far looks like this:
$ git diff --stat -C -M origin/master..origin/libxfs-3.19-update .... 150 files changed, 8295 insertions(+), 7378 deletions(-)
Cheers,
Dave.
Signature · 7 lines
Dave Chinner [email protected] _______________________________________________ xfs mailing list [email protected] http://oss.sgi.com/mailman/listinfo/xfs