Re: [PATCH v2 3/4] x86/tdx: Drop exported function tdx_quirk_reset_page()

[Ackerley Tng]

Yan Zhao <yan.y.zhao@xxxxxxxxx> writes:

> KVM invokes tdx_quirk_reset_page() to reset TDX control pages (including
> S-EPT pages, TDR page, etc.), as all those pages are allocated by KVM TDX
> and thus always have struct page.
>
> However, it's also reasonable for KVM to reset those TDX control pages via
> tdx_quirk_reset_paddr() directly, eliminating the need to export two
> parallel APIs. Keeping tdx_quirk_reset_page() as a one-line helper in the
> header file is also unnecessary.
>
> No functional change intended.
>
> Suggested-by: Paolo Bonzini <pbonzini@xxxxxxxxxx>
> Suggested-by: Xiaoyao Li <xiaoyao.li@xxxxxxxxx>
> Signed-off-by: Yan Zhao <yan.y.zhao@xxxxxxxxx>

Thanks for the cleanup!

Reviewed-by: Ackerley Tng <ackerleytng@xxxxxxxxxx>

>
> [...snip...]
>