[Devel] [PATCH RHEL COMMIT] net: silence high-order-allocation warning in sctp_pack_cookie()

Konstantin Khorenko khorenko at virtuozzo.com
Thu Sep 30 16:03:57 MSK 2021


The commit is pushed to "branch-rh9-5.14.vz9.1.x-ovz" and will appear at https://src.openvz.org/scm/ovz/vzkernel.git
after ark-5.14
------>
commit 61ff4f49284cb9028db3033bbbf253fb2417e7a8
Author: Vasily Averin <vvs at virtuozzo.com>
Date:   Thu Sep 30 16:03:57 2021 +0300

    net: silence high-order-allocation warning in sctp_pack_cookie()
    
    neoLTP sctp_big_chunk testcase reproduces high-order-allocation warning:
     -----------[ cut here ]-----------
     WARNING: CPU: 3 PID: 74823 at mm/page_alloc.c:3533 __alloc_pages_nodemask+0x1b1/0x600
     order 5 >= 3, gfp 0xc020
     Modules linked in: sctp
     CPU: 3 PID: 74823 Comm: sctp_big_chunk ve: 0 Kdump: loaded Tainted: G W ------------ 3.10.0-1127.18.2.vz7.163
     Hardware name: Virtuozzo OpenStack Compute, BIOS 1.11.0-2.vz7.1 04/01/2014
     Call Trace:
     <IRQ> [<ffffffff8a1b8a47>] dump_stack+0x19/0x1b
     [<ffffffff89a9d198>] __warn+0xd8/0x100
     [<ffffffff89a9d21f>] warn_slowpath_fmt+0x5f/0x80
     [<ffffffff89bdfaa1>] __alloc_pages_nodemask+0x1b1/0x600
     [<ffffffff89c35d38>] alloc_pages_current+0x98/0x110
     [<ffffffff89bff088>] kmalloc_order+0x18/0x40
     [<ffffffff89bff0d6>] kmalloc_order_trace+0x26/0xa0
     [<ffffffff89c454e1>] __kmalloc+0x281/0x2a0
     [<ffffffffc0a83244>] sctp_make_init_ack+0x104/0x6b0 [sctp]
     [<ffffffffc0a77036>] sctp_sf_do_5_1B_init+0x286/0x330 [sctp]
     [<ffffffffc0a7ab4d>] sctp_do_sm+0xad/0x350 [sctp]
     [<ffffffffc0a7e6f2>] sctp_endpoint_bh_rcv+0x122/0x240 [sctp]
     [<ffffffffc0a881b1>] sctp_inq_push+0x51/0x70 [sctp]
     [<ffffffffc0a9b53b>] sctp_rcv+0xa8b/0xbd0 [sctp]
     [<ffffffffc0a9e87e>] sctp6_rcv+0xe/0x20 [sctp]
     [<ffffffff8a153c07>] ip6_input_finish+0xd7/0x450
     [<ffffffff8a1544fa>] ip6_input+0x3a/0xb0
     [<ffffffff8a153a0e>] ip6_rcv_finish+0x3e/0xc0
     [<ffffffff8a1542a7>] ipv6_rcv+0x327/0x540
     [<ffffffff8a08a6b9>] __netif_receive_skb_core+0x729/0xa10
     [<ffffffff8a08a9b8>] __netif_receive_skb+0x18/0x60
     [<ffffffff8a08b99e>] process_backlog+0xae/0x180
     [<ffffffff8a08b06f>] net_rx_action+0x27f/0x3a0
     [<ffffffff8a1d1125>] __do_softirq+0x125/0x2bb
     [<ffffffff8a1cd54c>] call_softirq+0x1c/0x30
     <EOI> [<ffffffff89a2f645>] do_softirq+0x65/0xa0
     [<ffffffff89aa662b>] __local_bh_enable_ip+0x9b/0xb0
     [<ffffffff89aa6657>] local_bh_enable+0x17/0x20
     [<ffffffff8a150b07>] ip6_finish_output2+0x1a7/0x580
     [<ffffffff8a15319c>] ip6_finish_output+0x8c/0xf0
     [<ffffffff8a153257>] ip6_output+0x57/0x110
     [<ffffffff8a173007>] rawv6_sendmsg+0x6d7/0xcd0
     [<ffffffff8a113cf9>] inet_sendmsg+0x69/0xb0
     [<ffffffff8a069480>] sock_sendmsg+0xb0/0xf0
     [<ffffffff8a069ab1>] SYSC_sendto+0x121/0x1c0
     [<ffffffff8a06b6ce>] SyS_sendto+0xe/0x10
     [<ffffffff8a1cbf92>] system_call_fastpath+0x25/0x2a
     --[ end trace 49ad7969a95b86af ]--
    
    https://jira.sw.ru/browse/PSBM-122200
    Signed-off-by: Vasily Averin <vvs at virtuozzo.com>
    
    (cherry-picked from vz7 commit 9f3dae021be0badac6be21304646944ec5ac81bc)
    https://jira.sw.ru/browse/PSBM-127858
    Signed-off-by: Valeriy Vdovin <valeriy.vdovin at virtuozzo.com>
    
    (cherry picked from vz8 commit b7201e9581616c9fe9f5a57685d1ac052c3153ac)
    Signed-off-by: Andrey Zhadchenko <andrey.zhadchenko at virtuozzo.com>
---
 net/sctp/sm_make_chunk.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/sctp/sm_make_chunk.c b/net/sctp/sm_make_chunk.c
index 78929f86197c..d6b637a7f49c 100644
--- a/net/sctp/sm_make_chunk.c
+++ b/net/sctp/sm_make_chunk.c
@@ -1681,7 +1681,7 @@ static struct sctp_cookie_param *sctp_pack_cookie(
 	/* Clear this memory since we are sending this data structure
 	 * out on the network.
 	 */
-	retval = kzalloc(*cookie_len, GFP_ATOMIC);
+	retval = kzalloc(*cookie_len, GFP_ATOMIC|__GFP_ORDER_NOWARN);
 	if (!retval)
 		goto nodata;
 


More information about the Devel mailing list