| Message ID | 20190613161656.20765-3-julien.grall@arm.com (mailing list archive) |
|---|---|
| State | RFC |
| Headers | show |
| Series | arm64/sve: First steps towards optimizing syscalls | expand |
On Thu, Jun 13, 2019 at 05:16:50PM +0100, Julien Grall wrote: > The SVE state is saved by fpsimd_signal_preserve_current_state() and not > preserve_fpsimd_context(). Update the comment in preserve_sve_context to > reflect the current behavior. > > Signed-off-by: Julien Grall <julien.grall@arm.com> > --- > arch/arm64/kernel/signal.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/arch/arm64/kernel/signal.c b/arch/arm64/kernel/signal.c > index a9b0485df074..ab3e56bbfb07 100644 > --- a/arch/arm64/kernel/signal.c > +++ b/arch/arm64/kernel/signal.c > @@ -255,7 +255,8 @@ static int preserve_sve_context(struct sve_context __user *ctx) > if (vq) { > /* > * This assumes that the SVE state has already been saved to > - * the task struct by calling preserve_fpsimd_context(). > + * the task struct by calling the function > + * fpsimd_signal_preserve_current_state(). > */ > err |= __copy_to_user((char __user *)ctx + SVE_SIG_REGS_OFFSET, > current->thread.sve_state, Reviewed-by: Dave Martin <Dave.Martin@arm.com> Cheers ---Dave
diff --git a/arch/arm64/kernel/signal.c b/arch/arm64/kernel/signal.c index a9b0485df074..ab3e56bbfb07 100644 --- a/arch/arm64/kernel/signal.c +++ b/arch/arm64/kernel/signal.c @@ -255,7 +255,8 @@ static int preserve_sve_context(struct sve_context __user *ctx) if (vq) { /* * This assumes that the SVE state has already been saved to - * the task struct by calling preserve_fpsimd_context(). + * the task struct by calling the function + * fpsimd_signal_preserve_current_state(). */ err |= __copy_to_user((char __user *)ctx + SVE_SIG_REGS_OFFSET, current->thread.sve_state,
The SVE state is saved by fpsimd_signal_preserve_current_state() and not preserve_fpsimd_context(). Update the comment in preserve_sve_context to reflect the current behavior. Signed-off-by: Julien Grall <julien.grall@arm.com> --- arch/arm64/kernel/signal.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)