[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 2/5] hpet: entitle more irq pins for hpet
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH v5 2/5] hpet: entitle more irq pins for hpet |
Date: |
Mon, 30 Sep 2013 12:30:00 +0300 |
On Mon, Sep 30, 2013 at 11:06:48AM +0200, Paolo Bonzini wrote:
> Il 30/09/2013 11:06, Michael S. Tsirkin ha scritto:
> > On Mon, Sep 30, 2013 at 04:02:29PM +0800, liu ping fan wrote:
> >> On Sun, Sep 29, 2013 at 12:15 PM, Michael S. Tsirkin <address@hidden>
> >> wrote:
> >>> On Sun, Sep 29, 2013 at 11:49:41AM +0800, liu ping fan wrote:
> >>>> On Sun, Sep 29, 2013 at 3:56 AM, Michael S. Tsirkin <address@hidden>
> >>>> wrote:
> >>>>> On Thu, Sep 12, 2013 at 11:25:15AM +0800, Liu Ping Fan wrote:
> >>>>>> On PC, IRQ2/8 can be reserved for hpet timer 0/1. And pin 16~23
> >>>>>> of ioapic can be dynamically assigned to hpet as guest chooses.
> >>>>>> (Will enable them after introducing pc 1.6 compat)
> >>>>>>
> >>>>>> Signed-off-by: Liu Ping Fan <address@hidden>
> >>>>>> ---
> >>>>>> hw/timer/hpet.c | 13 +++++++++++--
> >>>>>> 1 file changed, 11 insertions(+), 2 deletions(-)
> >>>>>>
> >>>>>> diff --git a/hw/timer/hpet.c b/hw/timer/hpet.c
> >>>>>> index 8429eb3..46903b9 100644
> >>>>>> --- a/hw/timer/hpet.c
> >>>>>> +++ b/hw/timer/hpet.c
> >>>>>> @@ -25,6 +25,7 @@
> >>>>>> */
> >>>>>>
> >>>>>> #include "hw/hw.h"
> >>>>>> +#include "hw/boards.h"
> >>>>>> #include "hw/i386/pc.h"
> >>>>>> #include "ui/console.h"
> >>>>>> #include "qemu/timer.h"
> >>>>>> @@ -42,6 +43,12 @@
> >>>>>>
> >>>>>> #define HPET_MSI_SUPPORT 0
> >>>>>>
> >>>>>> +/* For bug compat, using only IRQ2. Soon it will be fixed as
> >>>>>> + * 0xff0104ULL, i.e using IRQ16~23, IRQ8 and IRQ2
> >>>>>
> >>>>> So users are expected to stick a bitmask of legal
> >>>>> pins here?
> >>>>> I think that's a bit too much rope to give to users.
> >>>>> Don't you think?
> >>>>>
> >>>> Sorry, not understand your meaning exactly. But the scene will be:
> >>>> guest kernel polls the ability bitmask, and pick up one pin which is
> >>>> not occupied or can be shared with the level-trigger and low-active.
> >>>> So is it rope?
> >>>
> >>> I merely say that it's better to make this a bool or bit property.
> >>> UINT32 is too much flexibility imho.
> >>>
> >> The interrupt capability is export to guest by register
> >> Tn_INT_ROUTE_CAP[63:32]. So it is useless to make them as a bit
> >> property. Do you think so?
> >>
> >> Regards
> >> Pingfan
> >
> > I think we merely need to support two modes:
> > - qemu 1.6 and older compatible
> > - compatible to actual hardware
> >
> > Why would we let users configure an arbitrary
> > configuration which isn't compatible to either
> > old qemu or real hardware?
>
> The actual setting depends on the chipset. For example, the "real"
> PIIX4 is the same as the QEMU PIIX4, only Q35 uses the new value.
>
> If in the future we had a chipset with more than 24 GSIs, you would have
> a third possibility.
>
> Paolo
I was really only talking about q35 here.
I thought it's ugly that users can control intcap
directly. Can object_set_property be used after
qdev_try_create?
PIIX has another issue:
the default value in hpet is really Q35 specific,
that's also kind of ugly, isn't it?
--
MST
- Re: [Qemu-devel] [PATCH v5 1/5] hpet: inverse polarity when pin above ISA_NUM_IRQS, (continued)
[Qemu-devel] [PATCH v5 2/5] hpet: entitle more irq pins for hpet, Liu Ping Fan, 2013/09/11
- Re: [Qemu-devel] [PATCH v5 2/5] hpet: entitle more irq pins for hpet, Michael S. Tsirkin, 2013/09/28
- Re: [Qemu-devel] [PATCH v5 2/5] hpet: entitle more irq pins for hpet, liu ping fan, 2013/09/28
- Re: [Qemu-devel] [PATCH v5 2/5] hpet: entitle more irq pins for hpet, Michael S. Tsirkin, 2013/09/29
- Re: [Qemu-devel] [PATCH v5 2/5] hpet: entitle more irq pins for hpet, liu ping fan, 2013/09/30
- Re: [Qemu-devel] [PATCH v5 2/5] hpet: entitle more irq pins for hpet, Michael S. Tsirkin, 2013/09/30
- Re: [Qemu-devel] [PATCH v5 2/5] hpet: entitle more irq pins for hpet, Paolo Bonzini, 2013/09/30
- Re: [Qemu-devel] [PATCH v5 2/5] hpet: entitle more irq pins for hpet,
Michael S. Tsirkin <=
- Re: [Qemu-devel] [PATCH v5 2/5] hpet: entitle more irq pins for hpet, Paolo Bonzini, 2013/09/30
- Re: [Qemu-devel] [PATCH v5 2/5] hpet: entitle more irq pins for hpet, Michael S. Tsirkin, 2013/09/30
[Qemu-devel] [PATCH v5 3/5] PC: use qdev_xx to create hpet instead of sysbus_create_xx, Liu Ping Fan, 2013/09/11
[Qemu-devel] [PATCH v5 4/5] PC: differentiate hpet's interrupt capability on piix and q35, Liu Ping Fan, 2013/09/11
[Qemu-devel] [PATCH v5 5/5] PC-1.6: add compatibility for hpet intcap on pc-q35-1.6, Liu Ping Fan, 2013/09/11
Re: [Qemu-devel] [PATCH v5 0/5] bugs fix for hpet, Paolo Bonzini, 2013/09/12
Re: [Qemu-devel] [PATCH v5 0/5] bugs fix for hpet, liu ping fan, 2013/09/25