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

On Tue, Nov 04, 2003 at 04:19:52PM +0000, David Laight wrote:
>=20
> Module Name:	src
> Committed By:	dsl
> Date:		Tue Nov  4 16:19:52 UTC 2003
>=20
> Modified Files:
> 	src/sys/sys: proc.h
>=20
> Log Message:
> Remove p_nras from struct proc - use LIST_EMPTY(&p->p_raslist) instead.
> Remove p_raslock and rename p_lwplock p_lock (one lock is enough).
> (pad fields left in struct proc to avoid kernel bump)

You changed field names and deleted one. You really should bump the
version. The point isn't just that you kept struct proc's layout the same,
it's that you renamed something.

While you've done an excelent job of maintaining binary compatability, you
have broken source compatability. The packages I'm familiar with that try=
=20
to keep lkms and drivers up to date use defines based on kernel version.=20
This change makes it so that that behavior (defines) doesn't work.

I'd say just bump the kernel version.

If we're getting worried about too many letters, then we (the developers)=
=20
should informally work on bunching changes together.

Take care,

Bill

--11Y7aswkeuHtSBEs
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.3 (NetBSD)

iD8DBQE/qpv5Wz+3JHUci9cRAswvAJ0fx5HopyKAwJEBXS9E+fndhf/ngACffrk1
ANO00jk48cbIPzrbndRNJt4=
=tyXI
-----END PGP SIGNATURE-----

--11Y7aswkeuHtSBEs--