Re: [PATCH 5/5] task_work: use TIF_NOTIFY_SIGNAL if available

Jens Axboe
2020-10-16T13:35:48Z

On 10/16/20 3:39 AM, Thomas Gleixner wrote:
> On Fri, Oct 16 2020 at 11:00, Thomas Gleixner wrote:
>> On Thu, Oct 15 2020 at 12:39, Jens Axboe wrote:
>>> On 10/15/20 9:49 AM, Oleg Nesterov wrote:
>> So if you change #2 to:
>>
>> Drop the CONFIG_GENERIC_ENTRY dependency, make _all_ architectures
>> use TIF_NOTIFY_SIGNAL and clean up the jobctl and whatever related
>> mess.
>>
>> and actually act apon it, then I'm fine with that approach.
>
> Which makes me rethink my view on Olegs suggestion:
>
>>>> You can simply nack the patch which adds TIF_NOTIFY_SIGNAL to
>>>> arch/xxx/include/asm/thread_info.h.
>
> That's a truly great suggestion:
>
> X86 is going to have that TIF bit once the above is available.
>
> I'm happy to help with the merge logistics of this.

Not really following this email...

But it seems to me that you're happy with approach 2, so I'll do
the following:

- Get rid of the CONFIG_GENERIC_ENTRY dependency for TIF_NOTIFY_SIGNAL
- Respin the arch additions and cleanups on top of that again

And hopefully we'll have something mergeable at that point. Once we
have this series merged somewhere (would be great if you could carry
it), I'll be talking to arch folks on the rest. Once archs have taken
the necessary bits, I'll be posting the third and final series which
is the cleanups that are currently sitting on top of the arch support.

--
Jens Axboe