Received: by 2002:a05:6358:3188:b0:123:57c1:9b43 with SMTP id q8csp4859940rwd; Tue, 23 May 2023 13:47:24 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ5JDUK3Lu2Wst6IoMuIH2+yoq2PL5w7qxgO9IjGTcgcMuVwKDIKo5lWoB3uneaZE+OMNJso X-Received: by 2002:a05:6a00:2384:b0:627:6328:79f1 with SMTP id f4-20020a056a00238400b00627632879f1mr306523pfc.34.1684874844647; Tue, 23 May 2023 13:47:24 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1684874844; cv=none; d=google.com; s=arc-20160816; b=QjuQjpDDYiNIuGmSZ5VapzLWC946O+GXT0GvPQlRgegXGEua14Xf+NtvUoGzSF2hVu 1bXyduO9kCSle2xWTujrP28uztO6czjvYEXm8KQik1BJG/G5a6yIVLbF6HQQvQhKG0fO Y/mL9wgqeQH+syjegy9dRVZszve2eiL24TqssRKRyUyJ9gmao4I5tDUgysgGbd1TuirS 5G7QbgW9BdvJEvd00Jt5SdNazVMAiFEQpk2lhx6BvPtAj8szQ2TEREpqEpimWtrm17YR 4Yb6wMdsCyymg84OlVjYWFN286Pf8FlL/v1f3Su75adVQIfj1zAZt4SkIGEiMVR5zIXf nwbg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=m69+0U+vf8WBXQfiW8cb5lnrUt7WhgTHWMAX6k595F4=; b=WW/TJFkTfPvOO3IoXUwYOjfBHmHVnILdYPDZPxVewMsmK0g0M2J5fKBotJBbXz+ZwI XnuYVqw2eaJGrB6fviqtEaRK3pD8E9un6gLHocUlZqDfSx1LQRMoMZBgU7oStbCW/bAB 71SgmHnQhZbccAPEa/d7c55XL7Pye/IxXieLsBVY0c6xwBn/RPAPBDsCQTqrR7WPA3eo CIuo95oKX2OpOcn/vhNSVFs4/lfbMyHn8kuNoGfZv69JgMDEHJUPke6ocEjwbPjY+tOA O/Q5G12ERiWdpzQmwOX0UCUDyHYvoVCjFAKSlzMNOthrECttDEJ9+5Zh5hw2pYsZBLER HCWg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20221208 header.b=VP0fHtjC; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id z10-20020aa79f8a000000b0063b64166faesi7041077pfr.157.2023.05.23.13.47.12; Tue, 23 May 2023 13:47:24 -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=@gmail.com header.s=20221208 header.b=VP0fHtjC; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238544AbjEWURM (ORCPT + 99 others); Tue, 23 May 2023 16:17:12 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59886 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233032AbjEWURJ (ORCPT ); Tue, 23 May 2023 16:17:09 -0400 Received: from mail-qk1-x730.google.com (mail-qk1-x730.google.com [IPv6:2607:f8b0:4864:20::730]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9BC47129; Tue, 23 May 2023 13:17:08 -0700 (PDT) Received: by mail-qk1-x730.google.com with SMTP id af79cd13be357-75b2a2bf757so20143585a.2; Tue, 23 May 2023 13:17:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1684873028; x=1687465028; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=m69+0U+vf8WBXQfiW8cb5lnrUt7WhgTHWMAX6k595F4=; b=VP0fHtjC3N4hCeuVqyg/C3oO7n9xQXxKL3dUNKvZ1gDMksuTVZyuxUuCYeoc/Gr0gS 2KaPT1FqB22o2RpQSu1A7kSB72uDMvOAPxpfFH47lJ9Y9nlrRBS3/GuxrQb2rIVCs/Fs q64me5UewXpu75kHm56eGiV61mfeSEG/GHEscko6K9JD/i9NzzzjSyyST0GIc1cdhvRh ClJI2qEQ+N3eryZTLIlj1H6yt++fiiWlQHm/nroZCswRWTK7Nh3LUAzB8a92qaHMsWLN uK5F8hIxZzIYbiMspMC8L3p+J3+j7aaKSgT2jXWC0OFO8EgmjT2uJ2ip0eySyFrNfWoL qqog== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1684873028; x=1687465028; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=m69+0U+vf8WBXQfiW8cb5lnrUt7WhgTHWMAX6k595F4=; b=A4LJOFtJtECbkFvmlwinYcRuft7wmOIKI5ZsdS1VO5zBJ+iVkhYTb/maI1cJZC7yaw i4zlhyBtYuN7uk9028Lm1Mt6nlRJvKIVb6bosY1Qx5JX+FfpZOHm+Ppn3m3jJbC4H+eq 8OrN0/7Hd1jtiv/VpE0QEbrZgzLtgWFC5d3BCj8nI+YpuQ8wHXr7jwulNj/x4X8i45cH +qfYnFkLfcPMs9sIHPSeD8uDYeO34c+fIvQvXvBhcrf5RN2L42NmORUmbv+l70FQNtr0 5OtunSUDG/masvARWBlzfNjHlVxp0aPwt36h+3IVjt1+PPbMIHJRbX43Ed0GlVnm+piI B54w== X-Gm-Message-State: AC+VfDz8UBpExFhakninHGSOrGZIBLXSnYyRSd1nOoMcoOgqZLThPuEI vMDXuQhfxFCDBkTuBjh52A== X-Received: by 2002:a37:a914:0:b0:75b:23a1:364b with SMTP id s20-20020a37a914000000b0075b23a1364bmr5512524qke.12.1684873027715; Tue, 23 May 2023 13:17:07 -0700 (PDT) Received: from C02FL77VMD6R.googleapis.com ([2600:1700:d860:12b0:c32:b55:eaec:a556]) by smtp.gmail.com with ESMTPSA id i28-20020a05620a145c00b00759169d0316sm2743370qkl.40.2023.05.23.13.17.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 23 May 2023 13:17:07 -0700 (PDT) Date: Tue, 23 May 2023 13:17:02 -0700 From: Peilin Ye To: "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Jamal Hadi Salim , Cong Wang , Jiri Pirko Cc: Peilin Ye , Daniel Borkmann , John Fastabend , Vlad Buslov , Pedro Tammela , Hillf Danton , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Cong Wang Subject: Re: [PATCH v4 net 0/6] net/sched: Fixes for sch_ingress and sch_clsact Message-ID: References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM, RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,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 Tue, May 23, 2023 at 12:12:39AM -0700, Peilin Ye wrote: > [1,2/6]: ingress and clsact Qdiscs should only be created under ffff:fff1 > [3/6]: Under ffff:fff1, only create ingress and clsact Qdiscs (for now, > at least) > [4/6]: After creating ingress and clsact Qdiscs under ffff:fff1, do not > graft them again to anywhere else (e.g. as the inner Qdisc of a > TBF Qdisc) > [5/6]: Prepare for [6/6], do not reuse that for-loop in qdisc_graft() > for ingress and clsact Qdiscs > [6/6]: Fix use-after-free [a] in mini_qdisc_pair_swap() In v5, I'll improve [6/6] according to Vlad's suggestion [a], and fix [1,2/6] according to Pedro's report [b]. [a] https://lore.kernel.org/r/87sfbnxhg7.fsf@nvidia.com/ [b] https://lore.kernel.org/r/e462a91e-8bea-8b72-481c-4a36699e4149@mojatatu.com/ Thanks, Peilin Ye