[PATCH] xfs: fix build without xfsstats

Eric Sandeen sandeen at sandeen.net
Fri Oct 16 19:08:01 CDT 2015



On 10/16/15 4:47 PM, Arnd Bergmann wrote:
> A recent rework of the xfsstats interface introduced a compile error
> for the procfs-disabled case:
> 
> fs/built-in.o: In function `xfs_free_ag_extent':
> fs/xfs/libxfs/xfs_alloc.c:1725: undefined reference to `xfsstats'
> fs/built-in.o: In function `xfs_alloc_ag_vextent':
> fs/xfs/libxfs/xfs_alloc.c:645: undefined reference to `xfsstats'
> fs/built-in.o: In function `xfs_attr_set':
> fs/xfs/libxfs/xfs_attr.c:353: undefined reference to `xfsstats'
> fs/built-in.o: In function `xfs_attr_remove':
> fs/xfs/libxfs/xfs_attr.c:483: undefined reference to `xfsstats'
> fs/built-in.o: In function `xfs_attr_get':
> fs/xfs/libxfs/xfs_attr.c:131: undefined reference to `xfsstats'
> fs/built-in.o:fs/xfs/libxfs/xfs_bmap.c:1461: more undefined references to `xfsstats' follow
> 
> This adds conditional compilation to each reference of the xfsstats
> global variable to get it to link again.
> 
> An alternative approach would be to always build the statistics
> implementation even when CONFIG_PROC_FS is disabled.

Yep, they live for realz in sysfs now, not procfs.

Reported a couple times already ;)  Dave has a patch at:

https://marc.info/?l=linux-xfs&m=144476889031721&w=2

Thanks,
-Eric



More information about the xfs mailing list