From bc5517d44c3c66a8da185ae3087e82b96c4a1c2c Mon Sep 17 00:00:00 2001 From: Konstantin Ushakov Date: Tue, 16 Jun 2026 12:06:37 +0000 Subject: [PATCH] lib/tapi: stop logging jump set/remove at INFO by default tapi_jmp.c defined TE_LOG_LEVEL 0xff -- the only file in the tree with that debug leftover -- forcing every log level on, so each tapi_jmp_push()/tapi_jmp_pop() emitted an INFO "Set"/"Remove jump point" line. Every TAPI_ON_JMP / CHECK_RC (and pyte's PYTE_GUARD) pushes a jump point, so the log filled with a set/remove pair per guarded call -- overwhelming in pyte-based suites. Demote the two per-call traces to VERB and set the file level to TE_LL_INFO, keeping the rare "Jump from" trace (logged only on an actual longjmp) visible. To debug push/pop, raise the level to include TE_LL_VERB. Link: https://github.com/ts-factory/test-environment/pull/15 Tested-by: Konstantin Ushakov Signed-off-by: Konstantin Ushakov --- lib/tapi/tapi_jmp.c | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/lib/tapi/tapi_jmp.c b/lib/tapi/tapi_jmp.c index bf799638d..d59d69f19 100644 --- a/lib/tapi/tapi_jmp.c +++ b/lib/tapi/tapi_jmp.c @@ -8,7 +8,13 @@ * Copyright (C) 2004-2022 OKTET Labs Ltd. All rights reserved. */ -#define TE_LOG_LEVEL 0xff +/* + * Keep INFO for the rare "Jump from ..." trace (logged only on an actual + * longjmp); the per-call set/remove traces below are VERB so they stay off + * by default. Previously 0xff forced every level on, which flooded the log + * with a set/remove pair for every guarded call (heavy in pyte suites). + */ +#define TE_LOG_LEVEL TE_LL_INFO #define TE_LGR_USER "TAPI Jumps" #include "te_config.h" @@ -172,7 +178,7 @@ tapi_jmp_push(const char *file, unsigned int lineno) p->file = file; p->lineno = lineno; - INFO("Set jump point %s:%u", file, lineno); + VERB("Set jump point %s:%u", file, lineno); return p; } @@ -198,7 +204,7 @@ tapi_jmp_pop(const char *file, unsigned int lineno) } SLIST_REMOVE(&ctx->stack, p, tapi_jmp_point, links); - INFO("Remove jump point %s:%u at %s:%u", + VERB("Remove jump point %s:%u at %s:%u", p->file, p->lineno, file, lineno); free(p);