Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 10 Aug 2013 22:35:06 +0300
From:      Konstantin Belousov <kostikbel@gmail.com>
To:        Peter Wemm <peter@wemm.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r254185 - head/sys/modules/random
Message-ID:  <20130810193506.GH4972@kib.kiev.ua>
In-Reply-To: <CAGE5yCrEr65m%2BohDWq_cROkXADNEcyFmDhubN_fh1g=m6V=icw@mail.gmail.com>
References:  <201308101823.r7AINS22019611@svn.freebsd.org> <CAGE5yCrEr65m%2BohDWq_cROkXADNEcyFmDhubN_fh1g=m6V=icw@mail.gmail.com>

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

--6mSV51rYMGA4/B6J
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sat, Aug 10, 2013 at 12:27:54PM -0700, Peter Wemm wrote:
> On Sat, Aug 10, 2013 at 11:23 AM, Konstantin Belousov <kib@freebsd.org> w=
rote:
> > Author: kib
> > Date: Sat Aug 10 18:23:28 2013
> > New Revision: 254185
> > URL: http://svnweb.freebsd.org/changeset/base/254185
> >
> > Log:
> >   Restore the ability to kldload random.ko, by linking in the newly
> >   added random_adaptors.c.
> >
> > Modified:
> >   head/sys/modules/random/Makefile
> >
> > Modified: head/sys/modules/random/Makefile
> > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D
> > --- head/sys/modules/random/Makefile    Sat Aug 10 18:23:18 2013       =
 (r254184)
> > +++ head/sys/modules/random/Makefile    Sat Aug 10 18:23:28 2013       =
 (r254185)
> > @@ -5,7 +5,7 @@
> >  .PATH: ${.CURDIR}/../../crypto/sha2
> >
> >  KMOD=3D  random
> > -SRCS=3D  randomdev.c probe.c
> > +SRCS=3D  randomdev.c random_adaptors.c probe.c
> >  .if ${MACHINE} =3D=3D "amd64" || ${MACHINE} =3D=3D "i386"
> >  SRCS+=3D nehemiah.c
> >  SRCS+=3D ivy.c
>=20
> However, random_adapters.c is "standard":
> # grep random files
> dev/random/harvest.c		standard
> dev/random/hash.c		optional random
> dev/random/probe.c		optional random
> dev/random/random_adaptors.c	standard
> dev/random/randomdev.c		optional random
> dev/random/randomdev_soft.c	optional random
> dev/random/yarrow.c		optional random
>=20
> I understand why harvest.c is standard - it's called by drivers.  But
> wouldn't your change to the module above duplicate the module list
> into two separate namespaces?

Oops, my change is wrong.  Apparently, I did not re-configured my kernel
where the random.ko module failed the load.

Reverting now, thank you for noticing.

--6mSV51rYMGA4/B6J
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.20 (FreeBSD)

iQIcBAEBAgAGBQJSBpXpAAoJEJDCuSvBvK1BU4AQAI2x97FytthZOBEIDqq9qaih
lvTD0hvL2+r7nLV3ut/4wMU/0ts3WW+MKaT+8br69NKvvNy1UhoKPpx+ar7hFPw2
QN54pGon3QLlC9GKaJwlJBFqqZaBQSDafC0IsK1UReBC6D2sgAfRgNu2Spxn+9J4
fIr6p8Y9lPvEtJyN2D9vkQx4W1WBY9s0a04urldbxNOSI9CaKOiLRorhLBcK5TYZ
fQK+1sXiJh2V8kllU90PJsVDc9+LL1F9OWgWM20DgyniYAC1BYI7J45oVYpOKq+b
7rwkUbhn8IT2dB48dYz/7E4eAOwNgCxvQZZsnhSrtL6m831DatbMkDyfQoJucxU+
QYlpqdM0S0agEGz3hck2g9aYXm0bUVgqKVqoD2bhhNJDvXh8XzNZBG3liIGkB5Or
DiqD/BOTNhgAnA9jPoQZJvC7uUtvowhS+1x11toT2/Pw4c6k6nOE3HTR86cO1lNm
37M3aasja/ypurlWSpZ70onfYrJP/oE4z4VAYWsiVLNqFoipmjH/arFxfFV/uO32
yLawwD0tIsvhCrsI3v7fE22bPosjkyRxlcEzffz9lHNk0jAIV9Mv9qxwU7XfeJ69
8Mghc2sm8WDJstFGaojLUQTQaa2a5t7D47cZ/uWcjRj4a/BPbakDuL4r9DKC2j4R
A8b9li6ffcnz4LH/R6ng
=Pwni
-----END PGP SIGNATURE-----

--6mSV51rYMGA4/B6J--



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