Page MenuHomeFreeBSD
Differential D27336

Make bsnmpclient(3) thread-safe
AbandonedPublic
Actions

Authored by trasz on Nov 23 2020, 1:46 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 5, 3:11 PM
Unknown Object (File)
Tue, Dec 2, 2:34 AM
Unknown Object (File)
Fri, Nov 28, 4:08 PM
Unknown Object (File)
Nov 3 2025, 9:15 PM
Unknown Object (File)
Nov 2 2025, 6:55 AM
Unknown Object (File)
Nov 2 2025, 6:55 AM
Unknown Object (File)
Nov 2 2025, 6:55 AM
Unknown Object (File)
Nov 2 2025, 6:54 AM
View All 100 Files
Subscribers
imp
NetApp

Details

Reviewers
harti
Commits
rG89e3d5671ba1: bsnmpclient(3): make it thread-safe
Summary

Make bsnmpclient(3) thread-safe.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 34978
Build 31971: arc lint + arc unit

Event Timeline

trasz created this revision.Nov 23 2020, 1:46 PM
Herald added a subscriber: imp. · View Herald TranscriptNov 23 2020, 1:46 PM
trasz requested review of this revision.Nov 23 2020, 1:46 PM
Harbormaster completed remote builds in B34978: Diff 79895.Nov 23 2020, 1:46 PM
trasz added a reviewer: harti.Nov 23 2020, 1:47 PM
trasz added a subscriber: NetApp.
trasz added a comment.Nov 24 2020, 10:41 AM

(Tinderboxed.)

harti accepted this revision.Nov 24 2020, 11:12 AM

For me this looks good. In the middle term (next year) I have a plan to get rid of static data in the library.

This revision is now accepted and ready to land.Nov 24 2020, 11:12 AM
Closed by commit rG89e3d5671ba1: bsnmpclient(3): make it thread-safe (authored by trasz). · Explain WhyDec 29 2020, 3:01 PM
This revision was automatically updated to reflect the committed changes.
trasz added a commit: rG89e3d5671ba1: bsnmpclient(3): make it thread-safe.
trasz added a reverting change: rGf53120073b32: Revert "bsnmpclient(3): make it thread-safe".Dec 29 2020, 7:59 PM
trasz added a comment.Dec 29 2020, 8:06 PM

... And reverted in f53120073b3 - mostly because, as Jessica pointed out, it would break existing applications which properly share the context between threads, eg by using explicit locking.

allanjude reopened this revision.Mar 2 2021, 11:08 PM
trasz abandoned this revision.Dec 29 2021, 9:06 PM

Revision Contents
Changeset List

PathSize
contrib/
bsnmp/
lib/
2 lines
2 lines
4 lines

Diff 79895

View Options

contrib/bsnmp/lib/asn1.c

Loading...
View Options

contrib/bsnmp/lib/snmpclient.h

Loading...
View Options

contrib/bsnmp/lib/snmpclient.c

Loading...