On 2/1/2024 10:02 AM, Harman Kalra wrote: > Following series adds support to enable creation of multiple representors > under one base device. There may be scenarios where port representors for > multiple PFs or VFs under PF are required and all these representor ports > created under a single pci device. Marvell CNXK port representor solution > is designed around this scenario where all representors are backed by a > single switch device. > > Earlier this change was implemented as part of the Marvell CNXK port > representor series but after suggestions from Thomas we would like > to propose these changes in common code. > https://patches.dpdk.org/project/dpdk/patch/20231219174003.72901-25-hka...@marvell.com/#166785 > > V5: > - Added test cases to demonstrate valid and invalid cases > - changed the tokenizing logic to address all valid cases > > V4: > - Used MT safe strtok_r in place of strtok > - Reworded some comments > > V3: > - Fix duplicate representor devarg key handling logic > > V2: > - Updated the multiple representor devarg pattern to list > i.e. representor=[pf[0-1],pf2vf[1,2-3],[4-5]] > - Introduced size of array as third argument to rte_eth_devargs_parse() > to avoid array corruption > - Squashed separate document patch > > > Harman Kalra (2): > ethdev: parsing multiple representor devargs string > test/devargs: add eth devargs parse cases >
Reviewed-by: Ferruh Yigit <ferruh.yi...@amd.com> Squashed patches while merging, Series applied to dpdk-next-net/main, thanks.
