Received: by 2002:a25:8b12:0:0:0:0:0 with SMTP id i18csp1762079ybl; Sat, 31 Aug 2019 01:44:22 -0700 (PDT) X-Google-Smtp-Source: APXvYqzRivh6+D4AYG5doGR42NO7POPOh3BXVP2ba/l8mcd+omlKnd0cY8NEQXLZrIrhtyY6D9gF X-Received: by 2002:a65:640a:: with SMTP id a10mr16176534pgv.338.1567241061844; Sat, 31 Aug 2019 01:44:21 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1567241061; cv=none; d=google.com; s=arc-20160816; b=yw3QYTNgH1GJZe1CEUjXGgEVGs04nRIvQjV//ABvv1Rj8hDuC8LodsMNgUJEKTItUA J3OhLZr06GNRgTWRxAszIaPcJSV2pjMqntvnM10hnLLCiUhYTIXBqVG+Pey6g1PN1NWw R9yxifXfZnD8Zzrl6OIXMGP3cSjqdNvt44gJUeQjxMmVeFRur/xyKH/udiHb1+Go+Wh+ I/CBcktJk/zbE99GcrlWDlETqlFuNl4EbEyxvcoaQzdM/9OpS7SEqJcgTp4cpXpdjsSw A/iJzEBmGEniXirGbfqLktwCI4Rn65l7kiAvKBvPSUPE72UlqwojJKnZDU2SE17wopLJ XQsQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=fTsEcdBcZJwCTgCpsZgxVPQTgOD5vn6bXCO0h7tpZzw=; b=OGB9nFqlZ6woRCGkQC02L6qQ4ltNOsm5Bt8iZ7dbBvM0jR8E8LL3ELm+tozsvvWDBY fY3Ue6zsduZYaEWDfP9mg4kqsoRhpSxVX+KEb5qj9CASuNC667H2U6b7onSPcfn6Il4c bWKonPFt1sLtOPhf/fYm01BXHz35657qqz7F0KhSgkW+T1giOAnjBSMy/esM0JNzQQ+Z xouYTOwnsX3IBuHGOvox8dxQP4iNz4JrRtF+L5OTjx8CDVrgA77END93dQQlzcrOCePD rUqrOJLObHTdzyw93jMVs29J8KAQMadqJz08cqEX1uV+22ZWX2/8vjs4bdhrXu604bR3 N91A== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id p5si254398pfn.191.2019.08.31.01.44.07; Sat, 31 Aug 2019 01:44:21 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727274AbfHaInU (ORCPT + 99 others); Sat, 31 Aug 2019 04:43:20 -0400 Received: from Chamillionaire.breakpoint.cc ([193.142.43.52]:60066 "EHLO Chamillionaire.breakpoint.cc" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726102AbfHaInU (ORCPT ); Sat, 31 Aug 2019 04:43:20 -0400 Received: from fw by Chamillionaire.breakpoint.cc with local (Exim 4.92) (envelope-from ) id 1i3yyZ-0000iW-BO; Sat, 31 Aug 2019 10:43:15 +0200 Date: Sat, 31 Aug 2019 10:43:15 +0200 From: Florian Westphal To: Leonardo Bras Cc: Florian Westphal , netfilter-devel@vger.kernel.org, coreteam@netfilter.org, bridge@lists.linux-foundation.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Pablo Neira Ayuso , Jozsef Kadlecsik , Roopa Prabhu , Nikolay Aleksandrov , "David S. Miller" Subject: Re: [PATCH v4 2/2] net: br_netfiler_hooks: Drops IPv6 packets if IPv6 module is not loaded Message-ID: <20190831084315.GU20113@breakpoint.cc> References: <20190830181354.26279-1-leonardo@linux.ibm.com> <20190830181354.26279-3-leonardo@linux.ibm.com> <20190830205541.GR20113@breakpoint.cc> <2ba876f9ad6597e640df68f09659dce3c4b5ce03.camel@linux.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <2ba876f9ad6597e640df68f09659dce3c4b5ce03.camel@linux.ibm.com> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Leonardo Bras wrote: > > There are two solutions: > > 1. The above patch, but use NF_ACCEPT instead > > 2. keep the DROP, but move it below the call_ip6tables test, > > so that users can tweak call-ip6tables to accept packets. > > Q: Does 2 mean that it will only be dropped if bridge intents to use > host's ip6tables? Else, it will be accepted by previous if? Yes, thats the idea: Let users decide if ipv6.disable or call-ip6tables is more important to them.