>-----Original Message-----
>From: Bruce Richardson <[email protected]>
>Sent: Thursday, February 10, 2022 5:02 PM
>To: Ankur Dwivedi <[email protected]>
>Cc: [email protected]; Nithin Kumar Dabilpuram <[email protected]>;
>Kiran Kumar Kokkilagadda <[email protected]>; Sunil Kumar Kori
><[email protected]>; Satha Koteswara Rao Kottidi
><[email protected]>; Jerin Jacob Kollanukkaran
><[email protected]>; Anoob Joseph <[email protected]>; Tejasree
>Kondoj <[email protected]>; [email protected]
>Subject: Re: [EXT] Re: [PATCH 0/2] add telemetry for inline IPsec
>
>On Thu, Feb 10, 2022 at 10:04:22AM +0000, Ankur Dwivedi wrote:
>> Hi Bruce,
>>
>> Please see my comment inline.
>>
>> Regards,
>> Ankur
>>
>> >-----Original Message-----
>> >From: Bruce Richardson <[email protected]>
>> >Sent: Thursday, February 10, 2022 2:42 PM
>> >To: Ankur Dwivedi <[email protected]>
>> >Cc: [email protected]; Nithin Kumar Dabilpuram
><[email protected]>;
>> >Kiran Kumar Kokkilagadda <[email protected]>; Sunil Kumar Kori
>> ><[email protected]>; Satha Koteswara Rao Kottidi
>> ><[email protected]>; Jerin Jacob Kollanukkaran
>> ><[email protected]>; Anoob Joseph <[email protected]>; Tejasree
>> >Kondoj <[email protected]>
>> >Subject: [EXT] Re: [PATCH 0/2] add telemetry for inline IPsec
>> >
>> >External Email
>> >
>> >---------------------------------------------------------------------
>> >- On Thu, Feb 10, 2022 at 12:23:32PM +0530, Ankur Dwivedi wrote:
>> >> Adds telemetry function for inline ipsec in net CNXK driver.
>> >>
>> >> This series depends on the following patch series:
>> >>
>> >> Series: Adding new features and improvements in cnxk crypto PMD
>> >> https://urldefense.proofpoint.com/v2/url?u=http-3A__patches.dpdk.or
>> >> g_p
>> >> roject_dpdk_list_-3Fseries-
>> >3D21433&d=DwIBAg&c=nKjWec2b6R0mOyPaz7xtfQ&r
>> >>
>>
>>=ILjiNF3GF25y6QdHZUxMl6JrStU0MIuCtO5dMzn3Ybk&m=szcLmV4s_NiVroE4
>B
>> >vC5yW6
>> >>
>>
>>litxm6Pfo2ymX7QhE1qDSpKTa56lnfmMqfnohRfI6&s=VtKHr9SAo3R0g7lGPoSJl
>g
>> >xVxn
>> >> r2t_TYhHbfH7rS4sE&e=
>> >>
>> >> Ankur Dwivedi (2): net/cnxk: add telemetry for inline IPsec for
>> >> cn9k
>> >> net/cnxk: add telemetry for inline IPsec for cn10k
>> >>
>> >Out of interest, is there any of this telemetry that could be made
>> >generic and apply at a device-class or library level rather than
>> >being tied to the cnxk driver only?
>> I could think of that the callback function registered in
>rte_telemetry_register_cmd() can be a generic one. The net device (eth_dev)
>may have ops registered, which can be called depending on dev->tx_offloads
>and dev->rx_offloads flags. However the data to be returned by telemetry
>might depend on the PMD.
>> Please let me know if this is possible.
>
>Thanks. So if I understand you correctly, the information returned would vary
>based on the offload flags only? If that is the case, it does sound like it
>could
>be generic, where the offload flags field in the return information identify
>what other fields are present in the reply. That would then make the
>telemetry generic and possible to implement at the ethdev level. Is that all
>correct?
Apart from offload flags, the information would vary depending on what a PMD is
willing to provide. That's why I was thinking ops callback function will be
required in
struct eth_dev_ops which will be PMD specific. This ops callback will be called
from
the registered telemetry callback.
