Skip to content

Commit

Permalink
Remove bad assert
Browse files Browse the repository at this point in the history
It is possible also in non-SMP case:
1. The process receives an exit signal and is set in status exiting
   and inserted into the run queue.
2. The distribution port exits before the process has been selected
   for execution and cannot remove the link half on the process since
   it is in status exiting.
3. Process is selected for execution and when removing this link half
   the distribution channel is gone!
  • Loading branch information
rickard-green committed Dec 20, 2011
1 parent c91492b commit d97c8d5
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions erts/emulator/beam/erl_process.c
Expand Up @@ -7980,15 +7980,6 @@ static void doit_exit_link(ErtsLink *lnk, void *vpcontext)
if (rlnk)
erts_destroy_link(rlnk);
erts_deref_dist_entry(dep);
} else {
#ifndef ERTS_SMP
/* XXX Is this possible? Shouldn't this link
previously have been removed if the node
had previously been disconnected. */
ASSERT(0);
#endif
/* This is possible when smp support has been enabled,
and dist port and process exits simultaneously. */
}
break;

Expand Down

0 comments on commit d97c8d5

Please sign in to comment.