From b2defec33a757c4f929b3d3705c3ec71ef2872b1 Mon Sep 17 00:00:00 2001 From: jdamick Date: Wed, 6 Mar 2024 13:57:12 -0500 Subject: [PATCH] removed the mutex locks with atomic bool (#6525) Signed-off-by: Jeffrey Damick Co-authored-by: Jeffrey Damick --- plugin/pkg/log/log.go | 18 +++++------------- 1 file changed, 5 insertions(+), 13 deletions(-) diff --git a/plugin/pkg/log/log.go b/plugin/pkg/log/log.go index 0589a3457..ad8d7ac3d 100644 --- a/plugin/pkg/log/log.go +++ b/plugin/pkg/log/log.go @@ -13,7 +13,7 @@ import ( "io" golog "log" "os" - "sync" + "sync/atomic" ) // D controls whether we should output debug logs. If true, we do, once set @@ -21,30 +21,22 @@ import ( var D = &d{} type d struct { - on bool - sync.RWMutex + on atomic.Bool } // Set enables debug logging. func (d *d) Set() { - d.Lock() - d.on = true - d.Unlock() + d.on.Store(true) } // Clear disables debug logging. func (d *d) Clear() { - d.Lock() - d.on = false - d.Unlock() + d.on.Store(false) } // Value returns if debug logging is enabled. func (d *d) Value() bool { - d.RLock() - b := d.on - d.RUnlock() - return b + return d.on.Load() } // logf calls log.Printf prefixed with level.