Re: [GIT PULL] Networking for 5.17

From: Linus Torvalds
Date: Tue Jan 11 2022 - 00:03:10 EST


On Mon, Jan 10, 2022 at 8:56 PM Jakub Kicinski <kuba@xxxxxxxxxx> wrote:
>
> The only explanation I can come up with it's that it was
> done so that people running olddefconfig right after the per-vendor
> split was introduced wouldn't lose all drivers.

Yeah, I think that was the reason. Don't hide existing drivers behind
vendors that got turned off.

I think it would probably be fine to have entirely new vendors (ie "no
existing drivers") default to 'n' when added, just to cut down on the
huge amount of driver questions.

But right now they all act the other way around, and are just a way to
explicitly turn off questions for a vendor you know you don't care
about.

It does perhaps get a bit confusing if some vendors are 'default y'
and others are 'default n'. It's easy enough to explain at the time
when that vendor is added, but then ten years passes, and somebody
asks why something like DEC or 3com is 'default y', while soem big new
vendor migth be 'default n'.

So I dunno.

Linus