[dpdk-dev] Survey for final decision about per-port offload API

[Patil, Harish]

-----Original Message-----
From: Thomas Monjalon <thomas at monjalon.net>
Date: Monday, April 2, 2018 at 1:10 PM
To: Harish Patil <Harish.Patil at cavium.com>
Cc: "dev at dpdk.org" <dev at dpdk.org>, Ajit Khaparde
<ajit.khaparde at broadcom.com>, "Jacob,  Jerin"
<Jerin.JacobKollanukkaran at cavium.com>, "Thotton, Shijith"
<Shijith.Thotton at cavium.com>, "Shukla, Santosh"
<Santosh.Shukla at cavium.com>, Rahul Lakkireddy
<rahul.lakkireddy at chelsio.com>, John Daley <johndale at cisco.com>, Wenzhuo
Lu <wenzhuo.lu at intel.com>, Konstantin Ananyev
<konstantin.ananyev at intel.com>, Beilei Xing <beilei.xing at intel.com>, Qi
Zhang <qi.z.zhang at intel.com>, Jingjing Wu <jingjing.wu at intel.com>, Adrien
Mazarguil <adrien.mazarguil at 6wind.com>, Nelio Laranjeiro
<nelio.laranjeiro at 6wind.com>, Yongseok Koh <yskoh at mellanox.com>, Shahaf
Shuler <shahafs at mellanox.com>, Tomasz Duszynski <tdu at semihalf.com>, Jianbo
Liu <jianbo.liu at arm.com>, Alejandro Lucero
<alejandro.lucero at netronome.com>, Hemant Agrawal <hemant.agrawal at nxp.com>,
Shreyansh Jain <shreyansh.jain at nxp.com>, "Mody, Rasesh"
<Rasesh.Mody at cavium.com>, Andrew Rybchenko <arybchenko at solarflare.com>,
Shrikrishna Khare <skhare at vmware.com>, Maxime Coquelin
<maxime.coquelin at redhat.com>, Allain Legacy <allain.legacy at windriver.com>,
Bruce Richardson <bruce.richardson at intel.com>, Gaetan Rivet
<gaetan.rivet at 6wind.com>, Olivier Matz <olivier.matz at 6wind.com>
Subject: Re: Survey for final decision about per-port offload API

>02/04/2018 19:39, Patil, Harish:
>> >It was also proposed to relax the API and allow "forgetting" port
>> >offloads in queue offloads:
>> >	http://dpdk.org/ml/archives/dev/2018-March/092978.html
>> >
>> >It would mean the offloads applied to a queue result of OR operation:
>> >	rte_eth_conf.[rt]xmode.offloads | rte_eth_[rt]xconf.offloads
>> >
>> >1/ Do you agree with above API change?
>> 
>> Yes. But pls confirm that this would still work properly if RX supports
>> port-only based offloads and doesn’t support queue based offloads at
>>all,
>> as advertised in dev_infos_get().
>> 
>> dev_info->rx_queue_offload_capa = 0 and
>> dev_info->rx_offload_capa = (DEV_RX_OFFLOAD_IPV4_CKSUM | ...);
>
>Yes, of course, no queue offload at all is possible.
>
Okay, thanks.
>