Skip to content
This repository was archived by the owner on Oct 28, 2024. It is now read-only.

Conversation

weissi
Copy link
Collaborator

@weissi weissi commented Feb 20, 2021

Motivation:

In case SwiftBacktrace fails, we get only very little output.
Unfortunately, some crucial information (like the errno) is missing and
in case mmap fails, we don't know if the problem is file I/O or
allocating memory.

Modification:

  • print errnos
  • distinguish the two mmaps

Result:

Easier debugging

@swift-server-bot
Copy link

Can one of the admins verify this patch?

4 similar comments
@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@weissi weissi requested a review from tomerd February 20, 2021 22:58
Motivation:

In case SwiftBacktrace fails, we get only very little output.
Unfortunately, some crucial information (like the errno) is missing and
in case `mmap` fails, we don't know if the problem is file I/O or
allocating memory.

Modification:

- print errnos
- distinguish the two mmaps

Result:

Easier debugging
@weissi
Copy link
Collaborator Author

weissi commented Feb 20, 2021

@swift-server-bot test this please

@weissi weissi added the patch-version-bump-only For PRs that when merged will only cause a bump of the patch version, ie. 1.0.x -> 1.0.(x+1) label Feb 21, 2021
@tomerd
Copy link
Collaborator

tomerd commented Feb 22, 2021

@swift-server-bot test this please

@tomerd
Copy link
Collaborator

tomerd commented Feb 22, 2021

@swift-server-bot add to allowlist

Copy link
Collaborator

@tomerd tomerd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is great

@weissi weissi merged commit 93b3d9a into swift-server:main Feb 22, 2021
@weissi weissi deleted the jw-error branch February 22, 2021 11:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
patch-version-bump-only For PRs that when merged will only cause a bump of the patch version, ie. 1.0.x -> 1.0.(x+1)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants