From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
To: "Maciej W. Rozycki" <macro@orcam.me.uk>
Cc: linux-input@vger.kernel.org, linux-kernel@vger.kernel.org,
Vojtech Pavlik <vojtech@ucw.cz>, Jiri Kosina <jikos@kernel.org>,
Benjamin Tissoires <bentiss@kernel.org>
Subject: Re: [RFC PATCH 0/5] Removal of a few obsolete input drivers
Date: Tue, 22 Oct 2024 13:55:58 -0700 [thread overview]
Message-ID: <ZxgRXmPZrhKUegon@google.com> (raw)
In-Reply-To: <alpine.DEB.2.21.2408131802050.59022@angie.orcam.me.uk>
On Thu, Aug 15, 2024 at 10:20:49PM +0100, Maciej W. Rozycki wrote:
> On Mon, 12 Aug 2024, Dmitry Torokhov wrote:
>
> > > > > Are these drivers broken, e.g. fail to compile or crash the system?
> > > >
> > > > I have no idea because I doubt that anyone has tested them since
> > > > forever.
> > >
> > > What's the rationale for your conclusion? How do you know nobody uses
> > > them?
> >
> > Because they are either require ISA add-on cards and it is quite hard to
> > find devices that still work, and are supported by the current kernel,
> > or internal peripherals in devices that are no longer useful. Do you
> > expect anyone using "Gateway AOL Connected Touchpad" in the year of our
> > Lord 2024?
>
> Maybe, maybe not.
>
> I certainly use Linux with actual ISA hardware, i.e. systems with ISA or
> EISA slots and option cards within, as well as other hardware dating back
> to 1989. I'm told people use Linux with m68k hardware going back in time
> even further. I don't use any of the bus mice themselves though (having
> had perhaps a more common serial mouse instead), but if the drivers build
> just fine, then I fail to see a reason to dump them.
OK, so here is an example:
https://lore.kernel.org/all/20241010194533.GA575181@bhelgaas/
We need to cleanup PCI core and the driver uses a hack. So we need
to patch it.
>
> > > > The same gain that we get from removing obsolete boards and
> > > > architectures - less maintenance burden, less work when we need to
> > > > change some APIs, less energy burnt by 0-day and other bots, CI systems,
> > > > etc, compiling useless drivers over and over and over.
> > >
> > > Well, you don't have do do anything about these drivers, do you? They
> > > don't scream for food. And as to the energy, well I doubt this really
> > > matters, the amount is noise lost in the overall consumption.
> >
> > I kind of do even if they did not require much involvement.
> >
> > Let me ask this: why do you want to keep them? Do you know of a large
> > (or small) userbase of bus mice enthusiasts? Note that it would be very
> > easy to "git revert" the removal if someone actually needs this.
>
> There is burden involved as well as repo clutter from going through an
> apply/revert cycle though.
You are assuming that somebody actually needs them and will have to
restore them.
>
> Sometimes we do want to discard code, because it causes burden elsewhere.
> It was the case with the removal of support for the original 80386 CPU due
> to its lack of user page write-protection in the kernel mode, which in
> turn required us to have explicit checks carefully sprinkled throughout
> and painfully maintained. That hindered generic code and was a good
> argument in favour to removal as soon as 80386 became unimportant enough.
>
> In this case the decision seems arbitrary, the presence of these drivers
> does not hurt anything else. I agree it might well be that nobody uses
> them anymore (though someone may come across a relevant piece of hardware
> anytime and wish to try it with Linux; I do it from time to time, and I
> also have old stuff even I'd like to write entirely new drivers for if I
> ever find some time for that, i.e. I have sorted higher priority stuff),
> which I can sort of recognise as an argument in favour of discarding them.
>
> I'm not entirely convinced it's enough of an argument by itself, however
> if there are other people who think otherwise, can we please at least do
> it in stages such as some other projects do? That is require an explicit
> action for any interested party to keep the drivers enabled, say by hiding
> them behind CONFIG_DEPRECATED or suchlike (with clear documentation saying
> it's for stuff slated for removal), wait a year or two, and only if nobody
> speaks out during that period, then actually retire the code in question?
I do not see how CONFIG_DEPRECATED help any better than revert. The
driver will disappear, people will start looking for it and will
complain on Linux Input/LKML. At which point we will revert either the
config change or driver removal patch.
If the argument that with config someone does not need git tree but
rather can work with a tarball I say I really do not care for this case.
>
> A part of the joy with Linux for me and I believe other people as well it
> has been the ability to do odd stuff just for the sake of it. It used not
> to be business back in 1990s and it still not is, not at least entirely,
> for such a Linux old-timer as I have now oddly enough become.
We are still willing to support old hardware, but only when it is
actually used.
Thanks.
--
Dmitry
prev parent reply other threads:[~2024-10-22 20:56 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-08 17:27 [RFC PATCH 0/5] Removal of a few obsolete input drivers Dmitry Torokhov
2024-08-08 17:27 ` [RFC PATCH 1/5] Input: inport - remove driver Dmitry Torokhov
2024-08-08 17:27 ` [RFC PATCH 2/5] Input: logibm " Dmitry Torokhov
2024-08-08 17:27 ` [RFC PATCH 3/5] Input: pc110pad " Dmitry Torokhov
2024-08-08 17:27 ` [RFC PATCH 4/5] Input: mk712 " Dmitry Torokhov
2024-08-08 17:27 ` [RFC PATCH 5/5] Input: ct82c710 " Dmitry Torokhov
2024-08-09 0:24 ` [RFC PATCH 0/5] Removal of a few obsolete input drivers Maciej W. Rozycki
2024-08-12 4:50 ` Dmitry Torokhov
2024-08-12 13:53 ` Maciej W. Rozycki
2024-08-12 16:46 ` Dmitry Torokhov
2024-08-15 21:20 ` Maciej W. Rozycki
2024-10-22 20:55 ` Dmitry Torokhov [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:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZxgRXmPZrhKUegon@google.com \
--to=dmitry.torokhov@gmail.com \
--cc=bentiss@kernel.org \
--cc=jikos@kernel.org \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=macro@orcam.me.uk \
--cc=vojtech@ucw.cz \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).