Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 31 Mar 2011 13:27:01 +0300
From:      Kostik Belousov <kostikbel@gmail.com>
To:        Pan Tsu <inyaoo@gmail.com>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r220158 - in head/sys: compat/freebsd32 kern sys
Message-ID:  <20110331102700.GH78089@deviant.kiev.zoral.com.ua>
In-Reply-To: <8662qzu5s7.fsf@gmail.com>
References:  <201103301446.p2UEkDDk066511@svn.freebsd.org> <8662qzu5s7.fsf@gmail.com>

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

--n/ThMq0iISHMWp2N
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, Mar 31, 2011 at 11:07:36AM +0400, Pan Tsu wrote:
> Konstantin Belousov <kib@FreeBSD.org> writes:
>=20
> > Author: kib
> > Date: Wed Mar 30 14:46:12 2011
> > New Revision: 220158
> > URL: http://svn.freebsd.org/changeset/base/220158
> >
> > Log:
> >   Provide compat32 shims for kldstat(2).
> >  =20
> >   Requested and tested by:	jpaetzel
> >   MFC after:	1 week
>=20
> It breaks loading modules from loader(8) for me. The box panics early
>=20
>   #13 0xffffffff804f2f2a in sysctl_find_oidname (name=3D0xffffffff80eec15=
6 "kstat", list=3D0x657a69735f796c)
>       at /usr/src/sys/kern/kern_sysctl.c:106
>   #14 0xffffffff804f3e69 in sysctl_register_oid (oidp=3D0xffffffff80ef0a8=
0)
>       at /usr/src/sys/kern/kern_sysctl.c:149
>   #15 0xffffffff804cb83c in linker_file_register_sysctls (lf=3DVariable "=
lf" is not available.
>   ) at /usr/src/sys/kern/kern_linker.c:301
>   #16 0xffffffff804cd112 in linker_preload (arg=3DVariable "arg" is not a=
vailable.
>   ) at /usr/src/sys/kern/kern_linker.c:1624
>   #17 0xffffffff80499ad7 in mi_startup () at /usr/src/sys/kern/init_main.=
c:258
>   #18 0xffffffff80299a3c in btext () at /usr/src/sys/amd64/amd64/locore.S=
:81
>   [...]
What supports your claim that the referenced commit breaks early
module loading ? You even did not provided the panic message, btw.

Commit only changed the wrapper around kldstat(2) syscall, that cannot
be used during bootstrap, and the changed code does not reference kldstat(9=
).
The later cannot prove that the change did not break something, but
due to its nature, it is unlikely.

--n/ThMq0iISHMWp2N
Content-Type: application/pgp-signature
Content-Disposition: inline

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

iEYEARECAAYFAk2UVvQACgkQC3+MBN1Mb4gqdgCfYY9BaWEtFbDTYm8/MUIPa8G9
aIQAn37G3mqOOmD8Mp2xQTUOQ997iuI1
=3m6w
-----END PGP SIGNATURE-----

--n/ThMq0iISHMWp2N--



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