[xdp-hints] Re: Interest in TX-side XDP metadata hints from Intel (+ Red Hat)

["Song, Yoong Siang" <yoong.siang.song@intel.com>]

On Saturday, September 9, 2023 12:38 AM Stanislav Fomichev <sdf@google.com> wrote:
>On Fri, Sep 8, 2023 at 7:18 AM Jesper Dangaard Brouer <hawk@kernel.org>
>wrote:
>>
>> Hi Stanislav,
>>
>> Sharing that Intel (+Red Hat) have an interest in getting XDP metadata
>> hints working on smaller devices used for "Edge Controls for Industrial"
>> (ECI)[1].
>>
>> What are your plans for your patchset[2] ?
>
>Sorry, I was OOO for two weeks and then got distracted by some internal work :-
>( I'll probably post another respin next week. I think I've addressed most of the
>Maciej's feedback, but the only thing that we need to move this forward is to
>support another device besides mlx5. Maciej was suggesting ice, but I don't have
>the hw :-( I have some idfp-capable devices, so I'll probably try to play with it
>instead (but the idpf driver is still not pulled).
>
>So if you want to help speed it up, we should probably focus first on getting some
>of the intel nics support tx metadata.
>To proceed, maybe I should post my next iteration and we can collaborate on the
>ice driver changes?
>Who should I CC explicitly?
>
Hi Stanislav,

I can help to speed it up. I don't have ice HW on my hand as well, so maybe we can start 
with igc drivers, followed by stmmac drivers.

Please post your next iteration, then we can collaborate based on that.

Thanks & Regards
Siang

>> My suggestion (to Intel) is that we implement the TX-timestamp (plus
>> LaunchTime) on top of Stanslav's patches[2].  For drivers
>> igc+igb+stmmac which I think will cover the tests[3] need by
>> Arun/Costin (?). And then post it upstream for review.
>>
>> The Intel test tool[3] isn't open, but it is an AF_XDP based Real-Time
>> Compute Performance (RTCP) application that show-case a real-time
>> control loop application.  The current code uses out-of-tree[4] "TXTIME"
>> LaunchTime feature, that want not upstreamed.
>>
>> Lets us instead implement a proper TX LaunchTime feature on top of [2].
>>
>> --Jesper
>> cc. XDP-hints ML as anyone is free to code this up, code help here[5]
>>
>>
>> [1] https://eci.intel.com/
>> [2]
>> https://lore.kernel.org/all/20230809165418.2831456-1-sdf@google.com/
>> [3] https://github.com/OTCShare2/rtcp-xdp
>> [4] https://lore.kernel.org/all/20210621230635.3a83851c@carbon/
>> [5]
>> https://github.com/xdp-project/xdp-project/blob/master/areas/tsn/code0
>> 1_follow_qdisc_TSN_offload.org#tx-time-to-hardware-driver-igc