mbox series

[0/3] Replace compilation time options with log level

Message ID 20181129203112.12222-1-dharmik.thakkar@arm.com (mailing list archive)
Headers
Series Replace compilation time options with log level |

Message

Dharmik Thakkar Nov. 29, 2018, 8:31 p.m. UTC 
  Currently, in test code, for printing extra information,
compilation time options are used. This approach does not compile the
code always. Macro needs to be set for the code to compile.

This patchset replaces compilation time option with log level based
approach, thus enabling compilation of code always. Log levels can be
set using eal parameters.

Dharmik Thakkar (3):
  test/hash: replace macro with log-level approach
  test/efd: enable unit test compilation always
  test/timer: enable unit test compilation always

 test/test/test_efd.c            | 24 ++++++++----------------
 test/test/test_hash.c           |  3 +--
 test/test/test_timer_racecond.c | 23 ++++++++++-------------
 3 files changed, 19 insertions(+), 31 deletions(-)

Comments

Ferruh Yigit Jan. 11, 2019, 4:30 p.m. UTC | #1 
On 11/29/2018 8:31 PM, Dharmik Thakkar wrote:
> Currently, in test code, for printing extra information,
> compilation time options are used. This approach does not compile the
> code always. Macro needs to be set for the code to compile.
> 
> This patchset replaces compilation time option with log level based
> approach, thus enabling compilation of code always. Log levels can be
> set using eal parameters.
> 
> Dharmik Thakkar (3):
>   test/hash: replace macro with log-level approach
>   test/efd: enable unit test compilation always
>   test/timer: enable unit test compilation always

For series,
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>