Re: CVS commit: src/sys

[Frank Kardel <kardel%netbsd.org@localhost>]

Hello Juergen !

Could you maybe re-check this change. With it I experience reliably a 
"locking against myself"
(option ) DIAGNOSTIC panic in  wapbl_begin() on a 4 CPU system when 
starting the
postgresql dbms (sorry no core).
I do not get that panic when I revert that change.

Best regards,
 Frank

Juergen Hannken-Illjes wrote:
> Module Name:    src
> Committed By:   hannken
> Date:           Fri Oct 10 09:21:58 UTC 2008
>
> Modified Files:
>         src/sys/miscfs/genfs: genfs_io.c
>         src/sys/ufs/ffs: ffs_extern.h ffs_vfsops.c ffs_vnops.c
>
> Log Message:
> Break a deadlock where one thread has a wapbl transaction, calls VOP_GETPAGES
> and wants to busy a page  while  another thread calls VOP_PUTPAGES on the same
> vnode, takes pages busy and wants to start a wapbl transaction.
>
> Reviewed by: Jason Thorpe <thorpej%netbsd.org@localhost>
>
>
> To generate a diff of this commit:
> cvs rdiff -r1.11 -r1.12 src/sys/miscfs/genfs/genfs_io.c
> cvs rdiff -r1.69 -r1.70 src/sys/ufs/ffs/ffs_extern.h
> cvs rdiff -r1.237 -r1.238 src/sys/ufs/ffs/ffs_vfsops.c
> cvs rdiff -r1.103 -r1.104 src/sys/ufs/ffs/ffs_vnops.c
>
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
>