Received: by 2002:a05:6358:5282:b0:b5:90e7:25cb with SMTP id g2csp2060577rwa; Mon, 22 Aug 2022 01:03:21 -0700 (PDT) X-Google-Smtp-Source: AA6agR6aJNRuHYQwvz0ViErgZY0hWZ4/is91+9zvYXMGNy2JObuWF+Cu2s9el6FVxD2lMdN4q9Ff X-Received: by 2002:a17:906:cc54:b0:73d:87f9:c0d4 with SMTP id mm20-20020a170906cc5400b0073d87f9c0d4mr1036812ejb.543.1661155401402; Mon, 22 Aug 2022 01:03:21 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1661155401; cv=none; d=google.com; s=arc-20160816; b=Hg3GkIasSSjp9j8qp5sgDvz2N7g+Q4l2gKaJhAP/ZmUkfjZ+2haCcLRDLczfYF2IEU wPR0+HcfemIrfqTkrXvKGHehX2cnhPb4B7Qi1soEafIrlvop1bafKIh/HjEEgmUHh055 sf/6whchvG8uikUNrHVYjzdYIjU6nfBqQ3oqz+/BBiV79GEB/iDSpLzdPmwSjqgA811F 6gRh5Fh6CxtIGxHmCCtaiTXXBOTP/TvYlBrNDMDyAfz8c6V2H3BbTV0ABT4QwH35NDwE 2IUh0zpPy4ra98BKdI5v8FThmhXROKRjcgbQfDhaM5UppMJXvom3XFbmxQj1YMDTh0tG v4rg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :date:message-id:dkim-signature; bh=Xt7sUJQ+aMJj9heYVbH58SruiAfLtzcBF4RZu/5f3ug=; b=DOjfdeYwLaInmSEtmHJoUBSFpZTB3ydGKVky481ke4HQHm/pR1yoNVqAePHz6VhXSP Hb/m1zefRcmU51oKOAfQNTfuIEUA3dmWRhwMOhEdBv11rqC7yjlb9ENttvbtBU8jDuOE Wz0zk/QvyVv0VILQF7CbYGzziTNEf0c5Q7hfEQZSUjFHHFeMNVvB8TbKHpktQcIcEcNR RraU9SRcQors9WhqcyLh7m48m8BnNh1nJrYH/cA0NJG/lUUPrsR08qeIbubIqzR24aKC +oQv1jm/LKpB+YLL7nh/ATrpRO8KmHjTbz6h+kCYZ/E/ef/FEDd+Mh1g2F63FCB+O8Xh 5VaA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@blackwall-org.20210112.gappssmtp.com header.s=20210112 header.b=rsY19QIa; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id hg3-20020a1709072cc300b0072b3ccd50dasi8647651ejc.905.2022.08.22.01.02.37; Mon, 22 Aug 2022 01:03:21 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@blackwall-org.20210112.gappssmtp.com header.s=20210112 header.b=rsY19QIa; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233436AbiHVHuI (ORCPT + 99 others); Mon, 22 Aug 2022 03:50:08 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44382 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233435AbiHVHuD (ORCPT ); Mon, 22 Aug 2022 03:50:03 -0400 Received: from mail-ed1-x535.google.com (mail-ed1-x535.google.com [IPv6:2a00:1450:4864:20::535]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BF9192A414 for ; Mon, 22 Aug 2022 00:49:58 -0700 (PDT) Received: by mail-ed1-x535.google.com with SMTP id z2so12759068edc.1 for ; Mon, 22 Aug 2022 00:49:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=blackwall-org.20210112.gappssmtp.com; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc; bh=Xt7sUJQ+aMJj9heYVbH58SruiAfLtzcBF4RZu/5f3ug=; b=rsY19QIao4S4MIiPcndmxJ/lUJkZ3wgbfJ6gAIwmeUjwbKR8ITGonQux7yCtKKClD+ /OwFji4+MLFoLb9jC0j99XndeYi/bccJ+jpgNFTc0lXYBYLaAleiwPdpFUm1/rQD9Hm5 lJ2FD4Ij183DVlHaU+nbQTYL4TBqq7iJlmS0NfCRDsk/mCJs0LDnQsvdHmAXn1YqXrTn 7tGsT8SXdSKh9CThquNhCv28umaIkvq5evvrYsNitj9UHlU1AFG0U8yCurPDAESw4LG3 b7joR24coTZYEcXXLGNzzpBPOLg6ZXvh6/rVGQOnA87+IWLBGtQhxgDxJHo19HF9MPg6 JKwA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc; bh=Xt7sUJQ+aMJj9heYVbH58SruiAfLtzcBF4RZu/5f3ug=; b=vxWhCkb1xvBLkupQpC+cPqYJ5Qk9lvu1Tz0p+fFd8vttwzfHVCNs3M12d2l7ndiqVE aG8ych+njjKbjYHlDLpO1XQWwIJZ2Z+RDdItDkmVW8EOTC1KlUG9k5SWio4rFDhQPNHe mbxKtnSnhlRn+eShV3ed6PSo0HaEK/uw6TL5Uk0cHkYO/rBUXHtJq0RsU/4XGW/hcgEq whZJ9uwel0jrgB/SfUKDbOlvbt02AQOVsyWoBGxQG3augizGDYieBXLsKQTqckfwpxNa hDnNFN2ETsz/bWAK277ftvMhxPu5lmNVaMyggtFF2XEalTwkei8pxXjL63esC7O6WH++ JZzQ== X-Gm-Message-State: ACgBeo0V0WWkNWOJ8UVYonvJmJbFAyVJ5bf+/xdd7iVmBeGiYoiGXDlU Bsf8dviw4k6br4B4y8rR3gTZ7A== X-Received: by 2002:aa7:cd79:0:b0:446:1fee:290e with SMTP id ca25-20020aa7cd79000000b004461fee290emr13736652edb.163.1661154597141; Mon, 22 Aug 2022 00:49:57 -0700 (PDT) Received: from [192.168.0.111] (87-243-81-1.ip.btc-net.bg. [87.243.81.1]) by smtp.gmail.com with ESMTPSA id bi4-20020a170906a24400b0072af4af2f46sm5764857ejb.74.2022.08.22.00.49.56 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 22 Aug 2022 00:49:56 -0700 (PDT) Message-ID: Date: Mon, 22 Aug 2022 10:49:55 +0300 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.10.0 Subject: Re: [PATCH net v2 RESEND] net: neigh: don't call kfree_skb() under spin_lock_irqsave() Content-Language: en-US To: Yang Yingliang , linux-kernel@vger.kernel.org, netdev@vger.kernel.org Cc: den@openvz.org, davem@davemloft.net, edumazet@google.com, kuba@kernel.org References: <20220822025346.3758558-1-yangyingliang@huawei.com> From: Nikolay Aleksandrov In-Reply-To: <20220822025346.3758558-1-yangyingliang@huawei.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_NONE, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 22/08/2022 05:53, Yang Yingliang wrote: > It is not allowed to call kfree_skb() from hardware interrupt > context or with interrupts being disabled. So add all skb to > a tmp list, then free them after spin_unlock_irqrestore() at > once. > > Fixes: 66ba215cb513 ("neigh: fix possible DoS due to net iface start/stop loop") > Suggested-by: Denis V. Lunev > Signed-off-by: Yang Yingliang > --- > v2: > move all skb to a tmp list, then free them after spin_unlock_irqrestore(). > --- > net/core/neighbour.c | 12 +++++++++--- > 1 file changed, 9 insertions(+), 3 deletions(-) > LGTM, Reviewed-by: Nikolay Aleksandrov