diff mbox series

[v3] x86/boot: Setup correctly fs segment for bogus_real_magic

Message ID 20241111132823.21658-1-frediano.ziglio@cloud.com (mailing list archive)
State New
Headers show
Series [v3] x86/boot: Setup correctly fs segment for bogus_real_magic | expand

Commit Message

Frediano Ziglio Nov. 11, 2024, 1:28 p.m. UTC
bogus_real_magic code uses fs segment so it should be initialised.

Fixes: d8c8fef09054 ("Provide basic Xen PM infrastructure")
Signed-off-by: Frediano Ziglio <frediano.ziglio@cloud.com>
---
Changes since v1:
- added "Fixes" comment;
- add initialisation, do not remove old one.

Changes since v2:
- move initialisation in bogus_real_magic.
---
 xen/arch/x86/boot/wakeup.S | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Comments

Jan Beulich Nov. 11, 2024, 2:18 p.m. UTC | #1
On 11.11.2024 14:28, Frediano Ziglio wrote:
> bogus_real_magic code uses fs segment so it should be initialised.
> 
> Fixes: d8c8fef09054 ("Provide basic Xen PM infrastructure")
> Signed-off-by: Frediano Ziglio <frediano.ziglio@cloud.com>

Reviewed-by: Jan Beulich <jbeulich@suse.com>
diff mbox series

Patch

diff --git a/xen/arch/x86/boot/wakeup.S b/xen/arch/x86/boot/wakeup.S
index 08447e1934..ff138314f5 100644
--- a/xen/arch/x86/boot/wakeup.S
+++ b/xen/arch/x86/boot/wakeup.S
@@ -77,8 +77,11 @@  check_vesaw:
         ret
 
 bogus_real_magic:
+        movw    $0xb800, %ax
+        movw    %ax, %fs
+1:
         movw    $0x0e00 + 'B', %fs:(0x12)
-        jmp     bogus_real_magic
+        jmp     1b
 
         .align 4
 real_magic:     .long 0x12345678