Commit message (Collapse) | Author | Age | |
---|---|---|---|
* | tcp_server: close client connections | Egor Tensin | 2023-07-05 |
| | | | | | I kinda forgot that I'm supposed to clean up client connections, and they immediately blew up in my face. | ||
* | command: dedupe the code | Egor Tensin | 2023-07-05 |
| | |||
* | tcp_server: keep track of client threads | Egor Tensin | 2023-07-05 |
| | | | | | | | This is a major change, obviously; brought to me by Valgrind, which noticed that we don't actually clean up after cimple-client threads. For a more thorough explanation, please see the added comment in tcp_server.c. | ||
* | sanitize #include-s | Egor Tensin | 2023-07-04 |
| | |||
* | worker: close the leftover descriptor | Egor Tensin | 2023-07-04 |
| | | | | | Thanks, Valgrind! As a note: if I think that Valgrind reports a false positive, chances are, it's not. | ||
* | move custom message parsing to a separate module | Egor Tensin | 2023-07-04 |
| | |||
* | storage_sqlite: refactoring | Egor Tensin | 2023-07-04 |
| | |||
* | sqlite: rename a column | Egor Tensin | 2023-07-04 |
| | |||
* | storage: mark completed runs as such | Egor Tensin | 2023-07-04 |
| | |||
* | storage: requeue old runs from storage on startup | Egor Tensin | 2023-07-04 |
| | |||
* | tcp_server: always clean up connection descriptors | Egor Tensin | 2023-07-04 |
| | |||
* | sqlite: store new runs in SQLite | Egor Tensin | 2023-07-04 |
| | |||
* | storage_sqlite: refactoring | Egor Tensin | 2023-07-04 |
| | |||
* | sqlite: add run status field | Egor Tensin | 2023-07-04 |
| | |||
* | fix a compiler warning | Egor Tensin | 2023-07-04 |
| | |||
* | appease clang-format | Egor Tensin | 2023-07-04 |
| | |||
* | use designated struct initializers more | Egor Tensin | 2023-06-30 |
| | |||
* | show git hash with --version | Egor Tensin | 2023-06-30 |
| | | | | Also, use cmake's configure_file to build string constants in. | ||
* | fix function names in error messages | Egor Tensin | 2023-06-30 |
| | |||
* | use __WNOTHREAD w/ waitpid | Egor Tensin | 2023-06-30 |
| | | | | | | | | | | I learned about this flag on my previous job; basically, it should be enabled by default IMO, so that the thread doesn't receive signals from other threads' children. Here, it doesn't matter too much, since we're waiting for a specific child. However, if we were to use waitpid(-1, ...), it would be essential to use this flag. Still, even here, it's good to have this on. | ||
* | log: minor refactoring | Egor Tensin | 2023-06-28 |
| | |||
* | process: fix function name in logs | Egor Tensin | 2023-06-28 |
| | |||
* | ci: environment array to execvpe must be NULL-terminated | Egor Tensin | 2023-06-28 |
| | | | | Thanks, Valgrind. | ||
* | log: support logging levels | Egor Tensin | 2023-06-28 |
| | |||
* | test: skip ci.sh w/ Valgrind | Egor Tensin | 2023-06-28 |
| | |||
* | valgrind: exit with an error after leak detection | Egor Tensin | 2023-06-28 |
| | | | | I cannot believe this isn't default. | ||
* | file: readlink(2) doesn't null-terminate | Egor Tensin | 2023-06-28 |
| | | | | Thanks, valgrind. | ||
* | minor refactoring | Egor Tensin | 2023-06-14 |
| | |||
* | cmake: fix grammar in option description | Egor Tensin | 2023-06-13 |
| | |||
* | cmake: why the hell would I need MSVC options | Egor Tensin | 2023-06-13 |
| | |||
* | minor refactoring | Egor Tensin | 2023-06-13 |
| | |||
* | signal: remove the stupid add_to_event_loop wrapper | Egor Tensin | 2023-06-13 |
| | |||
* | server: handle disconnected workers gracefully | Egor Tensin | 2023-06-13 |
| | |||
* | event_loop: support more event types | Egor Tensin | 2023-06-13 |
| | |||
* | event_loop: add event_loop_add_once | Egor Tensin | 2023-06-13 |
| | |||
* | signal: sigprocmask -> pthread_sigmask | Egor Tensin | 2023-06-13 |
| | |||
* | signal: refactoring | Egor Tensin | 2023-06-13 |
| | |||
* | sanitize #include-s | Egor Tensin | 2023-06-13 |
| | |||
* | signal: remove unused APIs | Egor Tensin | 2023-06-13 |
| | |||
* | event_loop: hide the API details | Egor Tensin | 2023-06-13 |
| | |||
* | use signalfd to stop on SIGTERM | Egor Tensin | 2023-06-13 |
| | | | | | | | | | | | | | | | | | | Is this an overkill? I don't know. The thing is, correctly intercepting SIGTERM (also SIGINT, etc.) is incredibly tricky. For example, before this commit, my I/O loops in server.c and worker.c were inherently racy. This was immediately obvious if you tried to run the tests. The tests (especially the Valgrind flavour) would run a worker, wait until it prints a "Waiting for a new command" line, and try to kill it using SIGTERM. The problem is, the global_stop_flag check could have already been executed by the worker, and it would hang forever in recv(). The solution seems to be to use signalfd and select()/poll(). I've never used either before, but it seems to work well enough - at least the very same tests pass and don't hang now. | ||
* | log: refactoring | Egor Tensin | 2023-06-12 |
| | |||
* | log: thread-safe logging | Egor Tensin | 2023-06-12 |
| | |||
* | log: prefix with thread ID | Egor Tensin | 2023-06-12 |
| | |||
* | command: fix a race condition | Egor Tensin | 2023-06-12 |
| | | | | | Modifying cmd_dispatcher fields like that make it inherently unsafe to call cmd_dispatcher_handle_conn concurrently. | ||
* | sqlite: fix generate-sql-header.py | Egor Tensin | 2023-06-12 |
| | |||
* | msg: rework some APIs | Egor Tensin | 2023-06-11 |
| | |||
* | worker_queue: fix worker_queue_add_last | Egor Tensin | 2023-06-09 |
| | |||
* | STAILQ -> SIMPLEQ | Egor Tensin | 2023-06-09 |
| | |||
* | msg: add msg_send_from_argv shortcut | Egor Tensin | 2023-06-09 |
| |