Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1583131 > unrolled thread

[PATCH 03/35] arm64: Convert remaining uses of pr_warning to pr_warn

Started byJoe Perches <joe@perches.com>
First post2017-02-17 08:20 +0100
Last post2017-02-17 12:00 +0100
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  [PATCH 03/35] arm64: Convert remaining uses of pr_warning to pr_warn Joe Perches <joe@perches.com> - 2017-02-17 08:20 +0100
    Re: [PATCH 03/35] arm64: Convert remaining uses of pr_warning to  pr_warn Mark Rutland <mark.rutland@arm.com> - 2017-02-17 12:00 +0100

#1583131 — [PATCH 03/35] arm64: Convert remaining uses of pr_warning to pr_warn

FromJoe Perches <joe@perches.com>
Date2017-02-17 08:20 +0100
Subject[PATCH 03/35] arm64: Convert remaining uses of pr_warning to pr_warn
Message-ID<tbKZI-176-31@gated-at.bofh.it>
To enable eventual removal of pr_warning

This makes pr_warn use consistent for arch/arm64

Prior to this patch, there were 7 uses of pr_warning and
27 uses of pr_warn in arch/arm64

Signed-off-by: Joe Perches <joe@perches.com>
---
 arch/arm64/include/asm/syscall.h  | 8 ++++----
 arch/arm64/kernel/hw_breakpoint.c | 8 ++++----
 arch/arm64/kernel/smp.c           | 4 ++--
 3 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/arch/arm64/include/asm/syscall.h b/arch/arm64/include/asm/syscall.h
index 709a574468f0..231f2c3fcf81 100644
--- a/arch/arm64/include/asm/syscall.h
+++ b/arch/arm64/include/asm/syscall.h
@@ -68,8 +68,8 @@ static inline void syscall_get_arguments(struct task_struct *task,
 	if (i + n > SYSCALL_MAX_ARGS) {
 		unsigned long *args_bad = args + SYSCALL_MAX_ARGS - i;
 		unsigned int n_bad = n + i - SYSCALL_MAX_ARGS;
-		pr_warning("%s called with max args %d, handling only %d\n",
-			   __func__, i + n, SYSCALL_MAX_ARGS);
+		pr_warn("%s called with max args %d, handling only %d\n",
+			__func__, i + n, SYSCALL_MAX_ARGS);
 		memset(args_bad, 0, n_bad * sizeof(args[0]));
 	}
 
@@ -92,8 +92,8 @@ static inline void syscall_set_arguments(struct task_struct *task,
 		return;
 
 	if (i + n > SYSCALL_MAX_ARGS) {
-		pr_warning("%s called with max args %d, handling only %d\n",
-			   __func__, i + n, SYSCALL_MAX_ARGS);
+		pr_warn("%s called with max args %d, handling only %d\n",
+			__func__, i + n, SYSCALL_MAX_ARGS);
 		n = SYSCALL_MAX_ARGS - i;
 	}
 
diff --git a/arch/arm64/kernel/hw_breakpoint.c b/arch/arm64/kernel/hw_breakpoint.c
index 0296e7924240..6fa79ffd0c43 100644
--- a/arch/arm64/kernel/hw_breakpoint.c
+++ b/arch/arm64/kernel/hw_breakpoint.c
@@ -62,7 +62,7 @@ int hw_breakpoint_slots(int type)
 	case TYPE_DATA:
 		return get_num_wrps();
 	default:
-		pr_warning("unknown slot type: %d\n", type);
+		pr_warn("unknown slot type: %d\n", type);
 		return 0;
 	}
 }
@@ -123,7 +123,7 @@ static u64 read_wb_reg(int reg, int n)
 	GEN_READ_WB_REG_CASES(AARCH64_DBG_REG_WVR, AARCH64_DBG_REG_NAME_WVR, val);
 	GEN_READ_WB_REG_CASES(AARCH64_DBG_REG_WCR, AARCH64_DBG_REG_NAME_WCR, val);
 	default:
-		pr_warning("attempt to read from unknown breakpoint register %d\n", n);
+		pr_warn("attempt to read from unknown breakpoint register %d\n", n);
 	}
 
 	return val;
@@ -138,7 +138,7 @@ static void write_wb_reg(int reg, int n, u64 val)
 	GEN_WRITE_WB_REG_CASES(AARCH64_DBG_REG_WVR, AARCH64_DBG_REG_NAME_WVR, val);
 	GEN_WRITE_WB_REG_CASES(AARCH64_DBG_REG_WCR, AARCH64_DBG_REG_NAME_WCR, val);
 	default:
-		pr_warning("attempt to write to unknown breakpoint register %d\n", n);
+		pr_warn("attempt to write to unknown breakpoint register %d\n", n);
 	}
 	isb();
 }
@@ -156,7 +156,7 @@ static enum dbg_active_el debug_exception_level(int privilege)
 	case AARCH64_BREAKPOINT_EL1:
 		return DBG_ACTIVE_EL1;
 	default:
-		pr_warning("invalid breakpoint privilege level %d\n", privilege);
+		pr_warn("invalid breakpoint privilege level %d\n", privilege);
 		return -EINVAL;
 	}
 }
diff --git a/arch/arm64/kernel/smp.c b/arch/arm64/kernel/smp.c
index 827d52d78b67..9c396e34c090 100644
--- a/arch/arm64/kernel/smp.c
+++ b/arch/arm64/kernel/smp.c
@@ -925,8 +925,8 @@ void smp_send_stop(void)
 		udelay(1);
 
 	if (num_online_cpus() > 1)
-		pr_warning("SMP: failed to stop secondary CPUs %*pbl\n",
-			   cpumask_pr_args(cpu_online_mask));
+		pr_warn("SMP: failed to stop secondary CPUs %*pbl\n",
+			cpumask_pr_args(cpu_online_mask));
 }
 
 /*
-- 
2.10.0.rc2.1.g053435c

[toc] | [next] | [standalone]

#1583319 — Re: [PATCH 03/35] arm64: Convert remaining uses of pr_warning to pr_warn

FromMark Rutland <mark.rutland@arm.com>
Date2017-02-17 12:00 +0100
SubjectRe: [PATCH 03/35] arm64: Convert remaining uses of pr_warning to pr_warn
Message-ID<tbOqC-3eL-23@gated-at.bofh.it>
In reply to#1583131
On Thu, Feb 16, 2017 at 11:11:16PM -0800, Joe Perches wrote:
> To enable eventual removal of pr_warning
> 
> This makes pr_warn use consistent for arch/arm64
> 
> Prior to this patch, there were 7 uses of pr_warning and
> 27 uses of pr_warn in arch/arm64
> 
> Signed-off-by: Joe Perches <joe@perches.com>

Looks nicer to me.

Acked-by: Mark Rutland <mark.rutland@arm.com>

Mark.

> ---
>  arch/arm64/include/asm/syscall.h  | 8 ++++----
>  arch/arm64/kernel/hw_breakpoint.c | 8 ++++----
>  arch/arm64/kernel/smp.c           | 4 ++--
>  3 files changed, 10 insertions(+), 10 deletions(-)
> 
> diff --git a/arch/arm64/include/asm/syscall.h b/arch/arm64/include/asm/syscall.h
> index 709a574468f0..231f2c3fcf81 100644
> --- a/arch/arm64/include/asm/syscall.h
> +++ b/arch/arm64/include/asm/syscall.h
> @@ -68,8 +68,8 @@ static inline void syscall_get_arguments(struct task_struct *task,
>  	if (i + n > SYSCALL_MAX_ARGS) {
>  		unsigned long *args_bad = args + SYSCALL_MAX_ARGS - i;
>  		unsigned int n_bad = n + i - SYSCALL_MAX_ARGS;
> -		pr_warning("%s called with max args %d, handling only %d\n",
> -			   __func__, i + n, SYSCALL_MAX_ARGS);
> +		pr_warn("%s called with max args %d, handling only %d\n",
> +			__func__, i + n, SYSCALL_MAX_ARGS);
>  		memset(args_bad, 0, n_bad * sizeof(args[0]));
>  	}
>  
> @@ -92,8 +92,8 @@ static inline void syscall_set_arguments(struct task_struct *task,
>  		return;
>  
>  	if (i + n > SYSCALL_MAX_ARGS) {
> -		pr_warning("%s called with max args %d, handling only %d\n",
> -			   __func__, i + n, SYSCALL_MAX_ARGS);
> +		pr_warn("%s called with max args %d, handling only %d\n",
> +			__func__, i + n, SYSCALL_MAX_ARGS);
>  		n = SYSCALL_MAX_ARGS - i;
>  	}
>  
> diff --git a/arch/arm64/kernel/hw_breakpoint.c b/arch/arm64/kernel/hw_breakpoint.c
> index 0296e7924240..6fa79ffd0c43 100644
> --- a/arch/arm64/kernel/hw_breakpoint.c
> +++ b/arch/arm64/kernel/hw_breakpoint.c
> @@ -62,7 +62,7 @@ int hw_breakpoint_slots(int type)
>  	case TYPE_DATA:
>  		return get_num_wrps();
>  	default:
> -		pr_warning("unknown slot type: %d\n", type);
> +		pr_warn("unknown slot type: %d\n", type);
>  		return 0;
>  	}
>  }
> @@ -123,7 +123,7 @@ static u64 read_wb_reg(int reg, int n)
>  	GEN_READ_WB_REG_CASES(AARCH64_DBG_REG_WVR, AARCH64_DBG_REG_NAME_WVR, val);
>  	GEN_READ_WB_REG_CASES(AARCH64_DBG_REG_WCR, AARCH64_DBG_REG_NAME_WCR, val);
>  	default:
> -		pr_warning("attempt to read from unknown breakpoint register %d\n", n);
> +		pr_warn("attempt to read from unknown breakpoint register %d\n", n);
>  	}
>  
>  	return val;
> @@ -138,7 +138,7 @@ static void write_wb_reg(int reg, int n, u64 val)
>  	GEN_WRITE_WB_REG_CASES(AARCH64_DBG_REG_WVR, AARCH64_DBG_REG_NAME_WVR, val);
>  	GEN_WRITE_WB_REG_CASES(AARCH64_DBG_REG_WCR, AARCH64_DBG_REG_NAME_WCR, val);
>  	default:
> -		pr_warning("attempt to write to unknown breakpoint register %d\n", n);
> +		pr_warn("attempt to write to unknown breakpoint register %d\n", n);
>  	}
>  	isb();
>  }
> @@ -156,7 +156,7 @@ static enum dbg_active_el debug_exception_level(int privilege)
>  	case AARCH64_BREAKPOINT_EL1:
>  		return DBG_ACTIVE_EL1;
>  	default:
> -		pr_warning("invalid breakpoint privilege level %d\n", privilege);
> +		pr_warn("invalid breakpoint privilege level %d\n", privilege);
>  		return -EINVAL;
>  	}
>  }
> diff --git a/arch/arm64/kernel/smp.c b/arch/arm64/kernel/smp.c
> index 827d52d78b67..9c396e34c090 100644
> --- a/arch/arm64/kernel/smp.c
> +++ b/arch/arm64/kernel/smp.c
> @@ -925,8 +925,8 @@ void smp_send_stop(void)
>  		udelay(1);
>  
>  	if (num_online_cpus() > 1)
> -		pr_warning("SMP: failed to stop secondary CPUs %*pbl\n",
> -			   cpumask_pr_args(cpu_online_mask));
> +		pr_warn("SMP: failed to stop secondary CPUs %*pbl\n",
> +			cpumask_pr_args(cpu_online_mask));
>  }
>  
>  /*
> -- 
> 2.10.0.rc2.1.g053435c
>

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web