Hi Stefano,
title: Is this the only violation we have in Xen? If so, then please add
the subsystem in the title.
On 11/05/2024 00:37, Stefano Stabellini wrote:
Change opt_conswitch to char to fix a violation of Rule 10.2.
Signed-off-by: Stefano Stabellini <stefano.stabell...@amd.com>
diff --git a/xen/drivers/char/console.c b/xen/drivers/char/console.c
index 2c363d9c1d..3a3a97bcbe 100644
--- a/xen/drivers/char/console.c
+++ b/xen/drivers/char/console.c
@@ -49,7 +49,7 @@ string_param("console", opt_console);
/* Char 1: CTRL+<char1> is used to switch console input between Xen and DOM0
*/
/* Char 2: If this character is 'x', then do not auto-switch to DOM0 when it
*/
/* boots. Any other value, or omitting the char, enables auto-switch
*/
-static unsigned char __read_mostly opt_conswitch[3] = "a";
+static char __read_mostly opt_conswitch[3] = "a";
Looking at the rest of the code, we have:
#define switch_code (opt_conswitch[0] - 'a' + 1)
Can you confirm whether this is not somehow adding a new violation?
Cheers,
--
Julien Grall