Date: Fri, 16 Jul 2010 11:09:44 +0300 From: Kostik Belousov <kostikbel@gmail.com> To: "Bjoern A. Zeeb" <bz@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, John Baldwin <jhb@freebsd.org> Subject: Re: svn commit: r210138 - in head/sys: kern sys Message-ID: <20100716080944.GA2381@deviant.kiev.zoral.com.ua> In-Reply-To: <20100716064257.S14969@maildrop.int.zabbadoz.net> References: <201007152024.o6FKObhM092035@svn.freebsd.org> <20100716064257.S14969@maildrop.int.zabbadoz.net>
next in thread | previous in thread | raw e-mail | index | archive | help
--RUvhGz2nhX7DIu1B Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Jul 16, 2010 at 06:44:39AM +0000, Bjoern A. Zeeb wrote: > On Thu, 15 Jul 2010, John Baldwin wrote: >=20 > >Author: jhb > >Date: Thu Jul 15 20:24:37 2010 > >New Revision: 210138 > >URL: http://svn.freebsd.org/changeset/base/210138 > > > >Log: > > Retire td_syscalls now that it is no longer needed. > > > >Modified: > > head/sys/kern/kern_thread.c > > head/sys/kern/subr_trap.c > > head/sys/sys/proc.h >=20 > Did you want to leave the field as a placehgolder or add a note > somewhere that people should do a full recompile because the layout > changed? Make depend should do the magic, right ? I was surprised to see that mips/arm did not touched td_syscalls, and I wonder how much other bugs are lurking in their implementation. >=20 >=20 > >Modified: head/sys/sys/proc.h > >=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/sys/proc.h Thu Jul 15 19:52:54 2010 (r210137) > >+++ head/sys/sys/proc.h Thu Jul 15 20:24:37 2010 (r210138) > >@@ -296,7 +296,6 @@ struct thread { > > struct mdthread td_md; /* (k) Any machine-dependent fields.=20 > > */ > > struct td_sched *td_sched; /* (*) Scheduler-specific data. */ > > struct kaudit_record *td_ar; /* (k) Active audit record, if any.=20 > > */ > >- int td_syscalls; /* per-thread syscall count (used by=20 > >NFS :)) */ > > struct lpohead td_lprof[2]; /* (a) lock profiling objects. */ > > struct kdtrace_thread *td_dtrace; /* (*) DTrace-specific data. */ > > int td_errno; /* Error returned by last syscall. */ > > >=20 > --=20 > Bjoern A. Zeeb From August on I will have a life. It's now up to you > to do the maths and count to 64. -- Bondorf, Germany, 14th June 2010 --RUvhGz2nhX7DIu1B Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (FreeBSD) iEUEARECAAYFAkxAE8cACgkQC3+MBN1Mb4gK1wCYuwUHib+490sYpf2+ZF1ywgCo fgCghg5r1cK8QctlFnT6CJysfO3xmyY= =mOqy -----END PGP SIGNATURE----- --RUvhGz2nhX7DIu1B--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20100716080944.GA2381>