From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-eopbgr60054.outbound.protection.outlook.com ([40.107.6.54]:44694 "EHLO EUR04-DB3-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753534AbeGESX4 (ORCPT ); Thu, 5 Jul 2018 14:23:56 -0400 Date: Thu, 5 Jul 2018 21:23:47 +0300 From: Ido Schimmel To: Greg KH Cc: stable@vger.kernel.org, jslaby@suse.cz, davem@davemloft.net, jiri@mellanox.com Subject: Re: [PATCH 4.9] mlxsw: spectrum: Forbid linking of VLAN devices to devices that have uppers Message-ID: <20180705182347.GA3072@splinter> References: <20180705172452.1454-1-idosch@mellanox.com> <20180705175221.GA29378@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180705175221.GA29378@kroah.com> Sender: stable-owner@vger.kernel.org List-ID: On Thu, Jul 05, 2018 at 07:52:21PM +0200, Greg KH wrote: > On Thu, Jul 05, 2018 at 08:24:52PM +0300, Ido Schimmel wrote: > > commit 25cc72a33835ed8a6f53180a822cadab855852ac upstream. > > > > Jiri Slaby noticed that the backport of upstream commit 25cc72a33835 > > ("mlxsw: spectrum: Forbid linking to devices that have uppers") to > > kernel 4.9.y introduced the same check twice in the same function > > instead of in two different places. > > > > Fix this by relocating one of the checks to its intended place, thus > > preventing unsupported configurations as described in the original > > commit. > > > > Fixes: 73ee5a73e75f ("mlxsw: spectrum: Forbid linking to devices that have uppers") > > Signed-off-by: Ido Schimmel > > Reported-by: Jiri Slaby > > --- > > Greg, I read Documentation/stable_kernel_rules.txt, so I hope the patch > > is fine. Please let me know if v2 is required. Thanks > > This commit is already in 4.9.51, why do you need/want it applied again? It wasn't clear to me if I need to specify "commit upstream" also for fixes of backports. You want me to post v2 without this line? Thanks