2021-02-02 05:54:47

by Yang Li

Subject: [PATCH] riscv: kasan: remove unneeded semicolon

Eliminate the following coccicheck warning:
./arch/riscv/mm/kasan_init.c:103:2-3: Unneeded semicolon

Reported-by: Abaci Robot <[email protected]>
Signed-off-by: Yang Li <[email protected]>
---
arch/riscv/mm/kasan_init.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/riscv/mm/kasan_init.c b/arch/riscv/mm/kasan_init.c
index a8a2ffd..fac437a 100644
--- a/arch/riscv/mm/kasan_init.c
+++ b/arch/riscv/mm/kasan_init.c
@@ -100,7 +100,7 @@ void __init kasan_init(void)
break;

populate(kasan_mem_to_shadow(start), kasan_mem_to_shadow(end));
- };
+ }

for (i = 0; i < PTRS_PER_PTE; i++)
set_pte(&kasan_early_shadow_pte[i],
--
1.8.3.1

2021-02-02 07:09:10

by Alexandre Ghiti

Subject: Re: [PATCH] riscv: kasan: remove unneeded semicolon

Hi Yang,

Le 2/2/21 ? 12:51 AM, Yang Li a ?crit?:
> Eliminate the following coccicheck warning:
> ./arch/riscv/mm/kasan_init.c:103:2-3: Unneeded semicolon
>
> Reported-by: Abaci Robot <[email protected]>
> Signed-off-by: Yang Li <[email protected]>
> ---
> arch/riscv/mm/kasan_init.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/riscv/mm/kasan_init.c b/arch/riscv/mm/kasan_init.c
> index a8a2ffd..fac437a 100644
> --- a/arch/riscv/mm/kasan_init.c
> +++ b/arch/riscv/mm/kasan_init.c
> @@ -100,7 +100,7 @@ void __init kasan_init(void)
> break;
>
> populate(kasan_mem_to_shadow(start), kasan_mem_to_shadow(end));
> - };
> + }
>
> for (i = 0; i < PTRS_PER_PTE; i++)
> set_pte(&kasan_early_shadow_pte[i],
>

Reviewed-by: Alexandre Ghiti <[email protected]>

Thanks,

Alex

2021-02-03 03:10:48

by Palmer Dabbelt

Subject: Re: [PATCH] riscv: kasan: remove unneeded semicolon

On Mon, 01 Feb 2021 21:51:59 PST (-0800), [email protected] wrote:
> Eliminate the following coccicheck warning:
> ./arch/riscv/mm/kasan_init.c:103:2-3: Unneeded semicolon
>
> Reported-by: Abaci Robot <[email protected]>
> Signed-off-by: Yang Li <[email protected]>
> ---
> arch/riscv/mm/kasan_init.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/riscv/mm/kasan_init.c b/arch/riscv/mm/kasan_init.c
> index a8a2ffd..fac437a 100644
> --- a/arch/riscv/mm/kasan_init.c
> +++ b/arch/riscv/mm/kasan_init.c
> @@ -100,7 +100,7 @@ void __init kasan_init(void)
> break;
>
> populate(kasan_mem_to_shadow(start), kasan_mem_to_shadow(end));
> - };
> + }
>
> for (i = 0; i < PTRS_PER_PTE; i++)
> set_pte(&kasan_early_shadow_pte[i],

Thanks, this is on fixes.