Skip to content
Snippets Groups Projects

cache: improve out-of-disk condition

Merged Vladimír Čunát requested to merge cache-sigbus into master
All threads resolved!

When suspect SIGBUS happens, print helpful error and try to remove the cache, so that the service might work again if auto-restarted.

Theoretically we could longjmp() out of the SIGBUS handler, but that would be rather messy, so let the process die.

Fixes https://gitlab.labs.nic.cz/knot/knot-resolver/issues/197 (in a sense).

Output:

SIGBUS received; this is most likely due to filling up the filesystem where cache resides.
Cache file deleted.
kresd can not recover reliably by itself, exiting.

(and the process exits as if SIGBUS had been unhandled)

Edited by Vladimír Čunát

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    • 8590f9f5 - fixup! cache: improve out-of-disk condition

    Compare with previous version

  • Vladimír Čunát marked as a Work In Progress from 8590f9f5

    marked as a Work In Progress from 8590f9f5

  • Hmm, pushing a --fixup commit auto-marks MR as WIP, apparently :-)

  • Petr Špaček unmarked as a Work In Progress

    unmarked as a Work In Progress

  • added 1 commit

    • ae9ea154 - cache: improve out-of-disk condition

    Compare with previous version

  • Vladimír Čunát resolved all discussions

    resolved all discussions

  • added 1 commit

    • efc08a64 - cache: improve out-of-disk condition

    Compare with previous version

  • Vladimír Čunát changed the description

    changed the description

  • Petr Špaček enabled an automatic merge when the pipeline for efc08a64 succeeds

    enabled an automatic merge when the pipeline for efc08a64 succeeds

  • merged

  • Petr Špaček mentioned in commit 88e78c66

    mentioned in commit 88e78c66

  • mentioned in commit f41ae543

  • Vladimír Čunát mentioned in merge request !677 (merged)

    mentioned in merge request !677 (merged)

  • mentioned in commit b4f1082c

  • Please register or sign in to reply
    Loading