Messages in this thread Patch in this message | ![/](/images/icornerl.gif) | | From | Jiapeng Chong <> | Subject | [PATCH] s390/vmem: Use swap() instead of open coding it | Date | Tue, 17 Jan 2023 14:02:23 +0800 |
| |
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 <abaci@linux.alibaba.com> Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com> --- 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
| ![\](/images/icornerr.gif) |