From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-f180.google.com (mail-wi0-f180.google.com [209.85.212.180]) by dpdk.org (Postfix) with ESMTP id 344CBB62 for ; Mon, 27 Apr 2015 18:09:38 +0200 (CEST) Received: by wiun10 with SMTP id n10so95810219wiu.1 for ; Mon, 27 Apr 2015 09:09:38 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:organization :user-agent:in-reply-to:references:mime-version :content-transfer-encoding:content-type; bh=rrFQQmRIJV+wMfZ8cpdZSmWrJGNFTRADMuriad3VwYw=; b=kZKlDr4qwiZnQH/1IV2MOs38bdMtgyYtpADTTRzuqSEj5cn0LbAAXyVgLYmmOdMbYX Vumm+ERxO+0PzWtKb1D3gEulFxIyontjZ40I9EwgZ8Mz3EDZQZqNKOrJwd3gQkBXl3hU /N5/N/KUpGei/+ODS+qziJQPntHEBeBf6jwFWLDyeIjHir7+tj6AIbViUiWwaJPP98Sp KOnL8StQJQwkgyKVeLCi+IR0umD4MxW9wrIn1n6euukR+D2aFbHghN/NsyyNgBFGojvm UN63y/VWz/fy3UaRa0+aFp7w2BbsRjCZ7+YqHp4tlh2BfyPg07p8Mip+XMGubUXViHiZ iNsQ== X-Gm-Message-State: ALoCoQn9I3IndHWfW9RNGRZ/tEsKGzW64hwRIaPClxXGY6YoQXcVNnf5Nku10TIS/5wiZhvzwi2P X-Received: by 10.180.101.3 with SMTP id fc3mr2625463wib.47.1430150978056; Mon, 27 Apr 2015 09:09:38 -0700 (PDT) Received: from xps13.localnet (6wind.net2.nerim.net. [213.41.180.237]) by mx.google.com with ESMTPSA id pm1sm11747422wjb.23.2015.04.27.09.09.36 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 27 Apr 2015 09:09:37 -0700 (PDT) From: Thomas Monjalon To: Venky Venkatesan , Neil Horman Date: Mon, 27 Apr 2015 18:08:55 +0200 Message-ID: <1815538.YzHUjUMMD8@xps13> Organization: 6WIND User-Agent: KMail/4.14.4 (Linux/3.18.4-1-ARCH; KDE/4.14.4; x86_64; ; ) In-Reply-To: <55352D47.8010906@intel.com> References: <1429539103-15885-1-git-send-email-thomas.monjalon@6wind.com> <20150420163348.GA19573@hmsreliant.think-freely.org> <55352D47.8010906@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [RFC PATCH] ethdev: remove old flow director API X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 27 Apr 2015 16:09:38 -0000 2015-04-20 09:45, Venky Venkatesan: > 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. +1 Agree with you :) The goal of this RFC proposal is to see how to progress on API cleanup. There are actually 2 parts: 1/ The flow director functions of rte_ethdev.h were only used for enic in DPDK 2.0. We could set a deprecation notice to remove them in DPDK 2.2. 2/ Some associated structures are also used for rte_eth_conf. My question was to check how it would be relevant to remove this rte_fdir_conf.