From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from imap1.codethink.co.uk ([176.9.8.82]:53774 "EHLO imap1.codethink.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726686AbeILGFM (ORCPT ); Wed, 12 Sep 2018 02:05:12 -0400 Message-ID: <1536714193.3024.184.camel@codethink.co.uk> Subject: Re: [PATCH 4.4 34/80] enic: handle mtu change for vf properly From: Ben Hutchings To: Greg Kroah-Hartman , linux-kernel@vger.kernel.org Cc: stable@vger.kernel.org, Govindarajulu Varadarajan , "David S. Miller" , Sasha Levin Date: Wed, 12 Sep 2018 02:03:13 +0100 In-Reply-To: <20180903164935.510879136@linuxfoundation.org> References: <20180903164934.171677301@linuxfoundation.org> <20180903164935.510879136@linuxfoundation.org> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: stable-owner@vger.kernel.org List-ID: On Mon, 2018-09-03 at 18:49 +0200, Greg Kroah-Hartman wrote: > 4.4-stable review patch.  If anyone has any objections, please let me know. > > ------------------ > > From: Govindarajulu Varadarajan > > [ Upstream commit ab123fe071c9aa9680ecd62eb080eb26cff4892c ] > > When driver gets notification for mtu change, driver does not handle it for > all RQs. It handles only RQ[0]. > > Fix is to use enic_change_mtu() interface to change mtu for vf. [...] This causes a assertion failure (noisy error logging, but not an oops) when the driver is probed. This was fixed upstream by: commit cb5c6568867325f9905e80c96531d963bec8e5ea Author: Govindarajulu Varadarajan Date:   Mon Jul 30 09:56:54 2018 -0700     enic: do not call enic_change_mtu in enic_probe which is now needed on the 3.18, 4.4, and 4.9 stable branches. Ben. -- Ben Hutchings, Software Developer   Codethink Ltd https://www.codethink.co.uk/ Dale House, 35 Dale Street Manchester, M1 2HF, United Kingdom