Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

boost::interprocess::scoped_lock application crash inside lock

I'm using boost::interprocess::scoped_lock, if the application crash for some reason inside the scope the mutex isn't released. Next time the application is executed (without restarting the computer), the mutex is locked.

How's this intended to work? I give a simple example of the code below.

{
    boost::interprocess::named_mutex lockMutex(boost::interprocess::open_or_create, "lockName");
    boost::interprocess::scoped_lock<boost::interprocess::named_mutex> lock(lockMutex);
    //crash here
}

I ended up doing a timeout like below. Anyone who could come up with a solution that doesn't limit the time for lock?

boost::interprocess::named_mutex named_mtx(boost::interprocess::open_or_create, lockName.c_str());

    while(true)
    {
        if(named_mtx.try_lock())
        {
            break;
        }

        if(!named_mtx.timed_lock(boost::get_system_time() + boost::posix_time::milliseconds(TIMEOUT_MILLISECONDS)))
        {
            named_mtx.unlock();
        }
    }
like image 226
thenail Avatar asked Nov 05 '22 20:11

thenail


1 Answers

That's seems perfectly logic to me :)

As your application crash, the mutex which maps to your OS interprocess communication mechanism (IPC) is not released. When your application restart it tries to get the mutex without success !

I suppose your application has different subsystems (processes) that need to be synchronized.

You have to devise a global policy in case of a crash of one of your subsystem to manage correctly the lock. For example, if one of your subsystem crash, it should try and unlock the mutex at startup. It can be tricky as other subsystems use that lock. Timeouts can help too. In any case, you have to devise the policy having in mind that any of your processes can crash while having locked the mutex...

Of course, if you do not need interprocess locking, use simple scoped locks :)

my2c

like image 188
neuro Avatar answered Nov 11 '22 20:11

neuro