Date: Mon, 9 Apr 2007 03:59:24 +0200 From: Max Laier <max@love2party.net> To: Pawel Jakub Dawidek <pjd@freebsd.org> Cc: freebsd-fs@freebsd.org, Dag-Erling Sm?rgrav <des@des.no>, freebsd-current@freebsd.org Subject: Re: ZFS committed to the FreeBSD base. Message-ID: <200704090359.30493.max@love2party.net> In-Reply-To: <20070409010703.GA74547@garage.freebsd.pl> References: <20070406025700.GB98545@garage.freebsd.pl> <20070408185312.GY63916@garage.freebsd.pl> <20070409010703.GA74547@garage.freebsd.pl>
next in thread | previous in thread | raw e-mail | index | archive | help
--nextPart1808063.uHuXMkA1X8
Content-Type: text/plain;
charset="iso-8859-6"
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline
On Monday 09 April 2007 03:07, Pawel Jakub Dawidek wrote:
> On Sun, Apr 08, 2007 at 08:53:12PM +0200, Pawel Jakub Dawidek wrote:
> > fzap_upgrade() changes type from 'zap_micro' to 'zap_fat' and union
> > is used for this (see
> > sys/contrib/opensolaris/uts/common/fs/zfs/sys/zap_impl.h), that's why
> > we see this trash:
> >
> > zap_num_entries_mtx =3D {lock_object =3D {lo_name =3D 0x70000 <Address
> > 0x70000 out of bounds>, lo_type =3D 0x0, lo_flags =3D 2155822976,
> > lo_witness_data =3D {lod_list =3D {stqe_next =3D 0x0}, lod_witness =3D =
0x0}},
> > sx_lock =3D 1, sx_recurse =3D 0},
> >
> > I already use kmem_zalloc() (note _z_) for zap allocation in
> > zap_micro.c, so Max is right, that we have to clear this structure
> > here.
> >
> > I'm quite tired of tracking such problems, because our mechanism for
> > detecting already initialized locks is too simple (based on one bit),
> > so I'd prefer to improve it, or just add bzero() to mutex_init().
>
> I just committed a fix. Now I do 13 bits check for already initialized
> locks detection instead of standard 1 bit check. Could you repeat your
> test?
Will do tomorrow. Thanks.
=2D-=20
/"\ Best regards, | mlaier@freebsd.org
\ / Max Laier | ICQ #67774661
X http://pf4freebsd.love2party.net/ | mlaier@EFnet
/ \ ASCII Ribbon Campaign | Against HTML Mail and News
--nextPart1808063.uHuXMkA1X8
Content-Type: application/pgp-signature
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (FreeBSD)
iD8DBQBGGZ4CXyyEoT62BG0RAsrPAJ4wzq6erlw5IQRpsViAS3uqDm9nKwCfRBvT
hmrRGWrdDA2eCrEF8WeBIzE=
=ujZh
-----END PGP SIGNATURE-----
--nextPart1808063.uHuXMkA1X8--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200704090359.30493.max>