On 2016-05-22 12:21, David Kiarie wrote:
> +static void amdvi_generate_msi_interrupt(AMDVIState *s)
> +{
> +    uint64_t addr;
> +    uint32_t data;
> +
> +    if ((pci_get_word(s->dev.config + s->dev.msi_cap + PCI_MSI_FLAGS) &
> +       PCI_MSI_FLAGS_ENABLE)) {
> +        addr = pci_get_quad(s->dev.config + s->dev.msi_cap +
> +                            PCI_MSI_ADDRESS_LO);
> +        data = pci_get_word(s->dev.config + s->dev.msi_cap +
> +                            PCI_MSI_DATA_64);

There is msi_enabled and msi_get_message - will make this a bit more
compact.

Jan