* [dpdk-dev] Remaining old offloading API usage in PMDs
@ 2018-05-11 10:45 Ferruh Yigit
2018-05-14 11:01 ` Shreyansh Jain
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Ferruh Yigit @ 2018-05-11 10:45 UTC (permalink / raw)
To: Thomas Monjalon, Andrew Rybchenko, Tetsuya Mukawa,
Maxime Coquelin, Hemant Agrawal, Shreyansh Jain,
Alejandro Lucero
Cc: dpdk-dev, Beilei Xing, Qi Zhang
Some PMDs are still using old offloading API related variables, thanks to Andrew
for reporting it.
Maintainers of related PMDs are cc'ed.
Please check following list and provide a fix ASAP for it, thanks.
# git grep header_split drivers/
drivers/net/i40e/i40e_rxtx_vec_common.h: if (rxmode->header_split == 1)
# git grep hw_vlan_filter drivers/
drivers/net/i40e/i40e_vf_representor.c: if
(ethdev->data->dev_conf.rxmode.hw_vlan_filter)
drivers/net/vhost/rte_eth_vhost.c: if (rxmode->hw_vlan_filter)
# git grep hw_vlan_strip drivers/
drivers/net/dpaa2/dpaa2_rxtx.c: if
(dev->data->dev_conf.rxmode.hw_vlan_strip)
drivers/net/i40e/i40e_vf_representor.c: if
(ethdev->data->dev_conf.rxmode.hw_vlan_strip)T
drivers/net/vhost/rte_eth_vhost.c: internal->vlan_strip =
rxmode->hw_vlan_strip;
# git grep hw_vlan_extend drivers/
drivers/net/i40e/i40e_rxtx_vec_common.h: if (rxmode->hw_vlan_extend == 1)
# git grep hw_strip_crc drivers/
drivers/net/nfp/nfp_net.c: rxq->crc_len = (uint8_t)
((dev->data->dev_conf.rxmode.hw_strip_crc) ? 0
drivers/net/sfc/sfc_rx.c: rxmode->hw_strip_crc = 1;
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] Remaining old offloading API usage in PMDs
2018-05-11 10:45 [dpdk-dev] Remaining old offloading API usage in PMDs Ferruh Yigit
@ 2018-05-14 11:01 ` Shreyansh Jain
2018-05-21 12:38 ` Ferruh Yigit
2018-05-22 14:12 ` Ferruh Yigit
2 siblings, 0 replies; 6+ messages in thread
From: Shreyansh Jain @ 2018-05-14 11:01 UTC (permalink / raw)
To: Ferruh Yigit
Cc: Thomas Monjalon, Andrew Rybchenko, Tetsuya Mukawa,
Maxime Coquelin, Hemant Agrawal, Alejandro Lucero, dpdk-dev,
Beilei Xing, Qi Zhang
On Friday 11 May 2018 04:15 PM, Ferruh Yigit wrote:
> Some PMDs are still using old offloading API related variables, thanks to Andrew
> for reporting it.
>
> Maintainers of related PMDs are cc'ed.
> Please check following list and provide a fix ASAP for it, thanks.
>
>
> # git grep header_split drivers/
> drivers/net/i40e/i40e_rxtx_vec_common.h: if (rxmode->header_split == 1)
>
> # git grep hw_vlan_filter drivers/
> drivers/net/i40e/i40e_vf_representor.c: if
> (ethdev->data->dev_conf.rxmode.hw_vlan_filter)
> drivers/net/vhost/rte_eth_vhost.c: if (rxmode->hw_vlan_filter)
>
> # git grep hw_vlan_strip drivers/
> drivers/net/dpaa2/dpaa2_rxtx.c: if
> (dev->data->dev_conf.rxmode.hw_vlan_strip)
For DPAA2, sent patch [1]
[1] http://dpdk.org/dev/patchwork/patch/39994/
Thanks for highlighting this miss.
[...]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] Remaining old offloading API usage in PMDs
2018-05-11 10:45 [dpdk-dev] Remaining old offloading API usage in PMDs Ferruh Yigit
2018-05-14 11:01 ` Shreyansh Jain
@ 2018-05-21 12:38 ` Ferruh Yigit
2018-05-21 14:16 ` Alejandro Lucero
2018-05-22 14:12 ` Ferruh Yigit
2 siblings, 1 reply; 6+ messages in thread
From: Ferruh Yigit @ 2018-05-21 12:38 UTC (permalink / raw)
To: Thomas Monjalon, Andrew Rybchenko, Tetsuya Mukawa,
Maxime Coquelin, Hemant Agrawal, Shreyansh Jain,
Alejandro Lucero
Cc: dpdk-dev, Beilei Xing, Qi Zhang
On 5/11/2018 11:45 AM, Ferruh Yigit wrote:
> Some PMDs are still using old offloading API related variables, thanks to Andrew
> for reporting it.
>
> Maintainers of related PMDs are cc'ed.
> Please check following list and provide a fix ASAP for it, thanks.
>
>
> # git grep header_split drivers/
> drivers/net/i40e/i40e_rxtx_vec_common.h: if (rxmode->header_split == 1)
>
> # git grep hw_vlan_filter drivers/
> drivers/net/i40e/i40e_vf_representor.c: if
> (ethdev->data->dev_conf.rxmode.hw_vlan_filter)
> drivers/net/vhost/rte_eth_vhost.c: if (rxmode->hw_vlan_filter)
>
> # git grep hw_vlan_strip drivers/
> drivers/net/dpaa2/dpaa2_rxtx.c: if
> (dev->data->dev_conf.rxmode.hw_vlan_strip)
> drivers/net/i40e/i40e_vf_representor.c: if
> (ethdev->data->dev_conf.rxmode.hw_vlan_strip)T
> drivers/net/vhost/rte_eth_vhost.c: internal->vlan_strip =
> rxmode->hw_vlan_strip;
>
> # git grep hw_vlan_extend drivers/
> drivers/net/i40e/i40e_rxtx_vec_common.h: if (rxmode->hw_vlan_extend == 1)
>
> # git grep hw_strip_crc drivers/
> drivers/net/nfp/nfp_net.c: rxq->crc_len = (uint8_t)
> ((dev->data->dev_conf.rxmode.hw_strip_crc) ? 0
> drivers/net/sfc/sfc_rx.c: rxmode->hw_strip_crc = 1;
>
Reminder of this one.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] Remaining old offloading API usage in PMDs
2018-05-21 12:38 ` Ferruh Yigit
@ 2018-05-21 14:16 ` Alejandro Lucero
2018-05-21 14:33 ` Ferruh Yigit
0 siblings, 1 reply; 6+ messages in thread
From: Alejandro Lucero @ 2018-05-21 14:16 UTC (permalink / raw)
To: Ferruh Yigit
Cc: Thomas Monjalon, Andrew Rybchenko, Tetsuya Mukawa,
Maxime Coquelin, Hemant Agrawal, Shreyansh Jain, dpdk-dev,
Beilei Xing, Qi Zhang
I completely missed this last bit in the NFP driver.
I will send a patch today.
By the way, will it go into 18.05? I ask because I have another patch
fixing a problem which would be good to have in 18.05 as well.
On Mon, May 21, 2018 at 1:38 PM, Ferruh Yigit <ferruh.yigit@intel.com>
wrote:
> On 5/11/2018 11:45 AM, Ferruh Yigit wrote:
> > Some PMDs are still using old offloading API related variables, thanks
> to Andrew
> > for reporting it.
> >
> > Maintainers of related PMDs are cc'ed.
> > Please check following list and provide a fix ASAP for it, thanks.
> >
> >
> > # git grep header_split drivers/
> > drivers/net/i40e/i40e_rxtx_vec_common.h: if
> (rxmode->header_split == 1)
> >
> > # git grep hw_vlan_filter drivers/
> > drivers/net/i40e/i40e_vf_representor.c: if
> > (ethdev->data->dev_conf.rxmode.hw_vlan_filter)
> > drivers/net/vhost/rte_eth_vhost.c: if (rxmode->hw_vlan_filter)
> >
> > # git grep hw_vlan_strip drivers/
> > drivers/net/dpaa2/dpaa2_rxtx.c: if
> > (dev->data->dev_conf.rxmode.hw_vlan_strip)
> > drivers/net/i40e/i40e_vf_representor.c: if
> > (ethdev->data->dev_conf.rxmode.hw_vlan_strip)T
> > drivers/net/vhost/rte_eth_vhost.c: internal->vlan_strip =
> > rxmode->hw_vlan_strip;
> >
> > # git grep hw_vlan_extend drivers/
> > drivers/net/i40e/i40e_rxtx_vec_common.h: if
> (rxmode->hw_vlan_extend == 1)
> >
> > # git grep hw_strip_crc drivers/
> > drivers/net/nfp/nfp_net.c: rxq->crc_len = (uint8_t)
> > ((dev->data->dev_conf.rxmode.hw_strip_crc) ? 0
> > drivers/net/sfc/sfc_rx.c: rxmode->hw_strip_crc = 1;
> >
>
> Reminder of this one.
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] Remaining old offloading API usage in PMDs
2018-05-21 14:16 ` Alejandro Lucero
@ 2018-05-21 14:33 ` Ferruh Yigit
0 siblings, 0 replies; 6+ messages in thread
From: Ferruh Yigit @ 2018-05-21 14:33 UTC (permalink / raw)
To: Alejandro Lucero
Cc: Thomas Monjalon, Andrew Rybchenko, Tetsuya Mukawa,
Maxime Coquelin, Hemant Agrawal, Shreyansh Jain, dpdk-dev,
Beilei Xing, Qi Zhang
On 5/21/2018 3:16 PM, Alejandro Lucero wrote:
> I completely missed this last bit in the NFP driver.
>
> I will send a patch today.
>
> By the way, will it go into 18.05? I ask because I have another patch fixing a
> problem which would be good to have in 18.05 as well.
Release is still open for fixes, specially for PMDs since their scope is
limited. After rc5 plan is to get only critical fixes.
>
> On Mon, May 21, 2018 at 1:38 PM, Ferruh Yigit <ferruh.yigit@intel.com
> <mailto:ferruh.yigit@intel.com>> wrote:
>
> On 5/11/2018 11:45 AM, Ferruh Yigit wrote:
> > Some PMDs are still using old offloading API related variables, thanks to
> Andrew
> > for reporting it.
> >
> > Maintainers of related PMDs are cc'ed.
> > Please check following list and provide a fix ASAP for it, thanks.
> >
> >
> > # git grep header_split drivers/
> > drivers/net/i40e/i40e_rxtx_vec_common.h: if (rxmode->header_split == 1)
> >
> > # git grep hw_vlan_filter drivers/
> > drivers/net/i40e/i40e_vf_representor.c: if
> > (ethdev->data->dev_conf.rxmode.hw_vlan_filter)
> > drivers/net/vhost/rte_eth_vhost.c: if (rxmode->hw_vlan_filter)
> >
> > # git grep hw_vlan_strip drivers/
> > drivers/net/dpaa2/dpaa2_rxtx.c: if
> > (dev->data->dev_conf.rxmode.hw_vlan_strip)
> > drivers/net/i40e/i40e_vf_representor.c: if
> > (ethdev->data->dev_conf.rxmode.hw_vlan_strip)T
> > drivers/net/vhost/rte_eth_vhost.c: internal->vlan_strip =
> > rxmode->hw_vlan_strip;
> >
> > # git grep hw_vlan_extend drivers/
> > drivers/net/i40e/i40e_rxtx_vec_common.h: if (rxmode->hw_vlan_extend
> == 1)
> >
> > # git grep hw_strip_crc drivers/
> > drivers/net/nfp/nfp_net.c: rxq->crc_len = (uint8_t)
> > ((dev->data->dev_conf.rxmode.hw_strip_crc) ? 0
> > drivers/net/sfc/sfc_rx.c: rxmode->hw_strip_crc = 1;
> >
>
> Reminder of this one.
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] Remaining old offloading API usage in PMDs
2018-05-11 10:45 [dpdk-dev] Remaining old offloading API usage in PMDs Ferruh Yigit
2018-05-14 11:01 ` Shreyansh Jain
2018-05-21 12:38 ` Ferruh Yigit
@ 2018-05-22 14:12 ` Ferruh Yigit
2 siblings, 0 replies; 6+ messages in thread
From: Ferruh Yigit @ 2018-05-22 14:12 UTC (permalink / raw)
To: Thomas Monjalon, Alejandro Lucero, Maciej Czekaj, Wenzhuo Lu,
Konstantin Ananyev, Qi Zhang, Xiao Wang, Santosh Shukla,
Jerin Jacob, Shijith Thotton, Srisivasubramanian Srinivasan
Cc: Andrew Rybchenko, Hemant Agrawal, Shreyansh Jain, dpdk-dev,
Beilei Xing, Tetsuya Mukawa, Maxime Coquelin
On 5/11/2018 11:45 AM, Ferruh Yigit wrote:
> Some PMDs are still using old offloading API related variables, thanks to Andrew
> for reporting it.
>
> Maintainers of related PMDs are cc'ed.
> Please check following list and provide a fix ASAP for it, thanks.
>
>
> # git grep header_split drivers/
> drivers/net/i40e/i40e_rxtx_vec_common.h: if (rxmode->header_split == 1)
>
> # git grep hw_vlan_filter drivers/
> drivers/net/i40e/i40e_vf_representor.c: if
> (ethdev->data->dev_conf.rxmode.hw_vlan_filter)
> drivers/net/vhost/rte_eth_vhost.c: if (rxmode->hw_vlan_filter)
>
> # git grep hw_vlan_strip drivers/
> drivers/net/dpaa2/dpaa2_rxtx.c: if
> (dev->data->dev_conf.rxmode.hw_vlan_strip)
> drivers/net/i40e/i40e_vf_representor.c: if
> (ethdev->data->dev_conf.rxmode.hw_vlan_strip)T
> drivers/net/vhost/rte_eth_vhost.c: internal->vlan_strip =
> rxmode->hw_vlan_strip;
>
> # git grep hw_vlan_extend drivers/
> drivers/net/i40e/i40e_rxtx_vec_common.h: if (rxmode->hw_vlan_extend == 1)
>
> # git grep hw_strip_crc drivers/
> drivers/net/nfp/nfp_net.c: rxq->crc_len = (uint8_t)
> ((dev->data->dev_conf.rxmode.hw_strip_crc) ? 0
> drivers/net/sfc/sfc_rx.c: rxmode->hw_strip_crc = 1;
>
Code (almost [1]) cleaned up from old API usage, thanks everybody.
But driver documentation still has some pointers from old API usage, Andrew also
reported some.
Related maintainers are cc'ed, can you please check and fix following usages:
doc/guides/prog_guide/poll_mode_drv.rst
331:For an application to use the Tx offloads API it should set the
``ETH_TXQ_FLAGS_IGNORE`` flag in the ``txq_flags`` field located in
``rte_eth_txconf`` struct.
332:In such cases it is not required to set other flags in ``txq_flags``.
doc/guides/nics/virtio.rst
237:* ``txq_flags`` is set to ``VIRTIO_SIMPLE_FLAGS`` (0xF01), which implies:
doc/guides/nics/ixgbe.rst
105:TX vPMD only works when txq_flags is set to IXGBE_SIMPLE_FLAGS.
110:* ETH_TXQ_FLAGS_NOMULTSEGS
112:* ETH_TXQ_FLAGS_NOVLANOFFL
114:* ETH_TXQ_FLAGS_NOXSUMSCTP
116:* ETH_TXQ_FLAGS_NOXSUMUDP
118:* ETH_TXQ_FLAGS_NOXSUMTCP
doc/guides/nics/fm10k.rst
109:TX vPMD only works when ``txq_flags`` is set to ``FM10K_SIMPLE_TX_FLAG``.
113:* ``ETH_TXQ_FLAGS_NOMULTSEGS``
115:* ``ETH_TXQ_FLAGS_NOVLANOFFL``
117:* ``ETH_TXQ_FLAGS_NOXSUMSCTP``
119:* ``ETH_TXQ_FLAGS_NOXSUMUDP``
121:* ``ETH_TXQ_FLAGS_NOXSUMTCP``
doc/guides/nics/thunderx.rst
323:``rxmode.hw_strip_crc`` member is set to 0 in ``struct rte_eth_conf``.
329:is fixed and cannot be changed. So, even when the ``rxmode.max_rx_pkt_len``
doc/guides/nics/ixgbe.rst
71:To guarantee the constraint, configuration flags in dev_conf.rxmode will be
checked:
133:In the configuration, ensure that port_conf.rxmode.hw_ip_checksum=0.
139:As in the case of l3fwd, set configure port_conf.rxmode.hw_ip_checksum=0 to
enable vPMD.
doc/guides/nics/fm10k.rst
82:To guarantee the constraint, the following configuration flags in
``dev_conf.rxmode``
153:``rxmode.hw_strip_crc`` member is set to 0 in ``struct rte_eth_conf``.
160:is fixed and cannot be changed. So, even when the ``rxmode.max_rx_pkt_len``
doc/guides/nics/octeontx.rst
169:``rxmode.hw_strip_crc`` member is set to 0 in ``struct rte_eth_conf``.
175:is fixed and cannot be changed. So, even when the ``rxmode.max_rx_pkt_len``
doc/guides/nics/liquidio.rst
205:interface. So, CRC will be stripped even when the ``rxmode.hw_strip_crc``
[1]
Waiting nfp.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-05-22 14:12 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-11 10:45 [dpdk-dev] Remaining old offloading API usage in PMDs Ferruh Yigit
2018-05-14 11:01 ` Shreyansh Jain
2018-05-21 12:38 ` Ferruh Yigit
2018-05-21 14:16 ` Alejandro Lucero
2018-05-21 14:33 ` Ferruh Yigit
2018-05-22 14:12 ` Ferruh Yigit
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).