xfs
[Top] [All Lists]

Re: [PATCH] xfs: fix build without xfsstats

To: xfs@xxxxxxxxxxx
Subject: Re: [PATCH] xfs: fix build without xfsstats
From: Eric Sandeen <sandeen@xxxxxxxxxxx>
Date: Fri, 16 Oct 2015 19:08:01 -0500
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <5578984.gxbqPUmtuT@wuerfel>
References: <5578984.gxbqPUmtuT@wuerfel>
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:38.0) Gecko/20100101 Thunderbird/38.3.0

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

<Prev in Thread] Current Thread [Next in Thread>