diff options
| author | Tom Rini <trini@konsulko.com> | 2025-06-18 08:05:29 -0600 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2025-06-18 08:05:29 -0600 |
| commit | ce2a7fcbd565d55b1d3fdc0e68edd50a21eca7d4 (patch) | |
| tree | 857f80e55d97e4170a71a02f90972f95522a6e21 /drivers/net/octeontx/nicvf_main.c | |
| parent | 5d7e003248ae836cbcc2b4c254901c1d85c85537 (diff) | |
| parent | 1d19bbcb68280814952bdc18d7dec6d083b2c0ef (diff) | |
Merge patch series "list.h/treewide: get rid of no-op prefetch()"
Rasmus Villemoes <ravi@prevas.dk> says:
While looking through list.h, I saw that the regular list_* helpers
(and one of the hlist_* ones) still contain the prefetch() that was
removed in linux 14 years ago. It doesn't do anything, but makes the
macros harder to read, so get rid of it, and the fallback, no-op
definition that they relied on. That requires removing a few uses
outside list.h as well.
checkpatch warns about some whitespace issues in list.h, but as I've
copied whole kerneldoc+#define blocks directly from the linux kernel,
I think it's better to just accept that so that we don't introduce
needless diffs. The "macro argument reuse" arguments should also be
ignored, as e.g. the "member" arguments are obviously always just bare
identifiers, and the "pos" arguments must be assigned to multiple
times.
Link: https://lore.kernel.org/r/20250507121246.518691-1-ravi@prevas.dk
Diffstat (limited to 'drivers/net/octeontx/nicvf_main.c')
| -rw-r--r-- | drivers/net/octeontx/nicvf_main.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/net/octeontx/nicvf_main.c b/drivers/net/octeontx/nicvf_main.c index 6e4d0a05121..27d0327c88a 100644 --- a/drivers/net/octeontx/nicvf_main.c +++ b/drivers/net/octeontx/nicvf_main.c @@ -279,8 +279,6 @@ int nicvf_cq_handler(struct nicvf *nic, void **ppkt, int *pkt_len) cq_desc = (struct cqe_rx_t *)GET_CQ_DESC(cq, cqe_head); cqe_head++; cqe_head &= (cq->dmem.q_len - 1); - /* Initiate prefetch for next descriptor */ - prefetch((struct cqe_rx_t *)GET_CQ_DESC(cq, cqe_head)); switch (cq_desc->cqe_type) { case CQE_TYPE_RX: |