Re: next/master bisection: baseline.login on minnowboard-turbot-E3826
From: Guillaume Tucker
Date: Thu Sep 23 2021 - 11:39:01 EST
Hi Thomas,
Please see the bisection report below about a kernel panic on
minnowboard-turbot-E3826 right after starting init.
Reports aren't automatically sent to the public while we're
trialing new bisection features on kernelci.org but this one
looks valid.
Some more details can be found here:
https://linux.kernelci.org/test/case/id/614c1a1e98c5ed9ffc99a2e9/
The same issue appears to be reproducible with QEMU:
https://linux.kernelci.org/test/case/id/614c71a50d39b53eee99a2f8/
Please let us know if you need help debugging the issue or if you
have a fix to try.
Best wishes,
Guillaume
GitHub: https://github.com/kernelci/kernelci-project/issues/53
On 23/09/2021 14:05, KernelCI bot wrote:
> * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
> * This automated bisection report was sent to you on the basis *
> * that you may be involved with the breaking commit it has *
> * found. No manual investigation has been done to verify it, *
> * and the root cause of the problem may be somewhere else. *
> * *
> * If you do send a fix, please include this trailer: *
> * Reported-by: "kernelci.org bot" <bot@xxxxxxxxxxxx> *
> * *
> * Hope this helps! *
> * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
>
> next/master bisection: baseline.login on minnowboard-turbot-E3826
>
> Summary:
> Start: bc5aa70f2699 Add linux-next specific files for 20210923
> Plain log: https://storage.kernelci.org/next/master/next-20210923/x86_64/x86_64_defconfig/clang-12/lab-collabora/baseline-minnowboard-turbot-E3826.txt
> HTML log: https://storage.kernelci.org/next/master/next-20210923/x86_64/x86_64_defconfig/clang-12/lab-collabora/baseline-minnowboard-turbot-E3826.html
> Result: 2af07f3a6e9f x86/fpu/signal: Change return type of copy_fpregs_to_sigframe() helpers to boolean
>
> Checks:
> revert: PASS
> verify: PASS
>
> Parameters:
> Tree: next
> URL: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
> Branch: master
> Target: minnowboard-turbot-E3826
> CPU arch: x86_64
> Lab: lab-collabora
> Compiler: clang-12
> Config: x86_64_defconfig
> Test case: baseline.login
>
> Breaking commit found:
>
> -------------------------------------------------------------------------------
> commit 2af07f3a6e9fb81331421ca24b26a96180d792dd
> Author: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Date: Wed Sep 8 15:29:34 2021 +0200
>
> x86/fpu/signal: Change return type of copy_fpregs_to_sigframe() helpers to boolean
>
> Now that copy_fpregs_to_sigframe() returns boolean the individual return
> codes in the related helper functions do not make sense anymore. Change
> them to return boolean success/fail.
>
> Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Signed-off-by: Borislav Petkov <bp@xxxxxxx>
> Link: https://lkml.kernel.org/r/20210908132525.794334915@xxxxxxxxxxxxx
>
> diff --git a/arch/x86/kernel/fpu/signal.c b/arch/x86/kernel/fpu/signal.c
> index 7ce396dcc942..1d10fe9b5b6b 100644
> --- a/arch/x86/kernel/fpu/signal.c
> +++ b/arch/x86/kernel/fpu/signal.c
> @@ -65,7 +65,7 @@ static inline int check_xstate_in_sigframe(struct fxregs_state __user *fxbuf,
> /*
> * Signal frame handlers.
> */
> -static inline int save_fsave_header(struct task_struct *tsk, void __user *buf)
> +static inline bool save_fsave_header(struct task_struct *tsk, void __user *buf)
> {
> if (use_fxsr()) {
> struct xregs_state *xsave = &tsk->thread.fpu.state.xsave;
> @@ -82,18 +82,19 @@ static inline int save_fsave_header(struct task_struct *tsk, void __user *buf)
> if (__copy_to_user(buf, &env, sizeof(env)) ||
> __put_user(xsave->i387.swd, &fp->status) ||
> __put_user(X86_FXSR_MAGIC, &fp->magic))
> - return -1;
> + return false;
> } else {
> struct fregs_state __user *fp = buf;
> u32 swd;
> +
> if (__get_user(swd, &fp->swd) || __put_user(swd, &fp->status))
> - return -1;
> + return false;
> }
>
> - return 0;
> + return true;
> }
>
> -static inline int save_xstate_epilog(void __user *buf, int ia32_frame)
> +static inline bool save_xstate_epilog(void __user *buf, int ia32_frame)
> {
> struct xregs_state __user *x = buf;
> struct _fpx_sw_bytes *sw_bytes;
> @@ -131,7 +132,7 @@ static inline int save_xstate_epilog(void __user *buf, int ia32_frame)
>
> err |= __put_user(xfeatures, (__u32 __user *)&x->header.xfeatures);
>
> - return err;
> + return !err;
> }
>
> static inline int copy_fpregs_to_sigframe(struct xregs_state __user *buf)
> @@ -218,10 +219,10 @@ bool copy_fpstate_to_sigframe(void __user *buf, void __user *buf_fx, int size)
> }
>
> /* Save the fsave header for the 32-bit frames. */
> - if ((ia32_fxstate || !use_fxsr()) && save_fsave_header(tsk, buf))
> + if ((ia32_fxstate || !use_fxsr()) && !save_fsave_header(tsk, buf))
> return false;
>
> - if (use_fxsr() && save_xstate_epilog(buf_fx, ia32_fxstate))
> + if (use_fxsr() && !save_xstate_epilog(buf_fx, ia32_fxstate))
> return false;
>
> return true;
> -------------------------------------------------------------------------------
>
>
> Git bisection log:
>
> -------------------------------------------------------------------------------
> git bisect start
> # good: [58e2cf5d794616b84f591d4d1276c8953278ce24] init: Revert accidental changes to print irqs_disabled()
> git bisect good 58e2cf5d794616b84f591d4d1276c8953278ce24
> # bad: [bc5aa70f26995791074a00da8b57ead3a9c1a646] Add linux-next specific files for 20210923
> git bisect bad bc5aa70f26995791074a00da8b57ead3a9c1a646
> # good: [18b9b2db40708ef6bbcaa9b8d8d10b7bb21e26e6] Merge branch 'nand/next' of git://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git
> git bisect good 18b9b2db40708ef6bbcaa9b8d8d10b7bb21e26e6
> # bad: [bcb42fa2db7240d8167cb1e7179f6377e468145e] Merge branch 'for-next' of git://git.kernel.org/pub/scm/linux/kernel/git/dennis/percpu.git
> git bisect bad bcb42fa2db7240d8167cb1e7179f6377e468145e
> # good: [c1d789f5e5155df5f629b95d93ac830fd7de97d6] Merge branch 'for-next' of git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git
> git bisect good c1d789f5e5155df5f629b95d93ac830fd7de97d6
> # good: [ba8b54a3a086472c81ba2b4cda62dbd4adab45f6] Merge branch 'next' of git://git.kernel.org/pub/scm/linux/kernel/git/pcmoore/audit.git
> git bisect good ba8b54a3a086472c81ba2b4cda62dbd4adab45f6
> # bad: [dcb7b66df80dc908f7dded6e8839093169e56a7b] Merge branch 'tip-x86-misc' into tip-auto-latest
> git bisect bad dcb7b66df80dc908f7dded6e8839093169e56a7b
> # good: [ccb4a51cd81f7ab035ba25df2c46f6f3c79f180d] Merge branch 'sched/core'
> git bisect good ccb4a51cd81f7ab035ba25df2c46f6f3c79f180d
> # bad: [1acd24c77ba4ec1f141f5175d19ae7275ce311d0] Merge branch 'x86/fpu'
> git bisect bad 1acd24c77ba4ec1f141f5175d19ae7275ce311d0
> # good: [fcfb7163329ce832aafef31f26345ef5e8642a17] x86/fpu/signal: Move xstate clearing out of copy_fpregs_to_sigframe()
> git bisect good fcfb7163329ce832aafef31f26345ef5e8642a17
> # bad: [be0040144152ed834c369a7830487e5ee4f27080] x86/fpu/signal: Change return code of check_xstate_in_sigframe() to boolean
> git bisect bad be0040144152ed834c369a7830487e5ee4f27080
> # bad: [ee4ecdfbd28954086a09740dc931c10c93e39370] x86/signal: Change return type of restore_sigcontext() to boolean
> git bisect bad ee4ecdfbd28954086a09740dc931c10c93e39370
> # bad: [2af07f3a6e9fb81331421ca24b26a96180d792dd] x86/fpu/signal: Change return type of copy_fpregs_to_sigframe() helpers to boolean
> git bisect bad 2af07f3a6e9fb81331421ca24b26a96180d792dd
> # good: [052adee668284b67105375c0a524f16a423f1424] x86/fpu/signal: Change return type of copy_fpstate_to_sigframe() to boolean
> git bisect good 052adee668284b67105375c0a524f16a423f1424
> # first bad commit: [2af07f3a6e9fb81331421ca24b26a96180d792dd] x86/fpu/signal: Change return type of copy_fpregs_to_sigframe() helpers to boolean
> -------------------------------------------------------------------------------