On Wed, 7 Sep 2022 11:59:52 -0400 Matthew Rosato <mjrosato@xxxxxxxxxxxxx> wrote: > The GAIT and all of its entries must be represented by physical > addresses as this structure is shared with underlying firmware. > We can keep a virtual address of the GAIT origin in order to > handle processing in the kernel, but when traversing the entries > we must again convert the physical AISB stored in that GAIT entry > into a virtual address in order to process it. > > Note: this currently doesn't fix a real bug, since virtual addresses > are indentical to physical ones. > > Reviewed-by: Pierre Morel <pmorel@xxxxxxxxxxxxx> > Acked-by: Nico Boehr <nrb@xxxxxxxxxxxxx> > Signed-off-by: Matthew Rosato <mjrosato@xxxxxxxxxxxxx> Reviewed-by: Claudio Imbrenda <imbrenda@xxxxxxxxxxxxx> > --- > arch/s390/kvm/interrupt.c | 2 +- > arch/s390/kvm/pci.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/s390/kvm/interrupt.c b/arch/s390/kvm/interrupt.c > index b9c944b262c7..ab569faf0df2 100644 > --- a/arch/s390/kvm/interrupt.c > +++ b/arch/s390/kvm/interrupt.c > @@ -3324,7 +3324,7 @@ static void aen_host_forward(unsigned long si) > if (gaite->count == 0) > return; > if (gaite->aisb != 0) > - set_bit_inv(gaite->aisbo, (unsigned long *)gaite->aisb); > + set_bit_inv(gaite->aisbo, phys_to_virt(gaite->aisb)); > > kvm = kvm_s390_pci_si_to_kvm(aift, si); > if (!kvm) > diff --git a/arch/s390/kvm/pci.c b/arch/s390/kvm/pci.c > index bb8c335d17b9..8cfa0b03ebbb 100644 > --- a/arch/s390/kvm/pci.c > +++ b/arch/s390/kvm/pci.c > @@ -71,7 +71,7 @@ static int zpci_setup_aipb(u8 nisc) > rc = -ENOMEM; > goto free_sbv; > } > - aift->gait = (struct zpci_gaite *)page_to_phys(page); > + aift->gait = (struct zpci_gaite *)page_to_virt(page); > > zpci_aipb->aipb.faisb = virt_to_phys(aift->sbv->vector); > zpci_aipb->aipb.gait = virt_to_phys(aift->gait);