lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-Id: <20170914000908.713-1-dave@stgolabs.net> Date: Wed, 13 Sep 2017 17:09:08 -0700 From: Davidlohr Bueso <dave@...olabs.net> To: akpm@...ux-foundation.org Cc: vbabka@...e.cz, parri.andrea@...il.com, dave@...olabs.net, linux-kernel@...r.kernel.org, Davidlohr Bueso <dbueso@...e.de> Subject: [PATCH] mm,compaction: serialize waitqueue_active() checks (for real) Andrea brought to my attention that the L->{L,S} guarantees are completely bogus for this case. I was looking at the diagram, from the offending commit, when that _is_ the race, we had the load reordered already. What we need is at least S->L semantics, thus simply use wq_has_sleeper() to serialize the call for good. Fixes: 46acef048a6 (mm,compaction: serialize waitqueue_active() checks) Reported-by: Andrea Parri <parri.andrea@...il.com> Signed-off-by: Davidlohr Bueso <dbueso@...e.de> --- mm/compaction.c | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/mm/compaction.c b/mm/compaction.c index fb548e4c7bd4..d1ad2cf6c758 100644 --- a/mm/compaction.c +++ b/mm/compaction.c @@ -1999,17 +1999,16 @@ void wakeup_kcompactd(pg_data_t *pgdat, int order, int classzone_idx) if (pgdat->kcompactd_max_order < order) pgdat->kcompactd_max_order = order; - /* - * Pairs with implicit barrier in wait_event_freezable() - * such that wakeups are not missed in the lockless - * waitqueue_active() call. - */ smp_acquire__after_ctrl_dep(); if (pgdat->kcompactd_classzone_idx > classzone_idx) pgdat->kcompactd_classzone_idx = classzone_idx; - if (!waitqueue_active(&pgdat->kcompactd_wait)) + /* + * Pairs with implicit barrier in wait_event_freezable() + * such that wakeups are not missed. + */ + if (!wq_has_sleeper(&pgdat->kcompactd_wait)) return; if (!kcompactd_node_suitable(pgdat)) -- 2.12.0
Powered by blists - more mailing lists