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;
Date:   Thu, 17 Jun 2021 19:41:55 +0300
From:   Vladimir Oltean <olteanv@gmail.com>
To:     Vadym Kochan <vadym.kochan@plvision.eu>
Cc:     "David S. Miller" <davem@davemloft.net>,
        Jakub Kicinski <kuba@kernel.org>,
        Jamal Hadi Salim <jhs@mojatatu.com>,
        Cong Wang <xiyou.wangcong@gmail.com>,
        Andrew Lunn <andrew@lunn.ch>,
        Serhiy Boiko <serhiy.boiko@plvision.eu>,
        Volodymyr Mytnyk <volodymyr.mytnyk@plvision.eu>,
        netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
        Boris Sukholitko <boris.sukholitko@broadcom.com>
Subject: Re: [PATCH net-next] net/sched: cls_flower: fix resetting of ether
 proto mask
Message-ID: <20210617164155.li3fct6ad45a6j7h@skbuf>
References: <20210617161435.8853-1-vadym.kochan@plvision.eu>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20210617161435.8853-1-vadym.kochan@plvision.eu>
Precedence: bulk

On Thu, Jun 17, 2021 at 07:14:35PM +0300, Vadym Kochan wrote:
> In case of matching 'protocol' the rule does not work:
> 
>     tc filter add dev $DEV ingress prio 1 protocol $PROTO flower skip_sw action drop
> 
> so clear the ether proto mask only for CVLAN case.
> 
> The issue was observed by testing on Marvell Prestera Switchdev driver
> with recent 'flower' offloading feature.
> 
> Fixes: 0dca2c7404a9 ("net/sched: cls_flower: Remove match on n_proto")
> CC: Boris Sukholitko <boris.sukholitko@broadcom.com>
> Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
> ---

You resent the patch very quickly, did you find out why Boris' patch was
working in the first place?