The fact that a boolean argument is present does not imply
that it's true.

This reverts commit 57cd74cf.

I promise to look after this once I got my Flexelint config back,
which, unfortunately, I lost during a reinstall. Yes, I do have
a backup, but cannot access it easily at the moment.

Fixes #2657.

For backend requests, we do not have the thread local storage of the
client thread available, so trq.req will normally be (nil).

Dump the busyobj back pointers to req/sess/wrk to maximize the
usefulness of panics.

Also reorder the panic by struct busyobj and add some more members
which may or may not be useful (but when we find out, it is likely
too late).

Also move the vmod list dump to the top level.

Noticed when trying to understand if #2667 was also a stack overflow
in the context of #2817

Related to #2810

If two blob pointers compare equal, the blobs are both the same and
equal - both being NULL is implied.

ALLOC_OBJ calls memset(0) already

Introduce VSIG which simply counts number of signals, which everybody
else can then look at.

If you use libvarnishapi and want your own signal handler, you must
increment these VSIG_{int|hup|term|usr1} counters yourself, for
libvarnishapi to work correctly.

The old VUT_Signal()/VUT_Signaled() API is retained and should be
deprecated in a future version.

Inspired by:	#2818

was lost in f2abaf86.

will continue looking backwards from 79687f13

Now that we include it in cache.h, it is required by vmods

Ref #2813

Avoid (potential) issues due to the monotonic clock not advancing for ~10
seconds such as for virtual machine snapshots or migrations.

In such a scenario, we would kill a child after cli_timeout anyway, so
using the same timeout by default for the watchdog also should reduce
the additional risk implied by it.

Ref	#2814

For healthy == IGNORE, we return before the call to shard_next(), so
at this point we do want to consider health state - and it makes
absolutely no sense to give up a healthy backend in favor of an un-
healthy one anyway.

Fixes #2823

(at leat on solaris) strerror() itself may fail for an out-of-memory
condition (because the localization code contains memory
allocations). In order to handle this situation, we need to save the
original errno because strerror() may also set errno.

Fixes #2815

This reverts commit ee33662a.

This may reduce our binary size.

seen by sun cc as:

"cache/cache_vrt_priv.c", line 54: syntax error:  empty declaration (E_EMPTY_DECLARATION)
"cache/cache_vrt_priv.c", line 109: syntax error:  empty declaration (E_EMPTY_DECLARATION)

The vcl and id members served no purpose except for debugging.

With this, we save 2 * sizeof(uintptr_t), which is exactly the
additional footprint introduced with red/black trees. So the
net footprint of a struct vrt_priv is now back to what it had
been before #2813.

For debugging, it should also be trivial to induce the context
from the address of the tree root.

O(n) does not scale for a high number of dynamic privs, so trade some
workspace for O(lg n)

Ref: #2812

sed 's:VRB_:VRBT_:g' -i "$FILES_CHANGED[@]"

Closes #2812

Fixes #2820