Messages in this thread | | | From | "Bae, Chang Seok" <> | Subject | Re: [PATCH v3 02/21] x86/fpu/xstate: Modify state copy helpers to handle both static and dynamic buffers | Date | Tue, 19 Jan 2021 18:50:52 +0000 |
| |
On Jan 15, 2021, at 04:50, Borislav Petkov <bp@suse.de> wrote: > On Wed, Dec 23, 2020 at 07:56:58AM -0800, Chang S. Bae wrote: >> >> -void copy_xstate_to_kernel(struct membuf to, struct xregs_state *xsave); >> -int copy_kernel_to_xstate(struct xregs_state *xsave, const void *kbuf); >> -int copy_user_to_xstate(struct xregs_state *xsave, const void __user *ubuf); >> -void copy_supervisor_to_kernel(struct xregs_state *xsave); >> +void copy_xstate_to_kernel(struct membuf to, struct fpu *fpu); >> +int copy_kernel_to_xstate(struct fpu *fpu, const void *kbuf); >> +int copy_user_to_xstate(struct fpu *fpu, const void __user *ubuf); >> +void copy_supervisor_to_kernel(struct fpu *fpu); > > Hmm, so those functions have "xstate" in the name because they took and > @xstate parameter. I guess not such a big deal you changing them, just > pointing out what the naming logic was.
I will add a sentence like this if looks fine:
"The copy functions used to have ‘xstate' in the name as they took a struct xregs_state * pointer."
Thanks, Chang | |