[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V10 7/9] hw/misc/pvpanic: preparing for adding
From: |
peng.hao2 |
Subject: |
Re: [Qemu-devel] [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
reason.
I think I can use a way like iommu, and I've thought about it in doing my
current way.
thanks.
>thanks
>-- PMM
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH V10 7/9] hw/misc/pvpanic: preparing for adding configure interface,
peng.hao2 <=