Re: [PATCH v10] um: Enable preemption in UML

2024-07-02 Thread Johannes Berg
On Tue, 2024-07-02 at 10:25 +0200, Johannes Berg wrote: > From: Anton Ivanov > > Since userspace state is saved in the MM process, kernel using > FPU still doesn't really need to do anything, so this really > is as simple as enabling preemption. The irq critical section > in sigio_handler() needs

Re: [PATCH v10] um: Enable preemption in UML

2024-07-02 Thread Anton Ivanov
On 02/07/2024 11:18, Johannes Berg wrote: On Tue, 2024-07-02 at 11:07 +0100, Anton Ivanov wrote: Is that all we have left? I think so, yes. I lost track of your mm patches and they do not apply cleanly to master at present. Mine actually mostly went away, in favour of Benjamin's, but

Re: [PATCH v10] um: Enable preemption in UML

2024-07-02 Thread Johannes Berg
On Tue, 2024-07-02 at 12:18 +0200, Johannes Berg wrote: > > We have 40 patches in patchwork now, including the 7 64-bit address > space ones from Benjamin where I think some rework is needed. I've > rebased/applied the other 33, I've just pushed that to the 'pending' > branch: > > https://git.ker

Re: [PATCH v10] um: Enable preemption in UML

2024-07-02 Thread Johannes Berg
On Tue, 2024-07-02 at 11:07 +0100, Anton Ivanov wrote: > > > Is that all we have left?  I think so, yes. > I lost track of your mm patches and they do not apply cleanly to master at > present. Mine actually mostly went away, in favour of Benjamin's, but his also didn't apply cleanly due to th

Re: [PATCH v10] um: Enable preemption in UML

2024-07-02 Thread Anton Ivanov
On 02/07/2024 09:25, Johannes Berg wrote: From: Anton Ivanov Since userspace state is saved in the MM process, kernel using FPU still doesn't really need to do anything, so this really is as simple as enabling preemption. The irq critical section in sigio_handler() needs preempt_disable()/pr

[PATCH v10] um: Enable preemption in UML

2024-07-02 Thread Johannes Berg
From: Anton Ivanov Since userspace state is saved in the MM process, kernel using FPU still doesn't really need to do anything, so this really is as simple as enabling preemption. The irq critical section in sigio_handler() needs preempt_disable()/preempt_enable(). Signed-off-by: Anton Ivanov [