Re: [PATCH v3 09/12] KVM: s390: add functions to (un)register GISC with GISA

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

 



On Thu, 29 Nov 2018 13:35:07 +0100
Michael Mueller <mimu@xxxxxxxxxxxxx> wrote:

> On 29.11.18 12:48, Cornelia Huck wrote:
> > On Thu, 29 Nov 2018 12:24:36 +0100
> > Michael Mueller <mimu@xxxxxxxxxxxxx> wrote:
> >   
> >> On 29.11.18 11:55, Cornelia Huck wrote:  
> >>> On Wed, 28 Nov 2018 11:19:40 +0100
> >>> Michael Mueller <mimu@xxxxxxxxxxxxx> wrote:
> >>>      
> >>>> Add the IAM (Interruption Alert Mask) to the architectue specific
> >>>> kvm struct. This mask in the GISA is used to define for which ISC
> >>>> a GIB alert can be issued.
> >>>>
> >>>> The functions kvm_s390_gisc_register() and kvm_s390_gisc_unregister()
> >>>> are used to (un)register a GISC (guest ISC) with a virtual machine and
> >>>> its GISA.
> >>>>
> >>>> Upon successful completion, kvm_s390_gisc_register() returns the
> >>>> ISC to be used for GIB alert interruptions. A negative return code
> >>>> indicates an error during registration.
> >>>>
> >>>> Signed-off-by: Michael Mueller <mimu@xxxxxxxxxxxxx>
> >>>> ---
> >>>>    arch/s390/include/asm/kvm_host.h |  6 ++++++
> >>>>    arch/s390/kvm/interrupt.c        | 44 ++++++++++++++++++++++++++++++++++++++++
> >>>>    2 files changed, 50 insertions(+)  
> >>>
> >>> I've started at this and the following patches and have not found a
> >>> place where the functions are called... leftover from a previous
> >>> version?  
> >>
> >> No, these functions are required by the related driver code.
> >>
> >> Pls. see:
> >>
> >> Message ID: 1542906675-7949-4-git-send-email-pmorel@xxxxxxxxxxxxx
> >>
> >> [v2,3/3] vfio: ap: AP Queue Interrupt Control VFIO ioctl calls
> >>  
> >>>      
> >>  
> > 
> > I guess they should go with that series, then.  
> 
> No, I don't think so. These functions are part of the
> *GIB infrastructure* and will by consumed e.g. by VFIO AP
> with irqs but also by PCI.

I disagree. The rest of this series lays the groundwork, this patch
exports two unused functions, and it is much clearer how they are
supposed to be used with an actual user. Of course PCI can use them
later on as well :)

But in the end, your call.



[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux