On Wed, Feb 22, 2023 at 04:55:02PM +0100, Nico Boehr wrote: > @@ -112,7 +112,7 @@ static int zpci_reset_aipb(u8 nisc) > return -EINVAL; > > aift->sbv = zpci_aif_sbv; > - aift->gait = (struct zpci_gaite *)zpci_aipb->aipb.gait; > + aift->gait = phys_to_virt(zpci_aipb->aipb.gait); > > return 0; > } With this change aift->gait would never be NULL. Does it work with line 125? 120 int kvm_s390_pci_aen_init(u8 nisc) 121 { 122 int rc = 0; 123 124 /* If already enabled for AEN, bail out now */ 125 if (aift->gait || aift->sbv) 126 return -EPERM;