Use active flag on each end of the channel

Change the way workers are signalled to exit

They now wait for all their channels to close, then signal themselves to exit via their event loop.

General cleanups in the worker code

Update comments, normalise debug messages.

Switch to atomics for shared fields

Minor cleanups in channel code

Use REQUESTOR in place of "master" and "network", and RESPONDER in place of "worker". The channel code will likely be reused very soon to create module specific thread pools to deal with blocking requests, and it's confusing to use usecase specific terms in generic code.

Move all size_t to uint64_t.  Guessing the original author was just being lazy and didn't want to type %"PRIu64 in a bunch of places.

Rename "ctx" to "uctx" because that's what it is...

Only emmit info messages about systemd if we're running under systemd

Worker starting messages are INFO level, exiting messages should be too

Free the snmp dictionaries

Missed debug build check

Still need to memset...

Address potential memory leak in track structures

Invalid  assert when module is exiting

...and a log message

Add a note...

Ignore sigterm earlier and send kill later

Stop the scheduler before sending the TERM signal

Fix segv

Needs somewhere to put the event

Better way of signalling termination

Add -e for triggering an exit after N seconds

Allow date and time_delta to be flipped

Only call fr_value_box_hton on numeric types

Add more context to assert

Fix struct names

Just use the list for counting workers

Rename schedule struct

Use destructor to enforce absolute free order

Rename local alloc function

run max_request timer no more than once every max_request time

this saves 1/3 to 1/2 in the test runs

move timer logic to function that implements the timer.

Handle OpenSSL init/free ourselves

Free the right thing

Avoid calling curl global init more than once

Fix minor leak on exit

Gotta catch em all...

Ignore until we have a proper fix

stack is void

Fix assert when trying to cancel new requests

Gracefully deal with trunk timer failures

Deal with failed timer events gracefully in connection.c

Don't crash on exit in threaded mode when using rlm_radius

Typo

Typo

Did not allocate this, so shouldn't be freeing it

causes crash on exit

Fix TOCTOU like issue in conn_closed caused by the a filedescriptor not being removed from the event loop before it's been freed

Quiet clang?

Fix assert in rlm_cache on unknown attributes

Don't set tmpl_da and immediately overwrite it

Don't emmit warnings if was can't find gigawords

Set conn to NULL after it's meant to have been freed

Halt if a connection attempt failed and no reconnect_delay was set

Rework time tracking so that it no longer uses an explicit tracking list

Move time tracking code into its own source file

Revert "add and use sanity check functions for dlists"

This reverts commit 147307e922828e5c183ece1c8c05cd699917a432.

nope

add and use sanity check functions for dlists

initialize to be the same as the worker tracking list

Don't allow the pool to start if we're in check config mode

Enable gperftools for eveloper builds

Barely noteworthy let alone error worthy

Not sure how it could ever be a garbage value, but oh well..

Add failed state to the trunk code

Partially mirrors connection's failed state, but only triggers if the connection immediately fails after opening.

remove Response-Packet-Type

now that it's unused everywhere

remove Response-Packet-Type

The 'Response-Packet-Type' was replaced by 'Packet-Type'

Part of a9e875e74af4b0407fd57cf350fe8203249e74ed

document Response-Packet-Type removal

remove old things

provisions for running many tests, one after the other.

in order to stress test the interpreter

switch to use Packet-Type for the reply

instead of Response-Packet-Type

Fix LD_PRELOAD

strdup filename in more places

remove unused variable

Revert "add API to clear request_data"

This reverts commit 33c8af9031618f293195802facfd5ab3cb3433dc.

no longer needed

Revert "add API to clear a pre-existing stack"

This reverts commit 1d764ca8062b0b73704df7acbd4ec8e7e70cab91.

not needed

Add the pointer to the handle, not the handle

typo.  Use the correct variable

snprintf() is expensive.

Using our own itoa() routine saves 200ms on a 15s run.

remove bad code

PCRE_ERROR_UNSET not available everywhere

move code to where it's used

add local free list for MD5 contexts

which saves 1/2s to 1s on the performance tests.

Because we don't control the content of OpenSSL's structures,
we can't use a dlist.  Instead, we just have a fixed-size local
array.  With provisions for all array entries being used.

In general, the code should allocate an MD5 context, use it,
and immediately free it.  i.e. *without* doing any IO while
holding the MD5 context.

Since the code does this already, it's OK.

don't call snprintf() in the hot path.

If we don't have debugging symbols enabled, then performance
is likely more important than debugging.  If debugging symbols
are enabled, then small performance hits aren't much of an issue.

Initialise is_closed to the correct value

Add basic I/O throughput test

Make trunk more permissive regarding what callbacks need to be provided

Typos

Add redis pipeline code

Untested... So probably doesn't work yet.

Pass around connection callbacks and configurations in structs

Makes it much easier to add new parameters later

Add a 'ignore a response' mini-API to the Redis I/O code

When a request is signalled to be freed we can now tell the redis pipeline code to ignore the response.

Unfortunately there's no way to explicitly cancel a command, but they're not usually very long running anyway.

Change how we deal with freeing connections

We now use the on_halt watcher as an indication the conn is ready to be freed, and that the tconn should also be freed.

This allows us to signal that the connection should be shutdown, and allows that process to complete before we attempt to free the conn or tconn.

The previous code just failed spectacularly... this is almost sane.

Allow request states to be signalled from outside of the demuxer

The demuxer isn't always needed or useful.

memcpy the trunk config

It's just easier, and there are fewer pointer derefs.

Enqueue return codes should be public...

Allow watchers to be removed by handlers without causing catastrophic failure

When a watcher is freed, the watcher call loop is now fully aware that this has happened, and will not attempt to call freed watchers.

Docs

Everything else uses entry...

Shift come code around

Add a deferred signal handling system to the connection API

It was a nightmare trying to reason about how signals would be delivered to the connections from within the handlers, so prevent that from happening, EVER.

Now the connection state machine completes whatever work it needs to, then at the end, checks to see if the handlers signalled it, and THEN, AND ONLY THEN does it process the signals.

The signals run through the public signalling interface, which should mean the connection is never allowed to make an invalid transition.  Signals that are invalid for the connections current state are just ignored.

reuse more memory

Make JIT stack size configurable, and keep the stack sizes consistent between library versions

Add missing errors, increase JIT stack size