From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754096AbcKRNBx (ORCPT ); Fri, 18 Nov 2016 08:01:53 -0500 Received: from mx0a-001b2d01.pphosted.com ([148.163.156.1]:37147 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752901AbcKRNBt (ORCPT ); Fri, 18 Nov 2016 08:01:49 -0500 Date: Fri, 18 Nov 2016 05:01:44 -0800 From: "Paul E. McKenney" To: Ding Tianhong Cc: josh@joshtriplett.org, rostedt@goodmis.org, mathieu.desnoyers@efficios.com, jiangshanlai@gmail.com, "linux-kernel@vger.kernel.org" , davem@davemloft.net Subject: Re: [PATCH] rcu: fix the OOM problem of huge IP abnormal packet traffic Reply-To: paulmck@linux.vnet.ibm.com References: <635ca612-370c-b6e4-7f2a-cba702dd0c4a@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <635ca612-370c-b6e4-7f2a-cba702dd0c4a@huawei.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-TM-AS-GCONF: 00 X-Content-Scanned: Fidelis XPS MAILER x-cbid: 16111813-0016-0000-0000-0000053357DF X-IBM-SpamModules-Scores: X-IBM-SpamModules-Versions: BY=3.00006099; HX=3.00000240; KW=3.00000007; PH=3.00000004; SC=3.00000189; SDB=6.00782394; UDB=6.00377537; IPR=6.00559863; BA=6.00004891; NDR=6.00000001; ZLA=6.00000005; ZF=6.00000009; ZB=6.00000000; ZP=6.00000000; ZH=6.00000000; ZU=6.00000002; MB=3.00013368; XFM=3.00000011; UTC=2016-11-18 13:01:47 X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused x-cbparentid: 16111813-0017-0000-0000-000034C07339 Message-Id: <20161118130144.GO3612@linux.vnet.ibm.com> X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2016-11-18_07:,, signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 suspectscore=2 malwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1609300000 definitions=main-1611180231 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Nov 18, 2016 at 08:40:09PM +0800, Ding Tianhong wrote: > The commit bedc196915 ("rcu: Fix soft lockup for rcu_nocb_kthread") > will introduce a new problem that when huge IP abnormal packet arrived, > it may cause OOM and break the kernel, just like this: > > [ 79.441538] mlx4_en: eth5: Leaving promiscuous mode steering mode:2 > [ 100.067032] ksoftirqd/0: page allocation failure: order:0, mode:0x120 > [ 100.067038] CPU: 0 PID: 3 Comm: ksoftirqd/0 Tainted: G OE ----V------- 3.10.0-327.28.3.28.x86_64 #1 > [ 100.067039] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS rel-1.9.1-0-gb3ef39f-20161018_184732-HGH1000003483 04/01/2014 > [ 100.067041] 0000000000000120 00000000b080d798 ffff8802afd5b968 ffffffff81638cb9 > [ 100.067045] ffff8802afd5b9f8 ffffffff81171380 0000000000000010 0000000000000000 > [ 100.067048] ffff8802befd8000 00000000ffffffff 0000000000000001 00000000b080d798 > [ 100.067050] Call Trace: > [ 100.067057] [] dump_stack+0x19/0x1b > [ 100.067062] [] warn_alloc_failed+0x110/0x180 > [ 100.067066] [] __alloc_pages_nodemask+0x9b6/0xba0 > [ 100.067070] [] ? skb_add_rx_frag+0x90/0xb0 > [ 100.067075] [] alloc_pages_current+0xaa/0x170 > [ 100.067080] [] mlx4_alloc_pages.isra.24+0x40/0x170 [mlx4_en] > [ 100.067083] [] mlx4_en_alloc_frags+0xdc/0x220 [mlx4_en] > [ 100.067086] [] ? __netif_receive_skb+0x18/0x60 > [ 100.067088] [] ? netif_receive_skb+0x40/0xc0 > [ 100.067092] [] mlx4_en_process_rx_cq+0x5f1/0xec0 [mlx4_en] > [ 100.067095] [] ? list_del+0xd/0x30 > [ 100.067098] [] ? __napi_complete+0x1f/0x30 > [ 100.067101] [] mlx4_en_poll_rx_cq+0x9f/0x170 [mlx4_en] > [ 100.067103] [] net_rx_action+0x152/0x240 > [ 100.067107] [] __do_softirq+0xef/0x280 > [ 100.067109] [] run_ksoftirqd+0x30/0x50 > [ 100.067114] [] smpboot_thread_fn+0xff/0x1a0 > [ 100.067117] [] ? schedule+0x29/0x70 > [ 100.067120] [] ? lg_double_unlock+0x90/0x90 > [ 100.067122] [] kthread+0xcf/0xe0 > [ 100.067124] [] ? kthread_create_on_node+0x140/0x140 > [ 100.067127] [] ret_from_fork+0x58/0x90 > [ 100.067129] [] ? kthread_create_on_node+0x140/0x140 > > ================================cut here===================================== > > The reason is that the huge abnormal IP packet will be received to net stack > and be dropped finally by dst_release, and the dst_release would use the rcuos > callback-offload kthread to free the packet, but the cond_resched_rcu_qs() will > calling do_softirq() to receive more and more IP abnormal packets which will be > throw into the RCU callbacks again later, the number of received packet is much > greater than the number of packets freed, it will exhaust the memory and then OOM, > so don't try to process any pending softirqs in the rcuos callback-offload kthread > is a more effective solution. OK, but we could still have softirqs processed by the grace-period kthread as a result of any number of other events. So this change might reduce the probability of this problem, but it doesn't eliminate it. How huge are these huge IP packets? Is the underlying problem that they are too large to use the memory-allocator fastpaths? Thanx, Paul > Fix commit bedc196915 ("rcu: Fix soft lockup for rcu_nocb_kthread") > Signed-off-by: Ding Tianhong > > Signed-off-by: Ding Tianhong > --- > kernel/rcu/tree_plugin.h | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/kernel/rcu/tree_plugin.h b/kernel/rcu/tree_plugin.h > index 85c5a88..760c3b5 100644 > --- a/kernel/rcu/tree_plugin.h > +++ b/kernel/rcu/tree_plugin.h > @@ -2172,8 +2172,7 @@ static int rcu_nocb_kthread(void *arg) > if (__rcu_reclaim(rdp->rsp->name, list)) > cl++; > c++; > - local_bh_enable(); > - cond_resched_rcu_qs(); > + _local_bh_enable(); > list = next; > } > trace_rcu_batch_end(rdp->rsp->name, c, !!list, 0, 0, 1); > -- > 1.9.0 > > >