2023-07-09 21:19:57

by Thomas Weißschuh

Subject: [PATCH v2 2/3] dyndbg: increase PREFIX_SIZE to 128

A follow-up patch will add the possibility to print the filename as part
of the prefix.
Increase the maximum prefix size to accommodate this.

Signed-off-by: Thomas Weißschuh <[email protected]>
---
lib/dynamic_debug.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c
index 71b22d206a1b..166229dfe171 100644
--- a/lib/dynamic_debug.c
+++ b/lib/dynamic_debug.c
@@ -808,7 +808,7 @@ const struct kernel_param_ops param_ops_dyndbg_classes = {
};
EXPORT_SYMBOL(param_ops_dyndbg_classes);

-#define PREFIX_SIZE 64
+#define PREFIX_SIZE 128

static int remaining(int wrote)
{

--
2.41.0

2023-07-10 19:36:02

by Luis Chamberlain

Subject: Re: [PATCH v2 2/3] dyndbg: increase PREFIX_SIZE to 128

On Sun, Jul 09, 2023 at 11:17:59PM +0200, Thomas Wei?schuh wrote:
> A follow-up patch will add the possibility to print the filename as part
> of the prefix.
> Increase the maximum prefix size to accommodate this.
>
> Signed-off-by: Thomas Wei?schuh <[email protected]>

Reviewed-by: Luis Chamberlain <[email protected]>

Luis