[Devel] [PATCH 0/2] fuse: fix signals handling while processing request

2016-10-13 Thread Stanislav Kinsburskiy
This patch fixes wrong SIGBUS result in page fault handler for fuse file, when process received a signal. https://jira.sw.ru/browse/PSBM-53581 --- Stanislav Kinsburskiy (2): new helper: wait_event_killable_exclusive() fuse: handle only fatal signals while waiting request answer fs

Re: [Devel] [PATCH 0/2] fuse: fix signals handling while processing request

2016-10-13 Thread Maxim Patlasov
Stas, The series look fine, so: Acked-by: Maxim Patlasov But, please, refine the description of the second patch. It must explain clearly why the patch fixes the problem: block_sigs() blocks ordinary non-fatal signals as expected, but surprisingly SIGTRAP is special: it does not matter w

Re: [Devel] [PATCH 0/2] fuse: fix signals handling while processing request

2016-10-14 Thread Stanislav Kinsburskiy
14.10.2016 02:23, Maxim Patlasov пишет: Stas, The series look fine, so: Acked-by: Maxim Patlasov But, please, refine the description of the second patch. It must explain clearly why the patch fixes the problem: block_sigs() blocks ordinary non-fatal signals as expected, but surprising

Re: [Devel] [PATCH 0/2] fuse: fix signals handling while processing request

2016-10-15 Thread Maxim Patlasov
Stas, On 10/14/2016 03:30 AM, Stanislav Kinsburskiy wrote: 14.10.2016 02:23, Maxim Patlasov пишет: Stas, The series look fine, so: Acked-by: Maxim Patlasov But, please, refine the description of the second patch. It must explain clearly why the patch fixes the problem: block_sigs()

Re: [Devel] [PATCH 0/2] fuse: fix signals handling while processing request

2016-10-17 Thread Stanislav Kinsburskiy
16.10.2016 05:21, Maxim Patlasov пишет: Stas, On 10/14/2016 03:30 AM, Stanislav Kinsburskiy wrote: 14.10.2016 02:23, Maxim Patlasov пишет: Stas, The series look fine, so: Acked-by: Maxim Patlasov But, please, refine the description of the second patch. It must explain clearly why

Re: [Devel] [PATCH 0/2] fuse: fix signals handling while processing request

2016-10-17 Thread Maxim Patlasov
Stas, Now, when we fully understand the patch, will you fix description of 98cbcb14d? The following does not seem correct: >IOW, any signal, arrived to the process, which does page fault handling on fuse >file, _before_ request_wait_answer() is called, will lead to request >int

Re: [Devel] [PATCH 0/2] fuse: fix signals handling while processing request

2016-10-17 Thread Stanislav Kinsburskiy
17 окт. 2016 г. 19:23 пользователь Maxim Patlasov написал: > > Stas, > > > Now, when we fully understand the patch, will you fix description of > 98cbcb14d? Yes, I"sent the update to Konstantin already. Thanks a lot again. The following does not seem correct: > > > >IOW, any signal, arrive