* [PATCH v2 0/1] MIPS: lantiq: enable all hardware interrupts on second VPE @ 2022-09-21 20:59 Aleksander Jan Bajkowski 2022-09-21 20:59 ` [PATCH v2 1/1] " Aleksander Jan Bajkowski 0 siblings, 1 reply; 3+ messages in thread From: Aleksander Jan Bajkowski @ 2022-09-21 20:59 UTC (permalink / raw) To: john, martin.blumenstingl, hauke, tsbogend, maz, linux-mips, linux-kernel Cc: Aleksander Jan Bajkowski v2: - switched to upstream recommendation. Aleksander Jan Bajkowski (1): MIPS: lantiq: enable all hardware interrupts on second VPE arch/mips/lantiq/prom.c | 26 ++++++++++++++++++++++++-- 1 file changed, 24 insertions(+), 2 deletions(-) -- 2.30.2 ^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH v2 1/1] MIPS: lantiq: enable all hardware interrupts on second VPE 2022-09-21 20:59 [PATCH v2 0/1] MIPS: lantiq: enable all hardware interrupts on second VPE Aleksander Jan Bajkowski @ 2022-09-21 20:59 ` Aleksander Jan Bajkowski 2022-09-24 10:45 ` Thomas Bogendoerfer 0 siblings, 1 reply; 3+ messages in thread From: Aleksander Jan Bajkowski @ 2022-09-21 20:59 UTC (permalink / raw) To: john, martin.blumenstingl, hauke, tsbogend, maz, linux-mips, linux-kernel Cc: Aleksander Jan Bajkowski This patch is needed to handle interrupts by the second VPE on the Lantiq ARX100, xRX200, xRX300 and xRX330 SoCs. Switching some ICU interrupts to the second VPE results in a hang. Currently, the vsmp_init_secondary() function is responsible for enabling these interrupts. It only enables Malta-specific interrupts (SW0, SW1, HW4 and HW5). The MIPS core has 8 interrupts defined. On Lantiq SoCs, hardware interrupts are wired to an ICU instance. Each VPE has an independent instance of the ICU. The mapping of the ICU interrupts is shown below: SW0(IP0) - IPI call, SW1(IP1) - IPI resched, HW0(IP2) - ICU 0-31, HW1(IP3) - ICU 32-63, HW2(IP4) - ICU 64-95, HW3(IP5) - ICU 96-127, HW4(IP6) - ICU 128-159, HW5(IP7) - timer. This patch enables all interrupt lines on the second VPE. This problem affects multithreaded SoCs with a custom interrupt controller. SOCs with 1004Kc core and newer use the MIPS GIC. At this point, I am aware that the Realtek RTL839x and RTL930x SoCs may need a similar fix. In the future, this may be replaced with some generic solution. Tested on Lantiq xRX200. Suggested-by: Thomas Bogendoerfer <tsbogend@alpha.franken.de> Signed-off-by: Aleksander Jan Bajkowski <olek2@wp.pl> --- arch/mips/lantiq/prom.c | 26 ++++++++++++++++++++++++-- 1 file changed, 24 insertions(+), 2 deletions(-) diff --git a/arch/mips/lantiq/prom.c b/arch/mips/lantiq/prom.c index c731082a0c42..be4829cc7a3a 100644 --- a/arch/mips/lantiq/prom.c +++ b/arch/mips/lantiq/prom.c @@ -34,6 +34,14 @@ unsigned long physical_memsize = 0L; */ static struct ltq_soc_info soc_info; +/* + * These structs are used to override vsmp_init_secondary() + */ +#if defined(CONFIG_MIPS_MT_SMP) +extern const struct plat_smp_ops vsmp_smp_ops; +static struct plat_smp_ops lantiq_smp_ops; +#endif + const char *get_system_type(void) { return soc_info.sys_type; @@ -84,6 +92,17 @@ void __init plat_mem_setup(void) __dt_setup_arch(dtb); } +#if defined(CONFIG_MIPS_MT_SMP) +static void lantiq_init_secondary(void) +{ + /* + * MIPS CPU startup function vsmp_init_secondary() will only + * enable some of the interrupts for the second CPU/VPE. + */ + set_c0_status(ST0_IM); +} +#endif + void __init prom_init(void) { /* call the soc specific detetcion code and get it to fill soc_info */ @@ -95,7 +114,10 @@ void __init prom_init(void) prom_init_cmdline(); #if defined(CONFIG_MIPS_MT_SMP) - if (register_vsmp_smp_ops()) - panic("failed to register_vsmp_smp_ops()"); + if (cpu_has_mipsmt) { + lantiq_smp_ops = vsmp_smp_ops; + lantiq_smp_ops.init_secondary = lantiq_init_secondary; + register_smp_ops(&lantiq_smp_ops); + } #endif } -- 2.30.2 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2 1/1] MIPS: lantiq: enable all hardware interrupts on second VPE 2022-09-21 20:59 ` [PATCH v2 1/1] " Aleksander Jan Bajkowski @ 2022-09-24 10:45 ` Thomas Bogendoerfer 0 siblings, 0 replies; 3+ messages in thread From: Thomas Bogendoerfer @ 2022-09-24 10:45 UTC (permalink / raw) To: Aleksander Jan Bajkowski Cc: john, martin.blumenstingl, hauke, maz, linux-mips, linux-kernel On Wed, Sep 21, 2022 at 10:59:44PM +0200, Aleksander Jan Bajkowski wrote: > This patch is needed to handle interrupts by the second VPE on the Lantiq > ARX100, xRX200, xRX300 and xRX330 SoCs. Switching some ICU interrupts to > the second VPE results in a hang. Currently, the vsmp_init_secondary() > function is responsible for enabling these interrupts. It only enables > Malta-specific interrupts (SW0, SW1, HW4 and HW5). > > The MIPS core has 8 interrupts defined. On Lantiq SoCs, hardware > interrupts are wired to an ICU instance. Each VPE has an independent > instance of the ICU. The mapping of the ICU interrupts is shown below: > SW0(IP0) - IPI call, > SW1(IP1) - IPI resched, > HW0(IP2) - ICU 0-31, > HW1(IP3) - ICU 32-63, > HW2(IP4) - ICU 64-95, > HW3(IP5) - ICU 96-127, > HW4(IP6) - ICU 128-159, > HW5(IP7) - timer. > > This patch enables all interrupt lines on the second VPE. > > This problem affects multithreaded SoCs with a custom interrupt controller. > SOCs with 1004Kc core and newer use the MIPS GIC. At this point, I am aware > that the Realtek RTL839x and RTL930x SoCs may need a similar fix. In the > future, this may be replaced with some generic solution. > > Tested on Lantiq xRX200. > > Suggested-by: Thomas Bogendoerfer <tsbogend@alpha.franken.de> > Signed-off-by: Aleksander Jan Bajkowski <olek2@wp.pl> > --- > arch/mips/lantiq/prom.c | 26 ++++++++++++++++++++++++-- > 1 file changed, 24 insertions(+), 2 deletions(-) applied to mips-next. Thomas. -- Crap can work. Given enough thrust pigs will fly, but it's not necessarily a good idea. [ RFC1925, 2.3 ] ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-09-24 12:06 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-09-21 20:59 [PATCH v2 0/1] MIPS: lantiq: enable all hardware interrupts on second VPE Aleksander Jan Bajkowski 2022-09-21 20:59 ` [PATCH v2 1/1] " Aleksander Jan Bajkowski 2022-09-24 10:45 ` Thomas Bogendoerfer
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).