lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-Id: <c5ea695e-8693-4033-9941-c582f1c6f6be@app.fastmail.com> Date: Mon, 27 Feb 2023 20:53:09 +0100 From: "Arnd Bergmann" <arnd@...db.de> To: "Oliver Hartkopp" <socketcan@...tkopp.net>, "Arnd Bergmann" <arnd@...nel.org>, "Dominik Brodowski" <linux@...inikbrodowski.net>, linux-kernel@...r.kernel.org Cc: "Bjorn Helgaas" <bhelgaas@...gle.com>, "Florian Fainelli" <f.fainelli@...il.com>, "Hartley Sweeten" <hsweeten@...ionengravers.com>, "Ian Abbott" <abbotti@....co.uk>, "Jakub Kicinski" <kuba@...nel.org>, "Kevin Cernekee" <cernekee@...il.com>, "Lukas Wunner" <lukas@...ner.de>, "Manuel Lauss" <manuel.lauss@...il.com>, "Olof Johansson" <olof@...om.net>, "Robert Jarzmik" <robert.jarzmik@...e.fr>, "YOKOTA Hiroshi" <yokota@...lab.is.tsukuba.ac.jp>, bcm-kernel-feedback-list@...adcom.com, linux-arm-kernel@...ts.infradead.org, linux-can@...r.kernel.org, linux-mips@...r.kernel.org, linux-pci@...r.kernel.org, linux-wireless@...r.kernel.org, Netdev <netdev@...r.kernel.org> Subject: Re: [RFC 0/6] pcmcia: separate 16-bit support from cardbus On Mon, Feb 27, 2023, at 20:07, Oliver Hartkopp wrote: > Hello Arnd, > > On 27.02.23 14:34, Arnd Bergmann wrote: >> From: Arnd Bergmann <arnd@...db.de> > > (..) > >> The remaining cardbus/yenta support is essentially a PCI hotplug driver >> with a slightly unusual sysfs interface, and it would still support all >> 32-bit cardbus hosts and cards, but no longer work with the even older >> 16-bit cards that require the pcmcia_driver infrastructure. > > I'm using a 2005 Samsung X20 laptop (Pentium M 1.6GHz, Centrino) with > PCMCIA (type 2) CAN bus cards: > > - EMS PCMCIA > https://elixir.bootlin.com/linux/latest/source/drivers/net/can/sja1000/ems_pcmcia.c > > - PEAK PCCard > https://elixir.bootlin.com/linux/latest/source/drivers/net/can/sja1000/peak_pcmcia.c > > As I still maintain the EMS PCMCIA and had to tweak and test a patch > recently (with a 5.16-rc2 kernel): > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/net/can/sja1000/ems_pcmcia.c?id=3ec6ca6b1a8e64389f0212b5a1b0f6fed1909e45 > > I assume these CAN bus PCMCIA interfaces won't work after your patch > set, right? Correct, the patch series in its current form breaks this since your laptop is cardbus compatible. The options I can see are: - abandon my series and keep everything unchanged, possibly removing some of the pcmcia drivers that Dominik identified as candidates - decide on a future timeline for when you are comfortable with discontinuing this setup and require any CAN users with cardbus laptops to move to USB or cardbus CAN adapters, apply the series then - duplicate the yenta_socket driver to have two variants of that, require the user to choose between the cardbus and the pcmcia variant depending on what card is going to be used. Can you give more background on who is using the EMS PCMCIA card? I.e. are there reasons to use this device on modern kernels with machines that could also support the USB, expresscard or cardbus variants, or are you likely the only one doing this for the purpose of maintaining the driver? Arnd
Powered by blists - more mailing lists