Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

kbuild: propagate CONFIG_WERROR to resolve_btfids #4701

Open
wants to merge 2 commits into
base: bpf-next_base
Choose a base branch
from

Conversation

kernel-patches-daemon-bpf-rc[bot]
Copy link

Pull request for series with
subject: kbuild: propagate CONFIG_WERROR to resolve_btfids
version: 1
url: https://patchwork.kernel.org/project/netdevbpf/list/?series=911973

@kernel-patches-daemon-bpf-rc
Copy link
Author

Upstream branch: 28eb75e
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=911973
version: 1

@kernel-patches-daemon-bpf-rc
Copy link
Author

At least one diff in series https://patchwork.kernel.org/project/netdevbpf/list/?series=911973 expired. Closing PR.

@kernel-patches-daemon-bpf-rc
Copy link
Author

Upstream branch: 27802ca
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=912629
version: 2

@kernel-patches-daemon-bpf-rc
Copy link
Author

Upstream branch: 27802ca
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=912629
version: 2

Currently warnings emitted by resolve_btfids are buried in the build log
and are slipping into mainline frequently.
Add an option to elevate warnings to hard errors so the CI bots can
catch any new warnings.

Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
Acked-by: Jiri Olsa <jolsa@kernel.org>
@kernel-patches-daemon-bpf-rc
Copy link
Author

Upstream branch: c8d02b5
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=912629
version: 2

Use CONFIG_WERROR to also fail on warnings emitted by resolve_btfids.
Allow the CI bots to prevent the introduction of new warnings.

Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
Acked-by: Jiri Olsa <jolsa@kernel.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant