From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=209.85.221.52 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1792498234; b=tozzdh2aRXN0ypngqsZUMRWa76B1/f0BhHIULUoSIce2P/UXqCq+UIyUI4yOOPfpTJie TKgu4tlipWSnpI8xAm7k+rxMRytJN6BiLG5JqwZzlG59D7ZkjGlyfUSdwQC2FS8eEoKoe VzrXP9MiGwKWJy8glvZoSSIppV4ZRqgwJbNBaG7HhT6gJ6n3rvXeOCP/eisfD/jT0WFXG TU7MMZU+Eolq/OwU68cimetqWOHxMz5j1MSkxqXZ6mI+hT32vbRpDsLIv1m6w1gOmjO/j s9WKkPJmZQSejdFJf0g0Xk1FpxHsKM+DkBYQWXE4KYb57clQRnEO1TOksz8EPnVOX/Q== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1792498234; h=DMARC-Filter:Received:DKIM-Signature:X-Google-DKIM-Signature: X-Forwarded-Encrypted:X-Gm-Message-State:X-Gm-Gg:X-Received:Received: Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; bh=TP3lHd7ZCrOKSupriqyX6PtdNuECuFfGjiY9AbgLC+E=; b=ry7LDlPjvwei/XsUBxxmJAKFbBYObo+vQk0ND7TP6yH1ldp97wqplRX9RdL6k3mANx9/ KrNAAGTKtGUma7ywoh7Yqc7mNFKBf5MjzvMs0eKscjYcPxbNjww0ILEQE1FnUUAzVxmYZ OthmFfKEjAmVNLTZjblcJkf2WpcbbOgoyfo+5DZCNRNgKe+7D5CxPK16gI7fjolgWIvq7 4rlLiBWzmGSxgfJahR9clygQVLbYkBfXVz7MtrQ8cIH2nqtylbKnPBQSd0R9Ic7vCAykQ OCek1IS1kXLPuNde33Ac2qMJUIfKKecIQ7vTPXqIqYFalX+6vQpJxaKI8BXwPuLs/2w== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass header.from=suse.com; spf=pass smtp.mailfrom=suse.com; dkim=pass header.d=suse.com header.i=@suse.com header.a=rsa-sha256 header.s=google header.b=RivgTgIb; arc=none smtp.remote-ip=209.85.221.52 Received: from mail-wr1-f52.google.com (mail-wr1-f52.google.com [209.85.221.52]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4D3F8606A323 for ; Tue, 20 Oct 2026 12:10:34 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.2 smtp.subspace.kernel.org 4D3F8606A323 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=suse.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key, unprotected) header.d=suse.com header.i=@suse.com header.a=rsa-sha256 header.s=google header.b=RivgTgIb Received: by mail-wr1-f52.google.com with SMTP id ffacd0b85a97d-48dc202a60eso865352f8f.3 for ; Tue, 20 Oct 2026 05:10:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=google; t=1792498232; x=1793103032; darn=vger.kernel.org; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:from:to:cc:subject :date:message-id:reply-to:content-type; bh=TP3lHd7ZCrOKSupriqyX6PtdNuECuFfGjiY9AbgLC+E=; b=RivgTgIbsmhNiozCHhxpiackjSI89ckS6Y0/0KLOUziRvM4YY48oqYzquDAmwWZxhT z+NtSzyVT6wtvVEoOSrlZA4GyMkaaKaTNHRw7RYCq3v2xkcNE6DcBAAVUL7AoGoCXHFV FxKoUgufgtJ4/RiMeKLE+/4EI1ck0CJf85jTFjtY7mVnDldDYTmsOJj9HHPz5nYoogJ6 CEd5O/nTy+QWVcbcVu+oICxT1Enorljx53dq3h9VwPWtVi02Ep9RULu+/vCV0TTM6EVf 1gFWUmOSxW+9zdkPPE0ep++ZsSK/6JQQtc8m9uc4rSpMRVZb3iuDcDyXWStqMOCppVA2 UItg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1792498232; x=1793103032; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=TP3lHd7ZCrOKSupriqyX6PtdNuECuFfGjiY9AbgLC+E=; b=2fFJtOn9nW6q4Jk9cn5jc55uYknR9P6Jaq3Z0GqwhTCPqWJxfAXFXWP9XY7dfpKfc8 qFcrcZDQ06G+ecKFf0xPfKvMsU4JZMh7dAQjAcV0aopnCiU3EmouC+0ECnrpwgceXxAO gR6c1y1jjzwBUe5Rg/k9WQpwNxynIgZn1eMxeffNUb1FfJn7mnMZbKjUpvP3Itzq48xk GZZqc0ByncRyivT/CsdWAqV70aFgK33PMdKTdPZJzz+GB6kUZV0cAdpArhygoDO9zvfI +HOOK+6bLAQHMNdW0OKZGjlb36Y4YElUkhY/ULGUKYSDnBiy1MB1jZ7akUIy93zZcUW1 bjSA== X-Forwarded-Encrypted: i=1; AKwUvBzP/0sdLk5qMB0bvy+aB9KXMgyQS3C4giCEW+Lob1eIObSo1ffH/xcmyiRoNbQVZWhAO3bg1LzdBi8=@vger.kernel.org X-Gm-Message-State: AFq9FYIpMW0hilFkhICUdvrCkx7YMX+DIpS4cxgcTuDuOGf4eVaPb3oz BLbZ8SogEV9i2R31HopB06esjwJKRP1SQA/mCqVxdA/Gh6y0CJraUYsvOWB9aJvasaU= X-Gm-Gg: AYBFou2lF/nAE7RfgLFZ43PQqJQ5Oquds5ek2zc0Mp2JFWwoa3DbYbwOn0iV1vesSf/ x+tTTVgrmYTW365Tet0hmtLqFkKqUkkaajnVOsKcLfz9bas8FCcIWko4u43Zqs4t1yAcGpjLKUU BUcUv+uCNCNB6fx+a/udMotavqSl1oRWu4EhcaNl1Rm11QL84NP4SbVxMcGxGz4gc7pmmsvlMQm 35gqGU8ZeeIZhg6xy5lxnAt3Kj8B9HwnWvziBrrgXofsGzDsd7H+ozKT9hSsHZwrkDxTdxioa+u 0AWhaW9ADlLBoTVLIF2ExAv8NQhY1PvhNc7xNeyJkLH2N32x4bsIFQazpr7ARKGU6/ZCfVy2OVP Gy4gXX3twxI44uUPIJDi7G0hg1/6Ms+GX93VyZDhhSD1a9/LDfgAuQgKXGzSEzCsgrqWneSZiuo GQvxuvSr1S8Q9KD+bL3R2z5Mj/w7nfDC3/UX/tw6NMyBwAxkai57tAw3hR8eXJD4cEGDnbewSIy qGqRruNuA8= X-Received: by 2002:a5d:5f51:0:b0:48e:412:3813 with SMTP id ffacd0b85a97d-48e07bb2fc7mr21324149f8f.20.1792498232549; Tue, 20 Oct 2026 05:10:32 -0700 (PDT) Received: from pathway.suse.cz ([176.114.240.130]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-48e1d823cfcsm5747053f8f.39.2026.10.20.05.10.31 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 20 Oct 2026 05:10:32 -0700 (PDT) Date: Tue, 20 Oct 2026 14:10:29 +0200 From: Petr Mladek To: Zack Rusin Cc: Andrew Morton , Baoquan He , Mike Rapoport , Pasha Tatashin , Pratyush Yadav , Dave Young , Borislav Petkov , Ajay Kaher , Alexey Makhalov , x86@kernel.org, Joel Granados , Thomas Gleixner , Ingo Molnar , Dave Hansen , "H. Peter Anvin" , virtualization@lists.linux.dev, bcm-kernel-feedback-list@broadcom.com, linux-kernel@vger.kernel.org, John Ogness , Steven Rostedt , Sergey Senozhatsky , Kees Cook , Jonathan Corbet , Bo Gan , Brennan Lamoreaux , kexec@lists.infradead.org, linux-doc@vger.kernel.org, "Guilherme G. Piccoli" , Maaz Mombasawala , Shuah Khan , Randy Dunlap , Stephen Brennan , Michael Kelley , Ian Forbes Subject: Re: [PATCH v2 2/6] crash: Notify pre-kdump callbacks before switching kernels Message-ID: References: <9ef573cdbbf5e659eea16fa23afeaf2488049198.1790014793.git.zack.rusin@broadcom.com> X-Mailing-List: linux-doc@vger.kernel.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <9ef573cdbbf5e659eea16fa23afeaf2488049198.1790014793.git.zack.rusin@broadcom.com> On Mon 2026-09-28 13:55:36, Zack Rusin wrote: > Fatal x86 oopses can call crash_kexec() without reaching panic(). Run the > pre-kdump chain from __crash_kexec() as well, after finding a loaded image > under the kexec lock and capturing the original registers. > > This covers direct crash entry without moving all panic notifiers ahead > of kdump. A missing image or failed trylock leaves callbacks uncalled; > the shared guard skips callbacks already invoked by panic(). > > Link: https://lore.kernel.org/r/aquttpMVKX8e6zGB@pathway.suse.cz > Signed-off-by: Zack Rusin > --- > New in v2: cover direct crash-kexec entry in a separate core patch. > > kernel/crash_core.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/kernel/crash_core.c b/kernel/crash_core.c > index 2b36aa9fade0..5d9fe9e00f39 100644 > --- a/kernel/crash_core.c > +++ b/kernel/crash_core.c > @@ -23,6 +23,7 @@ > #include > #include > #include > +#include > > #include > #include > @@ -139,6 +140,7 @@ void __noclone __crash_kexec(struct pt_regs *regs) > struct pt_regs fixed_regs; > > crash_setup_regs(&fixed_regs, regs); > + panic_notify_pre_kdump(NULL); I understand why you did this. But it makes the code really hard to follow: + This code path will be called also by the 1st __crash_kexec() in vpanic() when "_crash_kexec_post_notifiers == false". It is the default. + The panic_notify_pre_kdump(buf) in vpanic(), added by 1st patch, is called when "_crash_kexec_post_notifiers == true" or when __crash_kexec() returns from some reason. None of this is obvious. I am going to think more about it. Best Regards, Petr > crash_save_vmcoreinfo(); > machine_crash_shutdown(&fixed_regs); > crash_cma_clear_pending_dma();