mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2026-09-22 09:34:56 +02:00
inet: add indirect call wrapper for getfrag() calls
[ Upstream commit5204ccbfa2] UDP send path suffers from one indirect call to ip_generic_getfrag() We can use INDIRECT_CALL_1() to avoid it. Signed-off-by: Eric Dumazet <edumazet@google.com> Reviewed-by: Willem de Bruijn <willemb@google.com> Reviewed-by: Brian Vazquez <brianvv@google.com> Link: https://patch.msgid.link/20241203173617.2595451-1-edumazet@google.com Signed-off-by: Jakub Kicinski <kuba@kernel.org> Stable-dep-of:eca856950f("ipv4: account for fraggap on the paged allocation path") Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
65fb14cbeb
commit
f79f0db614
@@ -1168,7 +1168,10 @@ alloc_new_skb:
|
||||
/* [!] NOTE: copy will be negative if pagedlen>0
|
||||
* because then the equation reduces to -fraggap.
|
||||
*/
|
||||
if (copy > 0 && getfrag(from, data + transhdrlen, offset, copy, fraggap, skb) < 0) {
|
||||
if (copy > 0 &&
|
||||
INDIRECT_CALL_1(getfrag, ip_generic_getfrag,
|
||||
from, data + transhdrlen, offset,
|
||||
copy, fraggap, skb) < 0) {
|
||||
err = -EFAULT;
|
||||
kfree_skb(skb);
|
||||
goto error;
|
||||
@@ -1212,8 +1215,9 @@ alloc_new_skb:
|
||||
unsigned int off;
|
||||
|
||||
off = skb->len;
|
||||
if (getfrag(from, skb_put(skb, copy),
|
||||
offset, copy, off, skb) < 0) {
|
||||
if (INDIRECT_CALL_1(getfrag, ip_generic_getfrag,
|
||||
from, skb_put(skb, copy),
|
||||
offset, copy, off, skb) < 0) {
|
||||
__skb_trim(skb, off);
|
||||
err = -EFAULT;
|
||||
goto error;
|
||||
@@ -1253,7 +1257,8 @@ alloc_new_skb:
|
||||
get_page(pfrag->page);
|
||||
}
|
||||
copy = min_t(int, copy, pfrag->size - pfrag->offset);
|
||||
if (getfrag(from,
|
||||
if (INDIRECT_CALL_1(getfrag, ip_generic_getfrag,
|
||||
from,
|
||||
page_address(pfrag->page) + pfrag->offset,
|
||||
offset, copy, skb->len, skb) < 0)
|
||||
goto error_efault;
|
||||
|
||||
@@ -1695,8 +1695,9 @@ alloc_new_skb:
|
||||
pskb_trim_unique(skb_prev, maxfraglen);
|
||||
}
|
||||
if (copy > 0 &&
|
||||
getfrag(from, data + transhdrlen, offset,
|
||||
copy, fraggap, skb) < 0) {
|
||||
INDIRECT_CALL_1(getfrag, ip_generic_getfrag,
|
||||
from, data + transhdrlen, offset,
|
||||
copy, fraggap, skb) < 0) {
|
||||
err = -EFAULT;
|
||||
kfree_skb(skb);
|
||||
goto error;
|
||||
@@ -1740,8 +1741,9 @@ alloc_new_skb:
|
||||
unsigned int off;
|
||||
|
||||
off = skb->len;
|
||||
if (getfrag(from, skb_put(skb, copy),
|
||||
offset, copy, off, skb) < 0) {
|
||||
if (INDIRECT_CALL_1(getfrag, ip_generic_getfrag,
|
||||
from, skb_put(skb, copy),
|
||||
offset, copy, off, skb) < 0) {
|
||||
__skb_trim(skb, off);
|
||||
err = -EFAULT;
|
||||
goto error;
|
||||
@@ -1781,7 +1783,8 @@ alloc_new_skb:
|
||||
get_page(pfrag->page);
|
||||
}
|
||||
copy = min_t(int, copy, pfrag->size - pfrag->offset);
|
||||
if (getfrag(from,
|
||||
if (INDIRECT_CALL_1(getfrag, ip_generic_getfrag,
|
||||
from,
|
||||
page_address(pfrag->page) + pfrag->offset,
|
||||
offset, copy, skb->len, skb) < 0)
|
||||
goto error_efault;
|
||||
|
||||
Reference in New Issue
Block a user