Re: [PATCH v1 3/3] x86/msr: Convert a native_wrmsr() use to native_wrmsrq()

2025-05-19 Thread Xin Li
On 5/12/2025 1:45 AM, Xin Li (Intel) wrote: Convert a native_wrmsr() use to native_wrmsrq() to zap meaningless type conversions when a u64 MSR value is splitted into two u32. Signed-off-by: Xin Li (Intel) --- arch/x86/coco/sev/core.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-

Re: [PATCH v1 3/3] x86/msr: Convert a native_wrmsr() use to native_wrmsrq()

2025-05-17 Thread Xin Li
On 5/17/2025 6:21 AM, Ingo Molnar wrote: Ah, indeed, it's also a startup code wrapper, which wrmsrq() doesn't have at the moment. Fair enough. So you want me to drop this patch then? No, patch #3 is fine as-is in its -v1 form Thanks for confirming. I'll just update patch #2 as version v1A th

Re: [PATCH v1 3/3] x86/msr: Convert a native_wrmsr() use to native_wrmsrq()

2025-05-17 Thread Ingo Molnar
* Xin Li wrote: > > >>> On 5/15/2025 10:54 AM, Xin Li wrote: > >>> On 5/15/2025 8:27 AM, Ingo Molnar wrote: > > * Xin Li (Intel) wrote: > > > Convert a native_wrmsr() use to native_wrmsrq() to zap meaningless type > > conversions when a u64 MSR value is splitted into

Re: [PATCH v1 3/3] x86/msr: Convert a native_wrmsr() use to native_wrmsrq()

2025-05-17 Thread Xin Li
>>> On 5/15/2025 10:54 AM, Xin Li wrote: >>> On 5/15/2025 8:27 AM, Ingo Molnar wrote: * Xin Li (Intel) wrote: > Convert a native_wrmsr() use to native_wrmsrq() to zap meaningless type > conversions when a u64 MSR value is splitted into two u32. > BTW., at

Re: [PATCH v1 3/3] x86/msr: Convert a native_wrmsr() use to native_wrmsrq()

2025-05-17 Thread Ingo Molnar
* Xin Li wrote: > On 5/15/2025 10:54 AM, Xin Li wrote: > > On 5/15/2025 8:27 AM, Ingo Molnar wrote: > > > > > > * Xin Li (Intel) wrote: > > > > > > > Convert a native_wrmsr() use to native_wrmsrq() to zap meaningless type > > > > conversions when a u64 MSR value is splitted into two u32. > >

Re: [PATCH v1 3/3] x86/msr: Convert a native_wrmsr() use to native_wrmsrq()

2025-05-16 Thread Xin Li
On 5/15/2025 10:54 AM, Xin Li wrote: On 5/15/2025 8:27 AM, Ingo Molnar wrote: * Xin Li (Intel) wrote: Convert a native_wrmsr() use to native_wrmsrq() to zap meaningless type conversions when a u64 MSR value is splitted into two u32. BTW., at this point we should probably just replace sev_

Re: [PATCH v1 3/3] x86/msr: Convert a native_wrmsr() use to native_wrmsrq()

2025-05-15 Thread Xin Li
On 5/15/2025 8:27 AM, Ingo Molnar wrote: * Xin Li (Intel) wrote: Convert a native_wrmsr() use to native_wrmsrq() to zap meaningless type conversions when a u64 MSR value is splitted into two u32. BTW., at this point we should probably just replace sev_es_wr_ghcb_msr() calls with direct cal

Re: [PATCH v1 3/3] x86/msr: Convert a native_wrmsr() use to native_wrmsrq()

2025-05-15 Thread Ingo Molnar
* Xin Li (Intel) wrote: > Convert a native_wrmsr() use to native_wrmsrq() to zap meaningless type > conversions when a u64 MSR value is splitted into two u32. > > Signed-off-by: Xin Li (Intel) > --- > arch/x86/coco/sev/core.c | 7 +-- > 1 file changed, 1 insertion(+), 6 deletions(-) > >

[PATCH v1 3/3] x86/msr: Convert a native_wrmsr() use to native_wrmsrq()

2025-05-12 Thread Xin Li (Intel)
Convert a native_wrmsr() use to native_wrmsrq() to zap meaningless type conversions when a u64 MSR value is splitted into two u32. Signed-off-by: Xin Li (Intel) --- arch/x86/coco/sev/core.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/arch/x86/coco/sev/core.c b/arch/