Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Thread API wrappers required for thread logging #1054

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

ethouris
Copy link
Collaborator

@ethouris ethouris commented Jan 8, 2020

Depends: #1049

This applies part of the thread logging, which comprises thread API wrappers. This includes:

  • type aliases (currently just aliases) for pthread_mutex_t and phread_cond_t
  • functions operating on them for locking/unlocking and creation/destruction
  • shorthand functions for the threads:
    • jointhread: refers to pthread_join
    • isthread: checks if the thread id value is equal (taken as a good deal that thread isn't running)

Not provided wrappers for pthread_create and pthread_t. Possible wrappers for them should be more high level and therefore were not created. They should involve a wrapper over a handler function so that the possible thread logging on a new started thread or exited thread comes from the elevated handler.

This is the part that is only required for thread logging.

@ethouris ethouris added [core] Area: Changes in SRT library core Impact: Low Priority: High Type: Enhancement Indicates new feature requests labels Jan 8, 2020
Copy link
Collaborator

@maxsharabayko maxsharabayko left a comment

Choose a reason for hiding this comment

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

@ethouris This PR is already outdated and can be closed, right?

@ethouris
Copy link
Collaborator Author

It's long not updated, but the update is impossible until your fixes for timers are merged - in particular, the one that eliminates the type definition difference for monotonic condition variable.

@maxsharabayko maxsharabayko modified the milestones: v1.5.0, v1.5.1 Jul 27, 2020
@mbakholdina mbakholdina modified the milestones: v1.5.1, Backlog May 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[core] Area: Changes in SRT library core Priority: High Type: Enhancement Indicates new feature requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants