Differential D27336

Make bsnmpclient(3) thread-safe
AbandonedPublic
Actions

Authored by trasz on Nov 23 2020, 1:46 PM.

Tags

None

Referenced Files

	F106653915: D27336.diff
	Fri, Jan 3, 10:52 AM

	Unknown Object (File)
	Nov 15 2024, 7:55 AM

	Unknown Object (File)
	Nov 15 2024, 5:43 AM

	Unknown Object (File)
	Nov 7 2024, 2:38 AM

	Unknown Object (File)
	Sep 27 2024, 3:19 PM

	Unknown Object (File)
	Sep 26 2024, 4:00 PM

	Unknown Object (File)
	Sep 8 2024, 11:01 PM

	Unknown Object (File)
	Sep 8 2024, 6:52 PM

View All 58 Files

Subscribers

Details

Reviewers

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.

(Tinderboxed.)

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

... 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

Path

Size

contrib/

bsnmp/

lib/

2 lines

2 lines

4 lines

Diff 79895

contrib/bsnmp/lib/asn1.c

Loading...

contrib/bsnmp/lib/snmpclient.h

Loading...

contrib/bsnmp/lib/snmpclient.c

Loading...