APPLIED[linux-gcp]: [SRU][Bionic][linux/linux-gcp][PATCH 0/1] Fix xfs build error

Kleber Souza kleber.souza at canonical.com
Thu Sep 13 17:22:59 UTC 2018


On 09/13/18 18:56, Kleber Sacilotto de Souza wrote:
> This issue has real impact only on linux-gcp, which is the only kernel
> that enables CONFIG_XFS_ONLINE_SCRUB. So I'm proposing the fix for
> bionic/linux-gcp to be applied to the current cycle, and for
> bionic/linux to fix it on the main kernel on a following cycle.
> The patch applied for gcp will be gone away on the next rebase.
> 
> SRU Justification:
> 
> Impact:
> The backport of a78ee256c325 ("xfs: convert XFS_AGFL_SIZE to a helper function")
> done as part of "Bionic update: upstream stable patchset 2018-08-24"
> (bug 1788897) is incomplete, causing the build to fail if
> CONFIG_XFS_ONLINE_SCRUB is selected.
> 
> /tmp/kernel-kleber-79aa9f7-Y9dG/build/fs/xfs/scrub/agheader.c: In function 'xfs_scrub_walk_agfl':
> /tmp/kernel-kleber-79aa9f7-Y9dG/build/fs/xfs/scrub/agheader.c:96:24: error: implicit declaration of function 'XFS_AGFL_SIZE'; did you mean 'XFS_ACL_SIZE'? [-Werror=implicit-function-declaration]
>   for (i = flfirst; i < XFS_AGFL_SIZE(mp); i++) {
>                         ^~~~~~~~~~~~~
>                         XFS_ACL_SIZE
> 
> The backport didn't replace all the occurrences of the XFS_AGFL_SIZE macro
> by the xfs_agfl_size function.
> 
> Fix:
> A SAUCE patch is needed to fix the backport, given that the problem doesn't
> exist with mainline.
> 
> Testcase:
> Enable CONFIG_XFS_ONLINE_SCRUB and compile the kernel.
> 
> 
> Kleber Sacilotto de Souza (1):
>   UBUNTU: SAUCE: xfs: fix build error with CONFIG_XFS_ONLINE_SCRUB
>     enabled
> 
>  fs/xfs/scrub/agheader.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 

Applied to bionic/linux-gcp/master-next branch.

NOTE: not applied to bionic/linux/master-next.

Thanks,
Kleber




More information about the kernel-team mailing list