diff options
author | Benjamin Poirier <benjamin.poirier@gmail.com> | 2023-10-20 08:44:57 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2023-10-21 11:52:54 +0200 |
commit | 875be090928d19ff4ae7cbaadb54707abb3befdf (patch) | |
tree | 33b01cd452aa19eaef847c1d910355d4d74ac0ab /drivers/staging/qlge/TODO | |
parent | 88eddb0cce1c654090d94499b7c78085b23b7c2f (diff) | |
download | linux-875be090928d19ff4ae7cbaadb54707abb3befdf.tar.gz linux-875be090928d19ff4ae7cbaadb54707abb3befdf.tar.bz2 linux-875be090928d19ff4ae7cbaadb54707abb3befdf.zip |
staging: qlge: Retire the driver
No significant improvements have been done to this driver since commit
a7c3ddf29a78 ("staging: qlge: clean up debugging code in the QL_ALL_DUMP
ifdef land") in January 2021. The driver should not stay in staging
forever. Since it has been abandoned by the vendor and no one has stepped
up to maintain it, delete it.
If some users manifest themselves, the driver will be restored to
drivers/net/ as suggested in the linked message.
Link: https://lore.kernel.org/netdev/20231019074237.7ef255d7@kernel.org/
Suggested-by: Jakub Kicinski <kuba@kernel.org>
Cc: Manish Chopra <manishc@marvell.com>
Cc: Coiby Xu <coiby.xu@gmail.com>
Signed-off-by: Benjamin Poirier <benjamin.poirier@gmail.com>
Acked-by: Jakub Kicinski <kuba@kernel.org>
Link: https://lore.kernel.org/r/20231020124457.312449-3-benjamin.poirier@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/qlge/TODO')
-rw-r--r-- | drivers/staging/qlge/TODO | 28 |
1 files changed, 0 insertions, 28 deletions
diff --git a/drivers/staging/qlge/TODO b/drivers/staging/qlge/TODO deleted file mode 100644 index 7e277407033e..000000000000 --- a/drivers/staging/qlge/TODO +++ /dev/null @@ -1,28 +0,0 @@ -* commit 7c734359d350 ("qlge: Size RX buffers based on MTU.", v2.6.33-rc1) - introduced dead code in the receive routines, which should be rewritten - anyways by the admission of the author himself, see the comment above - qlge_build_rx_skb(). That function is now used exclusively to handle packets - that underwent header splitting but it still contains code to handle non - split cases. -* truesize accounting is incorrect (ex: a 9000B frame has skb->truesize 10280 - while containing two frags of order-1 allocations, ie. >16K) -* while in that area, using two 8k buffers to store one 9k frame is a poor - choice of buffer size. -* in the "chain of large buffers" case, the driver uses an skb allocated with - head room but only puts data in the frags. -* rename "rx" queues to "completion" queues. Calling tx completion queues "rx - queues" is confusing. -* struct rx_ring is used for rx and tx completions, with some members relevant - to one case only -* the flow control implementation in firmware is buggy (sends a flood of pause - frames, resets the link, device and driver buffer queues become - desynchronized), disable it by default -* the driver has a habit of using runtime checks where compile time checks are - possible (ex. qlge_free_rx_buffers()) -* reorder struct members to avoid holes if it doesn't impact performance -* use better-suited apis (ex. use pci_iomap() instead of ioremap()) -* remove duplicate and useless comments -* fix weird line wrapping (all over, ex. the qlge_set_routing_reg() calls in - qlge_set_multicast_list()). -* remove useless casts (ex. memset((void *)mac_iocb_ptr, ...)) -* fix checkpatch issues |