Skip to content

WIP macb stall patches #7667

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
nbuchwitz wants to merge 2 commits into
base: rpi-6.18.y
Choose a base branch
from
+19 −3
Draft

WIP macb stall patches #7667

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
f4d3c00
net: macb: reap lost TX completions on TX timeout
nbuchwitz Sep 30, 2026
e8ebfe7
net: macb: credit unsent frames to BQL in macb_tx_error_task()
nbuchwitz Sep 30, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 19 additions & 3 deletions drivers/net/ethernet/cadence/macb_main.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1357,10 +1357,8 @@ static void macb_tx_error_task(struct work_struct *work)
skb->data);
bp->dev->stats.tx_packets++;
queue->stats.tx_packets++;
packets++;
bp->dev->stats.tx_bytes += skb->len;
queue->stats.tx_bytes += skb->len;
bytes += skb->len;
}
} else {
/* "Buffers exhausted mid-frame" errors may only happen
Expand All @@ -1374,6 +1372,12 @@ static void macb_tx_error_task(struct work_struct *work)
desc->ctrl = ctrl | MACB_BIT(TX_USED);
}

/* BQL has to see every frame leaving the ring, sent or not */
if (skb) {
packets++;
bytes += skb->len;
}

macb_tx_unmap(bp, tx_skb, 0);
}

Expand Down Expand Up @@ -4699,8 +4703,20 @@ static int macb_setup_tc(struct net_device *dev, enum tc_setup_type type,
static void macb_tx_timeout(struct net_device *dev, unsigned int q)
{
struct macb *bp = netdev_priv(dev);
struct macb_queue *queue = &bp->queues[q];

macb_tx_restart(&bp->queues[q]);
macb_tx_restart(queue);

/* The frames may have gone out with their TCOMP interrupt lost.
* macb_tx_restart() does nothing then as TBQP already matches
* tx_head, so reap them from NAPI or the queue stays stopped.
*/
if (macb_tx_complete_pending(queue)) {
queue_writel(queue, IDR, MACB_BIT(TCOMP));
if (bp->caps & MACB_CAPS_ISR_CLEAR_ON_WRITE)
queue_writel(queue, ISR, MACB_BIT(TCOMP));
napi_schedule(&queue->napi_tx);
}
}

static const struct net_device_ops macb_netdev_ops = {
Expand Down
Loading