Re: [RFC PATCH] irqchip/gic-v3: Do not enable irqs when handling spurious interrups

Hello Marc, 在 2021/4/16 22:15, Marc Zyngier 写道: [+ Mark] On Fri, 16 Apr 2021 07:22:17 +0100, He Ying wrote: We found this problem in our kernel src tree: [ 14.816231] [ cut here ] [ 14.816231] kernel BUG at irq.c:99! [ 14.816232] Internal error: Oops - BUG: 0

[RFC PATCH] irqchip/gic-v3: Do not enable irqs when handling spurious interrups

f3117 ("arm64: Fix interrupt tracing in the presence of NMIs") Signed-off-by: He Ying --- drivers/irqchip/irq-gic-v3.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/irqchip/irq-gic-v3.c b/drivers/irqchip/irq-gic-v3.c index 94b89258d045..d3b52734a2c

[PATCH v2 -next] phy: phy-mtk-hdmi: Remove redundant dev_err call in mtk_hdmi_phy_probe()

There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Reported-by: Hulk Robot Signed-off-by: He Ying --- v2: - Use 'return PTR_ERR()' instead of 'ret = PTR_ERR();return ret;'. drivers/phy/mediatek/phy-mt

[PATCH v2 -next] phy: phy-mtk-mipi-dsi: Remove redundant dev_err call in mtk_mipi_tx_probe()

There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Reviewed-by: Chunfeng Yun Reported-by: Hulk Robot Signed-off-by: He Ying --- v2: - Use 'return PTR_ERR();' instead of 'ret = PTR_ERR();return ret;&

[PATCH resend] cpuidle: Fix ARM_QCOM_SPM_CPUIDLE configuration

undefined reference to `cpu_resume_arm' Note that cpu_resume_arm is defined when MMU is set. So, add dependency on MMU in ARM_QCOM_SPM_CPUIDLE configuration. Fixes: a871be6b8eee ("cpuidle: Convert Qualcomm SPM driver to a generic CPUidle driver") Reported-by: Hulk Robot Signed-off-

[PATCH V2] firmware: qcom-scm: Fix QCOM_SCM configuration

fined when HAVE_ARM_SMCCC is y. So add dependency on HAVE_ARM_SMCCC in QCOM_SCM configuration. Fixes: 916f743da354 ("firmware: qcom: scm: Move the scm driver to drivers/firmware") Reported-by: Hulk Robot Signed-off-by: He Ying --- V2: - Add Fixes tag drivers/firmware/Kconfig | 1 + 1

[PATCH] cpuidle: Fix ARM_QCOM_SPM_CPUIDLE configuration

undefined reference to `cpu_resume_arm' So, add dependency on MMU in ARM_QCOM_SPM_CPUIDLE configuration. Reported-by: Hulk Robot Signed-off-by: He Ying --- drivers/cpuidle/Kconfig.arm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/cpuidle/Kconfig.arm b/drivers/cpui

[PATCH] firmware: qcom-scm: Fix QCOM_SCM configuration

on HAVE_ARM_SMCCC in QCOM_SCM configuration. Reported-by: Hulk Robot Signed-off-by: He Ying --- drivers/firmware/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/firmware/Kconfig b/drivers/firmware/Kconfig index 3f14dffb9669..5dd19dbd67a3 100644 --- a/drivers/firmware/Kconfig +++ b/driv

[PATCH -next] docs: arm64: Fix a grammar error

depending -> depending on Reported-by: Hulk Robot Signed-off-by: He Ying --- Documentation/arm64/tagged-address-abi.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/arm64/tagged-address-abi.rst b/Documentation/arm64/tagged-address-abi.rst in

[PATCH] docs: powerpc: Fix misspellings and grammar errors

Reported-by: Hulk Robot Signed-off-by: He Ying --- Documentation/powerpc/booting.rst| 2 +- Documentation/powerpc/dawr-power9.rst| 2 +- Documentation/powerpc/eeh-pci-error-recovery.rst | 2 +- Documentation/powerpc/elfnote.rst| 2 +- Documentation

[PATCH V3 -next] powerpc: kernel/time.c - cleanup warnings

ve 'dtl_consumer' definition behind "include " because it is declared there. Reported-by: Hulk Robot Signed-off-by: He Ying --- V2: - Instead of including linux/mc146818rtc.h in powerpc kernel/time.c, declare rtc_lock in powerpc asm/time.h. V3: - Recover to V1, that is

[PATCH v2 -next] powerpc: kernel/time.c - cleanup warnings

Should it be static? Declare 'decrementer_max' and 'rtc_lock' in powerpc asm/time.h. Rename 'rtc_lock' in drviers/rtc/rtc-vr41xx.c to 'vr41xx_rtc_lock' to avoid the conflict with the variable in powerpc asm/time.h. Move 'dtl_consumer' definition behind "inc

[PATCH -next] powerpc: kernel/time.c - cleanup warnings

Should it be static? Declare 'decrementer_max' in arch/powerpc/include/asm/time.h. And include proper header in which 'rtc_lock' is declared. Move 'dtl_consumer' definition behind "include " because 'dtl_consumer' is declared there. Reported-by: Hulk Robot

[PATCH v2] smp: kernel/panic.c - silence warnings

s for these three functions in include/linux/smp.h. Reported-by: Hulk Robot Signed-off-by: He Ying --- V1->V2: - fix some misspellings include/linux/smp.h | 8 1 file changed, 8 insertions(+) diff --git a/include/linux/smp.h b/include/linux/smp.h index 70c6f6284dcf..27008a1c8111 10064

[PATCH] powerpc: arch/powerpc/kernel/setup_64.c - cleanup warnings

uaccess_flush' as static because they are not referenced outside the file. Include asm/security_features.h in which 'rfi_flush' is declared. Reported-by: Hulk Robot Signed-off-by: He Ying --- arch/powerpc/kernel/setup_64.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)

[PATCH] smp: kernel/panic.c - silence warnings

s for these three functions in include/linux/smp.h. Reported-by: Hulk Robot Signed-off-by: He Ying --- include/linux/smp.h | 8 1 file changed, 8 insertions(+) diff --git a/include/linux/smp.h b/include/linux/smp.h index 70c6f6284dcf..861a253cc179 100644 --- a/include/linux/smp.h +++

[PATCH] powerpc: define the variable 'uaccess_flush' as static

The variable 'uaccess_fulsh' is not referenced outside the file. Perhaps we should define it as static to avoid the warning as follows: arch/powerpc/kernel/setup_64.c:953:6: warning: symbol 'uaccess_flush' was not declared. Should it be static? Reported-by: Hulk Robot Si