Add comment about ObjSubscribeEvents() not being atomic.

ddeb2421 · Poul-Henning Kamp · f8936e78 · ddeb2421
Commit ddeb2421 authored Dec 04, 2023 by Poul-Henning Kamp
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 0 deletions

cache_obj.c bin/varnishd/cache/cache_obj.c +4 -0

No files found.
--- a/bin/varnishd/cache/cache_obj.c
+++ b/bin/varnishd/cache/cache_obj.c
@@ -672,6 +672,10 @@ static VTAILQ_HEAD(,oev_entry)		oev_list;
 static pthread_rwlock_t			oev_rwl;
 static unsigned				oev_mask;

+/*
+ * NB: ObjSubscribeEvents() is not atomic:
+ * oev_mask is checked optimistically in ObjSendEvent()
+ */
 uintptr_t
 ObjSubscribeEvents(obj_event_f *func, void *priv, unsigned mask)
 {