[ofa-general] lock dependency in ib_user_mad
Sean Hefty
sean.hefty at intel.com
Mon Jan 14 13:02:27 PST 2008
>@@ -703,7 +708,8 @@ static int ib_umad_unreg_agent(struct ib_umad_file *file,
>u32 __user *arg)
> if (get_user(id, arg))
> return -EFAULT;
>
>- down_write(&file->port->mutex);
>+ mutex_unlock(&file->port->file_mutex);
This should be mutex_lock().
The other changes look okay. With your latest patch and the change above, I was
not able to reproduce the lockdep warnings. (I've seen the warnings 3 or 4
times now, but it's not easy to reproduce.)
Thanks
- Sean
More information about the general
mailing list