Mailing List Archive

[master] ddeb24214 Add comment about ObjSubscribeEvents() not being atomic.
commit ddeb24214599de85f8e77be72ce9fc29d386a8a7
Author: Poul-Henning Kamp <phk@FreeBSD.org>
Date: Mon Dec 4 09:09:12 2023 +0000

Add comment about ObjSubscribeEvents() not being atomic.

diff --git a/bin/varnishd/cache/cache_obj.c b/bin/varnishd/cache/cache_obj.c
index d92fc1f0d..2ff54cb8e 100644
--- 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)
{
_______________________________________________
varnish-commit mailing list
varnish-commit@varnish-cache.org
https://www.varnish-cache.org/lists/mailman/listinfo/varnish-commit