stable rc 4.4 - v4.4.232-33-g0b3898baf614 - build breaks on arm64, arm, x86_64 and i386.

stable rc 4.4 - v4.4.232-33-g0b3898baf614 - build breaks on arm64, arm, x86_64 and i386.

[Naresh Kamboju]

stable rc 4.4 build breaks on arm64, arm, x86_64 and i386.

Here are the build log failures on arm64.
   git_repo: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
    target_arch: arm64
    toolchain: gcc-9
    git_short_log: 0b3898baf614 (\Linux 4.4.233-rc1\)
    git_sha: 0b3898baf61459e1f963dcf893b4683174668975
    git_describe: v4.4.232-33-g0b3898baf614
    kernel_version: 4.4.233-rc1

make -sk KBUILD_BUILD_USER=TuxBuild -C/linux -j16 ARCH=arm64
CROSS_COMPILE=aarch64-linux-gnu- HOSTCC=gcc CC="sccache
aarch64-linux-gnu-gcc" O=build Image
#
../arch/arm64/kernel/hw_breakpoint.c: In function ‘arch_bp_generic_fields’:
../arch/arm64/kernel/hw_breakpoint.c:348:5: note: parameter passing
for argument of type ‘struct arch_hw_breakpoint_ctrl’ changed in GCC
9.1
  348 | int arch_bp_generic_fields(struct arch_hw_breakpoint_ctrl ctrl,
      |     ^~~~~~~~~~~~~~~~~~~~~~
../fs/ext4/inode.c: In function ‘ext4_direct_IO’:
../fs/ext4/inode.c:3355:9: error: ‘offset’ redeclared as different
kind of symbol
 3355 |  loff_t offset = iocb->ki_pos;
      |         ^~~~~~
../fs/ext4/inode.c:3349:17: note: previous definition of ‘offset’ was here
 3349 |          loff_t offset)
      |          ~~~~~~~^~~~~~
make[3]: *** [../scripts/Makefile.build:277: fs/ext4/inode.o] Error 1
make[3]: Target '__build' not remade because of errors.
make[2]: *** [../scripts/Makefile.build:484: fs/ext4] Error 2
../drivers/net/ethernet/apm/xgene/xgene_enet_main.c:32:36: warning:
array ‘xgene_enet_acpi_match’ assumed to have one element
   32 | static const struct acpi_device_id xgene_enet_acpi_match[];
      |                                    ^~~~~~~~~~~~~~~~~~~~~
make[2]: Target '__build' not remade because of errors.
make[1]: *** [/linux/Makefile:1006: fs] Error 2
make[1]: Target 'Image' not remade because of errors.
make: *** [Makefile:152: sub-make] Error 2
make: Target 'Image' not remade because of errors.

-- 
Linaro LKFT
https://lkft.linaro.org

Re: stable rc 4.4 - v4.4.232-33-g0b3898baf614 - build breaks on arm64, arm, x86_64 and i386.

[Jan Kara]

On Wed 05-08-20 22:42:08, Naresh Kamboju wrote:
> stable rc 4.4 build breaks on arm64, arm, x86_64 and i386.
> 
> Here are the build log failures on arm64.
>    git_repo: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
>     target_arch: arm64
>     toolchain: gcc-9
>     git_short_log: 0b3898baf614 (\Linux 4.4.233-rc1\)
>     git_sha: 0b3898baf61459e1f963dcf893b4683174668975
>     git_describe: v4.4.232-33-g0b3898baf614
>     kernel_version: 4.4.233-rc1
> 
> make -sk KBUILD_BUILD_USER=TuxBuild -C/linux -j16 ARCH=arm64
> CROSS_COMPILE=aarch64-linux-gnu- HOSTCC=gcc CC="sccache
> aarch64-linux-gnu-gcc" O=build Image
> #
> ../arch/arm64/kernel/hw_breakpoint.c: In function ‘arch_bp_generic_fields’:
> ../arch/arm64/kernel/hw_breakpoint.c:348:5: note: parameter passing
> for argument of type ‘struct arch_hw_breakpoint_ctrl’ changed in GCC
> 9.1
>   348 | int arch_bp_generic_fields(struct arch_hw_breakpoint_ctrl ctrl,
>       |     ^~~~~~~~~~~~~~~~~~~~~~
> ../fs/ext4/inode.c: In function ‘ext4_direct_IO’:
> ../fs/ext4/inode.c:3355:9: error: ‘offset’ redeclared as different
> kind of symbol
>  3355 |  loff_t offset = iocb->ki_pos;
>       |         ^~~~~~
> ../fs/ext4/inode.c:3349:17: note: previous definition of ‘offset’ was here
>  3349 |          loff_t offset)
>       |          ~~~~~~~^~~~~~

This looks like a breakage from "ext4: fix direct I/O read error" where for
4.4 the patch needs to be updated (addition of the line
"loff_t offset = iocb->ki_pos;" needs to be removed from the patch because
'offset' is already passed from the caller. Jiang, will you update the patch
for 4.4 kernels please?

								Honza

> make[3]: *** [../scripts/Makefile.build:277: fs/ext4/inode.o] Error 1
> make[3]: Target '__build' not remade because of errors.
> make[2]: *** [../scripts/Makefile.build:484: fs/ext4] Error 2
> ../drivers/net/ethernet/apm/xgene/xgene_enet_main.c:32:36: warning:
> array ‘xgene_enet_acpi_match’ assumed to have one element
>    32 | static const struct acpi_device_id xgene_enet_acpi_match[];
>       |                                    ^~~~~~~~~~~~~~~~~~~~~
> make[2]: Target '__build' not remade because of errors.
> make[1]: *** [/linux/Makefile:1006: fs] Error 2
> make[1]: Target 'Image' not remade because of errors.
> make: *** [Makefile:152: sub-make] Error 2
> make: Target 'Image' not remade because of errors.
> 
> -- 
> Linaro LKFT
> https://lkft.linaro.org
-- 
Jan Kara <jack@suse.com>
SUSE Labs, CR