Re: [PATCH 3/7] x86/fpu/xstate: Add supervisor PASID state for ENQCMD feature

From: Fenghua Yu
Date: Mon Apr 27 2020 - 16:34:39 EST


On Sun, Apr 26, 2020 at 01:17:11PM +0200, Thomas Gleixner wrote:
> Fenghua Yu <fenghua.yu@xxxxxxxxx> writes:
> > From: Yu-cheng Yu <yu-cheng.yu@xxxxxxxxx>
> >
> > The IA32_PASID MSR is used when a task submits work via the ENQCMD
> > instruction.
>
> Is used?
>
> > The per task MSR is stored in the task's supervisor FPU
>
> per task MSR? Lot's of MSRs ....
>
> > PASID state and is context switched by XSAVES/XRSTORS.
> >

Maybe change the commit messge to the following?

ENQCMD instruction reads PASID from IA32_PASID MSR. The MSR is stored
in the task's supervisor FPU PASID state and is context switched by
XSAVES/XRSTORS.

Thanks.

-Fenghua