18/05/2022 18:24, David Marchand пишет:
On Fri, May 13, 2022 at 12:10 PM Min Hu (Connor) <humi...@huawei.com> wrote:

   I think net/bonding offer 'API' for APP to use the bonding.
     and use the specific PMD as slave device.
   The software framwork is like:
    APP
    ethdev
    bonding PMD
    PMD
    hardware

so, I think cmdlines for testpmd should not put in net/bonding.be

Actually, I feel the same.
I do understand the intention, and I do realize it is just location,
but still doesn't look right for me.
can't we have a special sub-folder in testpmd instead?
Something like app/testpmd/driver_specific/(ixgbe)|(i40e)|(bonding)...



Sorry, but the distinction is vague.

Those commands are specific to this driver/library.
I don't see the problem with hosting the commands in the bonding driver/library.

This is still a RFC, I don't mind dropping this patch (in the end) if
others think it does not make sense.
For now I'll keep it in a v2 series fixing the registering issue in patch 1.



Reply via email to