On 21.06.2013, at 14:10, Benjamin Herrenschmidt wrote: > On Fri, 2013-06-21 at 14:05 +0200, Alexander Graf wrote: >> Oh, I thought that was obvious. BookE also has in-kernel emulation and >> also does set kvm_msi_via_irqfd_allowed. The code above is spapr >> specific and would simply rewrite valid BookE MPIC addresses into >> something broken. >> >> In fact, where does the spapr variable come from at all here? > > Right, so we are back to square #1 -> This should be a method of the > PCI Host bridge.
Not sure. We could just declare a "direct virq==irq" mode in which msi.data == virq == irq. No need for any translation then. Alex