On 04/08/15 20:36, Mike Holmes wrote:


On 8 April 2015 at 13:02, Zoltan Kiss <zoltan.k...@linaro.org <mailto:zoltan.k...@linaro.org>> wrote:

    Hi,

    OVS has a major performance issue with pktio at the moment: pktio
    always does parsing, but OVS does it for itself as well, and it is
    quite deeply woven into its code, so we can't easily modify it to
    use the ODP parsed data. Also, not every platform accelerate that
    (e.g. DPDK), at the moment it would make more sense to make
    parsing optional for pktio, so an application can opt not to do it.
    I can see two options now to define the API:
    - odp_pktio_open get a new bool parameter for this purpose
    - we create a new odp_pktio_enable/disable_parse function pair for
    this purpose

    Any opinions about these options?

I have another option

  * How about an argument to global_init that can change the behavior
    - unless this has to be per port in the same app and I think that
    is less likely.

That flag could even be an arg explicitly in the per platform flags <http://docs.opendataplane.org/linux-generic-doxygen-html/structodp__platform__init__t.html> for global_init and thus only for DPDK right now, this would allow progress and document it is a platform hack.


That should be in some variant. To turn on/off initialization of odp layers like crypto, timers, scheduler...

Maxim.


    Regards,

    Zoli
    _______________________________________________
    lng-odp mailing list
    lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>
    https://lists.linaro.org/mailman/listinfo/lng-odp




--
Mike Holmes
Technical Manager - Linaro Networking Group
Linaro.org <http://www.linaro.org/>***│ *Open source software for ARM SoCs



_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp

_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to