Received: by 2002:a05:6a10:22f:0:0:0:0 with SMTP id 15csp1836012pxk; Tue, 1 Sep 2020 08:56:17 -0700 (PDT) X-Google-Smtp-Source: ABdhPJz2QIBZlStCbJBt2lXjG9KUzXKy6MfH+kOv3SuVI3SxoXBXtWmMAYtYzQe74wWYXYKdjqSW X-Received: by 2002:a05:6402:1fb:: with SMTP id i27mr2352630edy.379.1598975776865; Tue, 01 Sep 2020 08:56:16 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1598975776; cv=none; d=google.com; s=arc-20160816; b=aqUtW4Nzv6CyK3wXn/qx7utYrnylGweO6XpgaFTSe9HZtZUUFyPVgv88x8Qd8+oA8L ac8A7LEhdhxPuqukA86TXd1a84WB/jmbA8kh0hdhLbSJYJMLTze4ERGq1Pb4W3/wW/DB BfcbJ7oXb55WdnA9nwNMsGEytKhqGnbB3EXtROSW/hEmohE9PjvNJnFlFn3gefZlRL5V iQ2/lsCsWtUWEZ50wYxmThVnks1fKmIEtJg+4w6VhlXws39dm9RF0ogqCHqY7Z8wNFrP CboQdnPsBoNwWp29/+sBUjSnOs1FbrB8uwV7pcdwfM7XtK7Z9gw3+U57Ayc5z/z+/zP7 2kmQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=jV8bljM3lK+GBB6jw/WspxqJuyJuh/lf4f+szOAk3Fw=; b=mg+Utn3ic7QW42aZGZwcn85f3mGnVQ8xfZIAVrJ6GHGIqb5HfqLih4qE+IYjfx4fiJ MOL7kfJ+oD/NkAYsJ/hsAUWUU9WETz7mRTvC2xb2KpG78v6IxHYCGk1I/t3L8TvpTli+ QCRgEdxxDjE0yck2B1cGA+qFXfv+Uvwz4AitOx12XON8UA6ZLAwhsu76CAg7zex5tp5Z t1b9WuiD94PykWXMScX7Eo6g8O41EHPh9wAIHj+1uIpylxkKD08AgN0ljeZ05gBXKPpq kQR+2dO6bG96OI/usJasu/vGt2JZdhu/IZmy8F6Owe41KvjrMAOQfiJZHd8wuKtN+g9J 5T2A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@google.com header.s=20161025 header.b=XZCGQwgw; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id h25si744105edq.433.2020.09.01.08.55.53; Tue, 01 Sep 2020 08:56:16 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@google.com header.s=20161025 header.b=XZCGQwgw; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729406AbgIAPuz (ORCPT + 99 others); Tue, 1 Sep 2020 11:50:55 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57354 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728988AbgIAPuh (ORCPT ); Tue, 1 Sep 2020 11:50:37 -0400 Received: from mail-qk1-x741.google.com (mail-qk1-x741.google.com [IPv6:2607:f8b0:4864:20::741]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5A9AFC06125C for ; Tue, 1 Sep 2020 08:50:31 -0700 (PDT) Received: by mail-qk1-x741.google.com with SMTP id v69so1391117qkb.7 for ; Tue, 01 Sep 2020 08:50:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=jV8bljM3lK+GBB6jw/WspxqJuyJuh/lf4f+szOAk3Fw=; b=XZCGQwgwBt860UII5cREakpxcHTR71LTVSlB36D/ju3AvUYviLZB4KikFD+0jTIyh4 elj6nui5j6HeINBSDmNWHrIdG34Uv3wns0bdy9KTaoVYV+kisqyRNmJSOgrYLuePTrad lA3IdMFJZtWBxBXHGSaG/pYJoPXIiHRGQ618S7d0TpMG/bMbJyArJDrv/7cZrSJ2Yojz mFwtdlJXC2TzgWARoJGIRfZ659fxiIKfJZwTq6NzfUmje2J3LyDHznp3jhgy78USqdaf r3yjjGj0lEMDSnlPQp5OvFZlLTcgShRmFUDnTX8/kuG4fVTST5BA+0DKvJ/FMGBpjY70 VFZQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=jV8bljM3lK+GBB6jw/WspxqJuyJuh/lf4f+szOAk3Fw=; b=VmxZpA6kCXCVhj+AzXTw1Ls0EwoBTbv+ZTL4EzSSBouiWcyf8rZGFmpFPDphgwvVaQ 0Hy8Vc39+R/D0b+SDF/sDksKm9woMRYdyUDMVEj49eRXsGulcCEhud9uHGHjN6J0IvEW qRRHayYm0/6AXM9aiE/3zl7Z6i3HOfpARkt4gQPHwXlshA1+2vVQVs1EHr/czDHVb4dr NcaVFfCoFXKLMnEIzpl1DILAOuXAdTKhpsFqLKcWW30ZDCr0iXioEpSBC6TSFGUgW5K7 TCJ3eSyp32SfH23FGot0okBSFuuFDwdiDpA+pZvMnr/XcGOvBTUMkbmo/JjiDcB0HAUH pQSA== X-Gm-Message-State: AOAM5338YReU4oISCEu4tXydsG6DrqkwfpO6x6Tj+x9t0e0BAxbhpsXJ zFLxGHZowRaafqwQPOOnpBDpcI8nvonnnZxyMSMCQQ== X-Received: by 2002:a37:9d4:: with SMTP id 203mr2555955qkj.277.1598975430108; Tue, 01 Sep 2020 08:50:30 -0700 (PDT) MIME-Version: 1.0 References: <20200901065758.1141786-1-brianvv@google.com> <52832405-fa37-38fb-b8fb-d7bd7a0d1d52@gmail.com> In-Reply-To: <52832405-fa37-38fb-b8fb-d7bd7a0d1d52@gmail.com> From: Brian Vazquez Date: Tue, 1 Sep 2020 08:50:18 -0700 Message-ID: Subject: Re: [PATCH] net: ipv6: fix __rt6_purge_dflt_routers when forwarding is not set on all ifaces To: David Ahern Cc: Eric Dumazet , Brian Vazquez , "David S . Miller" , LKML , netdev , David Ahern Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hey David, On Tue, Sep 1, 2020 at 7:57 AM David Ahern wrote: > > On 9/1/20 1:56 AM, Eric Dumazet wrote: > > On Tue, Sep 1, 2020 at 8:58 AM Brian Vazquez wrote: > >> > >> The problem is exposed when the system has multiple ifaces and > >> forwarding is enabled on a subset of them, __rt6_purge_dflt_routers will > >> clean the default route on all the ifaces which is not desired. > >> > >> This patches fixes that by cleaning only the routes where the iface has > >> forwarding enabled. > >> > >> Fixes: z ("net: ipv6: Fix processing of RAs in presence of VRF") > > are you sure that is a Fixes tag for this problem? looking at that > change it only handles RA for tables beyond the main table; it does not > change the logic of how many or which routes are purged. That commit also added RT6_TABLE_HAS_DFLT_ROUTER so I thought that was the commit needed to be mentioned. But probably it shouldn't? Also Am I missing something or this is only called on on the sysctl path? > > >