On Wed, 5 Feb 2020 12:46:39 +0100 Michael Mueller <mimu@xxxxxxxxxxxxx> wrote: > On 05.02.20 12:31, Cornelia Huck wrote: > > On Mon, 3 Feb 2020 08:19:35 -0500 > > Christian Borntraeger <borntraeger@xxxxxxxxxx> wrote: > > > >> From: Michael Mueller <mimu@xxxxxxxxxxxxx> > >> > >> The patch implements interruption injection for the following > >> list of interruption types: > >> > >> - I/O > >> __deliver_io (III) > >> > >> - External > >> __deliver_cpu_timer (IEI) > >> __deliver_ckc (IEI) > >> __deliver_emergency_signal (IEI) > >> __deliver_external_call (IEI) > >> > >> - cpu restart > >> __deliver_restart (IRI) > > > > Hm... what do 'III', 'IEI', and 'IRI' stand for? > > that's the kind of interruption injection being used: > > inject io interruption > inject external interruption > inject restart interruption So, maybe make this: - I/O (uses inject io interruption) __ deliver_io - External (uses inject external interruption) (and so on) I find using the acronyms without explanation very confusing. > > > > >> > >> The service interrupt is handled in a followup patch. > >> > >> Signed-off-by: Michael Mueller <mimu@xxxxxxxxxxxxx> > >> Signed-off-by: Christian Borntraeger <borntraeger@xxxxxxxxxx> > >> [fixes] > >> --- > >> arch/s390/include/asm/kvm_host.h | 8 +++ > >> arch/s390/kvm/interrupt.c | 93 ++++++++++++++++++++++---------- > >> 2 files changed, 74 insertions(+), 27 deletions(-)