Re: [PATCH 1/2] x86/tdx: Add helper to query maximum TD Quote size

From: Peter Fang

Date: Tue Jun 23 2026 - 00:44:37 EST


On Fri, Jun 12, 2026 at 10:25:03PM +0800, Xiaoyao Li wrote:
> >
> > Assisted-by: Claude:claude-opus-4-7
> > Assisted-by: GitHub Copilot:gpt-5.4
> > Signed-off-by: Peter Fang <peter.fang@xxxxxxxxx>
>
> Reviewed-by: Xiaoyao Li <xiaoyao.li@xxxxxxxxx>

Thanks for the review Xiaoyao!

>
> I have another nit other than Kiryl's
>
> > +u32 tdx_get_max_quote_size(void)
> > +{
> > + u64 val, ret;
> > +
> > + ret = tdg_vm_rd(TDCS_QUOTE_MAX_SIZE, &val);
> > +
> > + return ret ? 0 : (u32)val;
> > +}
> > +EXPORT_SYMBOL_GPL(tdx_get_max_quote_size);
>
> Do we need to start to use
>
> EXPORT_SYMBOL_FOR_MODULES(tdx_get_max_quote_size, "tdx-guest") ?
>

This makes sense. But can we use a follow-up patch to improve this file
later? Right now there are only EXPORT_SYMBOL_GPL() usages, so using
EXPORT_SYMBOL_FOR_MODULES() here might look inconsistent.