RE: [PATCH v3 0/2] devtools: add tracepoint check in checkpatch

[Ankur Dwivedi <adwivedi@marvell.com>]

Hi Thomas,

Please let me know if this series can be included in 22.11 rc3.

Regards,
Ankur

>-----Original Message-----
>From: Ankur Dwivedi <adwivedi@marvell.com>
>Sent: Saturday, October 15, 2022 6:28 PM
>To: dev@dpdk.org
>Cc: thomas@monjalon.net; david.marchand@redhat.com; Akhil Goyal
><gakhil@marvell.com>; royzhang1980@gmail.com; Amit Prakash Shukla
><amitprakashs@marvell.com>; Jerin Jacob Kollanukkaran
><jerinj@marvell.com>; Ankur Dwivedi <adwivedi@marvell.com>
>Subject: [PATCH v3 0/2] devtools: add tracepoint check in checkpatch
>
>This patch series adds a validation in checkpatch tool to check if tracepoint is
>present in any new function added in cryptodev library.
>
>The first patch in the series moves build_map_changes function from check-
>symbol-change.sh to a new file build-symbol-map.sh.
>
>The second patch in the series adds a new script file check-tracepoint.sh which
>is called from checkpatch tool. The check-tracepoint.sh contains the code to
>detect the presence of tracepoint in a new function added to cryptodev library.
>
>v3:
> - Split the v2 patch into 2 patches.
> - The file common-func.sh is renamed to build-symbol-map.sh.
> - Removed check-tracepoint.py file.
> - Code improvements in check-tracepoint.sh.
>
>v2:
> - Add check for parent directory.
>
>Ankur Dwivedi (2):
>  devtools: move build symbol map function
>  devtools: add tracepoint check in checkpatch
>
> devtools/build-symbol-map.sh    |  77 +++++++++++++++++++
> devtools/check-symbol-change.sh |  76 +------------------
> devtools/check-tracepoint.sh    | 129 ++++++++++++++++++++++++++++++++
> devtools/checkpatches.sh        |   9 +++
> devtools/trace-skiplist.txt     |   0
> 5 files changed, 217 insertions(+), 74 deletions(-)  create mode 100644
>devtools/build-symbol-map.sh  create mode 100755 devtools/check-
>tracepoint.sh  create mode 100644 devtools/trace-skiplist.txt
>
>--
>2.28.0