Re: CVS commit: src/external/cddl/osnet/dist/uts/common/fs/zfs

[David Laight <david%l8s.co.uk@localhost>]

On Wed, Oct 28, 2009 at 11:44:51PM +0000, Adam Hamsik wrote:
> Module Name:  src
> Committed By: haad
> Date:         Wed Oct 28 23:44:51 UTC 2009
> 
> Modified Files:
>       src/external/cddl/osnet/dist/uts/common/fs/zfs: zfs_vnops.c zfs_znode.c
> 
> Log Message:
> Add workaround about zfs vnode reclaiming deadlock by checking if we don't
> ehld ZFS_MUTEX_OBJ already. If we can lock OBJ_MUTEX deffer execution of
> zfs_zinactive to taskq. Code was inspired by FreeBSD zfs_freebsd_reclaim.

This doesn't sound right at all ...

If the 'If we can lock OBJ_MUTEX' test is a try_mutex_enter() then
it may fail because another lwp owns it ...
I hope it is only the checkin comment that is inverted!

        David

-- 
David Laight: david%l8s.co.uk@localhost