Re: [PATCH v5 5/7] hw/isa/vt82c686: Work around missing level sensitive irq in i8259 model

2023-03-02 Thread BALATON Zoltan
On Thu, 2 Mar 2023, David Woodhouse wrote: On Wed, 2023-03-01 at 23:47 +0100, BALATON Zoltan wrote: On Wed, 1 Mar 2023, David Woodhouse wrote: On Wed, 2023-03-01 at 19:01 +0100, BALATON Zoltan wrote: It isn't a *correct* fix without a little bit more typing, but does this make it work?

Re: [PATCH v5 5/7] hw/isa/vt82c686: Work around missing level sensitive irq in i8259 model

2023-03-02 Thread David Woodhouse
On Wed, 2023-03-01 at 23:47 +0100, BALATON Zoltan wrote: > On Wed, 1 Mar 2023, David Woodhouse wrote: > > On Wed, 2023-03-01 at 19:01 +0100, BALATON Zoltan wrote: > > > > > > > It isn't a *correct* fix without a little bit more typing, but does > > > > this make it work? > > > > > > > > diff

Re: [PATCH v5 5/7] hw/isa/vt82c686: Work around missing level sensitive irq in i8259 model

2023-03-01 Thread BALATON Zoltan
On Wed, 1 Mar 2023, David Woodhouse wrote: On Wed, 2023-03-01 at 19:01 +0100, BALATON Zoltan wrote: It isn't a *correct* fix without a little bit more typing, but does this make it work? diff --git a/hw/intc/i8259.c b/hw/intc/i8259.c index 17910f3bcb..36ebcff025 100644 --- a/hw/intc/i8259.c

Re: [PATCH v5 5/7] hw/isa/vt82c686: Work around missing level sensitive irq in i8259 model

2023-03-01 Thread David Woodhouse
On Wed, 2023-03-01 at 19:01 +0100, BALATON Zoltan wrote: > > > It isn't a *correct* fix without a little bit more typing, but does > > this make it work? > > > > diff --git a/hw/intc/i8259.c b/hw/intc/i8259.c > > index 17910f3bcb..36ebcff025 100644 > > --- a/hw/intc/i8259.c > > +++

Re: [PATCH v5 5/7] hw/isa/vt82c686: Work around missing level sensitive irq in i8259 model

2023-03-01 Thread Bernhard Beschow
Am 1. März 2023 14:05:31 UTC schrieb David Woodhouse : >On Wed, 2023-03-01 at 14:18 +0100, BALATON Zoltan wrote: >> > Are you sure the PIC ELCR is actually set for the lines you're having >> > trouble with? Is that something the Pegasos SmartFirmware would have >> > done, and MorphOS is

Re: [PATCH v5 5/7] hw/isa/vt82c686: Work around missing level sensitive irq in i8259 model

2023-03-01 Thread BALATON Zoltan
On Wed, 1 Mar 2023, David Woodhouse wrote: On Wed, 2023-03-01 at 14:18 +0100, BALATON Zoltan wrote: Are you sure the PIC ELCR is actually set for the lines you're having trouble with? Is that something the Pegasos SmartFirmware would have done, and MorphOS is expecting to inherit but isn't

Re: [PATCH v5 5/7] hw/isa/vt82c686: Work around missing level sensitive irq in i8259 model

2023-03-01 Thread David Woodhouse
On Wed, 2023-03-01 at 14:18 +0100, BALATON Zoltan wrote: > > Are you sure the PIC ELCR is actually set for the lines you're having > > trouble with? Is that something the Pegasos SmartFirmware would have > > done, and MorphOS is expecting to inherit but isn't actually setting up > > for itself? >

Re: [PATCH v5 5/7] hw/isa/vt82c686: Work around missing level sensitive irq in i8259 model

2023-03-01 Thread BALATON Zoltan
On Wed, 1 Mar 2023, David Woodhouse wrote: On Wed, 2023-03-01 at 12:27 +0100, BALATON Zoltan wrote: On Wed, 1 Mar 2023, Bernhard Beschow wrote: Am 1. März 2023 00:17:11 UTC schrieb BALATON Zoltan : MorphOS sets the ISA PIC to level sensitive mode but QEMU does not support that so this causes

Re: [PATCH v5 5/7] hw/isa/vt82c686: Work around missing level sensitive irq in i8259 model

2023-03-01 Thread David Woodhouse
On Wed, 2023-03-01 at 12:27 +0100, BALATON Zoltan wrote: > On Wed, 1 Mar 2023, Bernhard Beschow wrote: > > Am 1. März 2023 00:17:11 UTC schrieb BALATON Zoltan : > > > MorphOS sets the ISA PIC to level sensitive mode but QEMU does > > > not > > > support that so this causes a freeze if multiple

Re: [PATCH v5 5/7] hw/isa/vt82c686: Work around missing level sensitive irq in i8259 model

2023-03-01 Thread BALATON Zoltan
On Wed, 1 Mar 2023, Bernhard Beschow wrote: Am 1. März 2023 00:17:11 UTC schrieb BALATON Zoltan : MorphOS sets the ISA PIC to level sensitive mode but QEMU does not support that so this causes a freeze if multiple devices try to raise a shared interrupt. Work around it by lowering the interrupt

Re: [PATCH v5 5/7] hw/isa/vt82c686: Work around missing level sensitive irq in i8259 model

2023-02-28 Thread Bernhard Beschow
Am 1. März 2023 00:17:11 UTC schrieb BALATON Zoltan : >MorphOS sets the ISA PIC to level sensitive mode but QEMU does not >support that so this causes a freeze if multiple devices try to raise >a shared interrupt. Work around it by lowering the interrupt before >raising it again if it is

[PATCH v5 5/7] hw/isa/vt82c686: Work around missing level sensitive irq in i8259 model

2023-02-28 Thread BALATON Zoltan
MorphOS sets the ISA PIC to level sensitive mode but QEMU does not support that so this causes a freeze if multiple devices try to raise a shared interrupt. Work around it by lowering the interrupt before raising it again if it is already raised. This could be reverted when the i8259 model is