Don't restart in deterministic stages. Don't restart where there's no restart safety. Make stage names unique

[tokatoka]

In order to make retryrestarting helper or exec counting helper work it is necessary that every created instance of stage have unique names to begin with.

[tokatoka]

Reasons for choosing the restarting helper

• calibration: deterministic, don't restart
• colorization: deterministic, don't restart
• concolic{tracing, mutation}: deterministic, don't restart
• ClosureStage, PushAdapater: there's no guarantee if the closure or pushed stage is restart-safe, don't restart
• sync stage: deterministic, don't restart
• tracing,aflpptracing: deterministic, don't restart

[domenukk]

At this point we could also decide to make handles always use ints instead of stings(?)

Would mean hashtable lookups get faster

[tokatoka]

it needs to be string as aarnav wants it to be similar to afl++ right?

[domenukk]

I mean we can still have the name, but hashtable lookups could be int

[tokatoka]

I mean we can still have the name, but hashtable lookups could be int

Maybe in a diffrent PR. but it's possible. We'd need change in trait Named for this. because we'll have name() for human-readable name, and another instance_id() to get the unique id for all the modules.

[tokatoka]

It fails because of network error, not my code's fault