[PATCH] x86/mwait-idle: add missing newline

Roger Pau Monne posted 1 patch 1 year, 10 months ago
Test gitlab-ci failed
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/20220516103601.25671-1-roger.pau@citrix.com
xen/arch/x86/cpu/mwait-idle.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] x86/mwait-idle: add missing newline
Posted by Roger Pau Monne 1 year, 10 months ago
Fixes: 5a211704e8 ('mwait-idle: prevent SKL-H boot failure when C8+C9+C10 enabled')
Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
---
 xen/arch/x86/cpu/mwait-idle.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xen/arch/x86/cpu/mwait-idle.c b/xen/arch/x86/cpu/mwait-idle.c
index 6add64dc5f..5d77672f6b 100644
--- a/xen/arch/x86/cpu/mwait-idle.c
+++ b/xen/arch/x86/cpu/mwait-idle.c
@@ -1363,7 +1363,7 @@ static int cf_check mwait_idle_cpu_init(
 		/* if state marked as disabled, skip it */
 		if (cpuidle_state_table[cstate].flags &
 		    CPUIDLE_FLAG_DISABLED) {
-			printk(XENLOG_DEBUG PREFIX "state %s is disabled",
+			printk(XENLOG_DEBUG PREFIX "state %s is disabled\n",
 			       cpuidle_state_table[cstate].name);
 			continue;
 		}
-- 
2.36.0


Re: [PATCH] x86/mwait-idle: add missing newline
Posted by Jan Beulich 1 year, 10 months ago
On 16.05.2022 12:36, Roger Pau Monne wrote:
> Fixes: 5a211704e8 ('mwait-idle: prevent SKL-H boot failure when C8+C9+C10 enabled')
> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>

Thanks for spotting. I wonder whether we shouldn't mention the Linux
commit (654d08a42a56) which did this as a "side effect". Obviously
the fix here isn't really a port thereof.

Jan

> --- a/xen/arch/x86/cpu/mwait-idle.c
> +++ b/xen/arch/x86/cpu/mwait-idle.c
> @@ -1363,7 +1363,7 @@ static int cf_check mwait_idle_cpu_init(
>  		/* if state marked as disabled, skip it */
>  		if (cpuidle_state_table[cstate].flags &
>  		    CPUIDLE_FLAG_DISABLED) {
> -			printk(XENLOG_DEBUG PREFIX "state %s is disabled",
> +			printk(XENLOG_DEBUG PREFIX "state %s is disabled\n",
>  			       cpuidle_state_table[cstate].name);
>  			continue;
>  		}
Re: [PATCH] x86/mwait-idle: add missing newline
Posted by Andrew Cooper 1 year, 10 months ago
On 16/05/2022 11:36, Roger Pau Monne wrote:
> Fixes: 5a211704e8 ('mwait-idle: prevent SKL-H boot failure when C8+C9+C10 enabled')
> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>

Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>