Netdev Archive on
help / color / mirror / Atom feed
From: Linus Torvalds <>
To: Jakub Kicinski <>, Thomas Gleixner <>
Cc: David Miller <>,
	Netdev <>,
	Linux Kernel Mailing List <>
Subject: Re: [GIT PULL] Networking for 5.17
Date: Mon, 10 Jan 2022 19:31:30 -0800	[thread overview]
Message-ID: <> (raw)
In-Reply-To: <>

On Sun, Jan 9, 2022 at 6:52 PM Jakub Kicinski <> wrote:
> At the time of writing we have one known conflict (/build failure)
> with tip, Stephen's resolution looks good:

Ok, the trees that trigger the conflict haven't actually hit my tree
yet, so I'll see that particular conflict later.

I assume I'll get the irq_set_affinity_and_hint() patches from Thomas
at some point - Thomas, can  you make sure to remind me of this
conflict, because this is exactly the kind of thing I would catch on
my home machine due to doing full builds, but that I will probably
miss if I'm on the road.

I'm home for a couple more days and will try to do as much of the
merge window heavy lifting as possible before my travels start, but
we'll see..

>   git:// tags/5.17-net-next

Merged. But I now note that this actually triggers an error when
building with clang:

  net/netfilter/nf_tables_api.c:8278:4: error: variable 'data_size' is
uninitialized when used here [-Werror,-Wuninitialized]
                          data_size += sizeof(*prule) + rule->dlen;

and I think clang is entirely right.

Sadly, I didn't actually notice that before having done the merge, so
I'll have to do the fixup as a separate commit.

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.

I also notice that NET_VENDOR_VERTEXCOM defaults to 'n'. That's fine
by me, but it seems unusual. Normally the 'enable vendor XYZ' tend to
default to 'y'. But for unusual (and new) vendors, maybe that 'n' is
the right thing to avoid unnecessary questions.

And maybe that NET_VENDOR_xyz thing has happened many times before,
and I just haven't happened to notice...


  parent reply	other threads:[~2022-01-11  3:31 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 [this message]
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

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='' \ \ \ \ \ \ \
    --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).