From owner-svn-src-all@FreeBSD.ORG Fri Dec 28 07:51:44 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 C7F7EE87 for ; Fri, 28 Dec 2012 07:51:44 +0000 (UTC) (envelope-from juli@clockworksquid.com) Received: from mail-yh0-f45.google.com (mail-yh0-f45.google.com [209.85.213.45]) by mx1.freebsd.org (Postfix) with ESMTP id 79CDC8FC0A for ; Fri, 28 Dec 2012 07:51:44 +0000 (UTC) Received: by mail-yh0-f45.google.com with SMTP id p34so1765561yhp.18 for ; Thu, 27 Dec 2012 23:51:38 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:sender:in-reply-to:references:from:date :x-google-sender-auth:message-id:subject:to:cc:content-type :x-gm-message-state; bh=WRT0Q4fhaF3Cho5ineLHlxa+M7MpoJqLId666yuhseM=; b=KQZJppAprlLbt/cAI0T3VtqCCcSB+Am2Ac4ycYNnghGtCQQjYAaYX97WE60uxLHsSi LmfxuANZz/NJBW5qrXaWnYQT3ndbTGqUhDPoe0dFjaFO2syehfayC8OVdgbpA9eR81K2 L+oSprIFQ3KG6R15RSWBtO55cMdG9C5/uZYfLhbn2zw5fzh/5nFiaCtyaeNt+sAjZhxW c7g9rj8lfHBIFs0YTfrAOxi8IEG3EE+TI2zVa9ndWz+zgSIkxRjp7jXcvwwWd3uUq/Ub yib1fKYIiLjRRhPnblgCTAwaXTgouuRzdoCMsksdI6uGszOnfR6LeNS1Si9YO7ApOg/s t8ZA== Received: by 10.236.137.172 with SMTP id y32mr28662925yhi.124.1356680763505; Thu, 27 Dec 2012 23:46:03 -0800 (PST) MIME-Version: 1.0 Sender: juli@clockworksquid.com Received: by 10.147.119.38 with HTTP; Thu, 27 Dec 2012 23:45:43 -0800 (PST) In-Reply-To: <201212280652.qBS6qrj7072246@svn.freebsd.org> References: <201212280652.qBS6qrj7072246@svn.freebsd.org> From: Juli Mallett Date: Thu, 27 Dec 2012 23:45:43 -0800 X-Google-Sender-Auth: 0zW-SNbbf7Wg1g1fn1VOOJDvvtQ Message-ID: Subject: Re: svn commit: r244765 - in head/sys: kern tools To: Oleksandr Tymoshenko Content-Type: text/plain; charset=UTF-8 X-Gm-Message-State: ALoCoQkd4BN+LOdnczO43R/ceLpx1p7wG1rPN/9oxi2fk4ONH0+KDr1ejYSRkdSZnFPRw3zN5/Ji Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@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 07:51:44 -0000 Seems like uintmax_t and %ju might be better? At least, truncating a syscall arg to a pointer seems like it could be misleading (e.g. on MIPS n32, even though it's an abomination.) On Thu, Dec 27, 2012 at 10:52 PM, Oleksandr Tymoshenko wrote: > Author: gonzo > Date: Fri Dec 28 06:52:53 2012 > New Revision: 244765 > URL: http://svnweb.freebsd.org/changeset/base/244765 > > Log: > Fix build on ARM (and probably other platforms) > > Modified: > head/sys/kern/subr_syscall.c > head/sys/tools/vnode_if.awk > > Modified: head/sys/kern/subr_syscall.c > ============================================================================== > --- head/sys/kern/subr_syscall.c Fri Dec 28 05:48:44 2012 (r244764) > +++ head/sys/kern/subr_syscall.c Fri Dec 28 06:52:53 2012 (r244765) > @@ -78,7 +78,7 @@ syscallenter(struct thread *td, struct s > ktrsyscall(sa->code, sa->narg, sa->args); > #endif > KTR_START4(KTR_SYSC, "syscall", syscallname(p, sa->code), > - td, "pid:%d", td->td_proc->p_pid, "arg0:%p", sa->args[0], > + (uintptr_t)td, "pid:%d", td->td_proc->p_pid, "arg0:%p", sa->args[0], > "arg1:%p", sa->args[1], "arg2:%p", sa->args[2]); > > if (error == 0) { > @@ -152,7 +152,7 @@ syscallenter(struct thread *td, struct s > } > retval: > KTR_STOP4(KTR_SYSC, "syscall", syscallname(p, sa->code), > - td, "pid:%d", td->td_proc->p_pid, "error:%d", error, > + (uintptr_t)td, "pid:%d", td->td_proc->p_pid, "error:%d", error, > "retval0:%#lx", td->td_retval[0], "retval1:%#lx", > td->td_retval[1]); > if (traced) { > > Modified: head/sys/tools/vnode_if.awk > ============================================================================== > --- head/sys/tools/vnode_if.awk Fri Dec 28 05:48:44 2012 (r244764) > +++ head/sys/tools/vnode_if.awk Fri Dec 28 06:52:53 2012 (r244765) > @@ -270,7 +270,7 @@ while ((getline < srcfile) > 0) { > ctrargs = 4; > else > ctrargs = numargs; > - ctrstr = ctrargs "(KTR_VOP, \"VOP\", \"" uname "\", a,\n\t "; > + ctrstr = ctrargs "(KTR_VOP, \"VOP\", \"" uname "\", (uintptr_t)a,\n\t "; > ctrstr = ctrstr "\"" args[0] ":0x%jX\", (uintptr_t)a->a_" args[0]; > for (i = 1; i < ctrargs; ++i) > ctrstr = ctrstr ", \"" args[i] ":0x%jX\", a->a_" args[i];