Re: [RFC 00/13] [net-next] drivers/net/Space.c cleanup

Paul Gortmaker
2021-05-17T14:47:08Z

[[RFC 00/13] [net-next] drivers/net/Space.c cleanup] On 16/05/2021 (Sun 00:13) Arnd Bergmann wrote:

> From: Arnd Bergmann <arnd@arndb.de>

[...]

> * Alternatively, the fact that the ISA drivers have never been cleaned
> up can be seen as an indication that there isn't really much remaining
> interest in them. We could move them to drivers/staging along with the
> consolidated contents of drivers/net/Space.c and see if anyone still
> uses them and eventually remove the ones that nobody has.
> I can see that Paul Gortmaker removed a number of less common ISA
> ethernet drivers in 2013, but at the time left these because they
> were possibly still relevant.

It was a thin/remote possibility, even then: From v3.9-rc1~139^2~288

In theory, it is possible that someone could still be running one of these
12+ year old P3 machines and want 3.9+ bleeding edge kernels (but unlikely).
In light of the above (remote) possibility, we can defer the removal of some
ISA network drivers that were highly popular and well tested. Typically
that means the stuff more from the mid to late '90s, some with ISA PnP
support, like the 3c509, the wd/SMC 8390 based stuff, PCnet/lance etc.

Now those 12y old machines are 20y old, and bleeding edge for this discussion
will be the the v5.14+ kernel.

Leaving the more popular cards was a concession to making progress vs.
having the whole cleanup blocked by individuals who haven't yet realized
that using ancient hardware is best done (only done?) with ancient kernels.

Maybe things are better now; people are putting more consideration to
the future of the kernel, rather than clinging to times long past?
We've since managed to delete several complete old arch dirs; which I
had previously thought impossible. I couldn't even git rid of the x86
EISA support code six years ago.[1]

I'd be willing to do a "Phase 2" of 930d52c012b8 ISA net delete; I'm
not sure the bounce through stable on the way out does much other than
muddy the git history. I'd be tempted to just propose the individual
deletes and see where that goes....

Paul.
--

[1] https://lwn.net/Articles/630115/

>
> * If we end up moving the cops localtalk driver to staging, support
> for localtalk devices (though probably not appletalk over ethernet)
> can arguably meet the same fate.
>
> If someone wants to work on those follow-ups or thinks they are a
> good idea, let me know, otherwise I'd leave it at this cleanup.
>
> Arnd
>
> Arnd Bergmann (13):
> [net-next] bcmgenet: remove call to netdev_boot_setup_check
> [net-next] natsemi: sonic: stop calling netdev_boot_setup_check
> [net-next] appletalk: ltpc: remove static probing
> [net-next] 3c509: stop calling netdev_boot_setup_check
> [net-next] cs89x0: rework driver configuration
> [net-next] m68k: remove legacy probing
> [net-next] move netdev_boot_setup into Space.c
> [net-next] make legacy ISA probe optional
> [net-next] wan: remove stale Kconfig entries
> [net-next] wan: remove sbni/granch driver
> [net-next] wan: hostess_sv11: use module_init/module_exit helpers
> [net-next] ethernet: isa: convert to module_init/module_exit
> [net-next] 8390: xsurf100: avoid including lib8390.c
>
> .../admin-guide/kernel-parameters.txt | 2 -
> drivers/net/Kconfig | 7 +
> drivers/net/Makefile | 3 +-
> drivers/net/Space.c | 178 +-
> drivers/net/appletalk/Kconfig | 4 +-
> drivers/net/appletalk/ltpc.c | 7 +-
> drivers/net/ethernet/3com/3c509.c | 3 -
> drivers/net/ethernet/3com/3c515.c | 3 +-
> drivers/net/ethernet/3com/Kconfig | 1 +
> drivers/net/ethernet/8390/Kconfig | 3 +
> drivers/net/ethernet/8390/Makefile | 2 +-
> drivers/net/ethernet/8390/apne.c | 11 +-
> drivers/net/ethernet/8390/ne.c | 5 +-
> drivers/net/ethernet/8390/smc-ultra.c | 9 +-
> drivers/net/ethernet/8390/wd.c | 7 +-
> drivers/net/ethernet/8390/xsurf100.c | 7 +-
> drivers/net/ethernet/amd/Kconfig | 2 +
> drivers/net/ethernet/amd/atarilance.c | 11 +-
> drivers/net/ethernet/amd/lance.c | 6 +-
> drivers/net/ethernet/amd/mvme147.c | 16 +-
> drivers/net/ethernet/amd/ni65.c | 6 +-
> drivers/net/ethernet/amd/sun3lance.c | 19 +-
> .../net/ethernet/broadcom/genet/bcmgenet.c | 2 -
> drivers/net/ethernet/cirrus/Kconfig | 27 +-
> drivers/net/ethernet/cirrus/cs89x0.c | 31 +-
> drivers/net/ethernet/i825xx/82596.c | 24 +-
> drivers/net/ethernet/i825xx/sun3_82586.c | 17 +-
> drivers/net/ethernet/natsemi/jazzsonic.c | 2 -
> drivers/net/ethernet/natsemi/xtsonic.c | 1 -
> drivers/net/ethernet/smsc/Kconfig | 1 +
> drivers/net/ethernet/smsc/smc9194.c | 6 +-
> drivers/net/wan/Kconfig | 51 -
> drivers/net/wan/Makefile | 1 -
> drivers/net/wan/hostess_sv11.c | 6 +-
> drivers/net/wan/sbni.c | 1638 -----------------
> drivers/net/wan/sbni.h | 147 --
> include/linux/netdevice.h | 13 -
> include/net/Space.h | 10 -
> net/core/dev.c | 125 --
> net/ethernet/eth.c | 2 -
> 40 files changed, 259 insertions(+), 2157 deletions(-)
> delete mode 100644 drivers/net/wan/sbni.c
> delete mode 100644 drivers/net/wan/sbni.h
>
> --
> 2.29.2
>
> Cc: Paul Gortmaker <paul.gortmaker@windriver.com>
> Cc: "Maciej W. Rozycki" <macro@orcam.me.uk>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Doug Berger <opendmb@gmail.com>
> Cc: Florian Fainelli <f.fainelli@gmail.com>
> Cc: Sam Creasey <sammy@sammy.net>
> Cc: Geert Uytterhoeven <geert@linux-m68k.org>
> Cc: Finn Thain <fthain@telegraphics.com.au>
> Cc: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
> Cc: Andrew Lunn <andrew@lunn.ch>
> Cc: Alexei Starovoitov <ast@kernel.org>
> Cc: Eric Dumazet <edumazet@google.com>
> Cc: Andrii Nakryiko <andriin@fb.com>
> Cc: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> Cc: netdev@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Cc: bcm-kernel-feedback-list@broadcom.com