Re: [RFC v2 28/32] x86/tdx: Make pages shared in ioremap()

[Kuppuswamy, Sathyanarayanan]

Hi Dave,

On 5/7/21 3:38 PM, Andi Kleen wrote:
> On 5/7/2021 2:55 PM, Dave Hansen wrote:
> > On 4/26/21 11:01 AM, Kuppuswamy Sathyanarayanan wrote:
> > >   static unsigned int __ioremap_check_encrypted(struct resource *res)
> > >   {
> > > -    if (!sev_active())
> > > +    if (!sev_active() && !is_tdx_guest())
> > >           return 0;
> > I think it's time to come up with a real name for all of the code that's
> > under: (sev_active() || is_tdx_guest()).
> >
> > "encrypted" isn't it, for sure.
>
> I called it protected_guest() in some other patches.

If you are also fine with above mentioned function name, I can include it
in this series. Since we have many use cases of above condition, it will
be useful define it as helper function.

> -Andi

--
Sathyanarayanan Kuppuswamy
Linux Kernel Developer