Re: [PATCH] md: don't use flush_signals in userspace processes

2017-06-08 Thread Henrique de Moraes Holschuh
On Fri, 09 Jun 2017, NeilBrown wrote: > Or maybe it could be discarded - the md_check_recovery() thing. > The idea was that if you alt-sysrq-K to kill all processes, md arrays > would go into immediate-safe-mode where the metadata is marked clean > immediately after writes finish, rather than waiti

Re: [PATCH] md: don't use flush_signals in userspace processes

2017-06-08 Thread NeilBrown
On Thu, Jun 08 2017, Mikulas Patocka wrote: > On Thu, 8 Jun 2017, NeilBrown wrote: > >> On Wed, Jun 07 2017, Mikulas Patocka wrote: >> >> > The function flush_signals clears all pending signals for the process. It >> > may be used by kernel threads when we need to prepare a kernel thread for >> >

Re: [PATCH] md: don't use flush_signals in userspace processes

2017-06-08 Thread NeilBrown
On Thu, Jun 08 2017, Shaohua Li wrote: > On Fri, Jun 09, 2017 at 07:24:29AM +1000, Neil Brown wrote: >> On Thu, Jun 08 2017, Mikulas Patocka wrote: >> >> > On Thu, 8 Jun 2017, Shaohua Li wrote: >> > >> >> On Thu, Jun 08, 2017 at 04:59:03PM +1000, Neil Brown wrote: >> >> > On Wed, Jun 07 2017, Mik

Re: [PATCH] md: don't use flush_signals in userspace processes

2017-06-08 Thread Shaohua Li
On Fri, Jun 09, 2017 at 07:24:29AM +1000, Neil Brown wrote: > On Thu, Jun 08 2017, Mikulas Patocka wrote: > > > On Thu, 8 Jun 2017, Shaohua Li wrote: > > > >> On Thu, Jun 08, 2017 at 04:59:03PM +1000, Neil Brown wrote: > >> > On Wed, Jun 07 2017, Mikulas Patocka wrote: > >> > > >> > > The functio

Re: [PATCH] md: don't use flush_signals in userspace processes

2017-06-08 Thread Mikulas Patocka
On Thu, 8 Jun 2017, NeilBrown wrote: > On Wed, Jun 07 2017, Mikulas Patocka wrote: > > > The function flush_signals clears all pending signals for the process. It > > may be used by kernel threads when we need to prepare a kernel thread for > > responding to signals. However using this function

Re: [PATCH] md: don't use flush_signals in userspace processes

2017-06-08 Thread NeilBrown
On Thu, Jun 08 2017, Mikulas Patocka wrote: > On Thu, 8 Jun 2017, Shaohua Li wrote: > >> On Thu, Jun 08, 2017 at 04:59:03PM +1000, Neil Brown wrote: >> > On Wed, Jun 07 2017, Mikulas Patocka wrote: >> > >> > > The function flush_signals clears all pending signals for the process. It >> > > may be

Re: [PATCH] md: don't use flush_signals in userspace processes

2017-06-08 Thread Mikulas Patocka
On Thu, 8 Jun 2017, Shaohua Li wrote: > On Thu, Jun 08, 2017 at 04:59:03PM +1000, Neil Brown wrote: > > On Wed, Jun 07 2017, Mikulas Patocka wrote: > > > > > The function flush_signals clears all pending signals for the process. It > > > may be used by kernel threads when we need to prepare a k

Re: [PATCH] md: don't use flush_signals in userspace processes

2017-06-07 Thread NeilBrown
On Wed, Jun 07 2017, Mikulas Patocka wrote: > The function flush_signals clears all pending signals for the process. It > may be used by kernel threads when we need to prepare a kernel thread for > responding to signals. However using this function for an userspaces > processes is incorrect - clea

[PATCH] md: don't use flush_signals in userspace processes

2017-06-07 Thread Mikulas Patocka
The function flush_signals clears all pending signals for the process. It may be used by kernel threads when we need to prepare a kernel thread for responding to signals. However using this function for an userspaces processes is incorrect - clearing signals without the program expecting it can cau