docs for daemon: better visibility for control sockets

See merge request !682

Vladimír Čunát authored 6 years ago and Tomas Krizek committed 6 years ago

- promote the part into a section
- move it right after the section about interactive CLI
- minor reformulations, location of the sockets, etc.
- fix location of the daemon-supervised anchor

daemon/worker: minor refactoring of qr_task_step

See merge request !687

daemon: changes in TCP connection timeouting

See merge request !686

daemon: TCP connection timeouting has changed; connection was closed after peer's inactivity before, now it is closed after incativity in both directions (peer->kresd, kresd->peer); prevents connection from closing before answer sent to client

It's actually possible to start queries without any cache open,
e.g. add `resolve('.', kres.type.DNSKEY)` into your configuration.
If that happens, avoid the cache module dereferencing a NULL pointer.

Using 2kB as start is a bit too much, and it was showing as 0.5% CPU
on malloc() called during trie_it_begin().  Let's start at 0.5 kB,
as it can grow anyway (only negligible in profile now).

ci: use 256 jobs for TLS forward test

See merge request !683

- mainly the daemon/session.* files are separated,
  moving lots of logic from daemon/worker.*;
- lib/generic/queue.* are added;
- verbose logging gets different IDs;
- various minor changes around.

Grigorii Demidov authored 6 years ago and Vladimír Čunát committed 6 years ago

Unificate handling of TCP timeouts for input and output connections; avoid redundant code; bugfixing

The typing around queue_it_begin() isn't ideal,
but I'm unable to come with anything better.
I'm really missing generics for these things.

- align the numbers to 5+2 digits (pad with zeros), and use unsigned
- add numbers for the '(NOT) classified as SECURE' message

lint:clang-scan-build reported:
> warning: The code calls sizeof() on a pointer type.
> This can produce an unexpected result
but in our case it's intentional.
(Yes, using pointers as keys in trie isn't very pretty.)

Only count incoming requests into this,
as before "some simplifications" commit.

Grigorii Demidov authored 6 years ago and Vladimír Čunát committed 6 years ago

daemon/worker: avoid usage of qr_tasklist in struct request_ctx as it is redundant; use direct pointer to task instead

There are some (other) tests in knot-dns repo,
but having some more shouldn't hurt, especially as
we've added some new functions now.

It's focused on FIFO queue usage.

FIXME: unit tests

This prompted a slight reorganization inside trie.c