Re: [RFC] libxfs kernel infrastructure (was [XFS updates] XFS development tree branch, xfs-libxfs-in-kernel-RFC, created. xfs-for-linus-3.15-rc2-52-g6579dd8)

Christoph Hellwig
2014-05-06T08:37:48Z

I like this in general, but one major and one minor issue with
the include files:

- headers that just include other headers are a bad idea in general.
Either they are dependent enough that they should be merged, or
they are not, in which case they shouldn't.

In this case it seems like we should temporarily provide a
xfs_mount.h stub in userspace, and just leave all the includes
for things in libxfs.h as they were. That doesn't preclude further
merging of the headers into more sensible ones as we've started
with the disk formats.

- do we really need the separate include/ dir? That always annoys
me when editing code. It makes sense for something that is a real
public interface, which this is not.

Also is libxfs/ really the right name? libxfs in userspace has quite
a bit more code than this, so maybe we should just called this "shared"
for the shared user/kernel code?