From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jakub Kicinski Subject: Re: [patch net-next 0/9] mlxsw: Support firmware flash Date: Mon, 29 May 2017 14:40:27 -0700 Message-ID: <20170529144027.747fc1c6@cakuba.lan> References: <20170523.113859.1803057381093280239.davem@davemloft.net> <0daa5c5a-377c-767f-ea19-26c1f22bd30c@mellanox.com> <20170528171735.0e44878d@cakuba.lan> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Yotam Gigi , David Miller , Yuval.Mintz@cavium.com, Jiri Pirko , Linux Netdev List , Ido Schimmel , mlxsw , ben@decadent.org.uk To: Or Gerlitz Return-path: Received: from mx3.wp.pl ([212.77.101.9]:6918 "EHLO mx3.wp.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751050AbdE2Vkg (ORCPT ); Mon, 29 May 2017 17:40:36 -0400 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: [swapping Ben's email] On Mon, 29 May 2017 11:52:27 +0300, Or Gerlitz wrote: > On Mon, May 29, 2017 at 3:17 AM, Jakub Kicinski wrote: > > On Sun, 28 May 2017 10:26:49 +0300, Yotam Gigi wrote: > > >> This problem is even more relevant in the Mellanox HCA driver team, which would > >> like to use that code in order to burn the HCA firmware, but not intend to > >> trigger it on boot time, which means that must have a way for the user to > >> trigger it. > > > What would the requirements for the HCA team be? Is it about loading > > different code or loading HW settings? > > For the NIC, as of the (happily growing) large open-systems by nature > install base, we > can't effort for the driver not to load when the currently burned fw > isn't the latest or > the system doesn't have the latest libfirmware clone. We do have to > keep the hassle of > compatibility with older FWs, etc. As part of the driver/FW API > design we use cap bits > for that matter, and never ask/branch on specific FW or HW versions. > > We do want to let user work with the kernel w.o the need to have them > install MLNX or anyone's > tool suit and hence the large value we find in the ethtool flashing of > FW image (the patches > are ready...) Thanks for the explanations Or and Yotam! That does sound like a plain version upgrade problem.