Re: [PATCH 1/2] workqueue: Add new function mod_fwd_delayed_work()

From: Mark Brown
Date: Tue Mar 07 2017 - 15:07:34 EST


On Mon, Feb 27, 2017 at 08:17:10PM +0100, Harald Geyer wrote:
> Mark Brown writes:

> > I'd need to figure out exactly what the restrictions are and like I say
> > the name of the function itself is confusing, I suspect because it
> > predates SMP.

> I guess you know that, but just to avoid any confusion: The bug in the
> regulator code is not related to SMP at all.

It's not? Oh. I had formed the impression that this was a race
condition.

> > Actually yes, though not immediately. Another option is to just rename
> > the current function and all the callers en masse then add a new, safe
> > mod_delayed_work().

> Okay by me. I'm removing the issue from my todo list and hand it over
> to you and your minions ... :)

Well, we need to figure out what the desired solution is!

Attachment: signature.asc
Description: PGP signature