From owner-svn-src-all@FreeBSD.ORG Sat Oct 8 21:02:08 2011 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id D41DD106566C; Sat, 8 Oct 2011 21:02:08 +0000 (UTC) (envelope-from des@FreeBSD.org) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:4f8:fff6::2c]) by mx1.freebsd.org (Postfix) with ESMTP id C33558FC0A; Sat, 8 Oct 2011 21:02:08 +0000 (UTC) Received: from svn.freebsd.org (localhost [127.0.0.1]) by svn.freebsd.org (8.14.4/8.14.4) with ESMTP id p98L28gf078425; Sat, 8 Oct 2011 21:02:08 GMT (envelope-from des@svn.freebsd.org) Received: (from des@localhost) by svn.freebsd.org (8.14.4/8.14.4/Submit) id p98L28gZ078423; Sat, 8 Oct 2011 21:02:08 GMT (envelope-from des@svn.freebsd.org) Message-Id: <201110082102.p98L28gZ078423@svn.freebsd.org> From: Dag-Erling Smorgrav Date: Sat, 8 Oct 2011 21:02:08 +0000 (UTC) To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org X-SVN-Group: head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Cc: Subject: svn commit: r226164 - head/usr.bin/kdump X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 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: Sat, 08 Oct 2011 21:02:09 -0000 Author: des Date: Sat Oct 8 21:02:08 2011 New Revision: 226164 URL: http://svn.freebsd.org/changeset/base/226164 Log: Address some of bde@'s concerns with the new code. Modified: head/usr.bin/kdump/kdump.c Modified: head/usr.bin/kdump/kdump.c ============================================================================== --- head/usr.bin/kdump/kdump.c Sat Oct 8 18:29:30 2011 (r226163) +++ head/usr.bin/kdump/kdump.c Sat Oct 8 21:02:08 2011 (r226164) @@ -110,16 +110,15 @@ struct ktr_header ktr_header; #define TIME_FORMAT "%b %e %T %Y" #define eqs(s1, s2) (strcmp((s1), (s2)) == 0) -#define print_number(i,n,c) \ - do { \ - if (decimal) \ - printf("%c%jd", c, (intmax_t)*i); \ - else \ - printf("%c%#jx", c, (intmax_t)*i); \ - i++; \ - n--; \ - c = ','; \ - } while (0) +#define print_number(i,n,c) do { \ + if (decimal) \ + printf("%c%jd", c, (intmax_t)*i); \ + else \ + printf("%c%#jx", c, (intmax_t)*i); \ + i++; \ + n--; \ + c = ','; \ +} while (0) #if defined(__amd64__) || defined(__i386__) @@ -513,7 +512,7 @@ ktrsyscall(struct ktr_syscall *ktr, u_in } case SYS_ptrace: putchar('('); - ptraceopname((intmax_t)*ip); + ptraceopname(*ip); c = ','; ip++; narg--; @@ -522,14 +521,14 @@ ktrsyscall(struct ktr_syscall *ktr, u_in case SYS_eaccess: print_number(ip, narg, c); putchar(','); - accessmodename((intmax_t)*ip); + accessmodename(*ip); ip++; narg--; break; case SYS_open: print_number(ip, narg, c); putchar(','); - flagsandmodename((int)ip[0], (int)ip[1], decimal); + flagsandmodename(ip[0], ip[1], decimal); ip += 2; narg -= 2; break; @@ -537,7 +536,7 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - wait4optname((intmax_t)*ip); + wait4optname(*ip); ip++; narg--; break; @@ -546,14 +545,14 @@ ktrsyscall(struct ktr_syscall *ktr, u_in case SYS_lchmod: print_number(ip, narg, c); putchar(','); - modename((intmax_t)*ip); + modename(*ip); ip++; narg--; break; case SYS_mknod: print_number(ip, narg, c); putchar(','); - modename((intmax_t)*ip); + modename(*ip); ip++; narg--; break; @@ -561,7 +560,7 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - getfsstatflagsname((intmax_t)*ip); + getfsstatflagsname(*ip); ip++; narg--; break; @@ -569,14 +568,14 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - mountflagsname((intmax_t)*ip); + mountflagsname(*ip); ip++; narg--; break; case SYS_unmount: print_number(ip, narg, c); putchar(','); - mountflagsname((intmax_t)*ip); + mountflagsname(*ip); ip++; narg--; break; @@ -585,7 +584,7 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - sendrecvflagsname((int)*ip); + sendrecvflagsname(*ip); ip++; narg--; break; @@ -595,7 +594,7 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - sendrecvflagsname((int)*ip); + sendrecvflagsname(*ip); ip++; narg--; break; @@ -604,26 +603,26 @@ ktrsyscall(struct ktr_syscall *ktr, u_in case SYS_lchflags: print_number(ip, narg, c); putchar(','); - modename((intmax_t)*ip); + modename(*ip); ip++; narg--; break; case SYS_kill: print_number(ip, narg, c); putchar(','); - signame((int)*ip); + signame(*ip); ip++; narg--; break; case SYS_reboot: putchar('('); - rebootoptname((intmax_t)*ip); + rebootoptname(*ip); ip++; narg--; break; case SYS_umask: putchar('('); - modename((intmax_t)*ip); + modename(*ip); ip++; narg--; break; @@ -631,7 +630,7 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - msyncflagsname((intmax_t)*ip); + msyncflagsname(*ip); ip++; narg--; break; @@ -640,11 +639,11 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - mmapprotname((intmax_t)*ip); + mmapprotname(*ip); putchar(','); ip++; narg--; - mmapflagsname((intmax_t)*ip); + mmapflagsname(*ip); ip++; narg--; break; @@ -653,11 +652,11 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - mmapprotname((intmax_t)*ip); + mmapprotname(*ip); putchar(','); ip++; narg--; - mmapflagsname((intmax_t)*ip); + mmapflagsname(*ip); ip++; narg--; break; @@ -665,7 +664,7 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - mmapprotname((intmax_t)*ip); + mmapprotname(*ip); ip++; narg--; break; @@ -673,7 +672,7 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - madvisebehavname((intmax_t)*ip); + madvisebehavname(*ip); ip++; narg--; break; @@ -681,32 +680,32 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - prioname((intmax_t)*ip); + prioname(*ip); ip++; narg--; break; case SYS_fcntl: print_number(ip, narg, c); putchar(','); - fcntlcmdname((int)ip[0], (int)ip[1], decimal); + fcntlcmdname(ip[0], ip[1], decimal); ip += 2; narg -= 2; break; case SYS_socket: { int sockdomain; putchar('('); - sockdomain=(intmax_t)*ip; + sockdomain = *ip; sockdomainname(sockdomain); ip++; narg--; putchar(','); - socktypename((intmax_t)*ip); + socktypename(*ip); ip++; narg--; if (sockdomain == PF_INET || sockdomain == PF_INET6) { putchar(','); - sockipprotoname((intmax_t)*ip); + sockipprotoname(*ip); ip++; narg--; } @@ -717,12 +716,12 @@ ktrsyscall(struct ktr_syscall *ktr, u_in case SYS_getsockopt: print_number(ip, narg, c); putchar(','); - sockoptlevelname((int)*ip, decimal); + sockoptlevelname(*ip, decimal); if (*ip == SOL_SOCKET) { ip++; narg--; putchar(','); - sockoptname((intmax_t)*ip); + sockoptname(*ip); } ip++; narg--; @@ -734,7 +733,7 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - whencename((intmax_t)*ip); + whencename(*ip); ip++; narg--; break; @@ -744,14 +743,14 @@ ktrsyscall(struct ktr_syscall *ktr, u_in /* Hidden 'pad' argument, not in lseek(2) */ print_number(ip, narg, c); putchar(','); - whencename((intmax_t)*ip); + whencename(*ip); ip++; narg--; break; case SYS_flock: print_number(ip, narg, c); putchar(','); - flockname((intmax_t)*ip); + flockname(*ip); ip++; narg--; break; @@ -759,24 +758,24 @@ ktrsyscall(struct ktr_syscall *ktr, u_in case SYS_mkdir: print_number(ip, narg, c); putchar(','); - modename((intmax_t)*ip); + modename(*ip); ip++; narg--; break; case SYS_shutdown: print_number(ip, narg, c); putchar(','); - shutdownhowname((intmax_t)*ip); + shutdownhowname(*ip); ip++; narg--; break; case SYS_socketpair: putchar('('); - sockdomainname((intmax_t)*ip); + sockdomainname(*ip); ip++; narg--; putchar(','); - socktypename((intmax_t)*ip); + socktypename(*ip); ip++; narg--; c = ','; @@ -784,7 +783,7 @@ ktrsyscall(struct ktr_syscall *ktr, u_in case SYS_getrlimit: case SYS_setrlimit: putchar('('); - rlimitname((intmax_t)*ip); + rlimitname(*ip); ip++; narg--; c = ','; @@ -792,21 +791,21 @@ ktrsyscall(struct ktr_syscall *ktr, u_in case SYS_quotactl: print_number(ip, narg, c); putchar(','); - quotactlname((intmax_t)*ip); + quotactlname(*ip); ip++; narg--; c = ','; break; case SYS_nfssvc: putchar('('); - nfssvcname((intmax_t)*ip); + nfssvcname(*ip); ip++; narg--; c = ','; break; case SYS_rtprio: putchar('('); - rtprioname((int)*ip); + rtprioname(*ip); ip++; narg--; c = ','; @@ -815,7 +814,7 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - semctlname((int)*ip); + semctlname(*ip); ip++; narg--; break; @@ -823,14 +822,14 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - semgetname((int)*ip); + semgetname(*ip); ip++; narg--; break; case SYS_msgctl: print_number(ip, narg, c); putchar(','); - shmctlname((int)*ip); + shmctlname(*ip); ip++; narg--; break; @@ -838,14 +837,14 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - shmatname((intmax_t)*ip); + shmatname(*ip); ip++; narg--; break; case SYS_shmctl: print_number(ip, narg, c); putchar(','); - shmctlname((int)*ip); + shmctlname(*ip); ip++; narg--; break; @@ -853,41 +852,41 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - minheritname((intmax_t)*ip); + minheritname(*ip); ip++; narg--; break; case SYS_rfork: putchar('('); - rforkname((intmax_t)*ip); + rforkname(*ip); ip++; narg--; c = ','; break; case SYS_lio_listio: putchar('('); - lio_listioname((intmax_t)*ip); + lio_listioname(*ip); ip++; narg--; c = ','; break; case SYS_mlockall: putchar('('); - mlockallname((intmax_t)*ip); + mlockallname(*ip); ip++; narg--; break; case SYS_sched_setscheduler: print_number(ip, narg, c); putchar(','); - schedpolicyname((intmax_t)*ip); + schedpolicyname(*ip); ip++; narg--; break; case SYS_sched_get_priority_max: case SYS_sched_get_priority_min: putchar('('); - schedpolicyname((intmax_t)*ip); + schedpolicyname(*ip); ip++; narg--; break; @@ -899,20 +898,20 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - sendfileflagsname((intmax_t)*ip); + sendfileflagsname(*ip); ip++; narg--; break; case SYS_kldsym: print_number(ip, narg, c); putchar(','); - kldsymcmdname((intmax_t)*ip); + kldsymcmdname(*ip); ip++; narg--; break; case SYS_sigprocmask: putchar('('); - sigprocmaskhowname((intmax_t)*ip); + sigprocmaskhowname(*ip); ip++; narg--; c = ','; @@ -931,13 +930,13 @@ ktrsyscall(struct ktr_syscall *ktr, u_in case SYS___acl_aclcheck_link: print_number(ip, narg, c); putchar(','); - acltypename((intmax_t)*ip); + acltypename(*ip); ip++; narg--; break; case SYS_sigaction: putchar('('); - signame((int)*ip); + signame(*ip); ip++; narg--; c = ','; @@ -945,7 +944,7 @@ ktrsyscall(struct ktr_syscall *ktr, u_in case SYS_extattrctl: print_number(ip, narg, c); putchar(','); - extattrctlname((intmax_t)*ip); + extattrctlname(*ip); ip++; narg--; break; @@ -953,7 +952,7 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - mountflagsname((intmax_t)*ip); + mountflagsname(*ip); ip++; narg--; break; @@ -961,28 +960,28 @@ ktrsyscall(struct ktr_syscall *ktr, u_in print_number(ip, narg, c); print_number(ip, narg, c); putchar(','); - thrcreateflagsname((intmax_t)*ip); + thrcreateflagsname(*ip); ip++; narg--; break; case SYS_thr_kill: print_number(ip, narg, c); putchar(','); - signame((int)*ip); + signame(*ip); ip++; narg--; break; case SYS_kldunloadf: print_number(ip, narg, c); putchar(','); - kldunloadfflagsname((intmax_t)*ip); + kldunloadfflagsname(*ip); ip++; narg--; break; case SYS_cap_new: print_number(ip, narg, c); putchar(','); - capname((intmax_t)*ip); + capname(*ip); ip++; narg--; break;