Re: [PATCH 0/3] adding MSI/MSIX for PCI on POWER
On 27.06.2012, at 23:32, Benjamin Herrenschmidt wrote: > On Wed, 2012-06-27 at 16:43 +0200, Alexander Graf wrote: >> On 14.06.2012, at 06:29, Alexey Kardashevskiy wrote: >> >>> The following patches add MSIX support for PCI on POWER. >>> The first aim is virtio-pci so it was tested. It will also support >>> VFIO when it becomes available in public. >> >> Wrong CC list. This has nothing to do at all with KVM, but you should >> have CC'ed the qemu-ppc mailing list. > > I wasn't even aware of a qemu-ppc list myself :-) This is definitely > worth CCing kvm-ppc at least since it will affect VFIO. The patches in question were all for emulated devices, so I wouldn't agree 100% on that assessment. However, it doesn't hurt to CC the kvm list on these. It does hurt to not CC the respective maintainers of the code in question though :). Alex -- To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/3] adding MSI/MSIX for PCI on POWER
On Wed, 2012-06-27 at 16:43 +0200, Alexander Graf wrote: > On 14.06.2012, at 06:29, Alexey Kardashevskiy wrote: > > > The following patches add MSIX support for PCI on POWER. > > The first aim is virtio-pci so it was tested. It will also support > > VFIO when it becomes available in public. > > Wrong CC list. This has nothing to do at all with KVM, but you should > have CC'ed the qemu-ppc mailing list. I wasn't even aware of a qemu-ppc list myself :-) This is definitely worth CCing kvm-ppc at least since it will affect VFIO. > Also, this code touches generic MSI functionality, so I'd like at > least an ACK from Michael. Cheers, Ben. > > Alex > > > > > Alexey Kardashevskiy (3): > > msi/msix: added functions to API to set up message address and data > > pseries: added allocator for a block of IRQs > > pseries pci: added MSI/MSIX support > > > > hw/msi.c | 14 +++ > > hw/msi.h |1 + > > hw/msix.c | 10 ++ > > hw/msix.h |3 + > > hw/spapr.c | 26 +- > > hw/spapr.h |1 + > > hw/spapr_pci.c | 266 > +-- > > hw/spapr_pci.h | 13 +++- > > trace-events |9 ++ > > 9 files changed, 331 insertions(+), 12 deletions(-) > > > > -- > > 1.7.7.3 > > > > -- > > To unsubscribe from this list: send the line "unsubscribe kvm-ppc" > in > > the body of a message to majord...@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/3] adding MSI/MSIX for PCI on POWER
On 14.06.2012, at 06:29, Alexey Kardashevskiy wrote: > The following patches add MSIX support for PCI on POWER. > The first aim is virtio-pci so it was tested. It will also support > VFIO when it becomes available in public. Wrong CC list. This has nothing to do at all with KVM, but you should have CC'ed the qemu-ppc mailing list. Also, this code touches generic MSI functionality, so I'd like at least an ACK from Michael. Alex > > Alexey Kardashevskiy (3): > msi/msix: added functions to API to set up message address and data > pseries: added allocator for a block of IRQs > pseries pci: added MSI/MSIX support > > hw/msi.c | 14 +++ > hw/msi.h |1 + > hw/msix.c | 10 ++ > hw/msix.h |3 + > hw/spapr.c | 26 +- > hw/spapr.h |1 + > hw/spapr_pci.c | 266 +-- > hw/spapr_pci.h | 13 +++- > trace-events |9 ++ > 9 files changed, 331 insertions(+), 12 deletions(-) > > -- > 1.7.7.3 > > -- > To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/3] adding MSI/MSIX for PCI on POWER
Forgot to CC: someone :) On 14/06/12 14:29, Alexey Kardashevskiy wrote: > The following patches add MSIX support for PCI on POWER. > The first aim is virtio-pci so it was tested. It will also support > VFIO when it becomes available in public. > > Alexey Kardashevskiy (3): > msi/msix: added functions to API to set up message address and data > pseries: added allocator for a block of IRQs > pseries pci: added MSI/MSIX support > > hw/msi.c | 14 +++ > hw/msi.h |1 + > hw/msix.c | 10 ++ > hw/msix.h |3 + > hw/spapr.c | 26 +- > hw/spapr.h |1 + > hw/spapr_pci.c | 266 +-- > hw/spapr_pci.h | 13 +++- > trace-events |9 ++ > 9 files changed, 331 insertions(+), 12 deletions(-) > -- Alexey -- To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH 0/3] adding MSI/MSIX for PCI on POWER
The following patches add MSIX support for PCI on POWER. The first aim is virtio-pci so it was tested. It will also support VFIO when it becomes available in public. Alexey Kardashevskiy (3): msi/msix: added functions to API to set up message address and data pseries: added allocator for a block of IRQs pseries pci: added MSI/MSIX support hw/msi.c | 14 +++ hw/msi.h |1 + hw/msix.c | 10 ++ hw/msix.h |3 + hw/spapr.c | 26 +- hw/spapr.h |1 + hw/spapr_pci.c | 266 +-- hw/spapr_pci.h | 13 +++- trace-events |9 ++ 9 files changed, 331 insertions(+), 12 deletions(-) -- 1.7.7.3 -- To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html