From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH net-next v8 00/19] new ETHTOOL_GSETTINGS/SSETTINGS API Date: Wed, 10 Feb 2016 16:56:42 -0800 Message-ID: <20160210165642.6f8576d2@xeon-e3> References: <1455064168-5102-1-git-send-email-ddecotig@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: "David S. Miller" , Ben Hutchings , linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-mips-6z/3iImG2C8G8FEW9MqTrA@public.gmane.org, fcoe-devel-s9riP+hp16TNLxjTenLetw@public.gmane.org, Andrew Morton , Tejun Heo , Eric Dumazet , Eugenia Emantayev , Or Gerlitz , Ido Shamay , Joe Perches , Saeed Mahameed , Govindarajulu Varadarajan <_govind-KK0ffGbhmjU@public.gmane.org>, Venkata Duvvuru , Jeff Kirsher , Pravin B Shelar , Ed Swierk , Robert Love , "James E.J. Bottomley" , Yuval Mintz , Rasmus Villemoes Return-path: In-Reply-To: <1455064168-5102-1-git-send-email-ddecotig-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Sender: linux-api-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: netdev.vger.kernel.org On Tue, 9 Feb 2016 16:29:09 -0800 David Decotigny wrote: > Along the way, I chose to drop in the new structure the 3 ethtool_cmd > fields marked "deprecated" (transceiver/maxrxpkt/maxtxpkt). They are > still available for old drivers via the (old) ETHTOOL_GSET/SSET API, > but are not available to drivers that switch to new API. Of those 3 > fields, ethtool_cmd::transceiver seems to be still actively used by > several drivers, maybe we should not consider this field deprecated? > The 2 other fields are basically not used. This transition requires > some care in the way old and new ethtool talk to the kernel. Please just fix old drivers. It is perfectly acceptable to break any out of tree drivers.