From: Dan Carpenter <dan.carpenter@linaro.org> To: Arnd Bergmann <arnd@arndb.de> Cc: Philipp Hortmann <philipp.g.hortmann@gmail.com>, Arnd Bergmann <arnd@kernel.org>, Kalle Valo <kvalo@kernel.org>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Nicolas Ferre <nicolas.ferre@microchip.com>, Alexandre Belloni <alexandre.belloni@bootlin.com>, Claudiu Beznea <claudiu.beznea@tuxon.dev>, Jakub Kicinski <kuba@kernel.org>, Pavel Machek <pavel@ucw.cz>, "David S . Miller" <davem@davemloft.net>, Larry Finger <Larry.Finger@lwfinger.net>, linux-kernel@vger.kernel.org, linux-wireless@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-staging@lists.linux.dev Subject: Re: [PATCH] [RFC] wireless: move obsolete drivers to staging Date: Thu, 12 Oct 2023 09:23:12 +0300 [thread overview] Message-ID: <428ef899-a5c4-4d93-beea-afd7ab6f6634@kadam.mountain> (raw) In-Reply-To: <db98d9ac-7650-4a72-8eb9-4def1f17ea0d@app.fastmail.com> On Wed, Oct 11, 2023 at 10:22:32PM +0200, Arnd Bergmann wrote: > On Wed, Oct 11, 2023, at 20:13, Philipp Hortmann wrote: > > On 10/10/23 17:27, Arnd Bergmann wrote: > >> From: Arnd Bergmann <arnd@arndb.de> While looking at the old drivers > >> using the obsolete .ndo_do_ioctl() callback, I found a number of network > >> drivers that are especially obsolete, in particular for 802.11b > >> (11Mbit/s) or even older wireless networks, using non-busmaster > >> ISA/PCMCIA style bus interfaces, and using the legacy wireless extension > >> ioctls rather than the netlink interfaces that were meant to replace > >> them in 2007. All of these drivers are obsolete or orphaned. We had > >> previously discussed this topic, but nobody ever moved the files, so I > >> now went through the list to my best knowledge. These are the drivers > >> that I would classify as "probably unused" by now: > > > > I found a USB WLAN Stick with a rtl8192u. I got it last Saturday and > > found out that the firmware is missing in my ubuntu 20.04. I found it on > > the web and fixed it. When I started the driver my computer crashed. The > > missing part was: priv->priv_wq = alloc_workqueue("priv_wq", 0, 0); > > Fixing this the next error was a network = kzalloc(sizeof(*network), > > GFP_KERNEL); in wrong context with leads to a crash of my computer. > > Fixing this leads to another issue which lets my computer crash. > > > > For me the firmware of rtl8192u was intentionally missing because of the > > issues with the driver. > > > > What this has to do with your question? > > Can we check for missing firmware in main distributions to know which > > drivers are considered to be old and unused? > > Nice, thanks so much for testing. > > I see the two bugs were introduced in 2016 by commit 1761a85c3bed3 > ("staging: rtl8192u: Remove create_workqueue()") This one never made it to lore... I think we wouldn't have merged it without a lore review these days. (There was a lot of moaning and complaining at the time). > and in 2021 by > commit 061e390b7c87f ("staging: rtl8192u: ieee80211_softmac: Move a > large data struct onto the heap"), so it's been broken for a while. :/ No way would I have seen this in review. Smatch is supposed to find some of these "sleeping in invalid context" warnings but the sleeping in IRQ stuff doesn't work and hasn't been released. regards, dan carpenter
WARNING: multiple messages have this Message-ID (diff)
From: Dan Carpenter <dan.carpenter@linaro.org> To: Arnd Bergmann <arnd@arndb.de> Cc: Arnd Bergmann <arnd@kernel.org>, Alexandre Belloni <alexandre.belloni@bootlin.com>, Philipp Hortmann <philipp.g.hortmann@gmail.com>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Kalle Valo <kvalo@kernel.org>, linux-staging@lists.linux.dev, linux-wireless@vger.kernel.org, Claudiu Beznea <claudiu.beznea@tuxon.dev>, linux-arm-kernel@lists.infradead.org, Pavel Machek <pavel@ucw.cz>, Jakub Kicinski <kuba@kernel.org>, "David S . Miller" <davem@davemloft.net>, linux-kernel@vger.kernel.org, Larry Finger <Larry.Finger@lwfinger.net> Subject: Re: [PATCH] [RFC] wireless: move obsolete drivers to staging Date: Thu, 12 Oct 2023 09:23:12 +0300 [thread overview] Message-ID: <428ef899-a5c4-4d93-beea-afd7ab6f6634@kadam.mountain> (raw) In-Reply-To: <db98d9ac-7650-4a72-8eb9-4def1f17ea0d@app.fastmail.com> On Wed, Oct 11, 2023 at 10:22:32PM +0200, Arnd Bergmann wrote: > On Wed, Oct 11, 2023, at 20:13, Philipp Hortmann wrote: > > On 10/10/23 17:27, Arnd Bergmann wrote: > >> From: Arnd Bergmann <arnd@arndb.de> While looking at the old drivers > >> using the obsolete .ndo_do_ioctl() callback, I found a number of network > >> drivers that are especially obsolete, in particular for 802.11b > >> (11Mbit/s) or even older wireless networks, using non-busmaster > >> ISA/PCMCIA style bus interfaces, and using the legacy wireless extension > >> ioctls rather than the netlink interfaces that were meant to replace > >> them in 2007. All of these drivers are obsolete or orphaned. We had > >> previously discussed this topic, but nobody ever moved the files, so I > >> now went through the list to my best knowledge. These are the drivers > >> that I would classify as "probably unused" by now: > > > > I found a USB WLAN Stick with a rtl8192u. I got it last Saturday and > > found out that the firmware is missing in my ubuntu 20.04. I found it on > > the web and fixed it. When I started the driver my computer crashed. The > > missing part was: priv->priv_wq = alloc_workqueue("priv_wq", 0, 0); > > Fixing this the next error was a network = kzalloc(sizeof(*network), > > GFP_KERNEL); in wrong context with leads to a crash of my computer. > > Fixing this leads to another issue which lets my computer crash. > > > > For me the firmware of rtl8192u was intentionally missing because of the > > issues with the driver. > > > > What this has to do with your question? > > Can we check for missing firmware in main distributions to know which > > drivers are considered to be old and unused? > > Nice, thanks so much for testing. > > I see the two bugs were introduced in 2016 by commit 1761a85c3bed3 > ("staging: rtl8192u: Remove create_workqueue()") This one never made it to lore... I think we wouldn't have merged it without a lore review these days. (There was a lot of moaning and complaining at the time). > and in 2021 by > commit 061e390b7c87f ("staging: rtl8192u: ieee80211_softmac: Move a > large data struct onto the heap"), so it's been broken for a while. :/ No way would I have seen this in review. Smatch is supposed to find some of these "sleeping in invalid context" warnings but the sleeping in IRQ stuff doesn't work and hasn't been released. regards, dan carpenter _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2023-10-12 6:23 UTC|newest] Thread overview: 71+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-10-10 15:27 [PATCH] [RFC] wireless: move obsolete drivers to staging Arnd Bergmann 2023-10-10 15:27 ` Arnd Bergmann 2023-10-10 16:08 ` Greg Kroah-Hartman 2023-10-10 16:08 ` Greg Kroah-Hartman 2023-10-11 5:40 ` Kalle Valo 2023-10-11 5:40 ` Kalle Valo 2023-10-11 6:16 ` Greg Kroah-Hartman 2023-10-11 6:16 ` Greg Kroah-Hartman 2023-10-11 7:29 ` Kalle Valo 2023-10-11 7:29 ` Kalle Valo 2023-10-11 15:09 ` Jakub Kicinski 2023-10-11 15:09 ` Jakub Kicinski 2023-10-12 11:47 ` Kalle Valo 2023-10-12 11:47 ` Kalle Valo 2023-10-12 14:36 ` Arnd Bergmann 2023-10-12 14:36 ` Arnd Bergmann 2023-10-12 14:48 ` Kalle Valo 2023-10-12 14:48 ` Kalle Valo 2023-10-12 15:11 ` Geert Uytterhoeven 2023-10-12 15:11 ` Geert Uytterhoeven 2023-10-12 15:11 ` Geert Uytterhoeven 2023-10-12 15:28 ` Johannes Berg 2023-10-12 15:28 ` Johannes Berg 2023-10-12 15:28 ` Johannes Berg 2023-10-12 15:39 ` Geert Uytterhoeven 2023-10-12 15:39 ` Geert Uytterhoeven 2023-10-12 15:39 ` Geert Uytterhoeven 2023-10-12 15:41 ` Johannes Berg 2023-10-12 15:41 ` Johannes Berg 2023-10-12 15:41 ` Johannes Berg 2023-10-12 16:36 ` Geoff Levand 2023-10-12 16:36 ` Geoff Levand 2023-10-12 16:36 ` Geoff Levand 2023-10-13 15:44 ` Arnd Bergmann 2023-10-13 15:44 ` Arnd Bergmann 2023-10-13 15:44 ` Arnd Bergmann 2023-10-13 18:02 ` Johannes Berg 2023-10-13 18:02 ` Johannes Berg 2023-10-13 18:02 ` Johannes Berg 2023-10-18 20:37 ` Jeff Johnson 2023-10-18 20:37 ` Jeff Johnson 2023-10-18 20:37 ` Jeff Johnson 2023-10-12 16:16 ` Philipp Hortmann 2023-10-12 16:16 ` Philipp Hortmann 2023-10-12 16:18 ` Dan Williams 2023-10-12 16:18 ` Dan Williams 2023-10-11 7:24 ` Arnd Bergmann 2023-10-11 7:24 ` Arnd Bergmann 2023-10-11 7:41 ` Greg Kroah-Hartman 2023-10-11 7:41 ` Greg Kroah-Hartman 2023-10-11 8:44 ` Kalle Valo 2023-10-11 8:44 ` Kalle Valo 2023-10-11 9:01 ` Arnd Bergmann 2023-10-11 9:01 ` Arnd Bergmann 2023-10-11 18:13 ` Philipp Hortmann 2023-10-11 18:13 ` Philipp Hortmann 2023-10-11 20:22 ` Arnd Bergmann 2023-10-11 20:22 ` Arnd Bergmann 2023-10-12 6:23 ` Dan Carpenter [this message] 2023-10-12 6:23 ` Dan Carpenter 2023-10-13 6:42 ` Philipp Hortmann 2023-10-13 6:42 ` Philipp Hortmann 2023-10-13 15:36 ` Arnd Bergmann 2023-10-13 15:36 ` Arnd Bergmann 2023-10-13 16:24 ` Larry Finger 2023-10-13 16:24 ` Larry Finger 2023-10-15 18:42 ` Philipp Hortmann 2023-10-15 18:42 ` Philipp Hortmann 2023-10-18 16:05 ` Arnd Bergmann 2023-10-18 16:05 ` Arnd Bergmann 2023-10-13 11:43 ` kernel test robot
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=428ef899-a5c4-4d93-beea-afd7ab6f6634@kadam.mountain \ --to=dan.carpenter@linaro.org \ --cc=Larry.Finger@lwfinger.net \ --cc=alexandre.belloni@bootlin.com \ --cc=arnd@arndb.de \ --cc=arnd@kernel.org \ --cc=claudiu.beznea@tuxon.dev \ --cc=davem@davemloft.net \ --cc=gregkh@linuxfoundation.org \ --cc=kuba@kernel.org \ --cc=kvalo@kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-staging@lists.linux.dev \ --cc=linux-wireless@vger.kernel.org \ --cc=nicolas.ferre@microchip.com \ --cc=pavel@ucw.cz \ --cc=philipp.g.hortmann@gmail.com \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.