From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-16.1 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 60E28C433ED for ; Tue, 18 May 2021 14:28:41 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 3340C6109F for ; Tue, 18 May 2021 14:28:41 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231800AbhERO35 (ORCPT ); Tue, 18 May 2021 10:29:57 -0400 Received: from mail.kernel.org ([198.145.29.99]:59452 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231704AbhERO3u (ORCPT ); Tue, 18 May 2021 10:29:50 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 0BDFB6109F; Tue, 18 May 2021 14:28:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1621348112; bh=O7IMIQvzJLMltit9Z4tUy5pRmXeTCujbBxsU2HAhLTY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=KiHCIHQoVtGNkRsyVekLnFCx5W9eArs2b0zwmKkI9zdNyyUL2TTtXH6Aqc5xdyXz4 5j1Q/DXTtt/5MNDo3fzndah7KmEa1dBgp9UMkr5BEpFwMNJVdKfMJdR4RM3AHPxkCc 8s3PB1QxDdyvZ81/hcm8dmMWojGEf1EJUapA3mlw= Date: Tue, 18 May 2021 16:28:30 +0200 From: Greg Kroah-Hartman To: Rudi Heitbaum Cc: linux-kernel@vger.kernel.org, stable@vger.kernel.org, Florian Fainelli , Vladimir Oltean , "David S. Miller" , Sasha Levin Subject: Re: [PATCH 5.12 081/363] net: bridge: propagate error code and extack from br_mc_disabled_update Message-ID: References: <20210517140302.508966430@linuxfoundation.org> <20210517140305.339768334@linuxfoundation.org> <20210518122449.GA65@ec3d6f83b95b> <20210518141238.GA20@416f1e4b4f0c> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210518141238.GA20@416f1e4b4f0c> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, May 18, 2021 at 02:12:42PM +0000, Rudi Heitbaum wrote: > On Tue, May 18, 2021 at 02:39:43PM +0200, Greg Kroah-Hartman wrote: > > On Tue, May 18, 2021 at 12:24:57PM +0000, Rudi Heitbaum wrote: > > > On Mon, May 17, 2021 at 03:59:07PM +0200, Greg Kroah-Hartman wrote: > > > > From: Florian Fainelli > > > > > > > > [ Upstream commit ae1ea84b33dab45c7b6c1754231ebda5959b504c ] > > > > > > > > Some Ethernet switches might only be able to support disabling multicast > > > > snooping globally, which is an issue for example when several bridges > > > > span the same physical device and request contradictory settings. > > > > > > > > Propagate the return value of br_mc_disabled_update() such that this > > > > limitation is transmitted correctly to user-space. > > > > > > > > Signed-off-by: Florian Fainelli > > > > Signed-off-by: Vladimir Oltean > > > > Signed-off-by: David S. Miller > > > > Signed-off-by: Sasha Levin > > > > --- > > > > net/bridge/br_multicast.c | 28 +++++++++++++++++++++------- > > > > net/bridge/br_netlink.c | 4 +++- > > > > net/bridge/br_private.h | 3 ++- > > > > net/bridge/br_sysfs_br.c | 8 +------- > > > > 4 files changed, 27 insertions(+), 16 deletions(-) > > > > > > This patch results in docker failing to start, and a regression between > > > 5.12.4 and 5.12.5-rc1 > > > > > > A working dmesg output is like: > > > > > > [ 11.545255] device eth0 entered promiscuous mode > > > [ 11.693848] process 'docker/tmp/qemu-check643160757/check' started with executable stack > > > [ 17.233059] br-92020c7e3aea: port 1(veth17a0552) entered blocking state > > > [ 17.233065] br-92020c7e3aea: port 1(veth17a0552) entered disabled state > > > [ 17.233098] device veth17a0552 entered promiscuous mode > > > [ 17.292839] docker0: port 2(veth9d227f5) entered blocking state > > > [ 17.292848] docker0: port 2(veth9d227f5) entered disabled state > > > [ 17.292946] device veth9d227f5 entered promiscuous mode > > > [ 17.293070] docker0: port 2(veth9d227f5) entered blocking state > > > [ 17.293075] docker0: port 2(veth9d227f5) entered forwarding state > > > > > > with this patch "device veth17a0552 entered promiscuous mode" never > > > shows up. > > > > > > the docker error itself is: > > > > > > docker: Error response from daemon: failed to create endpoint > > > sleepy_dijkstra on network bridge: adding interface veth8cbd8f9 to > > > bridge docker0 failed: operation not supported. > > > > Ick. > > > > Does 5.13-rc1 also show this same problem? > > > > And thanks for testing! > > > > greg k-h > > Hi Greg, > > I can confirm that docker starts correctly with 5.13-rc1 > > # dmesg | head > [ 0.000000] Linux version 5.13.0-rc1 (rudi@0e5f93d4a8a2) (x86_64-libreelec-linux-gnu-gcc-10.3.0 (GCC) 10.3.0, GNU ld (GNU Binutils) 2.36.1) #1 SMP Tue May 18 14:00:41 UTC 2021 > [ 0.000000] Command line: BOOT_IMAGE=/KERNEL boot=LABEL=LIBREELEC disk=LABEL=STORAGE i915.enable_guc=2 quiet > ... > [ 11.214582] docker0: port 1(veth2b37ac4) entered blocking state > [ 11.214589] docker0: port 1(veth2b37ac4) entered disabled state > [ 11.214649] device veth2b37ac4 entered promiscuous mode > [ 11.214752] docker0: port 1(veth2b37ac4) entered blocking state > [ 11.214755] docker0: port 1(veth2b37ac4) entered forwarding state Great, can you try 5.12.5-rc2 now? thanks, greg k-h