notify/retry

See merge request !1443

Libor Peltan authored 3 years ago and Daniel Salzman committed 2 years ago

...as dynarray itself had been moved from contrib

JP Mens authored 2 years ago and Daniel Salzman committed 2 years ago

closes #801

The number of UMEM frames have been decreased as it doesn't seem it helped the performance.
As a consequence the overall memory consumption decreases.

Also a hint on the fill ring size from the xdpsock_user.c sample has been considered.

After the change the following assert sometimes occured when running
kxdpgun in the emulated XDP mode.

knot_xdp_recv_finish():
  const uint32_t reserved = xsk_ring_prod__reserve(fq, count, &idx);
  assert(reserved == count);

This reverts commit f5f4ee9f.

JP Mens authored 2 years ago and Daniel Salzman committed 2 years ago

closes #798

If a packet was successfully alloced by knot_xdp_send_alloc(),
there must be a room in the TX ring for it.

events: improve mutexing

See merge request !1454

Libor Peltan authored 2 years ago and Daniel Salzman committed 2 years ago

This fixes a bug when zone_reload_modules only waits for
an event to be finished, but following reschedule is not
synchronized and might access memory freed by zone_free.

tests-extra XFR refactoring

See merge request !1453

This is a slight simplification of the test.

This is needed even without Valgrind when running massively in parallel

On slaves one common ACL for all actions has been replaced with a sequence
of ACLs for notify, one rule for one master.

the heap-element-replace bug fix

See merge request !1452

David Vasek authored 3 years ago and Libor Peltan committed 2 years ago

This avoids the timeouts in master and as a result the "TCP, terminated inactive client"
messages in the master's log are eliminated. Slave seems to be busy for too long when signing
the zones under Valgrind.

David Vasek authored 3 years ago and Libor Peltan committed 2 years ago

In contrast with Knot coding style, most single line "if" constructions retained.

David Vasek authored 3 years ago and Libor Peltan committed 2 years ago

Possible as a result of modified heap_replace().