[LU-1117] lov_getref/putref incorrectly rely on a mutex protection

Wed, 28 Feb 2018 20:29:18 +0000

lov_getref has this code:

static void lov_getref(struct obd_device *obd)
{
        struct lov_obd *lov = &obd->u.lov;

        /* nobody gets through here until lov_putref is done */
        cfs_mutex_down(&lov->lov_lock);
        cfs_atomic_inc(&lov->lov_refcount);
        cfs_mutex_up(&lov->lov_lock);
        return;
}

I imagine Nathan tried to avoid some sort of a race while working on bug 9439, but in reality whatever was the race he imagined, it's still there.

At the very least we need to just get rid of this locking here and in lov_putref() and also get rid of lov_lock() in total I suspect.

Whamcloud Community JIRA

[LU-1117] lov_getref/putref incorrectly rely on a mutex protection