On Thu, Apr 10, 2025 at 11:04:23AM -0700, Nathan Chancellor wrote:
Hi Greg,
On Tue, Apr 08, 2025 at 05:15:46PM +0200, Greg Kroah-Hartman wrote:
On Tue, Apr 08, 2025 at 05:55:57AM -0700, Vishal Annapurve wrote:
Hi Greg,
This patch depends on commit 22cc5ca5de52 ("x86/paravirt: Move halt paravirt calls under CONFIG_PARAVIRT"). I will respond to the other thread with a patch for commit 22cc5ca5de52 after resolving conflicts.
That commit is already in this series, do I need to add it again? :)
Is it? I don't see it in the stable review on lore and it does not look like it made it into 6.13.11 final because the build errors if CONFIG_PARAVIRT_XXL is disabled (for example, allmodconfig with CONFIG_XEN_PV disabled):
arch/x86/coco/tdx/tdx.c: In function 'tdx_early_init': arch/x86/coco/tdx/tdx.c:1107:19: error: 'struct pv_irq_ops' has no member named 'safe_halt' 1107 | pv_ops.irq.safe_halt = tdx_safe_halt; | ^ arch/x86/coco/tdx/tdx.c:1108:19: error: 'struct pv_irq_ops' has no member named 'halt' 1108 | pv_ops.irq.halt = tdx_halt; | ^
This was initially reported downstream by Marcus: https://github.com/ClangBuiltLinux/linux/issues/2081
Now queued up, thanks.
greg k-h