Re: CVS commit: src/sys/nfs

["J. Hannken-Illjes" <hannken%eis.cs.tu-bs.de@localhost>]

On Nov 15, 2011, at 4:08 AM, YAMAMOTO Takashi wrote:

> hi,
> 
>> Module Name: src
>> Committed By:        hannken
>> Date:                Sun Oct 30 12:00:28 UTC 2011
>> 
>> Modified Files:
>>      src/sys/nfs: nfs_serv.c
>> 
>> Log Message:
>> VOP_GETATTR() needs a shared lock at least.
> 
> this seems trying to lock the directory while holding its child locked.
> ie. wrong locking order

Seems I overlooked one case, the change on top of nfsrv_lookup().  All other
changes should use the right locking order.

--
Juergen Hannken-Illjes - hannken%eis.cs.tu-bs.de@localhost - TU Braunschweig 
(Germany)