[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/3] Change qemu_set_irq() to return status info
From: |
Paul Brook |
Subject: |
Re: [Qemu-devel] [PATCH 1/3] Change qemu_set_irq() to return status information. |
Date: |
Sun, 29 Jun 2008 19:11:01 +0100 |
User-agent: |
KMail/1.9.9 |
On Sunday 29 June 2008, Gleb Natapov wrote:
> On Sun, Jun 29, 2008 at 03:38:10PM +0100, Paul Brook wrote:
> > On Sunday 29 June 2008, Gleb Natapov wrote:
> > > The return value is less then zero if interrupt is masked, zero if it
> > > is known that interrupt is lost (due to coalescing) or greater then
> > > zero if interrupt is delivered or was successfully queued for delivery
> > > by interrupt controller. Device emulation can use this info as it
> > > pleases. Included patch adds detection of interrupt coalescing into PIC
> > > and APIC code for edge triggered interrupts.
> >
> > This is woefully incomplete, and obviously hasn't been tested on anything
> > other than x86 targets.
>
> Yes, you are right. It was not tested on anything other than x86. Do you
> see why this approach will not work on other architectures? Can you
> elaborate on what current patch is missing for other architectures
> support?
Well, if nothing else there's 40+ interrupt controllers that need fixing up
before it'll even compile cleanly (mismatching function prototypes are IMHO
not acceptable).
> The initial goal is to fix RTC/PIT problem on x86 while do not
> hurt any other architectures in any way.
Well, you're introducing a fair amount of churn, so it'd better work for other
architectures too. Otherwise we're liable to have to rewrite it later. I
can't say offhand whether your approach will work on other architectures.
qemu has quite a wide variety of interrupt controllers and timers, you should
check whether they fit into your model.
Paul
- [Qemu-devel] [PATCH 2/3] Fix time drift problem under high load when PIT is in use., (continued)
- Re: [Qemu-devel] [PATCH 1/3] Change qemu_set_irq() to return status information., Paul Brook, 2008/06/29
- Re: [Qemu-devel] [PATCH 1/3] Change qemu_set_irq() to return status information., Gleb Natapov, 2008/06/29
- Re: [Qemu-devel] [PATCH 1/3] Change qemu_set_irq() to return status information.,
Paul Brook <=
- Re: [Qemu-devel] [PATCH 1/3] Change qemu_set_irq() to return status information., Gleb Natapov, 2008/06/29
- Re: [Qemu-devel] [PATCH 1/3] Change qemu_set_irq() to return status information., Paul Brook, 2008/06/29
- Re: [Qemu-devel] [PATCH 1/3] Change qemu_set_irq() to return status information., Gleb Natapov, 2008/06/29
- Re: [Qemu-devel] [PATCH 1/3] Change qemu_set_irq() to return status information., Gleb Natapov, 2008/06/30
- Re: [Qemu-devel] [PATCH 1/3] Change qemu_set_irq() to return status information., Paul Brook, 2008/06/30
- Re: [Qemu-devel] [PATCH 1/3] Change qemu_set_irq() to return status information., Gleb Natapov, 2008/06/30
- Re: [Qemu-devel] [PATCH 1/3] Change qemu_set_irq() to return status information., Paul Brook, 2008/06/30
- [Qemu-devel] Re: [PATCH 1/3] Change qemu_set_irq() to return status information., Jan Kiszka, 2008/06/29
- Re: [Qemu-devel] Re: [PATCH 1/3] Change qemu_set_irq() to return status information., Paul Brook, 2008/06/29
- Re: [Qemu-devel] Re: [PATCH 1/3] Change qemu_set_irq() to return status information., Dor Laor, 2008/06/29