Messages in this thread | | | Subject | Re: [PATCH v3] kvm/x86 : Remove redundant function implement | From | Paolo Bonzini <> | Date | Thu, 21 May 2020 15:36:37 +0200 |
| |
On 21/05/20 11:27, Vitaly Kuznetsov wrote: > 彭浩(Richard) <richard.peng@oppo.com> writes: > >> pic_in_kernel(),ioapic_in_kernel() and irqchip_kernel() have the >> same implementation. > 'pic_in_kernel()' name is misleading, one may think this is about lapic > and it's not. Also, ioapic_in_kernel() doesn't have that many users, can > we maybe converge on using irqchip_*() functions everywhere? >
Richard's patch was my recommendation actually.
PIC is not the LAPIC and not the IOAPIC; even though right now the implementation is the same for pic_in_kernel and ioapic_in_kernel and irqchip_kernel(), that's more or less an implementation detail.
Paolo
| |