Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 12 Nov 2018 16:50:24 +0000
From:      Brooks Davis <brooks@freebsd.org>
To:        Eitan Adler <lists@eitanadler.com>
Cc:        Brooks Davis <brooks@freebsd.org>, src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r340294 - head/sys/compat/freebsd32
Message-ID:  <20181112165024.GC66623@spindle.one-eyed-alien.net>
In-Reply-To: <CAF6rxgmjGQz%2B1wgE%2BUKzm%2B7K-wJHBNTaTEUaPgHsRN5eB256PQ@mail.gmail.com>
References:  <201811091803.wA9I31ie014903@repo.freebsd.org> <20181109181901.GA61841@spindle.one-eyed-alien.net> <CAF6rxgmjGQz%2B1wgE%2BUKzm%2B7K-wJHBNTaTEUaPgHsRN5eB256PQ@mail.gmail.com>

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

--C7zPtVaVf+AK4Oqc
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sat, Nov 10, 2018 at 12:18:13AM -0800, Eitan Adler wrote:
> On Fri, 9 Nov 2018 at 10:19, Brooks Davis <brooks@freebsd.org> wrote:
> >
> > On Fri, Nov 09, 2018 at 06:03:01PM +0000, Brooks Davis wrote:
> > > Author: brooks
> > > Date: Fri Nov  9 18:03:01 2018
> > > New Revision: 340294
> > > URL: https://svnweb.freebsd.org/changeset/base/340294
> > >
> > > Log:
> > >   Fix a number of bugs in freebsd32's capabilities.conf.
> > >
> > >   Bugs range from failure to update after changing syscall implementa=
ion
> > >   names to using the wrong name.  Somewhat confusingly, the name in
> > >   capabilities.conf is exactly the string that appears in syscalls.ma=
ster,
> > >   not the name with a COMPAT* prefix which is the actual function nam=
e.
> > >
> > >   Found while making a change to use the default capabilities.conf.
>=20
> Is there any automated way to detect this in the future? Can we add
> some test that detect non-existent syscalls or the like?

https://reviews.freebsd.org/D17932 fixes most of this.  It wouldn't be
too hard to keep track of which syscalls were emitted and check that
against the set from capabilities.conf.  I'll keep that in mind.

-- Brooks

--C7zPtVaVf+AK4Oqc
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEcBAEBAgAGBQJb6a9PAAoJEKzQXbSebgfAW4cIAJT9Kx4h33Ot0XFw/oCduNsX
AM2zwRmX5cbejqzr8A4hGN5khczXPzJRqFIg8IJAa8pQ1ZIpVSsfwX5F/EYsW0Fz
gJUe+wNqm51dPqUZTCyXpUFApqTiPAHjZYU3c2DwhUJu/wfnsI9dqO1r0I3ZCxMK
GM7fTVTpU5cNrk2OdZdOsFxe4Bpu2tfgq/PfMwedlQQovu/2X9KGV+JPC/YNRhKt
oq8qenKXx0y5aT0tM/mgnPRjbMyyHAkY3dshvq+AcKCMMVvHkVePjldA1lEX8r+W
0CpkearE65JMzEpjHnTKFlN3UqQ97Gu0gWc7IR3lAvVqW05EsAoJLXm9rF2tRz4=
=5ZxW
-----END PGP SIGNATURE-----

--C7zPtVaVf+AK4Oqc--



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