Skip to content
Snippets Groups Projects

concurrent CTL

Merged Libor Peltan requested to merge concurrent_ctl into master
1 unresolved thread

Merge request reports

Pipeline #129565 passed

Pipeline passed for e4cb0df9 on concurrent_ctl

Approval is optional

Merged by Daniel SalzmanDaniel Salzman 7 months ago (Sep 1, 2024 5:04pm UTC)

Merge details

  • Changes merged into master with fcaf9f3d.
  • Deleted the source branch.
  • Auto-merge enabled

Pipeline #129566 failed

Pipeline failed for fcaf9f3d on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
356 pthread_cond_broadcast(&cctx->cond);
357 pthread_mutex_unlock(&cctx->mutex);
358 (void)pthread_join(cctx->thread, NULL);
359
360 assert(cctx->state == CONCURRENT_FINISHED);
361 knot_ctl_free(cctx->ctl);
362 pthread_mutex_destroy(&concurrent_ctxs[i].mutex);
363 pthread_cond_destroy(&concurrent_ctxs[i].cond);
364 }
365 }
366
367 static void *ctl_process_thread(void *arg)
368 {
369 concurrent_ctl_ctx_t *ctx = arg;
370 rcu_register_thread();
371 setup_signals(); // in fact, this blocks common signals so that they arrive to main thread instead of this one
  • Jan Doskočil added 1 commit

    added 1 commit

    • c6145f3b - fixup! concurrent CTL: force using one thread when conf txn open...

    Compare with previous version

  • added 1 commit

    • 3f7ba562 - fixup! concurrent CTL: force using one thread when conf txn open...

    Compare with previous version

  • Jan Doskočil added 1 commit

    added 1 commit

    • 29342e97 - WIP: signal safe pthread_create

    Compare with previous version

  • Jan Doskočil added 1 commit

    added 1 commit

    • 97892cdb - fixup! concurrent CTL: force using one thread when conf txn open...

    Compare with previous version

  • Libor Peltan added 1 commit

    added 1 commit

    • e946901e - concurrent CTL: disable signals temporarily when launching subthreads to avoid losing them

    Compare with previous version

  • Libor Peltan added 1 commit

    added 1 commit

    • 8dd82af5 - concurrent CTL: disable signals temporarily when launching subthreads to avoid losing them

    Compare with previous version

  • added 1 commit

    • 61db1ff9 - fixup! fixup! concurrent CTL: force using one thread when conf txn open...

    Compare with previous version

  • added 1 commit

    • 2bf9f3ac - fixup! fixup! concurrent CTL: force using one thread when conf txn open...

    Compare with previous version

  • added 1 commit

    • 145a7f56 - fixup! ctl: implemented multi-threaded (mutexted) CTL handling

    Compare with previous version

  • added 1 commit

    • c2516ea7 - ctl: use pthread_rwlock_lock() instead of pthread_rwlock_timedwrlock() on macOS

    Compare with previous version

  • added 1 commit

    • 4de77b09 - ctl: use pthread_rwlock_lock() instead of pthread_rwlock_timedwrlock() on macOS

    Compare with previous version

  • Daniel Salzman added 17 commits

    added 17 commits

    • 4de77b09...4d1bbe95 - 9 commits from branch master
    • 93b9c09d - ctl/zone-ksk-submitted: avoid deadlock on DNSSEC event when frozen...
    • bdc4cd1d - ctl: prohibit opening zone-begin when blocking command is running
    • f48fbc45 - ctl: implemented multi-threaded (mutexted) CTL handling
    • 287b23e9 - concurrent CTL: force using one thread when conf txn open...
    • cfc00cc3 - zone: use knot_atomic_ptr_t for backup_ctx
    • 4bbd7b5a - concurrent CTL: disable signals temporarily when launching subthreads to avoid losing them
    • 42384527 - ctl: add a workaround for macOS
    • 5f2f7a24 - concurrent CTL: bugfix: ctl_globals have to exist separately for each CTL thread

    Compare with previous version

  • Daniel Salzman added 8 commits

    added 8 commits

    • 2fdadb4e - ctl/zone-ksk-submitted: avoid deadlock on DNSSEC event when frozen...
    • e4f39905 - ctl: prohibit opening zone-begin when blocking command is running
    • 4468af58 - ctl: implemented multi-threaded (mutexted) CTL handling
    • 69328dd7 - concurrent CTL: force using one thread when conf txn open...
    • 2fcd2134 - zone: use knot_atomic_ptr_t for backup_ctx
    • c557191c - concurrent CTL: disable signals temporarily when launching subthreads to avoid losing them
    • fda1c01f - ctl: add a workaround for macOS
    • e4cb0df9 - concurrent CTL: bugfix: ctl_globals have to exist separately for each CTL thread

    Compare with previous version

  • Daniel Salzman marked this merge request as ready

    marked this merge request as ready

  • Daniel Salzman enabled an automatic merge when the pipeline for e4cb0df9 succeeds

    enabled an automatic merge when the pipeline for e4cb0df9 succeeds

  • Daniel Salzman mentioned in commit fcaf9f3d

    mentioned in commit fcaf9f3d

  • Please register or sign in to reply
    Loading