Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: +D7Hnh0IgXzlwn2vvsWCEvmOP/cYgLU6EmcwEugAp1z81dCOJSJSkzrtX294Br9VwVc/mSMcLi
 2rEPI5Vy+qps3J9BRRdXpn+ozd04ZQw2gLIe+ZXhhp/9knjohQkTJBzrdmoOEtRlSAnp0PglAP
 WwEtDVoCIl+OvrjCC9KYaWcxjuDsrIUT+6rwTmrOJ3wJvTyjG91gSmL2FHjuYXv0IkXlZa0gFq
 il4s3NgtjDMFKKAslqjAhBgCY6UmcMf+iyBySis3bCMeP6l74Zdhg7GfDSMGXw9nCsPb/tOsGf
 loI=
From:   "ito-yuichi@fujitsu.com" <ito-yuichi@fujitsu.com>
To:     "ito-yuichi@fujitsu.com" <ito-yuichi@fujitsu.com>,
        "maz@kernel.org" <maz@kernel.org>,
        "sumit.garg@linaro.org" <sumit.garg@linaro.org>,
        "tglx@linutronix.de" <tglx@linutronix.de>,
        "jason@lakedaemon.net" <jason@lakedaemon.net>,
        "catalin.marinas@arm.com" <catalin.marinas@arm.com>,
        "will@kernel.org" <will@kernel.org>
CC:     "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH 0/2] Enable support IPI_CPU_CRASH_STOP to be pseudo-NMI
Thread-Topic: [PATCH 0/2] Enable support IPI_CPU_CRASH_STOP to be pseudo-NMI
Thread-Index: AQHWki1OzvWvE6qquUyffKCpvoCHGKl9XDZA
Date:   Mon, 28 Sep 2020 02:43:10 +0000
Message-ID: <OSAPR01MB42279E9BFA43419F49231ECE8F350@OSAPR01MB4227.jpnprd01.prod.outlook.com>
References: <20200924044236.1245808-1-ito-yuichi@fujitsu.com>
In-Reply-To: <20200924044236.1245808-1-ito-yuichi@fujitsu.com>
Accept-Language: ja-JP, en-US
Content-Language: ja-JP
Content-Type: text/plain; charset="iso-2022-jp"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: f8ff0d11-3cc4-4a1c-59aa-08d863583d1a
X-MS-Exchange-CrossTenant-originalarrivaltime: 28 Sep 2020 02:43:10.5564
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: a19f121d-81e1-4858-a9d8-736e267fd4c7
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: As85vt5EyUILmEaRQlzk/GvRkoJVpC4sAYIpwO+UCZ3jGKZ+iY64XWwQi8H4sFaRvgnUGReBjaOV62mZsS1CsTt+XshOVP2PQ8pYRy4lJVw=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: OSAPR01MB3492
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Marc, Sumit

I would appreciate if you have any advice on this patch.

Yuichi Ito

> -----Original Message-----
> From: Yuichi Ito <ito-yuichi@fujitsu.com>
> Sent: Thursday, September 24, 2020 1:43 PM
> To: maz@kernel.org; sumit.garg@linaro.org; tglx@linutronix.de;
> jason@lakedaemon.net; catalin.marinas@arm.com; will@kernel.org
> Cc: linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org; I=
to,
> Yuichi/=1B$B0KF#=1B(B =1B$BM-0l=1B(B <ito-yuichi@fujitsu.com>
> Subject: [PATCH 0/2] Enable support IPI_CPU_CRASH_STOP to be
> pseudo-NMI
>=20
> Enable support IPI_CPU_CRASH_STOP to be pseudo-NMI
>=20
> This patchset enables IPI_CPU_CRASH_STOP IPI to be pseudo-NMI.
> This allows kdump to collect system information even when the CPU is in a
> HARDLOCKUP state.
>=20
> Only IPI_CPU_CRASH_STOP uses NMI and the other IPIs remain normal
> IRQs.
>=20
> The patch has been tested on ThunderX.
>=20
> This patch assumes Marc's latest IPIs patch-set. [1] It also uses some of
> Sumit's IPI patch set for NMI.[2]
>=20
> [1]
> https://lore.kernel.org/linux-arm-kernel/20200901144324.1071694-1-maz@ke
> rnel.org/
> [2]
> https://lore.kernel.org/linux-arm-kernel/1599830924-13990-3-git-send-emai=
l
> -sumit.garg@linaro.org/
>=20
> $ echo 1 > /proc/sys/kernel/panic_on_rcu_stal
> $ echo HARDLOCKUP > /sys/kernel/debug/provoke-crash/DIRECT
>    : kernel panics and crash kernel boot
>    : makedumpfile saves the system state at HARDLOCKUP in vmcore.
>=20
> crash utility:
> crash> bt
>   PID: 3213   TASK: fffffd001adc5940  CPU: 8   COMMAND: "bash"
>   #0 [fffffe0022fefcf0] lkdtm_HARDLOCKUP at fffffe0010888ab4
>   #1 [fffffe0022fefd10] lkdtm_do_action at fffffe00108882bc
>   #2 [fffffe0022fefd20] direct_entry at fffffe0010888720
>   #3 [fffffe0022fefd70] full_proxy_write at fffffe001058cfe4
>   #4 [fffffe0022fefdb0] vfs_write at fffffe00104a4c2c
>   #5 [fffffe0022fefdf0] ksys_write at fffffe00104a4f0c
>   #6 [fffffe0022fefe40] __arm64_sys_write at fffffe00104a4fbc
>   #7 [fffffe0022fefe50] el0_svc_common.constprop.0 at fffffe0010159e38
>   #8 [fffffe0022fefe80] do_el0_svc at fffffe0010159fa0
>   #9 [fffffe0022fefe90] el0_svc at fffffe00101481d0
>   #10 [fffffe0022fefea0] el0_sync_handler at fffffe00101484b4
>   #11 [fffffe0022fefff0] el0_sync at fffffe0010142b7c
>=20
>=20
> Sumit Garg (1):
>   irqchip/gic-v3: Enable support for SGIs to act as NMIs
>=20
> Yuichi Ito (1):
>   Register IPI_CPU_CRASH_STOP IPI as pseudo-NMI
>=20
>  arch/arm64/kernel/smp.c      | 39
> ++++++++++++++++++++++++++++--------
>  drivers/irqchip/irq-gic-v3.c | 13 ++++++++++--
>  2 files changed, 42 insertions(+), 10 deletions(-)
>=20
> --
> 2.25.1