mbox series

[SRU,G,H,0/3] disable building bpf selftests

Message ID 20201214220040.10644-1-kamal@canonical.com
Headers show
Series disable building bpf selftests | expand

Message

Kamal Mostafa Dec. 14, 2020, 10 p.m. UTC
BugLink: https://bugs.launchpad.net/bugs/1908144

Building tools/testing/selftests/bpf/ requires dependencies which cannot
be easily satisfied by our build process. The problem is well-described
by the pending upstream patch:

https://lore.kernel.org/bpf/20201210185233.28091-1-broonie@kernel.org/
From: Mark Brown <broonie@kernel.org>
Subject: [PATCH] selftests: Skip BPF seftests by default

To effect the same for {groovy, hirsute} we must instead make a
debian/rules change via "UBUNTU: disable building bpf selftests (no
VMLINUX_BTF)".

But lets also pick up the upstream change above, as well as my related
"clarify" change, also pending upstream.

These three patches should be applied to {groovy, hirsute}:

  UBUNTU: SAUCE: selftests/bpf: clarify build error if no vmlinux
  UBUNTU: SAUCE: selftests: Skip BPF seftests by default
  UBUNTU: disable building bpf selftests (no VMLINUX_BTF)

 -Kamal

---

Kamal Mostafa (2):
  UBUNTU: SAUCE: selftests/bpf: clarify build error if no vmlinux
  UBUNTU: disable building bpf selftests (no VMLINUX_BTF)

Mark Brown (1):
  UBUNTU: SAUCE: selftests: Skip BPF seftests by default

 debian/rules.d/0-common-vars.mk      | 2 +-
 tools/testing/selftests/Makefile     | 6 ++++--
 tools/testing/selftests/bpf/Makefile | 3 +++
 3 files changed, 8 insertions(+), 3 deletions(-)

Comments

Andrea Righi Dec. 15, 2020, 8:28 a.m. UTC | #1
On Mon, Dec 14, 2020 at 02:00:37PM -0800, Kamal Mostafa wrote:
> BugLink: https://bugs.launchpad.net/bugs/1908144
> 
> Building tools/testing/selftests/bpf/ requires dependencies which cannot
> be easily satisfied by our build process. The problem is well-described
> by the pending upstream patch:
> 
> https://lore.kernel.org/bpf/20201210185233.28091-1-broonie@kernel.org/
> From: Mark Brown <broonie@kernel.org>
> Subject: [PATCH] selftests: Skip BPF seftests by default
> 
> To effect the same for {groovy, hirsute} we must instead make a
> debian/rules change via "UBUNTU: disable building bpf selftests (no
> VMLINUX_BTF)".
> 
> But lets also pick up the upstream change above, as well as my related
> "clarify" change, also pending upstream.
> 
> These three patches should be applied to {groovy, hirsute}:
> 
>   UBUNTU: SAUCE: selftests/bpf: clarify build error if no vmlinux
>   UBUNTU: SAUCE: selftests: Skip BPF seftests by default
>   UBUNTU: disable building bpf selftests (no VMLINUX_BTF)
> 
>  -Kamal

It looks like a reasonable change to me. Thanks!

Acked-by: Andrea Righi <andrea.righi@canonical.com>
Andrea Righi Dec. 15, 2020, 8:29 a.m. UTC | #2
On Mon, Dec 14, 2020 at 02:00:37PM -0800, Kamal Mostafa wrote:
> BugLink: https://bugs.launchpad.net/bugs/1908144
> 
> Building tools/testing/selftests/bpf/ requires dependencies which cannot
> be easily satisfied by our build process. The problem is well-described
> by the pending upstream patch:
> 
> https://lore.kernel.org/bpf/20201210185233.28091-1-broonie@kernel.org/
> From: Mark Brown <broonie@kernel.org>
> Subject: [PATCH] selftests: Skip BPF seftests by default
> 
> To effect the same for {groovy, hirsute} we must instead make a
> debian/rules change via "UBUNTU: disable building bpf selftests (no
> VMLINUX_BTF)".
> 
> But lets also pick up the upstream change above, as well as my related
> "clarify" change, also pending upstream.
> 
> These three patches should be applied to {groovy, hirsute}:
> 
>   UBUNTU: SAUCE: selftests/bpf: clarify build error if no vmlinux
>   UBUNTU: SAUCE: selftests: Skip BPF seftests by default
>   UBUNTU: disable building bpf selftests (no VMLINUX_BTF)
> 
>  -Kamal

Applied to unstable.

-Andrea
Kleber Sacilotto de Souza Dec. 15, 2020, 10:08 a.m. UTC | #3
On 14.12.20 23:00, Kamal Mostafa wrote:
> BugLink: https://bugs.launchpad.net/bugs/1908144
> 
> Building tools/testing/selftests/bpf/ requires dependencies which cannot
> be easily satisfied by our build process. The problem is well-described
> by the pending upstream patch:
> 
> https://lore.kernel.org/bpf/20201210185233.28091-1-broonie@kernel.org/
> From: Mark Brown <broonie@kernel.org>
> Subject: [PATCH] selftests: Skip BPF seftests by default
> 
> To effect the same for {groovy, hirsute} we must instead make a
> debian/rules change via "UBUNTU: disable building bpf selftests (no
> VMLINUX_BTF)".
> 
> But lets also pick up the upstream change above, as well as my related
> "clarify" change, also pending upstream.
> 
> These three patches should be applied to {groovy, hirsute}:
> 
>    UBUNTU: SAUCE: selftests/bpf: clarify build error if no vmlinux
>    UBUNTU: SAUCE: selftests: Skip BPF seftests by default
>    UBUNTU: disable building bpf selftests (no VMLINUX_BTF)
> 
>   -Kamal
> 
> ---
> 
> Kamal Mostafa (2):
>    UBUNTU: SAUCE: selftests/bpf: clarify build error if no vmlinux
>    UBUNTU: disable building bpf selftests (no VMLINUX_BTF)
> 
> Mark Brown (1):
>    UBUNTU: SAUCE: selftests: Skip BPF seftests by default
> 
>   debian/rules.d/0-common-vars.mk      | 2 +-
>   tools/testing/selftests/Makefile     | 6 ++++--
>   tools/testing/selftests/bpf/Makefile | 3 +++
>   3 files changed, 8 insertions(+), 3 deletions(-)
> 

bpf selftests will be prone again to build breakage detected only after
the kernels are already built, but I also don't see a better solution
for now.

Acked-by: Kleber Sacilotto de Souza <kleber.souza@canonical.com>
Kamal Mostafa Dec. 15, 2020, 5:26 p.m. UTC | #4
Please hold on applying this series (upstream wants to massage my
"clarify build error" patch).

 -Kamal

On Mon, Dec 14, 2020 at 02:00:37PM -0800, Kamal Mostafa wrote:
> BugLink: https://bugs.launchpad.net/bugs/1908144
> 
> Building tools/testing/selftests/bpf/ requires dependencies which cannot
> be easily satisfied by our build process. The problem is well-described
> by the pending upstream patch:
> 
> https://lore.kernel.org/bpf/20201210185233.28091-1-broonie@kernel.org/
> From: Mark Brown <broonie@kernel.org>
> Subject: [PATCH] selftests: Skip BPF seftests by default
> 
> To effect the same for {groovy, hirsute} we must instead make a
> debian/rules change via "UBUNTU: disable building bpf selftests (no
> VMLINUX_BTF)".
> 
> But lets also pick up the upstream change above, as well as my related
> "clarify" change, also pending upstream.
> 
> These three patches should be applied to {groovy, hirsute}:
> 
>   UBUNTU: SAUCE: selftests/bpf: clarify build error if no vmlinux
>   UBUNTU: SAUCE: selftests: Skip BPF seftests by default
>   UBUNTU: disable building bpf selftests (no VMLINUX_BTF)
> 
>  -Kamal
> 
> ---
> 
> Kamal Mostafa (2):
>   UBUNTU: SAUCE: selftests/bpf: clarify build error if no vmlinux
>   UBUNTU: disable building bpf selftests (no VMLINUX_BTF)
> 
> Mark Brown (1):
>   UBUNTU: SAUCE: selftests: Skip BPF seftests by default
> 
>  debian/rules.d/0-common-vars.mk      | 2 +-
>  tools/testing/selftests/Makefile     | 6 ++++--
>  tools/testing/selftests/bpf/Makefile | 3 +++
>  3 files changed, 8 insertions(+), 3 deletions(-)
> 
> -- 
> 2.17.1
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
Kamal Mostafa Dec. 16, 2020, 9:33 p.m. UTC | #5
[v2] on the way.

 -Kamal

On Mon, Dec 14, 2020 at 02:00:37PM -0800, Kamal Mostafa wrote:
> BugLink: https://bugs.launchpad.net/bugs/1908144
> 
> Building tools/testing/selftests/bpf/ requires dependencies which cannot
> be easily satisfied by our build process. The problem is well-described
> by the pending upstream patch:
> 
> https://lore.kernel.org/bpf/20201210185233.28091-1-broonie@kernel.org/
> From: Mark Brown <broonie@kernel.org>
> Subject: [PATCH] selftests: Skip BPF seftests by default
> 
> To effect the same for {groovy, hirsute} we must instead make a
> debian/rules change via "UBUNTU: disable building bpf selftests (no
> VMLINUX_BTF)".
> 
> But lets also pick up the upstream change above, as well as my related
> "clarify" change, also pending upstream.
> 
> These three patches should be applied to {groovy, hirsute}:
> 
>   UBUNTU: SAUCE: selftests/bpf: clarify build error if no vmlinux
>   UBUNTU: SAUCE: selftests: Skip BPF seftests by default
>   UBUNTU: disable building bpf selftests (no VMLINUX_BTF)
> 
>  -Kamal
> 
> ---
> 
> Kamal Mostafa (2):
>   UBUNTU: SAUCE: selftests/bpf: clarify build error if no vmlinux
>   UBUNTU: disable building bpf selftests (no VMLINUX_BTF)
> 
> Mark Brown (1):
>   UBUNTU: SAUCE: selftests: Skip BPF seftests by default
> 
>  debian/rules.d/0-common-vars.mk      | 2 +-
>  tools/testing/selftests/Makefile     | 6 ++++--
>  tools/testing/selftests/bpf/Makefile | 3 +++
>  3 files changed, 8 insertions(+), 3 deletions(-)
> 
> -- 
> 2.17.1
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team