[Top][All Lists]

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

Re: [Qemu-arm] [PATCH V10 7/9] hw/misc/pvpanic: preparing for adding co

From: peng.hao2
Subject: Re: [Qemu-arm] [PATCH V10 7/9] hw/misc/pvpanic: preparing for adding configure interface
Date: Sat, 1 Dec 2018 17:11:55 +0800 (CST)

>On Wed, 28 Nov 2018 at 03:50, Peng Hao <address@hidden> wrote:
>> Prepare for pvpanic-mmio configure interface.
>> Signed-off-by: Peng Hao <address@hidden>
>> ---
>>  hw/arm/sysbus-fdt.c |  2 ++
>>  hw/arm/virt.c       |  2 ++
>>  hw/misc/pvpanic.c   | 11 +++++++++--
>>  3 files changed, 13 insertions(+), 2 deletions(-)
>This looks rather odd. The sysbus-fdt.c code is for
>handling devices that we must pass through from the
>host system (including pulling in device tree fragments
>from the host). The pvpanic device isn't a host pass
>through device so I don't think it should be touching
>that code at all. It's just a simple MMIO device.
>I suspect the reason you've done this is that you're
>trying to get "-device pvpanic" to work on the command
>line. I recommend not trying to get that to work.
>MMIO devices are not pluggable devices, and -device
>is not expected to work with them.
yes, I use -device pvpanic-mmio. Thank you for pointing this out.
 I also felt a little odd when I realized it. But I didn't think about the 
I think I can use a way like iommu, and I've thought about it in doing my 
current way.
>-- PMM

reply via email to

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