[v2] devtools/cocci: added script for ethdev namespace

Message ID 20211122175322.88911-1-aman.deep.singh@intel.com (mailing list archive)
State Rejected, archived
Headers
Series [v2] devtools/cocci: added script for ethdev namespace |

Checks

Context Check Description
ci/checkpatch warning coding style issues
ci/Intel-compilation warning apply issues
ci/iol-testing warning apply patch failure

Commit Message

Singh, Aman Deep Nov. 22, 2021, 5:53 p.m. UTC
  The cocci script is to help add prefix 'RTE_ETH' namespace to enum
& macro of ethdev library. It helps in automating these changes for
applications. The script won't make changes in the code comment part.
Usage: spatch <script path> <app path>

Signed-off-by: Aman Singh <aman.deep.singh@intel.com>

---
v2:
* Added support for struct changes
* Removed backward compatibility code gen part

 devtools/cocci/namespace_ethdev.cocci | 34 ++++++++++++---------------
 1 file changed, 15 insertions(+), 19 deletions(-)
  

Patch

diff --git a/devtools/cocci/namespace_ethdev.cocci b/devtools/cocci/namespace_ethdev.cocci
index 383db454f0..8bb189da5b 100644
--- a/devtools/cocci/namespace_ethdev.cocci
+++ b/devtools/cocci/namespace_ethdev.cocci
@@ -1,7 +1,7 @@ 
 @rule1@
-identifier I =~  "^(RTE_FC_|ETH_MQ_|ETH_RSS_|DEV_RX_|DEV_TX_|ETH_LINK_|RTE_RETA|
-ETH_SPEED|RTE_TUNNEL|ETH_VLAN|ETH_4|ETH_8|ETH_16|ETH_32|ETH_64|RTE_FDIR|RTE_L2|
-ETH_DCB|ETH_MIRROR|ETH_VMDQ|ETH_NUM|ETH_QINQ|rte_fdir)";
+identifier I =~  "^(RTE_FC_|ETH_MQ_|ETH_RSS|DEV_RX_|DEV_TX_|ETH_LINK|RTE_RETA|
+|ETH_DCB|RTE_TUNNEL|ETH_VLAN|ETH_4|ETH_8|ETH_16|ETH_32|ETH_64|RTE_FDIR|RTE_L2|
+|ETH_SPEED|ETH_MIRROR|ETH_VMDQ|ETH_NUM|ETH_QINQ|ETH_MAX_)";
 @@
 I
 
@@ -10,11 +10,10 @@  I << rule1.I;
 J;
 @@
 coccinelle .J="RTE_ETH_" + I[4:];
-if I.isupper() == False:
-	coccinelle .J="rte_eth_" + I[4:];
 
-exception_matches = ["ETH_RSS_MODE","ETH_VLAN_FILTER_ANY","ETH_VLAN_FILTER_SPEC",
-"ETH_VLAN_FILTER_CLASSIFY","ETH_RSS_UPDATE","RTE_FDIR_MODE"]
+exception_matches = ["ETH_VLAN_FILTER_CLASSIFY","ETH_VLAN_FILTER_ANY",
+"ETH_VLAN_FILTER_SPEC","ETH_RSS_MODE","ETH_RSS_UPDATE","RTE_FDIR_MODE",
+"RTE_FDIR_NO","RTE_FDIR_REPORT"]
 
 if any(x in I for x in exception_matches):
         coccinelle .J= I;
@@ -26,25 +25,22 @@  identifier p.J;
 - I
 + J
 
+
+// Below rule for structures only
 @rule2@
-identifier A  =~  "^(RTE_FC_|ETH_MQ_|ETH_RSS_|DEV_RX_|DEV_TX_|ETH_LINK_|RTE_RETA|
-ETH_SPEED|RTE_TUNNEL|ETH_VLAN|ETH_4|ETH_8|ETH_16|ETH_32|ETH_64|RTE_FDIR|RTE_L2|
-ETH_DCB|ETH_MIRROR|ETH_VMDQ|ETH_NUM|ETH_QINQ)";
-expression B ;
+identifier A  =~  "rte_fdir_conf|rte_intr_conf";
 @@
-#define A B
+struct A
 
 @ script : python p2@
 A << rule2.A;
-K;
+B;
 @@
-coccinelle .K="RTE_ETH_" + A[4:];
+coccinelle .B="rte_eth_" + A[4:];
 
 @ identifier2@
 identifier rule2.A;
-expression rule2.B;
-identifier p2.K;
+identifier p2.B;
 @@
-- #define A B
-+ #define K B
-+ #define A K
+- struct A
++ struct B