Sorry I was too busy last week to look at this.  The change you propose 
is correct.

jim

Pavel Filipensky wrote:

> Hi,
>
> can I get a code review for 6861594 NFSv4 client Deadlock: cycle in 
> blocking chain at nfs4_move_mi
>
> Webrev:
>
> http://cr.opensolaris.org/~pavelf/6861594/
>
>
> Background:
>
> If there are two threads doing a failover for differnet mounts at the 
> same time, both of them need to grab
> the s_lock for the new and the old nfs server. To prevent a deadlock 
> they must grab the s_locks it in the same order.
> nfs4_move_mi() is not safe since it does not guarantee the same order 
> and allows the deadlock to happen.
>
>
>
> Thanks,
> Pavel
>
>
> _______________________________________________
> nfs-discuss mailing list
> nfs-discuss at opensolaris.org



Reply via email to