Re: VOP_GETATTR: locking protocol change proposal

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

On Oct 9, 2011, at 5:02 PM, Emmanuel Dreyfus wrote:

> J. Hannken-Illjes <hannken%eis.cs.tu-bs.de@localhost> wrote:
> 
>> With the attached diff the locking protocol requests at least a shared
>> lock and all calls to VOP_GETATTR() outside of file systems respect it.
> 
> Please look at pn_sizemtx in src/fs/puffs/puffs_vnop.c

I know -- there are other places too.

> As I understand your change allows to remove it.
> VOP_(SETATTR|FSYNC|INACTIVE|WRITE) all operate of anexclusive locked
> vnode, right?

Yes.

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