mbox series

[v3,0/2] bbdev: dump debug information

Message ID 20240913231811.969965-1-nicolas.chautru@intel.com (mailing list archive)
Headers
Series bbdev: dump debug information |

Message

Nicolas Chautru Sept. 13, 2024, 11:18 p.m. UTC
v3: updated based on Maxime comments related to passing string
through function prototype. Thanks

v2: updated with comments from Hemant and rebased.

v1: Hi Maxime. 

Adding new support to support troubleshooting. This provides
to the application an API to dump into file information
to help troubleshoot issue on a queue. Some of it is tracked
at bbdev level and some extra information can be tracked
as an option at PMD level. This is for 24.11. 
In practive logging is not enabled at run time, but information
can be dump into file when an issue happens and requires extra
information. This collates now multiple source of errors detected
notably at driver level made more explicit.

Thanks,
Nic

Nicolas Chautru (2):
  bbdev: add new function to dump debug information
  baseband/acc: improvement to logging mechanism

 drivers/baseband/acc/acc_common.h  |  38 +++++
 drivers/baseband/acc/rte_vrb_pmd.c | 129 +++++++++++++----
 lib/bbdev/rte_bbdev.c              | 213 +++++++++++++++++++++++++++++
 lib/bbdev/rte_bbdev.h              |  47 +++++++
 lib/bbdev/rte_bbdev_pmd.h          |   9 ++
 lib/bbdev/version.map              |   4 +
 6 files changed, 413 insertions(+), 27 deletions(-)
  

Comments

Hemant Agrawal Sept. 20, 2024, 6:50 a.m. UTC | #1
Hi Nicolas,

     I have ack your patches.

  One suggestion, if you can use "--in-reply-to=" in you patch series, 
it will be helpful. This will link your new series with old series.


Regards

Hemant



On 14-09-2024 04:48, Nicolas Chautru wrote:
> v3: updated based on Maxime comments related to passing string
> through function prototype. Thanks
>
> v2: updated with comments from Hemant and rebased.
>
> v1: Hi Maxime.
>
> Adding new support to support troubleshooting. This provides
> to the application an API to dump into file information
> to help troubleshoot issue on a queue. Some of it is tracked
> at bbdev level and some extra information can be tracked
> as an option at PMD level. This is for 24.11.
> In practive logging is not enabled at run time, but information
> can be dump into file when an issue happens and requires extra
> information. This collates now multiple source of errors detected
> notably at driver level made more explicit.
>
> Thanks,
> Nic
>
> Nicolas Chautru (2):
>    bbdev: add new function to dump debug information
>    baseband/acc: improvement to logging mechanism
>
>   drivers/baseband/acc/acc_common.h  |  38 +++++
>   drivers/baseband/acc/rte_vrb_pmd.c | 129 +++++++++++++----
>   lib/bbdev/rte_bbdev.c              | 213 +++++++++++++++++++++++++++++
>   lib/bbdev/rte_bbdev.h              |  47 +++++++
>   lib/bbdev/rte_bbdev_pmd.h          |   9 ++
>   lib/bbdev/version.map              |   4 +
>   6 files changed, 413 insertions(+), 27 deletions(-)
>