summaryrefslogtreecommitdiffstats
path: root/runtime/wtp.c
Commit message (Collapse)AuthorAgeFilesLines
* new queue engine - initial commit (probably not 100% working!)Rainer Gerhards2009-10-141-28/+7
| | | | | | | | | | simplified and thus speeded up the queue engine, also fixed some potential race conditions (in very unusual shutdown conditions) along the way. The threading model has seriously changes, so there may be some regressions. NOTE: the code passed basic tests, but there is still more work and testing to be done. This commit should be treated with care.
* added some debug settings plus improved shutdown sequenceRainer Gerhards2009-10-131-2/+2
| | | | ... non-working version!
* bugfix and testbench improvementsRainer Gerhards2009-10-071-0/+3
| | | | | | | | - bugfix: solved potential (temporary) stall of messages when the queue was almost empty and few new data added (caused testbench to sometimes hang!) - fixed some race condition in testbench - added more elaborate diagnostics to parts of the testbench - solved a potential race inside the queue engine
* bugfix: potential race condition in debug modeRainer Gerhards2009-07-201-1/+2
|
* bugfix: minor static memory leak while reading configurationRainer Gerhards2009-07-201-13/+12
| | | | | This did NOT leak based on message volume. Also, did some cleanup during the commit.
* enhanced worker thread pool by atomic opsRainer Gerhards2009-07-201-26/+15
| | | | | ... greater performance and was able to remove a potential troublespot in a cancel cleanup handler.
* architecture change: queue now always has at least one worker threadRainer Gerhards2009-07-201-1/+5
| | | | | | ...if not running in direct mode. Previous versions could run without any active workers. This simplifies the code at a very small expense. See v5 compatibility note document for more in-depth discussion.
* more cleanup/simplification (forgot to remove one mutex lock)Rainer Gerhards2009-07-201-24/+1
|
* some more threading changesRainer Gerhards2009-07-201-18/+6
| | | | ... as well as some cleanup
* cleanup: removed no longer needed codeRainer Gerhards2009-07-201-18/+0
|
* removed mutex locksRainer Gerhards2009-07-171-30/+33
| | | | | ... by utilizing that we need to modify a state variable only in a sequential way during shutdown.
* further code simplificationRainer Gerhards2009-07-171-42/+22
| | | | | | | | | ... could even remove one mutex by using a better algorithm. I think I also spotted some situation in which a hang could have happened. As I can't fix it in v4 and less without moving to the new engine, I make no effort in testing this out. Hangs occur during shutdown, only (if at all). The code changes should also result in some mild performance improvement. Some bug potential, but overall the bug potential should have been greatly reduced.
* more code simplification, should also bring some performance enhancementRainer Gerhards2009-07-171-18/+9
| | | | reducing the number of thread cancellation state changes
* Merge branch 'beta'Rainer Gerhards2009-07-161-1/+5
|\ | | | | | | | | | | Conflicts: runtime/wti.c runtime/wtp.c
| * Merge branch 'v4-beta' into v4-develRainer Gerhards2009-07-161-2/+5
| |\
| | * calls to prctl() need to be based on configure results (cross-platform issue)Rainer Gerhards2009-07-161-2/+5
| | |
* | | some code simplificationRainer Gerhards2009-07-161-2/+2
| | |
* | | further reduction in code size/complexity due to design changeRainer Gerhards2009-07-081-30/+3
| | |
* | | simplified worker thread handlingRainer Gerhards2009-07-081-66/+2
| | | | | | | | | | | | | | | based on now working with detached threads. This is probably the biggest patch in this series and with large bug potential.
* | | optimizing queue thread handlingRainer Gerhards2009-07-081-2/+6
| | | | | | | | | | | | | | | | | | ... first commit in a series of more. Makes worker threads detached. Needs more testing (will be done soon) and if it works as expected, we can further reduce code.
* | | Merge branch 'v4-devel'Rainer Gerhards2009-07-081-1/+6
|\| | | | | | | | | | | | | | | | | Conflicts: runtime/debug.h runtime/stream.c
| * | some minor cleanupRainer Gerhards2009-07-061-1/+2
| | |
| * | first shot at asynchronous stream writer with timeout capabilityRainer Gerhards2009-07-061-1/+5
| |/ | | | | | | | | ... seems to work on quick testing, but needs a far more testing and improvement. Good milestone commit.
* | Merge branch 'master' into v5-develRainer Gerhards2009-06-251-0/+6
|\| | | | | | | | | Conflicts: runtime/queue.c
| * named queue worker threadsRainer Gerhards2009-06-251-0/+6
| | | | | | | | ... but I don't see the name anywhere...?
| * some memory accesses are now explicitely atomicRainer Gerhards2009-06-251-1/+2
| | | | | | | | | | ... as far as I think this mostly is to keep the thread debuggers happy
* | added a few atomic operationsRainer Gerhards2009-06-251-1/+2
| | | | | | | | | | mostly to get thread debugger errors clean (plus, of course, it makes things more deterministic)
* | Merge branch 'omfile' into tmpRainer Gerhards2009-06-221-1/+0
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was a complex manual merge, especially in action.c. So if there occur some problems, this would be a good point to start troubleshooting. I run a couple of tests before commiting and they all went well. Conflicts: action.c action.h runtime/queue.c runtime/queue.h runtime/wti.c runtime/wti.h
| * removed uniprocessor optimizationRainer Gerhards2009-06-191-9/+0
| | | | | | | | | | | | ... as it was not even optimal on uniprocessors any longer ;) I keep the config directive in, maybe we can utilize it again at some later point in time (questionable).
| * fixed a small memory leak...Rainer Gerhards2009-06-101-2/+1
| | | | | | | | and another problem, both introduced today. Also did some general cleanup.
* | some more fixes for queue engineRainer Gerhards2009-05-281-6/+0
| | | | | | | | The enhanced testbench now runs without failures, again
* | preserving current changesRainer Gerhards2009-05-281-6/+0
| | | | | | | | | | | | ... in preparation for some larger changes - I need to apply some serious design changes, as the current system does not play well at all with ultra-reliable queues. Will do that in a totally new version.
* | interim commit: working on failure casesRainer Gerhards2009-05-271-0/+15
| | | | | | | | slightly improved situation, would like to save it before carrying on
* | solved design issue with queue terminationRainer Gerhards2009-05-261-5/+8
| | | | | | | | | | | | | | | | | | | | | | ... and also improved the test suite. There is a design issue in the v3 queue engine that manifested to some serious problems with the new processing mode. However, in v3 shutdown may take eternally if a queue runs in DA mode, is configured to preserve data AND the action fails and retries immediately. There is no cure available for v3, it would require doing much of the work we have done on the new engine. The window of exposure, as one might guess from the description, is very small. That is probably the reason why we have not seen it in practice.
* | free last processed message in all casesRainer Gerhards2009-05-201-4/+3
| | | | | | | | | | | | | | | | | | so far, the last processed message was only freed when the next one was processed. This has been changed now. More precisely, a better algorithm has been selected for the queue worker process, which also involves less overhead than the previous one. The fix for "free last processed message" as then more or less a side-effect (easy to do) of the new algorithm.
* | yield() no longer needed on uniproc thanks to new algorithmsRainer Gerhards2009-05-201-12/+4
| |
* | fixed abort condition in DA modeRainer Gerhards2009-04-231-1/+1
| |
* | fixing a small (newly-introduced) memory leakRainer Gerhards2009-04-231-2/+1
| | | | | | | | | | ... plus simplifying free() calls after agreement on mailing list that we no longer need to check if the pointer is non-NULL
* | added $MainMsgQueueDequeueBatchSize and $ActionQueueDequeueBatchSize ↵Rainer Gerhards2009-04-231-2/+2
| | | | | | | | configuration directives
* | first attempt at dequeueing multiple batches inside the queueRainer Gerhards2009-04-221-1/+3
|/ | | | | | ... but this code has serious problems when terminating the queue, also it is far from being optimal. I will commit a series of patches (hopefully) as I am on the path to the final implementation.
* Merge branch 'beta'Rainer Gerhards2009-04-071-0/+8
|\ | | | | | | | | | | | | | | Conflicts: ChangeLog configure.ac doc/manual.html doc/rsyslog_conf.html
| * Merge branch 'v3-stable' into betaRainer Gerhards2009-04-021-3/+25
| |\ | | | | | | | | | | | | | | | | | | Conflicts: ChangeLog configure.ac doc/manual.html
| | * preparing for 3.20.5 releasev3.20.5Rainer Gerhards2009-04-021-0/+8
| | |
* | | Merge branch 'dapatch'Rainer Gerhards2009-03-251-3/+17
|\ \ \ | | |/ | |/|
| * | bugfix: potential abort with DA queue after high watermark is reachedRainer Gerhards2009-03-251-3/+17
| | | | | | | | | | | | | | | | | | There exists a race condition that can lead to a segfault. Thanks go to vbernetr, who performed the analysis and provided patch, which I only tweaked a very little bit.
* | | integrated various patches for solarisRainer Gerhards2009-03-051-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | Unfortunatley, I do not have the full list of contributors available. The patch set was compiled by Ben Taylor, and I made some further changes to adopt it to the news rsyslog branch. Others provided much of the base work, but I can not find the names of the original authors. If you happen to be one of them, please let me know so that I can give proper credits.
* | | added a setting "$OptimizeForUniprocessor"Rainer Gerhards2008-10-221-1/+15
| | | | | | | | | | | | | | | | | | ...to enable users to turn off pthread_yield calls which are counter-productive on multiprocessor machines (but have been shown to be useful on uniprocessors)
* | | Merge branch 'helgrind' into perfRainer Gerhards2008-10-021-2/+3
|\ \ \ | | |/ | |/|
| * | improved threadingRainer Gerhards2008-09-301-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | - changed sequence when awakening thread - removed no longer needed condition variable - EXPERIMENTALLY added mutex guarding to hostname lookups this is to be removed if it does not have any verifyable useful effect
* | | Merge branch 'helgrind' into perfRainer Gerhards2008-09-261-0/+2
|\| | | | | | | | | | | | | | | | | Conflicts: runtime/debug.c