From: Willy Tarreau <w@1wt.eu>
Date: Sat, 19 Oct 2024 13:12:47 +0000 (+0200)
Subject: MINOR: debug: make mark_tainted() return the previous value
X-Git-Tag: v3.1-dev11~103
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=8e048603d1bba0721433a5ae0480fbf1ab6f4897;p=thirdparty%2Fhaproxy.git

MINOR: debug: make mark_tainted() return the previous value

Since mark_tainted() uses atomic ops to update the tainted status, let's
make it return the prior value, which will allow the caller to detect
if it's the first one to set it or not.
---

diff --git a/include/haproxy/bug.h b/include/haproxy/bug.h
index f45fe347b4..5e683974fe 100644
--- a/include/haproxy/bug.h
+++ b/include/haproxy/bug.h
@@ -337,9 +337,9 @@ extern unsigned int tainted;
 
 void complain(int *counter, const char *msg, int taint);
 
-static inline void mark_tainted(const enum tainted_flags flag)
+static inline unsigned int mark_tainted(const enum tainted_flags flag)
 {
-	HA_ATOMIC_OR(&tainted, flag);
+	return HA_ATOMIC_FETCH_OR(&tainted, flag);
 }
 
 static inline unsigned int get_tainted()