On Tue, Jun 10, 2014 at 09:09:08AM -0500, Eric Sandeen wrote:
> On 6/10/14, 1:09 AM, Dave Chinner wrote:
> > There is a new version of this work (Eric's patches and the base
> > libxfs restructure) here:
> > git://oss.sgi.com/xfs/xfs.git xfs-libxfs-restructure
> > It's based on 3.15 with the current for-next branch merged into it
> > and then the changes done over the top, so it's about as up-to-date
> > as it can be. Comments welcome...
> Comment 1: doesn't build ;)
> make: *** No rule to make target `fs/xfs/xfs_dir2_readdir.o', needed by
> `fs/xfs/xfs.o'. Stop.
> make: *** Waiting for unfinished jobs....
> CC [M] fs/xfs/xfs_buf.o
> make: *** [_module_fs/xfs] Error 2
Ah, that's an issue....
> problems w/ xfs_rtbitmap.o as well, patch follows.
> Comment 2: Coverity thinks this adds about 25 defects, I'll have to go
> look at why...
Probably because it ignores issues based on path/filename, and the
> diff --git a/fs/xfs/Makefile b/fs/xfs/Makefile
> index a22a6b8..0b7b3b3 100644
> --- a/fs/xfs/Makefile
> +++ b/fs/xfs/Makefile
> @@ -43,6 +43,7 @@ xfs-y += $(addprefix libxfs/,
> xfs_dir2_data.o \
> xfs_dir2_leaf.o \
> xfs_dir2_node.o \
> + xfs_dir2_readdir.o \
No, the xfs_dir2_readdir.c file should not have been moved - it's
kernel only functionality.
> xfs_dir2_sf.o \
> xfs_dquot_buf.o \
> xfs_ialloc.o \
> @@ -55,6 +56,9 @@ xfs-y += $(addprefix libxfs/,
> xfs_trans_resv.o \
> +xfs-$(CONFIG_XFS_RT) += $(addprefix libxfs/, \
> + xfs_rtbitmap.o \
> + )
But that needs fixing, yes.
Makes me wonder why it linked here - must have found stale .o files
in the build area and linked them even though the files didn't get
Anyway, I'll fix it, and restart the errno negation rebase....