[ovs-dev] [PATCH v3] datapath: Simplify datapath locking.

Jesse Gross jesse at nicira.com
Wed Apr 10 19:01:55 UTC 2013


On Wed, Apr 10, 2013 at 11:40 AM, Pravin Shelar <pshelar at nicira.com> wrote:
> On Wed, Apr 10, 2013 at 9:01 AM, Jesse Gross <jesse at nicira.com> wrote:
>> By the way, I think we may be able to make the workqueue compatibility
>> code conditional on version again. The comment says that we were
>> having problems with waiting on genl mutex when using workqueues but
>> we shouldn't have to do that anymore.
>
>
> I guess this softlockup issue will move from genl_lock to ovs_mutex, as
> rehash_wq need to take ovs_mutex.  therefore we still need to use compat
> workq code.

Hmm, you're right, I found the original commit. I thought that it was
some other task holding genl that was causing the problem but it was
OVS. When we break down locking further we should be able to actually
fix this.



More information about the dev mailing list