Re: VFS_MOUNT reverses sense of IMNT_MPSAFE test?

Rhialto <[email protected]> Fri, 8 May 2026 12:06:06 +0200
Newsgroups gmane.os.netbsd.devel.kernel
Message-ID <[email protected]>
On Thu 07 May 2026 at 07:41:54 -0700, Konrad Schroder wrote:
> While hunting down a deadlock involving mount_null, I ran into a reversed
> test for IMNT_MPSAFE in VFS_MOUNT().  I'm not sure if this is what's causing
> my problem, but it's almost certainly wrong.  Is there something I'm
> missing, or should I commit?

The pattern you spotted indeed looks wrong compared to the other cases
in the same file.

I also had some deadlock when playing with bob some time ago. It does a
lot of null mounts, and if they should have locking but do not (because
of this bug) then this could definitely explain it. (But I don't know if
null mounts are supposed to be MPSAFE or not, so it could still be a
red herring).

-Olaf.
-- 
___ Olaf 'Rhialto' Seibert                            <rhialto/at/falu.nl>
\X/ There is no AI. There is just someone else's work.           --I. Rose
signature.asc (application/pgp-signature, 525 B)
-----BEGIN PGP SIGNATURE-----

iQFPBAEBCAA5FiEEFVAhiiWjqgwBVdQAmYnGRWHD+9MFAmn9tYkbFIAAAAAABAAO
bWFudTIsMi41KzEuMTIsMiwzAAoJEJmJxkVhw/vTVF4H/2aisTCVmCaX0ooEfd+F
/vupY/pX56qvKRlcj46Myr34rQEvYFmF13/uC6wM1FacN1rGMPckSMiphLW7OAm2
w1eEHQHk4MBcrzSpTUOOIGtcYAIUrXP15NvD11m4IW05bEmQXjzyFME49Etm/zMc
WKoPaCo5+N0gGDwQXEjLXU9G7JV4F49+O06/O30sx+DxnD5G5fuf/kV58aBnyS61
8JOz2lYhdHMvEFI8q0g2Zn1WoBHIaOK5eHiNoV9JOmGHxKetiY2h082qWnvGMjuF
iagzk3VfiaCJ6ER2rp2XFJWekVFMfYOJLgWINjsmAIGwWkoZI0EihmqrG2U3y8Dr
Tfo=
=7qAe
-----END PGP SIGNATURE-----