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

doc: Add some documentation around a new pthread call #15287

Merged
merged 1 commit into from
Feb 6, 2024

Conversation

donaldsharp
Copy link
Member

Not necessarily the correct place for this but there is no other place and it needs to be called out and I would rather have some documentation in place. Long term I would like to add a bunch of frr_pthread documentation but at this point in time it's not there. We can
re-arrange when that happens.

@@ -60,6 +60,7 @@ RCU is designed for read-heavy workloads where objects are updated relatively
rarely, but frequently accessed. Do *not* indiscriminately replace locking by
RCU patterns.

57a63d9009 zebra: eheck sscanf return value
Copy link
Contributor

Choose a reason for hiding this comment

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

I suppose this is not needed here.

Copy link
Member Author

Choose a reason for hiding this comment

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

yep fixed

Not necessarily the correct place for this but there
is no other place and it needs to be called out and I
would rather have some documentation in place.  Long
term I would like to add a bunch of frr_pthread documentation
but at this point in time it's not there.  We can
re-arrange when that happens.

Signed-off-by: Donald Sharp <[email protected]>
@github-actions github-actions bot added size/XS and removed size/S labels Feb 5, 2024
@ton31337 ton31337 added this to the 10.0 milestone Feb 5, 2024
@ton31337 ton31337 merged commit 969b2d4 into FRRouting:master Feb 6, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants