mbox

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

Message ID 20230303155811.2751210-1-adwivedi@marvell.com (mailing list archive)
Headers

Message

Ankur Dwivedi March 3, 2023, 3: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 ethdev 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 ethdev library.

v4:
 - Rebased on the recent next-net branch.
 - Refined logic to find function definition.
 - Updated year in the license in devtools/check-tracepoint.sh.
 - Removed cryptodev, added ethdev in libdir in
   devtools/check-tracepoint.sh. 

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    |  78 +++++++++++++++++
 devtools/check-symbol-change.sh |  76 +----------------
 devtools/check-tracepoint.sh    | 146 ++++++++++++++++++++++++++++++++
 devtools/checkpatches.sh        |   9 ++
 devtools/trace-skiplist.txt     |   0
 5 files changed, 235 insertions(+), 74 deletions(-)
 create mode 100755 devtools/build-symbol-map.sh
 create mode 100755 devtools/check-tracepoint.sh
 create mode 100644 devtools/trace-skiplist.txt