[dpdk-dev] [PATCH 0/6] fix port id type

[Ferruh Yigit]

On 10/13/2017 3:04 AM, Yang, Zhiyong wrote:
> Hi Ferruh,
> 
> The patch (ethdev: increase port_id range) tend to increase port id range in ethdev, not eventdev, 
> Do you mean we should change eventdev port id ? 
> 
> Before V2, that patchset includes eventdev port id extension. 
> 
> but Bruce reminded me about it in that patchset v2 thread and suggest to send a
> separate patch/patchset to fix it if required. So, I didn’t touch the event related files after v2.
> 
> If required, I can send eventdev port id range patch soon.

Sorry I just copy-paste the command output without parsing.

No we are not interested in eventdev ones, please ignore them, but in
the list there can be still ethdev related ones, you will need to check
them first.

Thanks,
ferruh

> 
> FYI.
> Discussion  in V2 patch
> 
>>>> Fix some checkpatch issues from the original code and remove some
>>>> unnecessary cast operations.
>>>>
>>>> Signed-off-by: Zhiyong Yang <zhiyong.yang at intel.com>
>>>> ---
>>>>  app/pdump/main.c                                  |   2 +-
>>>>  app/test-eventdev/test_perf_common.h              |   6 +-
>>>>  app/test-pmd/cmdline.c                            |   4 +-
>>>>  app/test-pmd/config.c                             |   4 +-
>>>>  app/test-pmd/testpmd.c                            |  18 +-
>>>>  app/test-pmd/testpmd.h                            |   2 +-
>>>>  drivers/event/octeontx/ssovf_evdev.c              |   6 +-
>>>>  drivers/event/octeontx/ssovf_evdev.h              |   4 +-
>>>>  drivers/event/skeleton/skeleton_eventdev.c        |   4 +-
>>>>  drivers/event/skeleton/skeleton_eventdev.h        |   2 +-
>>>>  drivers/event/sw/sw_evdev.c                       |   4 +-
>>>>  drivers/event/sw/sw_evdev.h                       |   6 +-
>>>>  drivers/event/sw/sw_evdev_xstats.c                |   8 +-
>>>
>>> These are not ethdev drivers, but eventdev ones. I don't think
>>> eventdev needs to go to 16-bit port numbers, but there is no harm in doing so.
>>> However, it would be best to modify eventdev drivers in a separate
>>> patch - or even patchset.
>>>
>>
>> I will remove them and focus on ethdev in the patchset.
> 
> Thanks
> Zhiyong
> 
> 
>> -----Original Message-----
>> From: Yigit, Ferruh
>> Sent: Friday, October 13, 2017 4:13 AM
>> To: Yang, Zhiyong <zhiyong.yang at intel.com>; dev at dpdk.org
>> Subject: Re: [dpdk-dev] [PATCH 0/6] fix port id type
>>
>> On 10/12/2017 7:03 PM, Ferruh Yigit wrote:
>>> On 10/12/2017 10:32 AM, Zhiyong Yang wrote:
>>>> Now ethdev port_id has been increased to uint16_t from uint8_t in the
>>>> below patch http://www.dpdk.org/dev/patchwork/patch/29418/
>>>>
>>>> but
>>>> 1. Some features which are applied were still developed based on
>>>> older version uint8_t port_id.
>>>> 2. Some places using port id such as testpmd are omitted in previous patch.
>>>> The patchset fixes some port id type issues.
>>>>
>>>> Zhiyong Yang (6):
>>>>   lib/librte_ether: fix port id type
>>>>   net/bnxt: fix port id type
>>>>   net/liquidio: fix port id type
>>>>   app/pdump: fix port id type
>>>>   app/proc_info: fix port id type
>>>>   app/testpmd: fix port id type
>>>
>>> Series applied to dpdk-next-net/master, thanks.
>>>
>>>
>>> Thanks for the patches!
>>> But I can see there are still more uint8_t usage [1], some looks valid.
>>>
>>> I aware this looks like never it will never end but would you mind
>>> doing one more go? It would be nice to get all fixed before rc1.
>>> And I hope we won't need to change port_id storage type again...
>>>
>>>
>>> [1]
>>> $ git grep -c "uint8_t port_id"
>>
>> And (git grep "uint8_t port") yields even more result.
>>
>>> app/test-eventdev/test_order_common.h:2
>>> app/test-eventdev/test_perf_common.h:2
>>> doc/guides/prog_guide/eventdev.rst:1
>>> doc/guides/prog_guide/poll_mode_drv.rst:1
>>> doc/guides/prog_guide/rte_flow.rst:6
>>> doc/guides/sample_app_ug/kernel_nic_interface.rst:4
>>> doc/guides/sample_app_ug/link_status_intr.rst:1
>>> drivers/event/dpaa2/dpaa2_eventdev.c:2
>>> drivers/event/octeontx/ssovf_evdev.c:2
>>> drivers/event/skeleton/skeleton_eventdev.c:2
>>> drivers/event/skeleton/skeleton_eventdev.h:1
>>> drivers/event/sw/sw_evdev.c:2
>>> drivers/event/sw/sw_evdev_xstats.c:1
>>> drivers/net/bonding/rte_eth_bond_pmd.c:1
>>> drivers/net/i40e/i40e_ethdev.c:1
>>> examples/ethtool/lib/rte_ethtool.c:21
>>> examples/ethtool/lib/rte_ethtool.h:21
>>> examples/eventdev_pipeline_sw_pmd/main.c:6
>>> examples/ipsec-secgw/ipsec-secgw.c:2
>>> examples/l2fwd-jobstats/main.c:1
>>> examples/quota_watermark/qw/init.c:2
>>> examples/quota_watermark/qw/init.h:2
>>> examples/quota_watermark/qw/main.c:1
>>> examples/server_node_efd/node/node.c:2
>>> examples/tep_termination/vxlan_setup.c:1
>>> examples/tep_termination/vxlan_setup.h:2
>>> lib/librte_eventdev/rte_event_eth_rx_adapter.c:1
>>> lib/librte_eventdev/rte_eventdev.c:7
>>> lib/librte_eventdev/rte_eventdev.h:11
>>> lib/librte_eventdev/rte_eventdev_pmd.h:2
>>> test/test/test_link_bonding.c:1
>>> test/test/test_link_bonding_mode4.c:2
>>> test/test/test_link_bonding_rssconf.c:3
>>>
>