* [PATCH RESEND] x86/asm: Use inout "+" asm onstraint modifiers in __iowrite32_copy()
@ 2025-11-10 22:31 Uros Bizjak
2026-01-20 20:51 ` H. Peter Anvin
0 siblings, 1 reply; 4+ messages in thread
From: Uros Bizjak @ 2025-11-10 22:31 UTC (permalink / raw)
To: x86, linux-kernel
Cc: Uros Bizjak, Thomas Gleixner, Ingo Molnar, Borislav Petkov,
Dave Hansen, H. Peter Anvin
Use inout "+" asm constraint modifiers to simplify asm operands.
No functional changes intended.
Signed-off-by: Uros Bizjak <ubizjak@gmail.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Dave Hansen <dave.hansen@linux.intel.com>
Cc: "H. Peter Anvin" <hpa@zytor.com>
---
arch/x86/include/asm/io.h | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/arch/x86/include/asm/io.h b/arch/x86/include/asm/io.h
index ca309a3227c7..2ea25745e059 100644
--- a/arch/x86/include/asm/io.h
+++ b/arch/x86/include/asm/io.h
@@ -218,9 +218,8 @@ static inline void __iowrite32_copy(void __iomem *to, const void *from,
size_t count)
{
asm volatile("rep movsl"
- : "=&c"(count), "=&D"(to), "=&S"(from)
- : "0"(count), "1"(to), "2"(from)
- : "memory");
+ : "+D"(to), "+S"(from), "+c"(count)
+ : : "memory");
}
#define __iowrite32_copy __iowrite32_copy
#endif
--
2.51.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH RESEND] x86/asm: Use inout "+" asm onstraint modifiers in __iowrite32_copy()
@ 2025-12-16 10:51 Uros Bizjak
2026-03-07 14:57 ` [tip: x86/cleanups] " tip-bot2 for Uros Bizjak
0 siblings, 1 reply; 4+ messages in thread
From: Uros Bizjak @ 2025-12-16 10:51 UTC (permalink / raw)
To: x86, linux-kernel
Cc: Uros Bizjak, Thomas Gleixner, Ingo Molnar, Borislav Petkov,
Dave Hansen, H. Peter Anvin
Use inout "+" asm constraint modifiers to simplify asm operands.
No functional changes intended.
Signed-off-by: Uros Bizjak <ubizjak@gmail.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Dave Hansen <dave.hansen@linux.intel.com>
Cc: "H. Peter Anvin" <hpa@zytor.com>
---
arch/x86/include/asm/io.h | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/arch/x86/include/asm/io.h b/arch/x86/include/asm/io.h
index ca309a3227c7..2ea25745e059 100644
--- a/arch/x86/include/asm/io.h
+++ b/arch/x86/include/asm/io.h
@@ -218,9 +218,8 @@ static inline void __iowrite32_copy(void __iomem *to, const void *from,
size_t count)
{
asm volatile("rep movsl"
- : "=&c"(count), "=&D"(to), "=&S"(from)
- : "0"(count), "1"(to), "2"(from)
- : "memory");
+ : "+D"(to), "+S"(from), "+c"(count)
+ : : "memory");
}
#define __iowrite32_copy __iowrite32_copy
#endif
--
2.52.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH RESEND] x86/asm: Use inout "+" asm onstraint modifiers in __iowrite32_copy()
2025-11-10 22:31 [PATCH RESEND] " Uros Bizjak
@ 2026-01-20 20:51 ` H. Peter Anvin
0 siblings, 0 replies; 4+ messages in thread
From: H. Peter Anvin @ 2026-01-20 20:51 UTC (permalink / raw)
To: Uros Bizjak, x86, linux-kernel
Cc: Thomas Gleixner, Ingo Molnar, Borislav Petkov, Dave Hansen
On 2025-11-10 14:31, Uros Bizjak wrote:
> Use inout "+" asm constraint modifiers to simplify asm operands.
>
> No functional changes intended.
>
> Signed-off-by: Uros Bizjak <ubizjak@gmail.com>
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Cc: Ingo Molnar <mingo@kernel.org>
> Cc: Borislav Petkov <bp@alien8.de>
> Cc: Dave Hansen <dave.hansen@linux.intel.com>
> Cc: "H. Peter Anvin" <hpa@zytor.com>
> ---
> arch/x86/include/asm/io.h | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
Looks good to me.
Reviewed-by: H. Peter Anvin (Intel) <hpa@zytor.com>
> diff --git a/arch/x86/include/asm/io.h b/arch/x86/include/asm/io.h
> index ca309a3227c7..2ea25745e059 100644
> --- a/arch/x86/include/asm/io.h
> +++ b/arch/x86/include/asm/io.h
> @@ -218,9 +218,8 @@ static inline void __iowrite32_copy(void __iomem *to, const void *from,
> size_t count)
> {
> asm volatile("rep movsl"
> - : "=&c"(count), "=&D"(to), "=&S"(from)
> - : "0"(count), "1"(to), "2"(from)
> - : "memory");
> + : "+D"(to), "+S"(from), "+c"(count)
> + : : "memory");
> }
> #define __iowrite32_copy __iowrite32_copy
> #endif
^ permalink raw reply [flat|nested] 4+ messages in thread
* [tip: x86/cleanups] x86/asm: Use inout "+" asm onstraint modifiers in __iowrite32_copy()
2025-12-16 10:51 [PATCH RESEND] x86/asm: Use inout "+" asm onstraint modifiers in __iowrite32_copy() Uros Bizjak
@ 2026-03-07 14:57 ` tip-bot2 for Uros Bizjak
0 siblings, 0 replies; 4+ messages in thread
From: tip-bot2 for Uros Bizjak @ 2026-03-07 14:57 UTC (permalink / raw)
To: linux-tip-commits
Cc: Uros Bizjak, Borislav Petkov (AMD), H. Peter Anvin (Intel), x86,
linux-kernel
The following commit has been merged into the x86/cleanups branch of tip:
Commit-ID: 09fbb775f1d01945119c4a0be4afacf30cc86796
Gitweb: https://git.kernel.org/tip/09fbb775f1d01945119c4a0be4afacf30cc86796
Author: Uros Bizjak <ubizjak@gmail.com>
AuthorDate: Tue, 16 Dec 2025 11:51:20 +01:00
Committer: Borislav Petkov (AMD) <bp@alien8.de>
CommitterDate: Sat, 07 Mar 2026 15:46:10 +01:00
x86/asm: Use inout "+" asm onstraint modifiers in __iowrite32_copy()
Use inout "+" asm constraint modifiers to simplify asm operands.
No functional changes intended.
Signed-off-by: Uros Bizjak <ubizjak@gmail.com>
Signed-off-by: Borislav Petkov (AMD) <bp@alien8.de>
Reviewed-by: H. Peter Anvin (Intel) <hpa@zytor.com>
Link: https://patch.msgid.link/20251216105134.248196-1-ubizjak@gmail.com
---
arch/x86/include/asm/io.h | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/arch/x86/include/asm/io.h b/arch/x86/include/asm/io.h
index ca309a3..2ea2574 100644
--- a/arch/x86/include/asm/io.h
+++ b/arch/x86/include/asm/io.h
@@ -218,9 +218,8 @@ static inline void __iowrite32_copy(void __iomem *to, const void *from,
size_t count)
{
asm volatile("rep movsl"
- : "=&c"(count), "=&D"(to), "=&S"(from)
- : "0"(count), "1"(to), "2"(from)
- : "memory");
+ : "+D"(to), "+S"(from), "+c"(count)
+ : : "memory");
}
#define __iowrite32_copy __iowrite32_copy
#endif
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2026-03-07 14:57 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-12-16 10:51 [PATCH RESEND] x86/asm: Use inout "+" asm onstraint modifiers in __iowrite32_copy() Uros Bizjak
2026-03-07 14:57 ` [tip: x86/cleanups] " tip-bot2 for Uros Bizjak
-- strict thread matches above, loose matches on Subject: below --
2025-11-10 22:31 [PATCH RESEND] " Uros Bizjak
2026-01-20 20:51 ` H. Peter Anvin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox