Paolo Bonzini <pbonz...@redhat.com> writes:
> On 11/11/22 13:26, Alex Bennée wrote:
>> if (addr > 0xfff || !index) {
>> switch (attrs.requester_type) {
>> }
>> MSIMessage msi = { .address = addr, .data = val };
>> apic_send_msi(&msi);
>> return MEMTX_OK;
>> }
>
>
>> which at least gets things booting properly. Does this seem like a
>> better modelling of the APIC behaviour?
>
> Yes and you don't even need the "if", just do MTRT_CPU vs everything
> else.
Can the CPU trigger MSIs by writing to this area of memory? I went for
the explicit switch for clarity but are you saying:
if (attrs.requester_type != MTRT_CPU) {
MSIMessage msi = { .address = addr, .data = val };
apic_send_msi(&msi);
return MEMTX_OK;
} else {
return MEMTX_ACESSS_ERROR;
}
for the MSI range?
>
> Paolo
--
Alex Bennée
