Messages in this thread | | | From | "Bae, Chang Seok" <> | Subject | Re: [PATCH v3 08/21] x86/fpu/xstate: Define the scope of the initial xstate data | Date | Tue, 9 Feb 2021 15:38:58 +0000 |
| |
On Feb 9, 2021, at 04:49, Borislav Petkov <bp@suse.de> wrote: > On Mon, Feb 08, 2021 at 06:53:23PM +0000, Bae, Chang Seok wrote: > Yours does. So drop it from this one and from all the other patches as > it is causing more confusion than it is trying to dispel.
Okay.
>> I think they are in a different context. >> >> The helper indicates the mask for the ‘init_fpstate’ buffer. The rest is the >> initial mask value for the per-task xstate buffer. > > Wait, what? > > Are you trying to tell me that that helper will return different masks > depending on xfeatures_mask_user_dynamic, which changes in its lifetime?
At least in this series, no. But I thought it is possible in the future.
> Then drop that helper altogether - that is more confusion and the xstate > code is already confusing enough.
Okay.
>> Since you suggested to introduce get_xstate_buffer_attr(), how about replacing >> what you found with something like: >> >> get_xstate_buffer_attr(XSTATE_INIT_MASK) > > I'd prefer no helper at all but only comments above the usage site.
Yes, I will do that.
Thanks, Chang | |