[BUG] Linux-next fails to link on ARM in allyesconfig

* [BUG] Linux-next fails to link on ARM in allyesconfig
@ 2019-01-23 10:09 Maxim Kuvyrkov
  2019-01-23 10:12 ` Ard Biesheuvel
  2019-01-23 11:06 ` Eran Ben Elisha
  0 siblings, 2 replies; 3+ messages in thread
From: Maxim Kuvyrkov @ 2019-01-23 10:09 UTC (permalink / raw)
  To: Eran Ben Elisha
  Cc: Mark Brown, Arnd Bergmann, linux-arm-kernel, Ard Biesheuvel

Hi Eran,

It seems that your commit 880ee82f0313453ec5a6cb122866ac057263066b causes a link failure for ARM in allyesconfig.  This happens for at least gcc-8 and gcc-master compilers.
===
arm-linux-gnueabihf-ld: net/core/devlink.o: in function `devlink_health_buffers_create':
devlink.c:(.text+0x4eac): undefined reference to `__aeabi_uldivmod'
===

Would you please investigate?  Below is diagnostics from Linaro's automated bisection job.

Successfully identified regression in CI configuration gnu-release-arm-next-allyesconfig for linux between bad_rev 5b22549b8c0011b8cc81b6f4fcd082e5545c3225 and baseline_rev a37d50ca3b837c19a297f349365d11a20c1087d0 .

Culprit:
<cut>
commit 880ee82f0313453ec5a6cb122866ac057263066b
Author: Eran Ben Elisha <eranbe@mellanox.com>
Date:   Thu Jan 17 23:59:11 2019 +0200

   devlink: Add health reporter create/destroy functionality

   Devlink health reporter is an instance for reporting, diagnosing and
   recovering from run time errors discovered by the reporters.
   Define it's data structure and supported operations.
   In addition, expose devlink API to create and destroy a reporter.
   Each devlink instance will hold it's own reporters list.

   As part of the allocation, driver shall provide a set of callbacks which
   will be used the devlink in order to handle health reports and user
   commands related to this reporter. In addition, driver is entitled to
   provide some priv pointer, which can be fetched from the reporter by
   devlink_health_reporter_priv function.

   For each reporter, devlink will hold a metadata of statistics,
   buffers and status.

   Signed-off-by: Eran Ben Elisha <eranbe@mellanox.com>
   Reviewed-by: Moshe Shemesh <moshe@mellanox.com>
   Signed-off-by: David S. Miller <davem@davemloft.net>
</cut>

Results regressed from (for last_good == cb5ccfbe73b389470e1dc11061bb185ef4bc9aec)

<cut>
reset_artifacts:
-10
build_abe binutils:
-9
build_abe stage1:
-8
linux_n_obj:
all
</cut>

to (for first_bad == 880ee82f0313453ec5a6cb122866ac057263066b)

<cut>
reset_artifacts:
-10
build_abe binutils:
-9
build_abe stage1:
-8
linux_n_obj:
17403
</cut>

Artifacts of last_good build: https://ci.linaro.org/job/tcwg_kernel-bisect-gnu-release-arm-next-allyesconfig/12/artifact/artifacts/build-cb5ccfbe73b389470e1dc11061bb185ef4bc9aec/
Artifacts of first_bad build: https://ci.linaro.org/job/tcwg_kernel-bisect-gnu-release-arm-next-allyesconfig/12/artifact/artifacts/build-880ee82f0313453ec5a6cb122866ac057263066b/

Reproduce builds:
<cut>
mkdir investigate-linux-880ee82f0313453ec5a6cb122866ac057263066b
cd investigate-linux-880ee82f0313453ec5a6cb122866ac057263066b

git clone https://git.linaro.org/toolchain/jenkins-scripts

mkdir -p artifacts/manifests
curl -o artifacts/manifests/build-baseline.sh https://ci.linaro.org/job/tcwg_kernel-bisect-gnu-release-arm-next-allyesconfig/12/artifact/artifacts/manifests/build-baseline.sh
curl -o artifacts/manifests/build-parameters.sh https://ci.linaro.org/job/tcwg_kernel-bisect-gnu-release-arm-next-allyesconfig/12/artifact/artifacts/manifests/build-parameters.sh
curl -o artifacts/test.sh https://ci.linaro.org/job/tcwg_kernel-bisect-gnu-release-arm-next-allyesconfig/12/artifact/artifacts/test.sh
chmod +x artifacts/test.sh

# Reproduce the baseline build (build all pre-requisites)
./jenkins-scripts/tcwg_kernel-build.sh @@ artifacts/manifests/build-baseline.sh

cd linux

# Reproduce first_bad build
git checkout --detach 880ee82f0313453ec5a6cb122866ac057263066b
../artifacts/test.sh

# Reproduce last_good build
git checkout --detach cb5ccfbe73b389470e1dc11061bb185ef4bc9aec
../artifacts/test.sh

cd ..
</cut>

History of pending regressions and results: https://git.linaro.org/toolchain/ci/base-artifacts.git/log/?h=linaro-local/ci/tcwg_kernel/gnu-release-arm-next-allyesconfig

Bisect log: https://ci.linaro.org/job/tcwg_kernel-bisect-gnu-release-arm-next-allyesconfig/12/artifact/artifacts/bisect.log/*view*/
Artifacts: https://ci.linaro.org/job/tcwg_kernel-bisect-gnu-release-arm-next-allyesconfig/12/artifact/artifacts/
Build URL: https://ci.linaro.org/job/tcwg_kernel-bisect-gnu-release-arm-next-allyesconfig/12/
Build log: https://ci.linaro.org/job/tcwg_kernel-bisect-gnu-release-arm-next-allyesconfig/12/consoleText

--
Maxim Kuvyrkov
www.linaro.org

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 3+ messages in thread