Re: [PATCH 20/30] bsd-user/signal.c: core_dump_signal

2022-01-23 Thread Richard Henderson
On 1/10/22 3:19 AM, Warner Losh wrote: Returns 1 for signals that cause core files. Signed-off-by: Stacey Son Signed-off-by: Kyle Evans Signed-off-by: Warner Losh --- bsd-user/signal.c | 17 + 1 file changed, 17 insertions(+) Reviewed-by: Richard Henderson r~

Re: [PATCH 20/30] bsd-user/signal.c: core_dump_signal

2022-01-13 Thread Peter Maydell
On Thu, 13 Jan 2022 at 20:28, Warner Losh wrote: > On Thu, Jan 13, 2022 at 1:22 PM Peter Maydell > wrote: >> Code is fine, but since this is a static function with no callers >> the compiler is going to emit a warning about that. It's a small >> function, so the easiest thing is just to squash t

Re: [PATCH 20/30] bsd-user/signal.c: core_dump_signal

2022-01-13 Thread Warner Losh
On Thu, Jan 13, 2022 at 1:22 PM Peter Maydell wrote: > On Sun, 9 Jan 2022 at 16:48, Warner Losh wrote: > > > > Returns 1 for signals that cause core files. > > > > Signed-off-by: Stacey Son > > Signed-off-by: Kyle Evans > > Signed-off-by: Warner Losh > > --- > > bsd-user/signal.c | 17 ++

Re: [PATCH 20/30] bsd-user/signal.c: core_dump_signal

2022-01-13 Thread Peter Maydell
On Sun, 9 Jan 2022 at 16:48, Warner Losh wrote: > > Returns 1 for signals that cause core files. > > Signed-off-by: Stacey Son > Signed-off-by: Kyle Evans > Signed-off-by: Warner Losh > --- > bsd-user/signal.c | 17 + > 1 file changed, 17 insertions(+) > > diff --git a/bsd-user

[PATCH 20/30] bsd-user/signal.c: core_dump_signal

2022-01-09 Thread Warner Losh
Returns 1 for signals that cause core files. Signed-off-by: Stacey Son Signed-off-by: Kyle Evans Signed-off-by: Warner Losh --- bsd-user/signal.c | 17 + 1 file changed, 17 insertions(+) diff --git a/bsd-user/signal.c b/bsd-user/signal.c index a6e07277fb2..824535be8b8 100644 -