[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 0/4] xics: Eliminate unnecessary class
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH 0/4] xics: Eliminate unnecessary class |
Date: |
Tue, 24 Sep 2019 12:22:49 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
On 9/24/19 12:06 PM, Greg Kurz wrote:
> On Tue, 24 Sep 2019 11:47:51 +0200
> Philippe Mathieu-Daudé <address@hidden> wrote:
>
>> On 9/24/19 6:59 AM, David Gibson wrote:
>>> The XICS interrupt controller device used to have separate subtypes
>>> for the KVM and non-KVM variant of the device. That was a bad idea,
>>> because it leaked information that should be entirely host-side
>>> implementation specific to the kinda-sorta guest visible QOM class
>>> names.
>>>
>>> We eliminated the KVM specific class some time ago, but it's left
>>> behind a distinction between the TYPE_ICS_BASE abstract class and
>>> TYPE_ICS_SIMPLE subtype which no longer serves any purpose.
>>>
>>> This series collapses the two types back into one.
>>>
>>> David Gibson (4):
>>> xics: Eliminate 'reject', 'resend' and 'eoi' class hooks
>>> xics: Merge reset and realize hooks
>>> xics: Rename misleading ics_simple_*() functions
>>> xics: Merge TYPE_ICS_BASE and TYPE_ICS_SIMPLE classes
>>
>> Please remove the qemu_register_reset() call in hw/intc/xics.c,
>
> No. This is needed because the XICS devices don't sit in a bus and
> dc->reset doesn't get called by anyone.
Oh right, now I remember a previous discussion.
Please add a comment about this before the qemu_register_reset() call!
>> then for the series:
>> Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by stands :)
- Re: [PATCH 3/4] xics: Rename misleading ics_simple_*() functions, (continued)
Re: [PATCH 0/4] xics: Eliminate unnecessary class, Philippe Mathieu-Daudé, 2019/09/24