Thread (14 messages) 14 messages, 5 authors, 2025-07-26

Re: [PATCH] net: Revert tx queue length on partial failure in dev_qdisc_change_tx_queue_len()

From: Suchit K <hidden>
Date: 2025-07-23 18:17:21
Also in: linux-kernel-mentees, lkml

WRITE_ONCE() is missing.
quoted
+               while (i >= 0) {
+                       qdisc_change_tx_queue_len(dev, &dev->_tx[i]);
What happens if one of these calls fails ?

I think a fix will be more complicated...
Hi Eric,
Given that pfifo_fast_change_tx_queue_len is currently the only
implementation of change_tx_queue_len, would it be reasonable to
handle partial failures solely within pfifo_fast_change_tx_queue_len
(which in turn leads to skb_array_resize_multiple_bh)? In other words,
is it sufficient to modify only the underlying low level
implementation of pfifo_fast_change_tx_queue_len for partial failures,
given that it's the sole implementation of change_tx_queue_len?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help