annotate base/Serialiser.cpp @ 1857:14c776dad920

Make Serialiser cancellable while waiting for its lock
author Chris Cannam
date Thu, 14 May 2020 16:35:39 +0100
parents d095214ffbaf
children
rev   line source
Chris@297 1 /* -*- c-basic-offset: 4 indent-tabs-mode: nil -*- vi:set ts=8 sts=4 sw=4: */
Chris@297 2
Chris@297 3 /*
Chris@297 4 Sonic Visualiser
Chris@297 5 An audio file viewer and annotation editor.
Chris@297 6 Centre for Digital Music, Queen Mary, University of London.
Chris@297 7 This file copyright 2007 QMUL.
Chris@297 8
Chris@297 9 This program is free software; you can redistribute it and/or
Chris@297 10 modify it under the terms of the GNU General Public License as
Chris@297 11 published by the Free Software Foundation; either version 2 of the
Chris@297 12 License, or (at your option) any later version. See the file
Chris@297 13 COPYING included with this distribution for more information.
Chris@297 14 */
Chris@297 15
Chris@297 16 #include "Serialiser.h"
Chris@1857 17 #include "Debug.h"
Chris@297 18
Chris@406 19 #include <iostream>
Chris@406 20
Chris@297 21 QMutex
Chris@297 22 Serialiser::m_mapMutex;
Chris@297 23
Chris@297 24 std::map<QString, QMutex *>
Chris@297 25 Serialiser::m_mutexMap;
Chris@297 26
Chris@297 27 Serialiser::Serialiser(QString id) :
Chris@1857 28 Serialiser(id, nullptr) { }
Chris@1857 29
Chris@1857 30 Serialiser::Serialiser(QString id, const std::atomic<bool> *cancelled) :
Chris@1857 31 m_id(id),
Chris@1857 32 m_cancelled(cancelled),
Chris@1857 33 m_locked(false)
Chris@297 34 {
Chris@297 35 m_mapMutex.lock();
Chris@297 36
Chris@297 37 if (m_mutexMap.find(m_id) == m_mutexMap.end()) {
Chris@297 38 m_mutexMap[m_id] = new QMutex;
Chris@297 39 }
Chris@297 40
Chris@406 41 // The id mutexes are never deleted, so once we have a reference
Chris@406 42 // to the one we need, we can hold on to it while we release the
Chris@406 43 // map mutex. We need to release the map mutex, otherwise if the
Chris@406 44 // id mutex is currently held, it will never be released (because
Chris@406 45 // the destructor needs to hold the map mutex to release the id
Chris@406 46 // mutex).
Chris@406 47
Chris@406 48 QMutex *idMutex = m_mutexMap[m_id];
Chris@398 49
Chris@297 50 m_mapMutex.unlock();
Chris@406 51
Chris@1857 52 if (!m_cancelled) {
Chris@1857 53 idMutex->lock();
Chris@1857 54 m_locked = true;
Chris@1857 55 } else {
Chris@1857 56 // try to lock, polling the cancelled status occasionally
Chris@1857 57 while (!m_locked && ! *m_cancelled) {
Chris@1857 58 m_locked = idMutex->tryLock(500);
Chris@1857 59 if (*m_cancelled) {
Chris@1857 60 SVCERR << "Serialiser: cancelled!" << endl;
Chris@1857 61 }
Chris@1857 62 }
Chris@1857 63 }
Chris@297 64 }
Chris@297 65
Chris@297 66 Serialiser::~Serialiser()
Chris@297 67 {
Chris@398 68 m_mapMutex.lock();
Chris@1857 69
Chris@1857 70 if (m_locked) {
Chris@1857 71 m_mutexMap[m_id]->unlock();
Chris@1857 72 }
Chris@398 73
Chris@398 74 m_mapMutex.unlock();
Chris@297 75 }
Chris@297 76
Chris@297 77
Chris@297 78