mbox series

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

Message ID 20221015125826.27760-1-adwivedi@marvell.com (mailing list archive)
Headers
Series devtools: add tracepoint check in checkpatch |

Message

Ankur Dwivedi Oct. 15, 2022, 12:58 p.m. UTC 
  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

Comments

Ankur Dwivedi Nov. 2, 2022, 4:08 a.m. UTC | #1 
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