cimple/src/tcp_server.h, branch mesonMy little CI system (hopefully)http://git.tensin.name/cimple/atom?h=meson2026-09-21T17:19:05Zremove annoying trailing period from repo URLs2026-09-21T17:19:05ZEgor Tensinegor@tensin.name2026-09-21T17:19:05Zurn:sha1:588fd2262c2f65e237a5a4e4858e06b4b87aaae3finally, perfect .clang-format settings?2026-06-13T18:16:43ZEgor Tensinegor@tensin.name2026-06-13T18:16:43Zurn:sha1:d64e7bf3d93649b78231912f37606153e3d30665unify my .clang-format files2026-06-12T15:20:08ZEgor Tensinegor@tensin.name2026-06-12T15:20:08Zurn:sha1:1160cdb21e0b667269299853cb47a4aa88948dc3switch to egor@tensin.name2023-12-12T20:29:42ZEgor Tensinegor@tensin.name2023-12-12T20:29:42Zurn:sha1:82c5f204346def706fdc7ee28a8e2570fb880369tcp_server: keep track of client threads2023-07-05T14:18:05ZEgor TensinEgor.Tensin@gmail.com2023-07-05T13:30:57Zurn:sha1:8465f8181eda45e3d6cc5d6c3d08ca36db04763b
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.
use signalfd to stop on SIGTERM2023-06-12T23:37:08ZEgor TensinEgor.Tensin@gmail.com2023-06-11T23:42:08Zurn:sha1:48ce9170b057ddd2165b0239a92aede15849f7a3
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.
make struct tcp_server opaque2023-04-29T07:57:16ZEgor TensinEgor.Tensin@gmail.com2023-04-28T10:58:42Zurn:sha1:37895358b03b805f1887749c8dc73d1e075af810add copyright notices2022-12-02T15:00:29ZEgor TensinEgor.Tensin@gmail.com2022-12-02T15:00:29Zurn:sha1:9a0bca49f80f841f0689712b9bc8f5f69ad2b3e5net: rework API2022-08-25T08:56:03ZEgor TensinEgor.Tensin@gmail.com2022-08-25T08:47:06Zurn:sha1:08ee3cab641f367b642bc29f6540de6be85226bd
First, rename all API functions so that they start with net_.
Second, abstract the basic TCP server functionality into tcp_server.c.
This includes reworking net_accept so that it's a simple blocking
operation, and putting the callback stuff to tcp_server.c. Also, the
server now uses detached threads instead of fork(), since I want
connection handlers to share memory.