Hi, > -----Original Message----- > From: Thomas Monjalon <[email protected]> > Sent: Monday, April 12, 2021 3:26 PM > To: Tal Shnaiderman <[email protected]>; Raslan Darawsheh > <[email protected]> > Cc: [email protected]; Matan Azrad <[email protected]>; Asaf Penso > <[email protected]>; Wisam Monther <[email protected]>; > [email protected] > Subject: Re: [PATCH] common/mlx5: fix missing __rte_internal tags in > exported functions > > 12/04/2021 13:25, Raslan Darawsheh: > > Removed __ from the commit title to fix wrong headline format issue. > > Removing __ makes it wrong: rte_internal does not exist. > The idea of discouraging the use of underscore is to avoid > naming bits and bytes in the title. > Here it should be: "add missing internal tags to some functions" My suggestion was to make it like this: "fix missing rte internal tags in exported functions"
But, I find your suggestion even better, so I'd change it to that > > > > Several functions introduced in the addition of the Windows support to > > > mlx5 were missing the __rte_internal tag although being exported. > > Kindest regards Raslan Darawsheh
