diff options
| author | Thomas Huth <thuth@redhat.com> | 2023-06-27 09:47:02 +0200 |
|---|---|---|
| committer | Thomas Huth <thuth@redhat.com> | 2023-06-29 20:45:12 +0200 |
| commit | e31f08dc748bad980f2092be165bd091f9cf3c48 (patch) | |
| tree | 80be76785ed17a5916d7ef1740e0574336eda8a6 /tests/tcg | |
| parent | 74fe98ee7fb3344dbd085d1fa32c0dc2fc2c831f (diff) | |
| download | qemu-e31f08dc748bad980f2092be165bd091f9cf3c48.zip qemu-e31f08dc748bad980f2092be165bd091f9cf3c48.tar.gz qemu-e31f08dc748bad980f2092be165bd091f9cf3c48.tar.bz2 | |
pc-bios/s390-ccw: Move the stack array into start.S
The stack array is only referenced from the start-up code (which is
shared between the s390-ccw.img and the s390-netboot.img), but it is
currently declared twice, once in main.c and once in netmain.c.
It makes more sense to declare this in start.S instead - which will
also be helpful in the next patch, since we need to mention the .bss
section in start.S in that patch.
While we're at it, let's also drop the huge alignment of the stack,
since there is no technical requirement for aligning it to page
boundaries.
Message-Id: <20230627074703.99608-4-thuth@redhat.com>
Reviewed-by: Claudio Imbrenda <imbrenda@linux.ibm.com>
Reviewed-by: Eric Farman <farman@linux.ibm.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
Diffstat (limited to 'tests/tcg')
| -rw-r--r-- | tests/tcg/s390x/head64.S | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/tests/tcg/s390x/head64.S b/tests/tcg/s390x/head64.S index c6f36df..4fe2883 100644 --- a/tests/tcg/s390x/head64.S +++ b/tests/tcg/s390x/head64.S @@ -8,6 +8,8 @@ #include "../../../pc-bios/s390-ccw/start.S" #undef main +.text + main_pre: aghi %r15,-160 /* reserve stack for C code */ brasl %r14,sclp_setup @@ -24,8 +26,3 @@ success_psw: .quad 0x2000180000000,0xfff /* see is_special_wait_psw() */ failure_psw: .quad 0x2000180000000,0 /* disabled wait */ - - .section .bss - .align 0x1000 -stack: - .skip 0x8000 |