Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 9 Mar 2021 23:38:04 +0800
From:      Li-Wen Hsu <lwhsu@freebsd.org>
To:        Kyle Evans <kevans@freebsd.org>
Cc:        src-committers <src-committers@freebsd.org>, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org
Subject:   Re: git: 1ae20f7c70ea - main - kern: malloc: fix panic on M_WAITOK during THREAD_NO_SLEEPING()
Message-ID:  <CAKBkRUyxzJi65F3fPZ9mJT3VibjBD5F1SHjNMVGshFk6vPsqTA@mail.gmail.com>
In-Reply-To: <202103091117.129BHOZa042851@gitrepo.freebsd.org>
References:  <202103091117.129BHOZa042851@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Mar 9, 2021 at 7:18 PM Kyle Evans <kevans@freebsd.org> wrote:
>
> The branch main has been updated by kevans:
>
> URL: https://cgit.FreeBSD.org/src/commit/?id=1ae20f7c70ea16fa8b702e409030e170df4f5c13
>
> commit 1ae20f7c70ea16fa8b702e409030e170df4f5c13
> Author:     Kyle Evans <kevans@FreeBSD.org>
> AuthorDate: 2021-03-08 06:16:27 +0000
> Commit:     Kyle Evans <kevans@FreeBSD.org>
> CommitDate: 2021-03-09 11:16:39 +0000
>
>     kern: malloc: fix panic on M_WAITOK during THREAD_NO_SLEEPING()
>
>     Simple condition flip; we wanted to panic here after epoch_trace_list().
>
>     Reviewed by:    glebius, markj
>     MFC after:      3 days
>     Differential Revision:  https://reviews.freebsd.org/D29125
> ---
>  sys/kern/kern_malloc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/sys/kern/kern_malloc.c b/sys/kern/kern_malloc.c
> index 48383358e3ad..0d6f9dcfcab7 100644
> --- a/sys/kern/kern_malloc.c
> +++ b/sys/kern/kern_malloc.c
> @@ -537,7 +537,7 @@ malloc_dbg(caddr_t *vap, size_t *sizep, struct malloc_type *mtp,
>  #ifdef EPOCH_TRACE
>                         epoch_trace_list(curthread);
>  #endif
> -                       KASSERT(1,
> +                       KASSERT(0,
>                             ("malloc(M_WAITOK) with sleeping prohibited"));
>                 }
>         }

I guess this reveals a bug in gmirror?  Now the test case
sys/geom/class/mirror/8_test:main panics:

https://ci.freebsd.org/job/FreeBSD-main-amd64-test/17779/console
https://ci.freebsd.org/job/FreeBSD-main-i386-test/11831/console

Li-Wen



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAKBkRUyxzJi65F3fPZ9mJT3VibjBD5F1SHjNMVGshFk6vPsqTA>