From ce7bf1839d868fd829f0224b226da54612ac0e88 Mon Sep 17 00:00:00 2001 From: Simon Marlow Date: Mon, 23 Nov 2009 11:04:16 +0000 Subject: [PATCH] disable a false assertion, add a comment to explain why --- rts/sm/Scav.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/rts/sm/Scav.c b/rts/sm/Scav.c index a2ee1ce..466b9b4 100644 --- a/rts/sm/Scav.c +++ b/rts/sm/Scav.c @@ -1490,9 +1490,11 @@ scavenge_mutable_list(bdescr *bd, generation *gen) case TSO: { StgTSO *tso = (StgTSO *)p; if (tso->dirty == 0) { - // Must be on the mutable list because its link - // field is dirty. - ASSERT(tso->flags & TSO_LINK_DIRTY); + // Should be on the mutable list because its link + // field is dirty. However, in parallel GC we may + // have a thread on multiple mutable lists, so + // this assertion would be invalid: + // ASSERT(tso->flags & TSO_LINK_DIRTY); scavenge_TSO_link(tso); if (gct->failed_to_evac) { -- 1.7.10.4