From owner-svn-src-all@FreeBSD.ORG Fri Dec 28 02:29:28 2012 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 2F0D119A; Fri, 28 Dec 2012 02:29:28 +0000 (UTC) (envelope-from kostikbel@gmail.com) Received: from kib.kiev.ua (kib.kiev.ua [IPv6:2001:470:d5e7:1::1]) by mx1.freebsd.org (Postfix) with ESMTP id 767F08FC08; Fri, 28 Dec 2012 02:29:27 +0000 (UTC) Received: from tom.home (kostik@localhost [127.0.0.1]) by kib.kiev.ua (8.14.5/8.14.5) with ESMTP id qBS2TNft091031; Fri, 28 Dec 2012 04:29:23 +0200 (EET) (envelope-from kostikbel@gmail.com) DKIM-Filter: OpenDKIM Filter v2.7.3 kib.kiev.ua qBS2TNft091031 Received: (from kostik@localhost) by tom.home (8.14.5/8.14.5/Submit) id qBS2TNMh091030; Fri, 28 Dec 2012 04:29:23 +0200 (EET) (envelope-from kostikbel@gmail.com) X-Authentication-Warning: tom.home: kostik set sender to kostikbel@gmail.com using -f Date: Fri, 28 Dec 2012 04:29:23 +0200 From: Konstantin Belousov To: Oleksandr Tymoshenko Subject: Re: svn commit: r244445 - in head/sys: kern sys tools Message-ID: <20121228022923.GP82219@kib.kiev.ua> References: <201212192010.qBJKA0lb075959@svn.freebsd.org> <50DCFDEE.8080404@freebsd.org> <20121228021425.GO82219@kib.kiev.ua> <50DD031E.409@freebsd.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="Sk71+Upln2BLuDmg" Content-Disposition: inline In-Reply-To: <50DD031E.409@freebsd.org> User-Agent: Mutt/1.5.21 (2010-09-15) X-Spam-Status: No, score=-2.0 required=5.0 tests=ALL_TRUSTED,BAYES_00, DKIM_ADSP_CUSTOM_MED,FREEMAIL_FROM,NML_ADSP_CUSTOM_MED autolearn=no version=3.3.2 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on tom.home Cc: svn-src-head@freebsd.org, Jeff Roberson , src-committers@freebsd.org, svn-src-all@freebsd.org X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 28 Dec 2012 02:29:28 -0000 --Sk71+Upln2BLuDmg Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Dec 27, 2012 at 06:25:34PM -0800, Oleksandr Tymoshenko wrote: > On 12/27/2012 6:14 PM, Konstantin Belousov wrote: > > On Thu, Dec 27, 2012 at 06:03:26PM -0800, Oleksandr Tymoshenko wrote: > >> On 12/19/2012 12:10 PM, Jeff Roberson wrote: > >>> Author: jeff > >>> Date: Wed Dec 19 20:10:00 2012 > >>> New Revision: 244445 > >>> URL: http://svnweb.freebsd.org/changeset/base/244445 > >>> > >>> Log: > >>> - Add new machine parsable KTR macros for timing events. > >>> - Use this new format to automatically handle syscalls and VOPs.= This > >>> changes the earlier format but is still human readable. > >>> =20 > >>> Sponsored by: EMC / Isilon Storage Division > >> Jeff, > >> > >> please fix tinderbox breakage caused by this commit > >> > >> ARM kernel build with KTR option is still broken as of today: > >> > >> http://tinderbox.freebsd.org/tinderbox-head-ss-build-HEAD-arm-arm.brief > >> > >> In file included from /src/sys/arm/arm/trap.c:900: > >> /src/sys/arm/arm/../../kern/subr_syscall.c: In function 'syscallenter': > >> /src/sys/arm/arm/../../kern/subr_syscall.c:80: warning: cast from poin= ter to integer of different size [-Wpointer-to-int-cast] > >> /src/sys/arm/arm/../../kern/subr_syscall.c:154: warning: cast from poi= nter to integer of different size [-Wpointer-to-int-cast] > >> *** [trap.o] Error code 1 > >> > >> > >> Thank you > > I am running a fix through the make universe right now. > > If you can compile just ARM and confirm that gcc is silenced by this, > > I will commit it before make universe finishes. > > > > diff --git a/sys/kern/subr_syscall.c b/sys/kern/subr_syscall.c > > index 3dcfcf5..349b410 100644 > > --- a/sys/kern/subr_syscall.c > > +++ b/sys/kern/subr_syscall.c > > @@ -77,7 +77,7 @@ syscallenter(struct thread *td, struct syscall_args *= sa) > > if (KTRPOINT(td, KTR_SYSCALL)) > > ktrsyscall(sa->code, sa->narg, sa->args); > > #endif > > - KTR_START4(KTR_SYSC, "syscall", syscallname(p, sa->code), > > + KTR_START4(KTR_SYSC, "syscall", (uintptr_t)syscallname(p, sa->code), > > td, "pid:%d", td->td_proc->p_pid, "arg0:%p", sa->args[0], > > "arg1:%p", sa->args[1], "arg2:%p", sa->args[2]); > > =20 > > @@ -151,7 +151,7 @@ syscallenter(struct thread *td, struct syscall_args= *sa) > > syscall_thread_exit(td, sa->callp); > > } > > retval: > > - KTR_STOP4(KTR_SYSC, "syscall", syscallname(p, sa->code), > > + KTR_STOP4(KTR_SYSC, "syscall", (uintptr_t)syscallname(p, sa->code), > > td, "pid:%d", td->td_proc->p_pid, "error:%d", error, > > "retval0:%#lx", td->td_retval[0], "retval1:%#lx", > > td->td_retval[1]); >=20 > Didn't help. It's td, not sysclalname result that gcc complains about: >=20 > http://people.freebsd.org/~gonzo/patches/ktr-syscall-fix.diff Ok, commit ? Assuming it compiles. --Sk71+Upln2BLuDmg Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.19 (FreeBSD) iQIcBAEBAgAGBQJQ3QQCAAoJEJDCuSvBvK1BA/MP/izLo5XE9EbMCjtTbpsNdj8W Go5jzPY2JeMDgXuaoXuCjIxEEepJwVyOgeBui+7auChDwm0se42GgtnPmB0fiEKy +cDIdvLS/HZNeprq9thWnlz8r1+rl0O3Qm5Q+Jr9v15kpKrEFvf2FTQA3+8qm0FK TyUKh5KcZCIJ6lfACu3A6vFvPVEVF9syggjKbZcCjoDtb+BbCyokrMQKBLnEzhTA R1AfNcvGZP8+iSFdRfwM9CnVEt3xj3qLRzgjSRb0tOGr+fpwdA8DUj3zFWD5RIqr 6YwtVrGZab/L99W1vrXFeFY4aXFLeoQPkrQr2+cILFYDE+2Qmtzxi1bSv/EDhaT2 NGhaLpQQRuFBHce6LKK/2Qytzzz6NjQOKEE+XThZmw4cTcu4QETzY07xxHG90dwJ 6unfc9RMuc2XohixCm5pxTsCKo8B/G4tPcrCO8LgwsGuELmXK1y5t+B6SNOpj8KY QeOHAdDFNzuZqE+FKivR8e6EERh0xy8oQrjQXTPqk102yuLpHBKsC9vqIRm8fiCv +PoCdFRJ1GlDdDLBeIrZ8cI0GyR6LmvA4FFTY6Hx2JZJKVcY89CJGegm+Lv8BNrl fIaQd6CXuNPK3wV9TtxSHr9bjeEwpHBVRwE8rI64jnL/evYbcSdkVEOuvl+UtmjV kA+wzs8vAFIHmSWo9jCf =Zhyv -----END PGP SIGNATURE----- --Sk71+Upln2BLuDmg--