我有一个问题

黎明748

我觉得这里mutex_lock(&adb_mutex)主要是保护if（state），kfree(state)和 spin_lock_irqsave(&state->lock, flags)和spin_unlock_irqsave(&state->lock, flags);。这里操作file结构没有上锁，估计file结构之前也是已经获得锁了，没代码不清楚。spin就保护state结构。

adidiaos丶丶

那什么情况下会出现你说的“只保证这里的if语句只有一个路径可以执行，并不保证其他路径不会访问state结构”。我是觉得他这种写法好像是mutex只保证state不会被NULL，但是不保证state里面的内容不被修改，所以需要另加spin。如果是这样我就不明白为什么mutex不能保证state内容不被修改呢？

黎明748 发表于 2014-08-05 02:23
mutex_lock(&adb_mutex);只保证这里的if语句只有一个路径可以执行，并不保证其他路径不会访问state结构，所 ...

super皮波

回复 12# adidiaos丶丶
看这段代码感觉是spin_lock中保护的资源有可能在中断上半部会被访问到

   

super皮波

具体的你找找代码，看自旋锁所保护的资源到底在内核的什么路径下会被访问

黎明748

mutex保护的是临界区，spin锁才是保护结构。mutex怎么能保护state结构回复 12# adidiaos丶丶


   

adidiaos丶丶

恩，对头。回复 14# super皮波


   

adidiaos丶丶

我再研究研究。。。

黎明748 发表于 2014-08-05 12:50
mutex保护的是临界区，spin锁才是保护结构。mutex怎么能保护state结构回复 12# adidiaos丶丶