io_uring/zcrx: don't reload skb_shinfo

Keep skb_shinfo in a local variable so that it doesn't reload it on
every iteration of the loop.

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://patch.msgid.link/1e4c864a5ea639803c866aea70b8d5bf558f189f.1786108672.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
This commit is contained in:
Pavel Begunkov
2026-08-15 17:29:57 -06:00
committed by Jens Axboe
parent 7f32f17ffb
commit 2d837b9961
+5 -6
View File
@@ -1692,6 +1692,7 @@ io_zcrx_recv_skb(read_descriptor_t *desc, struct sk_buff *skb,
struct io_kiocb *req = args->req;
struct sk_buff *frag_iter;
unsigned start, start_off = offset;
struct skb_shared_info *shi;
int i, ret = 0;
len = min_t(size_t, len, desc->count);
@@ -1727,17 +1728,15 @@ io_zcrx_recv_skb(read_descriptor_t *desc, struct sk_buff *skb,
}
start = skb_headlen(skb);
shi = skb_shinfo(skb);
for (i = 0; i < skb_shinfo(skb)->nr_frags; i++) {
const skb_frag_t *frag;
unsigned frag_end;
for (i = 0; i < shi->nr_frags; i++) {
const skb_frag_t *frag = &shi->frags[i];
unsigned frag_end = start + skb_frag_size(frag);
if (WARN_ON(start > offset + len))
return -EFAULT;
frag = &skb_shinfo(skb)->frags[i];
frag_end = start + skb_frag_size(frag);
if (offset < frag_end) {
unsigned copy = min(frag_end - offset, len);
unsigned frag_off = offset - start;