On Fri, Jul 25, 2025 at 7:34 AM Michael Kelley <mhkli...@outlook.com> wrote:
>
> From: Tianyu Lan <ltyker...@gmail.com> Sent: Wednesday, July 23, 2025 12:03 PM
> >
>
> I'm nit-picking, but the patch Subject: line prefix should be "x86/hyperv:"
> for consistency with past patches. Note that there's no capitalization
> or dash. I know not everyone is consistent on this if you look at the git log
> for mshyperv.c, but I try to flag it and encourage consistency.
>
> > Hyper-V doesn't support auto-eoi with Secure AVIC.
> > So set the HV_DEPRECATING_AEOI_RECOMMENDED flag
> > to force writing the EIO register after handling an interrupt.
> >
> > Signed-off-by: Tianyu Lan <ti...@microsoft.com>
> > ---
> > arch/x86/kernel/cpu/mshyperv.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/arch/x86/kernel/cpu/mshyperv.c b/arch/x86/kernel/cpu/mshyperv.c
> > index c78f860419d6..8f029650f16c 100644
> > --- a/arch/x86/kernel/cpu/mshyperv.c
> > +++ b/arch/x86/kernel/cpu/mshyperv.c
> > @@ -463,6 +463,8 @@ static void __init ms_hyperv_init_platform(void)
> > ms_hyperv.max_vp_index, ms_hyperv.max_lp_index);
> >
> > hv_identify_partition_type();
> > + if (cc_platform_has(CC_ATTR_SNP_SECURE_AVIC))
> > + ms_hyperv.hints |= HV_DEPRECATING_AEOI_RECOMMENDED;
> >
> > if (ms_hyperv.hints & HV_X64_HYPERV_NESTED) {
> > hv_nested = true;
> > --
> > 2.25.1
> >
>
Hi Michael:
Thanks for your review. Will update.
--
Thanks
Tianyu Lan
