scanalysis: fixup spacing

test: sctest: fix warnings

Perhaps we wanted to print these variables? Anyway, just make them
global, so the warnings will go away:

  gcc -o sctest.o sctest.c -Wall -g -O3 -I.. -I../include -L.. -lmodel
  sctest.c: In function ‘b’:
  sctest.c:20:6: warning: unused variable ‘r1’ [-Wunused-variable]
  sctest.c: In function ‘c’:
  sctest.c:26:6: warning: unused variable ‘r2’ [-Wunused-variable]
  sctest.c: In function ‘d’:
  sctest.c:33:6: warning: unused variable ‘r3’ [-Wunused-variable]

Fix bug that prevents graph generation from compiling.
Check in test case that shows theorem that I've been trying to prove for 2 days is in fact not true (and thus difficult to prove).

action: improve "unintialized load" bug print

action: add get_{type,mo}_str() accessors

The big switch/case statements can be shortened and separated into their
own functions.

Merge remote-tracking branch 'private/master'

README: update text

bug...will sometimes print SC traces in a bad order

cleanup printing

Rework how we present non-SC traces...previous presentation didn't make reasons clear.

snapshot-interface: bugfix - terminate string from readlink()

readlink() doesn't terminate the string for us, so our string doesn't
match properly (it will have a little extra garbage at the end).

snapshot: debugging - show the snapshot regions that are added

snapshot-interface: don't snapshot libmodel.so

We don't need to snapshot our own globals; we are careful to only place
constant pointers in global memory.

snapshot: use perror()

main: support long options, improve help message

execution: improve documentation

is_yieldblocked() wasn't documented

check_action_enabled() had incorrect documentation, since I removed the
waiter lists

execution: improve yield-blocking structure

We should just check the yieldblock parameter within
ModelExecution::is_yieldblock().

TMP: improve debugging, remove print_trace()

Remove special cases for printing SC executions.

Just use happens before relation instead.  Also include a criteria to keep things in execution order as much as possible.

Add yield block support.  The idea is to not generate executions with yield actions.

While preventing yields doesn't preserve livelock, under the CHESS assumptions, it does allow us to reach all states.

datarace: bugfix - use proper multiple-inclusion guard

The DATARACE_H macro is never defined, so it never succeeds as a
multiple-inclusion guard.

stacktrace: remove usage of FILE in stacktrace

The original print_stacktrace() function required a FILE pointer, so we
manufactured one. We really should avoid the bugs that can come from the
allocation and buffering done with FILE, since this is used on error
paths.

Bug: ModelExecution had bogus model_params reference.

Fix.

datarace: bugfix - don't implicitly allocate global SnapVector

We do not want to pollute the user's snapshotting region with the
unrealizedraces SnapVector. Allocate it in initRaceDetector(), so that
it is placed on the model-checker's snapshotting heap.

datarace: don't export unrealized race vector

execution: bugfix - no action "conflicts" with itself

execution: bugfix - backtrack seq-cst fences properly

The commented-out ATOMIC_FENCE should not be commented out; the note
that "fences don't directly cause backtracking" is false for seq-cst
fences. We intend to backtrack for all seq-cst fences, so we need to
perform the appropriate 'get conflict' search for all fences.

execution: correct comment on check_current_action()

nodestack: localize the model-checker parameters

model: privatize ModelChecker::get_num_threads()

The public interface is now in ModelExecution.

nodestack: register ModelExecution class w/in NodeStack

promise: add max_available_thread_idx() interface

To remove an unnecessary use of global/public model->get_num_threads().

clockvector: don't use global get_num_threads()

model: remove leftover junk

execution: embed obj_map directly in class

This structure can be embedded as a full-fledged member, not a pointer.

hashtable: update documentation

Some of the existing documentation was incorrect or insufficient. Also,
add new documentation to describe the design "feature" of this table:
that 0/NULL is not a valid key.

hashtable: enforce, document non-zero keys

HashTable does not support a key of "zero." ...

execution: convert HashTable to SnapVector

We don't actually need a hash table for threads, since we allocate their
indeces contiguously, at least for now. Also, HashTable is really
designed for pointer-based keys and may not accept a 0-valued key. To
avoid these problems, just switch to a snapshotted vector.

action: bugfix - use non-zero fence "location"

Our HashTable does not support zero-based keys (e.g., NULL pointer as a
key). So as a hack, switch to use a small, arbitrary, non-zero location
instead of NULL (0).

execution: embed action_list as full member

execution: add const

execution: embed more data structures directly in class

execution: embed snapshotted data structures in class

execution: make structure snapshotting

All members are snapshot-able.

execution: move execution number back to ModelChecker class

clean out includes, etc.

model: embed the trace_analyses in the class

action: add <stdlib>

For exit() and EXIT_*

scanalysis: allocate structures as true members of class (not pointers)

They're all snapshotting, so it makes sense to make them legitimate
members of the SCAnalysis class.

scanalysis: fixup spacing

Merge cleanup branch

This branch separated ModelChecker into two separate classes/modules. It was a
big disruptive change, and it was not compile-able at some points (the BROKEN
commit and some of its successors). Hence the non-fast-forward merge here, to
record the divergent portion of history as special.

schedule: drop the ModelChecker::check_promises_thread_disabled interface

model / threads: remove global get_next_id() interface

promise: get reference to ModelExecution

scanalysis: use ModelExecution interfaces

scanalysis: install ModelExecution object in the analysis

execution: add 'const'

model: cleanup a few more interfaces

These public interfaces are no longer presentin ModelChecker.

model: add get_execution() interface

BROKEN: restructure much of ModelChecker as ModelExecution class

This will *not* compile fully. It is an interim step in which I move
much of the ModelChecker code into a separate class which does not need
to be globally-accessible. The next few commits should fix the compile
problems as I rewrite/remove some interfaces.

datarace: simplify raceCheck{Read,Write}() function interfaces

The user should not have to pass in a ClockVector.

datarace: make globals static

They're only global within this file.

model: drop public get_current_node() interface

model: remove public check_promises() interface

model: add synchronize() function

To prevent code duplication.

common: drop model_print_summary()

This can cause problems if we include it in the output of ASSERT().

promise: add Promise::get_index function

action: we don't need special cases for lock/join anymore

move license to LICENSE file

Merge cleanup code from Brian N.

traceanalysis: rename to remove '_'

bugmessage: move struct to header file

fix weird duplicated code

params: move model_params to header file

model: add too_many_steps()

model: fixup whitespace

scanalysis: fix warning

canalysis.cc: In member function ‘bool SCAnalysis::processRead(ModelAction*, ClockVector*)’:
scanalysis.cc:162:78: warning: suggest parentheses around ‘&&’ within ‘||’ [-Wparentheses]

scanalysis: remove whitespace

clean up printing a little for cycle cases...

more implementation of scanalysis...

implementation of scanalysis and printout code...  should work now

more implementation of scanalysis...
seems like it might work at this point, but need code to tell whether it is a SC trace or not and which parts are SC

towards supporting scanalysis...
added some accessor methods in model.cc for external analyses

add some support for traceanalysis plugins

add some support for traceanalysis plugins

improve documentation + Doxygen formatting

action: refactor, move VALUE_TRY{SUCCESS,FAILED} out of header

cyclegraph: kill unused promise_list_t type

snapshot: cleanup/document much of fork-based snapshotting

We can consolidate much of the fork-based snapshotting "context"
variables so that they are easier to understand; we essentially only
need two types of contexts, the "shared" context used for the client
program (i.e., the model-checker) and the "private" context, which
each snapshot process uses for running and managing its state. That way,
we only have a single process using the "shared" context at a time,
while the others reside in their private context.

Additionally, I add an "exit" context, which is used just to help with
terminating processes once they have to roll back.

Otherwise, this commit contains renaming and documenting of several
fields, variables, etc.

snapshot: use sizeof(*fork_snap)

model: remove DEBUG action print

A ModelAction should only be printed at certain points, where the action
is guaranteed to be consistent (e.g., its reads_from pointer is valid).
Rather than try to position this print opportunely, just drop it; we
already print out the whole trace, which should be sufficient.

snapshot: clean up fork-based error handling

This is an ugly attempt at error handling. Kill the useless variables,
reword the debug prints, and use perror().

action: correct RF capitalization

model: deadlock: print the culprit thread, when known

datarace: use variable-argument bug-printing

model: add variable arguments for bug messages

model: proactively build/prune is_enabled() set

We should proactively check whether threads are enabled, as this allows
the model-checker to more accurately reason about backtracking, fairness
(particularly, assigning thread priorities), and sleep sets.

This commit causes changes in observed executions for many test
programs. Often, this occurs for programs where the main thread
performs a JOIN on a child thread. Previously, we would only discover
that this thread should be disabled once the scheduler attempts to
execute the JOIN ModelAction. For the period of the trace in which this
disabling isn't discovered, we may try to set backtracking points for
the main thread. These executions just turn into infeasible or
sleep-set-redundant executions anyway.

This can also affect the assertion of thread priority when assigning
backtracking points. If a thread is supposedly "enabled" (but in fact is
not), then we may prevent other threads from running because this thread has
priority. But that thread cannot run, so instead we were doing some
funky things with the remaining ("low-priority") threads.

This fix remedies these sorts of issues by precisely computing the set
of threads which are disabled, waiting on other threads/locks.

model: pull thread control logic out of take_step(), check_current...()

Much of the scheduling and main control loop should be handled closer to
the top-level run() method. This is a start toward putting these
decisions at the appropriate level, making things more clear. Of course,
it's not complete.

snapshot: kill useless macro

We're not reusing this macro, so it's better just to use the standard
perror() followed by exit().

context: move Mac swapcontext() to separate compilation unit

We really don't want model_swapcontext() to ever be inlined on Mac OSX,
since this could potentially cause subtle getcontext()-related bugs to
resurface. Just make it a separate compilation unit (*.cc file).

common: improve redirect_output() error handling

Check the return code and errno results of all system/library calls.
Also delete the extraneous 'fd' variable and use the appropriate
STDOUT_FILENO macro instead of fileno(stdout).