qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH RFC V4 4/4] Add virt-v3 machine that uses GIC-50


From: Shlomo Pongratz
Subject: Re: [Qemu-devel] [PATCH RFC V4 4/4] Add virt-v3 machine that uses GIC-500
Date: Mon, 28 Sep 2015 18:20:19 +0300

Thanks,

I'll fix it and submit together with with the changes required by Peter.

Best regards,

S.P. 

On Thursday, September 24, 2015, Christopher Covington <address@hidden> wrote:
On 09/24/2015 02:03 PM, Christopher Covington wrote:
> Hi,
>
> On 09/17/2015 01:38 PM, Shlomo Pongratz wrote:
>> From: Pavel Fedin <address@hidden>
>>
>> I would like to offer this, slightly improved implementation. The key thing is a new
>> kernel_irqchip_type member in Machine class. Currently it it used only by virt machine for
>> its internal purposes, however in future it is to be passed to KVM in
>> kvm_irqchip_create(). The variable is defined as int in order to be architecture agnostic,
>> for potential future users.
>>
>> Signed-off-by: Pavel Fedin <address@hidden>
>> ---
>>  hw/arm/virt.c         | 72 +++++++++++++++++++++++++++++++++++++++++++++------
>>  include/hw/arm/fdt.h  |  2 ++
>>  include/hw/arm/virt.h |  1 +
>>  target-arm/machine.c  |  7 ++++-
>>  4 files changed, 73 insertions(+), 9 deletions(-)
>>
>> diff --git a/hw/arm/virt.c b/hw/arm/virt.c
>> index 4d15309..4c2ae7f 100644
>> --- a/hw/arm/virt.c
>> +++ b/hw/arm/virt.c
>
>> @@ -445,6 +462,14 @@ static void create_gic(VirtBoardInfo *vbi, qemu_irq *pic, int type, bool secure)
>>          sysbus_mmio_map(gicbusdev, 1, vbi->memmap[VIRT_GIC_CPU].base);
>>      }
>>
>> +    if (type == 3) {
>> +        /* Connect GIC to CPU */
>> +        for (i = 0; i < smp_cpus; i++) {
>> +            CPUState *cpu = qemu_get_cpu(i);
>> +            aatch64_registers_with_opaque_set(OBJECT(cpu), (void *)gicdev);
>
> Typo--should be "aarch64".
>
> With that, feel free to add the following if it's any use:
>
> Tested-by: Christopher Covington <address@hidden>

I originally tested building only for aarch64-softmmu, but I've now noticed a
build issue with arm-softmmu.

Christopher Covington

--
Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

reply via email to

[Prev in Thread] Current Thread [Next in Thread]