workqueue fixlets (Was: PM: Add suspend blocking work.)
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest2 patches 6 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/2 | workqueues: flush_delayed_work: keep the original workqueue for re-queueingkernel/ | +1 | 1 reply | |
| 2/2 | workqueues: export keventd_wqinclude/ kernel/ | +3 | 5 replies |
Reviews trailers per patch
1 of 2 patches has a Reviewed-by or Acked-by.
| Patch | Tejun Heo |
|---|---|
| 1/2 | A-b |
| 2/2 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH 0/2] workqueue fixlets (Was: PM: Add suspend blocking work.)
Quoted 7 lines · Oleg Nesterov
I don't really care, this change is trivial. But to me it makes more sense to push the trivial/simple changes ahead. Unless they really complicate the maintainging of the pending cmwq changes.
Hmm... Speaking about
keventd_wq, I just noticedflush_delayed_work()needs the fix:
So. I'd suggestd these 2 simple patches for now.
Please note that 2/2 is really trivial, it only adds EXPORT_SYMBOL but avoids the possible cleanups to minimize the conflicts with the pending cmwq changes. However, please feel free to ignore this patch.
As for 1/2, imho it is always better to fix the bug asap, even if it is minor.
Oleg.