Re: [kvm-unit-tests PATCH v4 4/9] s390x: Split assembly into multiple files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Kernel Development]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Info]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Linux Media]     [Device Mapper]

  Powered by Linux