* [PATCH net] net: tulip: dmfe: Disable PCI device on removal
@ 2026-09-15 2:03 Myeonghun Pak
2026-09-16 0:43 ` Jakub Kicinski
0 siblings, 1 reply; 2+ messages in thread
From: Myeonghun Pak @ 2026-09-15 2:03 UTC (permalink / raw)
To: netdev
Cc: Andrew Lunn, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, linux-parisc, linux-kernel, stable
dmfe_init_one() enables the PCI device and balances that reference on
probe failures, but dmfe_remove_one() leaves the device enabled.
Disable it after unregistering the network device and releasing the
driver's mappings, DMA buffers and PCI regions.
This issue is already present in the initial Git import; its pre-Git
introduction is unknown.
This issue was identified during our ongoing static-analysis research
while reviewing kernel code.
Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Cc: stable@vger.kernel.org
Assisted-by: LLM
Co-developed-by: Ijae Kim <ae878000@gmail.com>
Signed-off-by: Ijae Kim <ae878000@gmail.com>
Signed-off-by: Myeonghun Pak <mhun512@gmail.com>
---
drivers/net/ethernet/dec/tulip/dmfe.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/ethernet/dec/tulip/dmfe.c b/drivers/net/ethernet/dec/tulip/dmfe.c
index dabc091..dcf6848 100644
--- a/drivers/net/ethernet/dec/tulip/dmfe.c
+++ b/drivers/net/ethernet/dec/tulip/dmfe.c
@@ -528,6 +528,7 @@ static void dmfe_remove_one(struct pci_dev *pdev)
TX_BUF_ALLOC * TX_DESC_CNT + 4,
db->buf_pool_ptr, db->buf_pool_dma_ptr);
pci_release_regions(pdev);
+ pci_disable_device(pdev);
free_netdev(dev); /* free board information */
}
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH net] net: tulip: dmfe: Disable PCI device on removal
2026-09-15 2:03 [PATCH net] net: tulip: dmfe: Disable PCI device on removal Myeonghun Pak
@ 2026-09-16 0:43 ` Jakub Kicinski
0 siblings, 0 replies; 2+ messages in thread
From: Jakub Kicinski @ 2026-09-16 0:43 UTC (permalink / raw)
To: Myeonghun Pak
Cc: netdev, Andrew Lunn, David S. Miller, Eric Dumazet, Paolo Abeni,
linux-parisc, linux-kernel, stable
On Mon, 14 Sep 2026 22:03:59 -0400 Myeonghun Pak wrote:
> Subject: [PATCH net] net: tulip: dmfe: Disable PCI device on removal
Please stop wasting our time fixing bugs in 30 year old drivers that
nobody cares about.
--
pw-bot: reject
pv-bot: slop
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-09-16 0:43 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-09-15 2:03 [PATCH net] net: tulip: dmfe: Disable PCI device on removal Myeonghun Pak
2026-09-16 0:43 ` Jakub Kicinski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®