最小化锁争用c ++ std :: map

时间:2012-02-29 03:04:51

标签: c++ multithreading locking maps contention

我有std::map<int, Object*> ObjectMap。现在我需要更新地图并通过多个线程进行更新。因此,我们锁定地图以进行更新。但是每次更新都会导致冗长的计算,从而导致锁争用。

让我们考虑以下情况。

class Order  //Subject      
{ double _a, _b,_c;   
  std::vector<Customer* > _customers;  

  public:           
  void notify(int a, int b. int c)
  {    
     //update all customers via for loop. assume a for loop and iterator i    
     _customers[i] ->updateCustomer(a,b,c)
  }      

};
class SomeNetworkClass
{
private:
   std::map<int, Order*> _orders;
public:
   void updateOrder(int orderId, int a, int b, intc)
   {     
    //lock the map
      Order* order = _orders[orderId];
      order->notify();
    //release the lock
   }     
}

class Customer
{
public:
   void updateCustomer(int a,int b, int c)
   {
    //some lengthy function. just for this example. 
    //assume printing a, b and c multiple times
   }
}

每个客户也会更新一些涉及的计算。 现在这是一个简单的观察者模式。但是,每个观察者都有大量的观察者和巨大的计算,这是这种设计的杀手锏。锁争用在我的代码中上升。我认为这是一个实际问题,但人们使用更聪明的方式,我正在寻找那些更聪明的方法。我希望这次我有点清楚

由于 希夫

1 个答案:

答案 0 :(得分:0)

由于更新发生在地图的元素上,并且没有将地图作为参数,我假设地图是不变的。

我将结构可视化为每个地图ID的对象链。现在,如果链包含不同的条目(并且更新不访问其链外的任何元素或任何全局元素),您可以通过向每个链的根元素添加锁来逃避。

但是,如果链中的对象可能被共享,那么您就会遇到更困难的问题。在这种情况下,为每个对象添加一个锁就足够了。您可以证明,如果链的行为正确(每个节点都有一个子节点,但子节点可以共享),则必须以一致的顺序获取锁,这意味着不存在死锁的可能性。

如果链之间存在其他共享,那么遇到死锁的可能性很大。

假设您有案例2,那么您的代码将大致与此类似

class Object
{
   Object * next;
   Lock l;
   Data d;
   void update(Data d_new)
   {
     l.lock();
     d = d_new;
     next->update(d_new);
     l.unlock();
   }
};