On 13/01/2021 13.15, Janosch Frank wrote:
On 1/13/21 1:04 PM, Thomas Huth wrote:
On 12/01/2021 14.20, Janosch Frank wrote:
I've added too much to cstart64.S which is not start related
already. Now that I want to add even more code it's time to split
cstart64.S. lib.S has functions that are used in tests. macros.S
contains macros which are used in cstart64.S and lib.S
Signed-off-by: Janosch Frank <frankja@xxxxxxxxxxxxx>
Acked-by: Claudio Imbrenda <imbrenda@xxxxxxxxxxxxx>
---
s390x/Makefile | 6 +--
s390x/cstart64.S | 119 ++---------------------------------------------
s390x/lib.S | 65 ++++++++++++++++++++++++++
lib.S is a very generic name ... maybe rather use cpuasm.S or something similar?
instr.S ?
Hmm, no, if I read something like that, I'd expect wrapper functions for
single instructions, which is not what we have here.
Looking at the two functions, both are related to CPU stuff (reset and
state), so something with "cpu" in the name would be best, I think. Maybe
just cpu.S ?
Or if you intend to add non-CPU related stuff here later, maybe something
like misc.S ?
Or maybe entry.S to make it similar to the kernel?
No, entry.S sounds like a startup code, which we already have in cstart64.S,
so I'd rather avoid that name.
Thomas
