Re: [GIT PULL 09/11] KVM: s390: add bitmap for handling cpu-local interrupts

[Jens Freimann <jfrei@linux.vnet.ibm.com>]

Excerpts from Christian Borntraeger's message of 2014-11-29 22:05:10 +0100:
> Am 28.11.2014 um 18:18 schrieb Paolo Bonzini:
> > 
> > 
> > On 28/11/2014 14:25, Christian Borntraeger wrote:
> >>  
> >> +struct kvm_s390_irq_payload {
> >> +    struct kvm_s390_io_info io;
> >> +    struct kvm_s390_ext_info ext;
> >> +    struct kvm_s390_pgm_info pgm;
> >> +    struct kvm_s390_emerg_info emerg;
> >> +    struct kvm_s390_extcall_info extcall;
> >> +    struct kvm_s390_prefix_info prefix;
> >> +    struct kvm_s390_mchk_info mchk;
> >> +};
> >> +
> > 
> > struct or union?
> 
> struct. This is used for keeping the payload of the interrupts. Multiple different interrupts can be pending and most of them have payload - we want to keep everything.
> 
> Now, looking at that code again, as I/O is floating and emergency is also handled via a separate bitmap we could get rid of these two in a follow-up patch. Jens, can you have a look and prepare a followup-cleanup if appropriate?

I/O is floating and currently not used in that struct, but I have a patch 
series in the works to change floating interrupts as well and will need it then.

I will check if I can get rid of the emerg_info and sent a followup patch.

Jens

> Christian