From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH] doc: notice for changes in kni structures Date: Tue, 9 May 2017 14:42:47 +0100 Message-ID: <707fb30a-1fae-a4bf-8b53-dbf68752034c@intel.com> References: <1493811091-26226-1-git-send-email-hemant.agrawal@nxp.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org To: Hemant Agrawal Return-path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id B8E8C1C00 for ; Tue, 9 May 2017 15:42:49 +0200 (CEST) In-Reply-To: Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 5/8/2017 10:46 AM, Hemant Agrawal wrote: > On 5/4/2017 10:20 PM, Ferruh Yigit wrote: >> On 5/3/2017 12:31 PM, Hemant Agrawal wrote: >>> Signed-off-by: Hemant Agrawal >>> --- >>> doc/guides/rel_notes/deprecation.rst | 7 +++++++ >>> 1 file changed, 7 insertions(+) >>> >>> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst >>> index a3e7c72..0c1ef2c 100644 >>> --- a/doc/guides/rel_notes/deprecation.rst >>> +++ b/doc/guides/rel_notes/deprecation.rst >>> @@ -81,3 +81,10 @@ Deprecation Notices >>> >>> - ``rte_crpytodev_scheduler_mode_get``, replaced by ``rte_cryptodev_scheduler_mode_get`` >>> - ``rte_crpytodev_scheduler_mode_set``, replaced by ``rte_cryptodev_scheduler_mode_set`` >>> + >>> +* kni: additional functionality is planned to be added in kni to support mtu, macaddr, >>> + gso_size, promiscusity configuration. >>> + some of the kni structure will be changed to support additional functionality >>> + e.g ``rte_kni_request`` to support promiscusity`` and mac_addr, >> >> rte_kni_request is between KNI library and KNI kernel module, shouldn't >> be part of API. >> >>> + ``rte_kni_mbu`` to support the configured gso_size, >> >> Again, rte_kni_mbuf should be only concern of KNI kernel module. >> >>> + ``rte_kni_device_info`` and ``rte_kni_conf`` to also support mtu and macaddr. >> >> rte_kni_device_info also between KNI library and KNI kernel module. >> >> I think deprecation notice not required for above ones. >> >> But you KNI patchset updates rte_kni_conf and rte_kni_ops. >> These are part of KNI API and changing them cause ABI breakage, >> but if new fields appended in these structs, this will not cause an ABI >> breakage, and I think that is better to do instead of deprecation >> notice, what do you think? > > I agree. >> >> >> And apart from above ABI issues, >> adding new fields to "rte_kni_ops" means DPDK application that use KNI >> should implement them, right? > > Well, it depend, if the application is interested in this information or > not? > >> So this suggest everyone require to set promiscuity of KNI device should >> implement this. > > yes! > >> Can't we find another way that all can benefit from a common implementation? > > how you want it differently? Any ideas? Can having default implementations in librte_kni work? Would applications be doing something different, lets say to set MTU? > > >> >> Thanks, >> ferruh >> > >