Re: [PATCH] um: hostfs: catch EINTR and partial read/write

2024-01-05 Thread Benjamin Berg
On Thu, 2024-01-04 at 23:27 +0100, Richard Weinberger wrote: > On Fri, Nov 10, 2023 at 10:44 AM wrote: > > > > From: Benjamin Berg > > > > The UM kernel uses signals for various purposes (SIGALRM for > > scheduling > > for example). These signals are interrupts for the UM kernel, which > > shou

Re: [PATCH] um: hostfs: catch EINTR and partial read/write

2024-01-04 Thread Richard Weinberger
On Fri, Nov 10, 2023 at 10:44 AM wrote: > > From: Benjamin Berg > > The UM kernel uses signals for various purposes (SIGALRM for scheduling > for example). These signals are interrupts for the UM kernel, which > should not affect file system operations from userspace processes. > > Said different

Re: [PATCH] um: hostfs: catch EINTR and partial read/write

2023-11-10 Thread Benjamin Berg
On Fri, 2023-11-10 at 11:12 +, Anton Ivanov wrote: > > > On 10/11/2023 11:10, Benjamin Berg wrote: > > Hi, > > > > On Fri, 2023-11-10 at 10:56 +, Anton Ivanov wrote: > > > On 10/11/2023 10:42, Anton Ivanov wrote: > > > > [SNIP] > > > > > > > > If we are going to use this definition of C

Re: [PATCH] um: hostfs: catch EINTR and partial read/write

2023-11-10 Thread Anton Ivanov
On 10/11/2023 11:10, Benjamin Berg wrote: Hi, On Fri, 2023-11-10 at 10:56 +, Anton Ivanov wrote: On 10/11/2023 10:42, Anton Ivanov wrote: [SNIP] If we are going to use this definition of CATCH_EINTR throughout we might as well remove the partial read/write code in UBD and other places.

Re: [PATCH] um: hostfs: catch EINTR and partial read/write

2023-11-10 Thread Benjamin Berg
Hi, On Fri, 2023-11-10 at 10:56 +, Anton Ivanov wrote: > On 10/11/2023 10:42, Anton Ivanov wrote: > > [SNIP] > > > > If we are going to use this definition of CATCH_EINTR throughout we > > might as well remove the partial read/write code in UBD and other > > places. > > Actually - some of it

Re: [PATCH] um: hostfs: catch EINTR and partial read/write

2023-11-10 Thread Anton Ivanov
On 10/11/2023 10:42, Anton Ivanov wrote: On 10/11/2023 09:44, benja...@sipsolutions.net wrote: From: Benjamin Berg The UM kernel uses signals for various purposes (SIGALRM for scheduling for example). These signals are interrupts for the UM kernel, which should not affect file system opera

Re: [PATCH] um: hostfs: catch EINTR and partial read/write

2023-11-10 Thread Anton Ivanov
On 10/11/2023 09:44, benja...@sipsolutions.net wrote: From: Benjamin Berg The UM kernel uses signals for various purposes (SIGALRM for scheduling for example). These signals are interrupts for the UM kernel, which should not affect file system operations from userspace processes. Said diffe

[PATCH] um: hostfs: catch EINTR and partial read/write

2023-11-10 Thread benjamin
From: Benjamin Berg The UM kernel uses signals for various purposes (SIGALRM for scheduling for example). These signals are interrupts for the UM kernel, which should not affect file system operations from userspace processes. Said differently, in hostfs all operations are directly forwarded to