Received: by 2002:a05:6a10:22f:0:0:0:0 with SMTP id 15csp1554021pxk; Tue, 1 Sep 2020 01:22:10 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyTYbiK8+MRumwGWzdUus15055Vqhy+6X4HCPgzaWMLxM6SrDqvtvye9f0bEGbSCP8ITDXc X-Received: by 2002:a17:906:d206:: with SMTP id w6mr498437ejz.428.1598948530717; Tue, 01 Sep 2020 01:22:10 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1598948530; cv=none; d=google.com; s=arc-20160816; b=JZODarBwUTwQYEvy9JmhAlF7qyshO93RhEwkPFWkF+EvmoUDRTU3QoKva9SA+ggX1D 8vsuqojlPyBpdWhYzBVCqBH61rfcrnIKWLNnp28oYoOv2ckvRuh4X/jpAzDBkYKE5OM7 6x3hsZZ7Yvur4lJXxW0UTuR5h2xffotlyiVLwP+tfEvSPOukEhMPZX3j5c9EoCt+LDdf /F00nIHR93tu2h1IKE+hD0oeb1tL2cATDDnG0B76gXfW9txXy21qpve7suM4fDJY46xw KYv59XE/9rZl0LJZEZT5KHqwSHxGmv3K98rGk+cxpig3wK96zYLh3lO34KwoJ5MyxFj6 CpMQ== 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=GidK5vLHoMLtRO6oix32yakLUL0qUZ3WOH9e89I+SuQ=; b=ppuQhSjdwylc9ZGOlrIrH7sErtL+bY6izpGbmYsM8wteDwavUDdpIjtv90cKMpMX2K cX4ooeqcHcDgWSg7/rFx7ku8asE/WDoBhfsT28K5nkhjaT3WOpc4lPwAgwoQkFhZ8d/v c/0EZwshbh9LlYK4hTJ7rw9vOTpQ0wQ47LcLU5uCYLOqsYKDvLAhS1JBC18PdtaO5i/L rK5+OJgFNtVsvw0iaUNoIhyns6NzS3H3/xPS09qIeEswMMht8xUsTNA1dw9BaZcLcZOM DLyDsMVjZ8WuRv/UMRqj/Nz3Xdl/drp51nmoX+6s3t4apptjoxmQoKCK3XR0KjT//DoV 9DVg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@google.com header.s=20161025 header.b=JWDmWWI2; 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 co6si195755edb.366.2020.09.01.01.21.47; Tue, 01 Sep 2020 01:22:10 -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=JWDmWWI2; 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 S1726913AbgIAIUk (ORCPT + 99 others); Tue, 1 Sep 2020 04:20:40 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44272 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726503AbgIAIUj (ORCPT ); Tue, 1 Sep 2020 04:20:39 -0400 Received: from mail-il1-x141.google.com (mail-il1-x141.google.com [IPv6:2607:f8b0:4864:20::141]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5E768C061245 for ; Tue, 1 Sep 2020 01:20:39 -0700 (PDT) Received: by mail-il1-x141.google.com with SMTP id q6so429909ild.12 for ; Tue, 01 Sep 2020 01:20:39 -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=GidK5vLHoMLtRO6oix32yakLUL0qUZ3WOH9e89I+SuQ=; b=JWDmWWI2C+/q0ErrIxsPoQGB9SLKNmgMZwhlDmezzeSX5mGrH1H4d7Qza+mQ7Tft9/ nho0o7iksQUkSJNldr31Kt4cC20TXRffDpzHFSvCtXUMTgh8CO3qv5uzlGr15axSysJE /M0YbP5QOcxoxto7+xqf5A/mUF+lsCrMkQFu2XYxy2l5uSW8zHwKhgyUrsU15pUiKvEc CB2h4JMATfmHV2LuqLbrwRuYJQ2GeuRZ7lanR9ZCbTGdO0jZcV9pyv9nrPCf2+88o2pz KacrlNthIMzRA6kNRiI5vRVI4x3Q+lRf7ViIviUgJT0lIwkrlP7t9CV6s/qHIexreAUt 0JpA== 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=GidK5vLHoMLtRO6oix32yakLUL0qUZ3WOH9e89I+SuQ=; b=ZJOhOZ3RYB/1yTyfKasHJEE7qqBJHXgNb6a8R0/rSU89ypFRPxYfWmNSRkoXN1X4JR ifY8TOZdthe8aGf7xmCp+3L6vLLPUWebQzYWMMXxvI8PxWWuysHOUzpTPM8Xwl/Zl9qo PgePz0+NoYt4vuL3CBZNr9AmU1FFa3Rh/KgFN9xCj2rRXrqtWO5G2mLZX+CuIZE0VAbB fG/TXjtBGcyyJzDGZJSmYJXzMYGk6jyeu5T2mT0PMrHiKT7TTKsPrRR7iTg6qvK5i08F e6/mrtruFs/XMyL4u9qvI/P4G7QA8oEC3R9ZOdedHv5K9wN+D/2Y3fw7SBSF+UQZZW++ clZg== X-Gm-Message-State: AOAM531AsKmX5LFvLqtYcWiP057IrmtwPBTnhwl7d+ICkPBFFjTT6t8L H1Nou8fgcNCbGc76hlL28SkOaQic/euSd50pj7Eklw== X-Received: by 2002:a92:bb0e:: with SMTP id w14mr344384ili.68.1598948438458; Tue, 01 Sep 2020 01:20:38 -0700 (PDT) MIME-Version: 1.0 References: <20200901065758.1141786-1-brianvv@google.com> In-Reply-To: <20200901065758.1141786-1-brianvv@google.com> From: Eric Dumazet Date: Tue, 1 Sep 2020 10:20:27 +0200 Message-ID: Subject: Re: [PATCH] net: ipv6: fix __rt6_purge_dflt_routers when forwarding is not set on all ifaces To: Brian Vazquez Cc: 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 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. What is the exact problem you want to fix ? > > This patches fixes that by cleaning only the routes where the iface has > forwarding enabled. This seems like a new feature, and this might break some setups. linux always had removed any IPv6 address and default route it learnt via SLAAC. (this might be to conform to one RFC or known security rule). It would be nice to add a nice comment giving references. > > Fixes: 830218c1add1 ("net: ipv6: Fix processing of RAs in presence of VRF") Please provide a test, or a better commit message ? If your patch targets the net tree, then it should only fix a bug. Thanks.