[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 0/4] pc-bios/s390-ccw: Fixes and improvements for start.S
From: |
Thomas Huth |
Subject: |
[PATCH 0/4] pc-bios/s390-ccw: Fixes and improvements for start.S |
Date: |
Mon, 26 Jun 2023 15:21:34 +0200 |
Main motivation of this series was a bug that showed up when compiling
with Clang 16 and binutils 2.40 (which has been reported in Fedora ELN, see
https://bugzilla.redhat.com/show_bug.cgi?id=2216662). This is fixed in
the fourth patch.
While working on this issue, I came accross some other issues which I
address in the first three patches:
- Indentation is a mixture between tabs and spaces in start.S (patch 1)
- We do not set up a stack frame for the main() function, which could
cause memory corruption (patch 2)
- The stack is declared in multiple places, though it's only needed
in start.S (patch 3)
Thomas Huth (4):
pc-bios/s390-ccw: Fix indentation in start.S
pc-bios/s390-ccw: Provide space for initial stack frame in start.S
pc-bios/s390-ccw: Move the stack array into start.S
pc-bios/s390-ccw: Don't use __bss_start with the "larl" instruction
pc-bios/s390-ccw/s390-ccw.h | 1 -
pc-bios/s390-ccw/main.c | 1 -
pc-bios/s390-ccw/netmain.c | 1 -
pc-bios/s390-ccw/start.S | 143 +++++++++++++++++++-----------------
4 files changed, 75 insertions(+), 71 deletions(-)
--
2.39.3