github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-28 14:07:14 +00:00
Code Issues Projects Releases Wiki Activity

pkg/controller/garbagecollector: simplify mutexes.

Browse Source 
Similar to #29598, we can rely on the zero-value construction behavior
to embed `sync.Mutex` into parent structs.
This commit is contained in:
Matt T. Proud 2016-08-02 08:13:17 +02:00
parent 37d3d390df
commit 1ca99119fd
2 changed files with 5 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
pkg/controller/garbagecollector/garbagecollector.go
View File

@ -67,7 +67,7 @@ func (s objectReference) String() string {
type node struct { type node struct {
identity objectReference identity objectReference
// dependents will be read by the orphan() routine, we need to protect it with a lock. // dependents will be read by the orphan() routine, we need to protect it with a lock.
dependentsLock *sync.RWMutex dependentsLock sync.RWMutex
dependents map[*node]struct{} dependents map[*node]struct{}
// When processing an Update event, we need to compare the updated // When processing an Update event, we need to compare the updated
// ownerReferences with the owners recorded in the graph. // ownerReferences with the owners recorded in the graph.
@ -151,8 +151,7 @@ func (p *Propagator) addDependentToOwners(n *node, owners []metatypes.OwnerRefer
OwnerReference: owner, OwnerReference: owner,
Namespace: n.identity.Namespace, Namespace: n.identity.Namespace,
}, },
dependentsLock: &sync.RWMutex{}, dependents: make(map[*node]struct{}),
dependents: make(map[*node]struct{}),
} }
p.uidToNode.Write(ownerNode) p.uidToNode.Write(ownerNode)
p.gc.dirtyQueue.Add(ownerNode) p.gc.dirtyQueue.Add(ownerNode)
@ -378,9 +377,8 @@ func (p *Propagator) processEvent() {
}, },
Namespace: accessor.GetNamespace(), Namespace: accessor.GetNamespace(),
}, },
dependentsLock: &sync.RWMutex{}, dependents: make(map[*node]struct{}),
dependents: make(map[*node]struct{}), owners: accessor.GetOwnerReferences(),
owners: accessor.GetOwnerReferences(),
} }
p.insertNode(newNode) p.insertNode(newNode)
// the underlying delta_fifo may combine a creation and deletion into one event // the underlying delta_fifo may combine a creation and deletion into one event

2
pkg/controller/garbagecollector/garbagecollector_test.go
View File

@ -301,7 +301,7 @@ func TestDependentsRace(t *testing.T) {
} }
const updates = 100 const updates = 100
owner := &node{dependentsLock: &sync.RWMutex{}, dependents: make(map[*node]struct{})} owner := &node{dependents: make(map[*node]struct{})}
ownerUID := types.UID("owner") ownerUID := types.UID("owner")
gc.propagator.uidToNode.Write(owner) gc.propagator.uidToNode.Write(owner)
go func() { go func() {