TrueCloudLab/frostfs-node
20
1
Fork 27
Code Issues 95 Pull requests 4 Projects Releases 1 Activity Actions

[#472] blobstor: check for blobovnicza without mutex first

Browse source 
Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>
This commit is contained in:
Evgenii Stratonikov 2021-04-15 17:58:15 +03:00 committed by Alex Vanin
parent 828c8695c1
commit 62b8958177
1 changed files with 9 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
pkg/local_object_storage/blobstor/blobovnicza.go
View file

@ -807,11 +807,19 @@ func (b *blobovniczas) close() error {
// //
// If blobovnicza is already opened and cached, instance from cache is returned w/o changes. // If blobovnicza is already opened and cached, instance from cache is returned w/o changes.
func (b *blobovniczas) openBlobovnicza(p string) (*blobovnicza.Blobovnicza, error) { func (b *blobovniczas) openBlobovnicza(p string) (*blobovnicza.Blobovnicza, error) {
b.lruMtx.Lock()
v, ok := b.opened.Get(p)
b.lruMtx.Unlock()
if ok {
// blobovnicza should be opened in cache
return v.(*blobovnicza.Blobovnicza), nil
}
b.openMtx.Lock() b.openMtx.Lock()
defer b.openMtx.Unlock() defer b.openMtx.Unlock()
b.lruMtx.Lock() b.lruMtx.Lock()
v, ok := b.opened.Get(p) v, ok = b.opened.Get(p)
b.lruMtx.Unlock() b.lruMtx.Unlock()
if ok { if ok {
// blobovnicza should be opened in cache // blobovnicza should be opened in cache