Netdev Archive on
help / color / mirror / Atom feed
From: Nathan Chancellor <>
To: Linus Torvalds <>
Cc: Jakub Kicinski <>,
	Thomas Gleixner <>,
	David Miller <>,
	Netdev <>,
	Linux Kernel Mailing List <>,
Subject: Re: [GIT PULL] Networking for 5.17
Date: Tue, 11 Jan 2022 09:01:32 -0700	[thread overview]
Message-ID: <Yd2p3IbHJdzNok+1@archlinux-ax161> (raw)
In-Reply-To: <>

On Mon, Jan 10, 2022 at 07:31:30PM -0800, Linus Torvalds wrote:
> I really wish we had more automation doing clang builds. Yes, some
> parts of the kernel are still broken with clang, but a lot isn't, and
> this isn't the first time my clang build setup has found issues.

As far as I know, we have four major groups doing regular build testing
with clang:

* Intel's kernel test robot
* KernelCI
* RedHat's Continuous Kernel Integration (CKI)
* Linaro's Linux Kernel Functional Testing (LKFT)

I regularly check the daily -next report that we get from KernelCI to
see what breakage there is and triage it as needed. The rest email us as
things break. The Intel folks are the only ones building from the
mailing list as far as I can tell, everyone else mainly targets your
tree and/or -next.

I don't think this particular issue was an automation fail, more of a
timing one, as the warning was reported by the kernel test robot:

However, it was reported a little under a day after the patch hit the
mailing list according to the lore timestamps at the bottom, after it
had already been merged into net-next (it looks like they were applied
to the netfilter tree and merged into net-next within an hour or so).

Pablo did sent a follow up fix rather quickly, which I noticed because
my own local builds were broken.

Normally, I try to review patches like this so that the maintainers are
aware that the warning will break a build with CONFIG_WERROR. In this
case, I assumed that the netdev build tests would catch it and it would
be applied before the pull request was sent, as they have started
testing with clang and catching these warnings before accepting patches
but as Jakub said, that did not happen.

I'll try to keep an eye out for this stuff in the future, so that it is
dealt with by the time you get it, especially now that passing -Werror
is expected. Most standard arm64 and x86_64 configs should be completely
warning free with clang now, arm and some of the more exotic
architectures are still a WIP.


      parent reply	other threads:[~2022-01-11 16:01 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-10  2:52 Jakub Kicinski
2022-01-11  3:20 ` pr-tracker-bot
2022-01-11  3:31 ` Linus Torvalds
2022-01-11  3:42   ` Linus Torvalds
2022-01-11  4:56   ` Jakub Kicinski
2022-01-11  5:02     ` Linus Torvalds
2022-01-11 16:01   ` Nathan Chancellor [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Yd2p3IbHJdzNok+1@archlinux-ax161 \ \ \ \ \ \ \ \ \
    --subject='Re: [GIT PULL] Networking for 5.17' \

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).