Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 28 Feb 2010 20:07:28 +0200
From:      Kostik Belousov <kostikbel@gmail.com>
To:        Attilio Rao <attilio@freebsd.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r204470 - head/sys/fs/msdosfs
Message-ID:  <20100228180728.GP2489@deviant.kiev.zoral.com.ua>
In-Reply-To: <3bbf2fe11002281000i5a6ebb35seadd6cb8f506d65e@mail.gmail.com>
References:  <201002281713.o1SHD74Q040738@svn.freebsd.org> <3bbf2fe11002281000i5a6ebb35seadd6cb8f506d65e@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help

--IihjUyvzd0n5Ehsu
Content-Type: text/plain; charset=koi8-r
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sun, Feb 28, 2010 at 07:00:10PM +0100, Attilio Rao wrote:
> 2010/2/28 Konstantin Belousov <kib@freebsd.org>:
> > Author: kib
> > Date: Sun Feb 28 17:13:07 2010
> > New Revision: 204470
> > URL: http://svn.freebsd.org/changeset/base/204470
> >
> > Log:
> > =9AAdd per-mountpoint lockmgr lock for msdosfs. It is intended to be us=
ed
> > =9Aas fat bitmap lock and to replace global mutex protecting fileno rbt=
ree.
>=20
> Looking at this patch, it seems you can safely use a sx lock here and
> I would prefer that as we don't want to encourage more lockmgr to get
> in.
> If you are concerned about writer starvation in any way here,
> probabilly the right solution is to implement it in sx and remove
> lockmgr also from the export code.

There is no write starvation in the msdosfs case, because all locks
are acquired exclusively (this is not true for the export list locking,
and write starvation is real issue there).

I used lockmgr mostly because it is fs code.

--IihjUyvzd0n5Ehsu
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (FreeBSD)

iEYEARECAAYFAkuKsOAACgkQC3+MBN1Mb4g3TgCfXM3msusBi8gxeTtHkvBVoIIh
hTEAnA16WubWpC+4p7q8LbtHVp8D5SFj
=Pl/2
-----END PGP SIGNATURE-----

--IihjUyvzd0n5Ehsu--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20100228180728.GP2489>