diff mbox series

[v2,17/19] crypto: x86 - standardize suboptimal prints

Message ID 20221012215931.3896-18-elliott@hpe.com
State New
Headers show
Series [RFC,1/7] rcu: correct CONFIG_EXT_RCU_CPU_STALL_TIMEOUT descriptions | expand

Commit Message

Elliott, Robert (Servers) Oct. 12, 2022, 9:59 p.m. UTC
Reword prints that the module is not being loaded (although
it otherwise qualifies) because performance would be suboptimal
on the particular CPU model.

Although modules are not supposed to print unless they're loaded
and active, this is an existing exception.

Signed-off-by: Robert Elliott <elliott@hpe.com>
---
 arch/x86/crypto/blowfish_glue.c     | 5 +----
 arch/x86/crypto/camellia_glue.c     | 5 +----
 arch/x86/crypto/des3_ede_glue.c     | 2 +-
 arch/x86/crypto/twofish_glue_3way.c | 5 +----
 4 files changed, 4 insertions(+), 13 deletions(-)

Comments

Jason A. Donenfeld Oct. 13, 2022, 12:38 a.m. UTC | #1
On Wed, Oct 12, 2022 at 04:59:29PM -0500, Robert Elliott wrote:
> Reword prints that the module is not being loaded (although
> it otherwise qualifies) because performance would be suboptimal
> on the particular CPU model.
> 
> Although modules are not supposed to print unless they're loaded
> and active, this is an existing exception.
> 
> Signed-off-by: Robert Elliott <elliott@hpe.com>
> ---
>  arch/x86/crypto/blowfish_glue.c     | 5 +----
>  arch/x86/crypto/camellia_glue.c     | 5 +----
>  arch/x86/crypto/des3_ede_glue.c     | 2 +-
>  arch/x86/crypto/twofish_glue_3way.c | 5 +----
>  4 files changed, 4 insertions(+), 13 deletions(-)
> 
> diff --git a/arch/x86/crypto/blowfish_glue.c b/arch/x86/crypto/blowfish_glue.c
> index 27b7aed9a488..8d4ecf406dee 100644
> --- a/arch/x86/crypto/blowfish_glue.c
> +++ b/arch/x86/crypto/blowfish_glue.c
> @@ -320,10 +320,7 @@ static int __init blowfish_init(void)
>  		return -ENODEV;
>  
>  	if (!force && is_blacklisted_cpu()) {
> -		printk(KERN_INFO
> -			"blowfish-x86_64: performance on this CPU "
> -			"would be suboptimal: disabling "
> -			"blowfish-x86_64.\n");
> +		pr_info("CPU-optimized crypto module not loaded, crypto optimization performance on this CPU would be suboptimal\n");

This is a comma splice. You need some sort of conjunction.

But also: it's not like the user can do anything about this. Why clutter
dmesg? Seems like we'd be better off without these messages?
diff mbox series

Patch

diff --git a/arch/x86/crypto/blowfish_glue.c b/arch/x86/crypto/blowfish_glue.c
index 27b7aed9a488..8d4ecf406dee 100644
--- a/arch/x86/crypto/blowfish_glue.c
+++ b/arch/x86/crypto/blowfish_glue.c
@@ -320,10 +320,7 @@  static int __init blowfish_init(void)
 		return -ENODEV;
 
 	if (!force && is_blacklisted_cpu()) {
-		printk(KERN_INFO
-			"blowfish-x86_64: performance on this CPU "
-			"would be suboptimal: disabling "
-			"blowfish-x86_64.\n");
+		pr_info("CPU-optimized crypto module not loaded, crypto optimization performance on this CPU would be suboptimal\n");
 		return -ENODEV;
 	}
 
diff --git a/arch/x86/crypto/camellia_glue.c b/arch/x86/crypto/camellia_glue.c
index 94dd2973bb47..002a1e84b277 100644
--- a/arch/x86/crypto/camellia_glue.c
+++ b/arch/x86/crypto/camellia_glue.c
@@ -1394,10 +1394,7 @@  static int __init camellia_init(void)
 		return -ENODEV;
 
 	if (!force && is_blacklisted_cpu()) {
-		printk(KERN_INFO
-			"camellia-x86_64: performance on this CPU "
-			"would be suboptimal: disabling "
-			"camellia-x86_64.\n");
+		pr_info("CPU-optimized crypto module not loaded, crypto optimization performance on this CPU would be suboptimal\n");
 		return -ENODEV;
 	}
 
diff --git a/arch/x86/crypto/des3_ede_glue.c b/arch/x86/crypto/des3_ede_glue.c
index 7b4dd02007ed..b38ad3ec38e2 100644
--- a/arch/x86/crypto/des3_ede_glue.c
+++ b/arch/x86/crypto/des3_ede_glue.c
@@ -371,7 +371,7 @@  static int __init des3_ede_x86_init(void)
 		return -ENODEV;
 
 	if (!force && is_blacklisted_cpu()) {
-		pr_info("des3_ede-x86_64: performance on this CPU would be suboptimal: disabling des3_ede-x86_64.\n");
+		pr_info("CPU-optimized crypto module not loaded, crypto optimization performance on this CPU would be suboptimal\n");
 		return -ENODEV;
 	}
 
diff --git a/arch/x86/crypto/twofish_glue_3way.c b/arch/x86/crypto/twofish_glue_3way.c
index 2fde637b40c8..29c35d2aaeba 100644
--- a/arch/x86/crypto/twofish_glue_3way.c
+++ b/arch/x86/crypto/twofish_glue_3way.c
@@ -157,10 +157,7 @@  static int __init twofish_3way_init(void)
 		return -ENODEV;
 
 	if (!force && is_blacklisted_cpu()) {
-		printk(KERN_INFO
-			"twofish-x86_64-3way: performance on this CPU "
-			"would be suboptimal: disabling "
-			"twofish-x86_64-3way.\n");
+		pr_info("CPU-optimized crypto module not loaded, crypto optimization performance on this CPU would be suboptimal\n");
 		return -ENODEV;
 	}