[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/5] apic: add global apic_get_class()
From: |
Radim Krčmář |
Subject: |
Re: [Qemu-devel] [PATCH 1/5] apic: add global apic_get_class() |
Date: |
Tue, 27 Sep 2016 15:28:51 +0200 |
2016-09-23 17:17+0800, Peter Xu:
> On Thu, Sep 22, 2016 at 11:04:28PM +0200, Radim Krčmář wrote:
>> Every configuration has only up to one APIC class and we'll be extending
>> the class with a function that can be called without an instanced
>> object, so a direct access to the class is convenient.
>>
>> Signed-off-by: Radim Krčmář <address@hidden>
>> ---
>> diff --git a/hw/intc/apic_common.c b/hw/intc/apic_common.c
>> @@ -306,6 +314,12 @@ static void apic_common_realize(DeviceState *dev, Error
>> **errp)
>> info = APIC_COMMON_GET_CLASS(s);
>> info->realize(dev, errp);
>>
>> + if (apic_class && apic_class != info) {
>> + error_report("All APICs must be of the same class.");
>> + exit(1);
>> + }
>
> Can user trigger this error? If not, I'd prefer:
Shouldn't be able to.
> assert(!apic_class || apic_class == info);
I'll use that, thanks.
- [Qemu-devel] [PATCH 0/5] intel_iommu: fix EIM, Radim Krčmář, 2016/09/22
- [Qemu-devel] [PATCH 1/5] apic: add global apic_get_class(), Radim Krčmář, 2016/09/22
- [Qemu-devel] [PATCH 3/5] intel_iommu: pass whole remapped addresses to apic, Radim Krčmář, 2016/09/22
- [Qemu-devel] [PATCH 4/5] intel_iommu: add "eim" property, Radim Krčmář, 2016/09/22
- [Qemu-devel] [PATCH 2/5] apic: add send_msi() to APICCommonClass, Radim Krčmář, 2016/09/22
- [Qemu-devel] [PATCH 5/5] intel_iommu: do not allow EIM without KVM support, Radim Krčmář, 2016/09/22