From mboxrd@z Thu Jan 1 00:00:00 1970 From: Venky Venkatesan Subject: Re: [RFC PATCH] ethdev: remove old flow director API Date: Mon, 20 Apr 2015 09:45:59 -0700 Message-ID: <55352D47.8010906@intel.com> References: <1429539103-15885-1-git-send-email-thomas.monjalon@6wind.com> <20150420163348.GA19573@hmsreliant.think-freely.org> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit To: dev-VfR2kkLFssw@public.gmane.org Return-path: In-Reply-To: <20150420163348.GA19573-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" On 04/20/2015 09:33 AM, Neil Horman wrote: > On Mon, Apr 20, 2015 at 04:11:43PM +0200, Thomas Monjalon wrote: >> It's time to remove this old API. >> It seems some work is still needed to rely only on eth_ctrl API. >> At least ixgbe, i40e and testpmd must be fixed. >> Jingjing, do you think it's possible to remove all these structures >> from rte_ethdev.h? >> >> Thanks >> > NAK. > > I'm certainly not opposed to removing the API's if they are truly no longer > needed. But they have been codified as part of the ABI, so the deprecation > schedule needs to be followed. Given what you've said above, it seems like that > might be worthwhile anyway, as it will provide the needed runway to allow users > to convert to the new API. > > Neil +1 NAK. Agree with Neil.