2023-01-17 06:15:40

by Jiapeng Chong

[permalink] [raw]
Subject: [PATCH] s390/vmem: Use swap() instead of open coding it

Swap is a function interface that provides exchange function. To avoid
code duplication, we can use swap function.

./arch/s390/mm/vmem.c:680:10-11: WARNING opportunity for swap().

Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=3786
Reported-by: Abaci Robot <[email protected]>
Signed-off-by: Jiapeng Chong <[email protected]>
---
arch/s390/mm/vmem.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/arch/s390/mm/vmem.c b/arch/s390/mm/vmem.c
index 78d7768f93d7..774a71b94f5c 100644
--- a/arch/s390/mm/vmem.c
+++ b/arch/s390/mm/vmem.c
@@ -674,11 +674,8 @@ static void __init memblock_region_swap(void *a, void *b, int size)
{
struct memblock_region *r1 = a;
struct memblock_region *r2 = b;
- struct memblock_region swap;

- swap = *r1;
- *r1 = *r2;
- *r2 = swap;
+ swap(*r1, *r2);
}

/*
--
2.20.1.7.g153144c


2023-01-17 09:14:26

by Heiko Carstens

[permalink] [raw]
Subject: Re: [PATCH] s390/vmem: Use swap() instead of open coding it

On Tue, Jan 17, 2023 at 02:02:23PM +0800, Jiapeng Chong wrote:
> Swap is a function interface that provides exchange function. To avoid
> code duplication, we can use swap function.
>
> ./arch/s390/mm/vmem.c:680:10-11: WARNING opportunity for swap().
>
> Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=3786
> Reported-by: Abaci Robot <[email protected]>
> Signed-off-by: Jiapeng Chong <[email protected]>
> ---
> arch/s390/mm/vmem.c | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/arch/s390/mm/vmem.c b/arch/s390/mm/vmem.c
> index 78d7768f93d7..774a71b94f5c 100644
> --- a/arch/s390/mm/vmem.c
> +++ b/arch/s390/mm/vmem.c
> @@ -674,11 +674,8 @@ static void __init memblock_region_swap(void *a, void *b, int size)
> {
> struct memblock_region *r1 = a;
> struct memblock_region *r2 = b;
> - struct memblock_region swap;
>
> - swap = *r1;
> - *r1 = *r2;
> - *r2 = swap;
> + swap(*r1, *r2);
> }

Let's remove r1, and r2 as well. I applied the below:

From ff3ebe76f3b9481e6553bc2f05b52a266cd2e4a5 Mon Sep 17 00:00:00 2001
From: Jiapeng Chong <[email protected]>
Date: Tue, 17 Jan 2023 14:02:23 +0800
Subject: [PATCH] s390/vmem: use swap() instead of open coding it

Swap is a function interface that provides exchange function. To avoid
code duplication, we can use swap function.

./arch/s390/mm/vmem.c:680:10-11: WARNING opportunity for swap().

[[email protected]: get rid of all temp variables]
Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=3786
Reported-by: Abaci Robot <[email protected]>
Signed-off-by: Jiapeng Chong <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Heiko Carstens <[email protected]>
---
arch/s390/mm/vmem.c | 8 +-------
1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/arch/s390/mm/vmem.c b/arch/s390/mm/vmem.c
index 78d7768f93d7..15daf777cf41 100644
--- a/arch/s390/mm/vmem.c
+++ b/arch/s390/mm/vmem.c
@@ -672,13 +672,7 @@ static int __init memblock_region_cmp(const void *a, const void *b)

static void __init memblock_region_swap(void *a, void *b, int size)
{
- struct memblock_region *r1 = a;
- struct memblock_region *r2 = b;
- struct memblock_region swap;
-
- swap = *r1;
- *r1 = *r2;
- *r2 = swap;
+ swap(*(struct memblock_region *)a, *(struct memblock_region *)b);
}

/*
--
2.34.1