Re: [PATCH v5 2/7] Revert "hw/isa/vt82c686: Remove intermediate IRQ forwarder"

[BALATON Zoltan]

On Thu, 2 Mar 2023, Philippe Mathieu-Daudé wrote:
> On 1/3/23 01:33, BALATON Zoltan wrote:
> > On Wed, 1 Mar 2023, BALATON Zoltan wrote:
> > > This partially reverts commit bb98e0f59cde846666d9fddc60ae74ef7ddfca17
> > > keeping the rename of a state field but reverting other cahanges which
> > > break interrupts on pegasos2.
> >
> > I've found this with just booting the MorphOS iso which now hangs without 
> > this revert when trying to read from the ide device.
>
> Can you add an Avocado test booting the MorphOS iso?

I think you had a patch for that before, so you remember where to find it? 
I can't test that though as I don't have avocado and it does not come with 
my distro. So maybe if you can revive your patch and test it that might 
work better.

Regards,
BALATON Zoltan

> > I think I've mentioned that I've also tried this way first but then ended 
> > up adding this because it was needed in a review of the patch earlier but I 
> > can't find that message now. For now it seems the easiest is to revert this 
> > and think about it later.
> >
> > Regards,
> > BALATON Zoltan
> >
> > > Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
> > > ---
> > > hw/isa/vt82c686.c | 10 +++++++++-
> > > 1 file changed, 9 insertions(+), 1 deletion(-)