On 17/7/24 13:53, Jiaxun Yang wrote:


在2024年7月17日七月 下午5:22,maobibo写道:
On 2024/7/16 下午2:40, Philippe Mathieu-Daudé wrote:
On 16/7/24 03:29, maobibo wrote:


On 2024/7/16 上午9:04, maobibo wrote:


On 2024/7/15 下午11:17, Philippe Mathieu-Daudé wrote:
On 4/7/24 05:37, Bibo Mao wrote:
Now loongson ipi and loongarch ipi share the same code with different
macro, loongson ipi has its separate function such mmio region,
loongarch ipi has other requirement such as irqchip in kernel.

Interrupt irqchip has strong relationship with architecture, since
it sends irq to vcpu and interfaces to get irqchip register is also
architecture specific.

Here like other architectures, base class TYPE_LOONGSON_IPI_COMMON
is added, it comes from loongson ipi mostly. And it defined four
abstract
interfaces which can be used for MIPS 3A4000 and Loongarch 3A5000
machine,
also can be used for 3A5000 irqchip in kernel mode soon.

Also Loongarch ipi and loongson ipi device are added here, it inherits
from base class TYPE_LOONGSON_IPI_COMMON. Loongarch ipi is tested,
loongson ipi device only passes to compile and make check, it is not
tested.

Bibo Mao (4):
    hw/intc/loongson_ipi_common: Add loongson ipi common class
    hw/intc/loongarch_ipi: Add loongarch ipi support
    hw/loongarch/virt: Replace loongson ipi with loongarch ipi
    hw/intc/loongson_ipi: reconstruct driver inherit from common class

I'll try to respin a clearer v3.
I am ok with it since it solve the problem, and it is suitable for
9.1 release. Only that in the long time we hope that intc emulation
driver has common base class + tcg/kvm driver, similar with other
architecture.

Sorry for the confusion, I had thought it was another topic.

Thanks for pointing out the problem and welcome the v3 version.

Please do not post v3, let me post it.
Hi Philippe,

QEMU 9.1 is coming to soft frozen stage, do you have enough time working
on it?  Is it ok to use bugfix patch for 9.1 release version?

Yeah, trying to do my best here. Again I expect splitting your work
in multiple commits would have accelerated the review process.


Bug fix is always qualified between soft freeze and release.

I tested the series for MIPS yesterday and can confirm it do work.

Will give my tags on v3.

Thanks, I think I'm getting sick so I'll post what I did so far,
only build-tested so far, but my brain is not working well (fever?)
so I'll let Bibo/you resolve any issues with v3 (respinning if
required) and then either Song or me can queue the result next
week before the freeze.

Regards,

Phil.

https://lore.kernel.org/all/20240627125819.62779-2-phi...@linaro.org/

After 9.1 is released, there will be enough time for patch v3.

Regards
Bibo, Mao



Reply via email to