From owner-svn-src-stable-6@FreeBSD.ORG Mon May 18 07:55:20 2009 Return-Path: Delivered-To: svn-src-stable-6@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 98035106566B; Mon, 18 May 2009 07:55:20 +0000 (UTC) (envelope-from rwatson@FreeBSD.org) Received: from cyrus.watson.org (cyrus.watson.org [65.122.17.42]) by mx1.freebsd.org (Postfix) with ESMTP id 371738FC08; Mon, 18 May 2009 07:55:20 +0000 (UTC) (envelope-from rwatson@FreeBSD.org) Received: from fledge.watson.org (fledge.watson.org [65.122.17.41]) by cyrus.watson.org (Postfix) with ESMTPS id C295946B4C; Mon, 18 May 2009 03:55:19 -0400 (EDT) Date: Mon, 18 May 2009 08:55:19 +0100 (BST) From: Robert Watson X-X-Sender: robert@fledge.watson.org To: Attilio Rao In-Reply-To: <200905141336.n4EDadT4092402@svn.freebsd.org> Message-ID: References: <200905141336.n4EDadT4092402@svn.freebsd.org> User-Agent: Alpine 2.00 (BSF 1167 2008-08-23) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: svn-src-stable@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, svn-src-stable-6@freebsd.org Subject: Re: svn commit: r192098 - in stable/6: gnu/usr.bin/gdb/libgdb lib/libthr/thread lib/libthread_db lib/libthread_db/arch/alpha lib/libthread_db/arch/amd64 lib/libthread_db/arch/i386 lib/libthread_db/arch... X-BeenThere: svn-src-stable-6@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for only the 6-stable src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 18 May 2009 07:55:21 -0000 On Thu, 14 May 2009, Attilio Rao wrote: > Author: attilio > Date: Thu May 14 13:36:39 2009 > New Revision: 192098 > URL: http://svn.freebsd.org/changeset/base/192098 > > Log: > MFC libthread_db modifies until r181651: > - Introduce and use new functions pthread_{read, write}_* > - Move psaddr_t from void * to uintptr_t > - Fix some ABI mismatches > - Introduce WARN 6 for compilation The 6-STABLE amd64 build has been broken for four days now. Please fix, or back out this change. Robert N M Watson Computer Laboratory University of Cambridge > > Added: > stable/6/lib/libthread_db/libc_r_db.h (contents, props changed) > Modified: > stable/6/gnu/usr.bin/gdb/libgdb/fbsd-threads.c > stable/6/lib/libthr/thread/thr_event.c > stable/6/lib/libthread_db/Makefile > stable/6/lib/libthread_db/arch/alpha/libc_r_md.c > stable/6/lib/libthread_db/arch/alpha/libpthread_md.c > stable/6/lib/libthread_db/arch/amd64/libc_r_md.c > stable/6/lib/libthread_db/arch/amd64/libpthread_md.c > stable/6/lib/libthread_db/arch/i386/libc_r_md.c > stable/6/lib/libthread_db/arch/i386/libpthread_md.c > stable/6/lib/libthread_db/arch/ia64/libc_r_md.c > stable/6/lib/libthread_db/arch/ia64/libpthread_md.c > stable/6/lib/libthread_db/arch/sparc64/libc_r_md.c > stable/6/lib/libthread_db/arch/sparc64/libpthread_md.c > stable/6/lib/libthread_db/libc_r_db.c > stable/6/lib/libthread_db/libpthread_db.c > stable/6/lib/libthread_db/libpthread_db.h > stable/6/lib/libthread_db/libthr_db.c > stable/6/lib/libthread_db/thread_db.c > stable/6/lib/libthread_db/thread_db.h > stable/6/lib/libthread_db/thread_db_int.h > stable/6/sys/sys/procfs.h > > Modified: stable/6/gnu/usr.bin/gdb/libgdb/fbsd-threads.c > ============================================================================== > --- stable/6/gnu/usr.bin/gdb/libgdb/fbsd-threads.c Thu May 14 13:32:33 2009 (r192097) > +++ stable/6/gnu/usr.bin/gdb/libgdb/fbsd-threads.c Thu May 14 13:36:39 2009 (r192098) > @@ -710,7 +710,7 @@ check_event (ptid_t ptid) > error ("Cannot get thread event message: %s", > thread_db_err_str (err)); > } > - err = td_thr_get_info_p (msg.th_p, &ti); > + err = td_thr_get_info_p ((void*)(uintptr_t)msg.th_p, &ti); > if (err != TD_OK) > error ("Cannot get thread info: %s", thread_db_err_str (err)); > ptid = BUILD_THREAD (ti.ti_tid, GET_PID (ptid)); > @@ -720,7 +720,7 @@ check_event (ptid_t ptid) > /* We may already know about this thread, for instance when the > user has issued the `info threads' command before the SIGTRAP > for hitting the thread creation breakpoint was reported. */ > - attach_thread (ptid, msg.th_p, &ti, 1); > + attach_thread (ptid, (void *)(uintptr_t)msg.th_p, &ti, 1); > break; > case TD_DEATH: > if (!in_thread_list (ptid)) > @@ -1178,13 +1178,14 @@ fbsd_thread_pid_to_str (ptid_t ptid) > > if (ti.ti_lid != 0) > { > - snprintf (buf, sizeof (buf), "Thread %p (LWP %d)", > - th.th_thread, ti.ti_lid); > + snprintf (buf, sizeof (buf), "Thread %llx (LWP %d)", > + (unsigned long long)th.th_thread, ti.ti_lid); > } > else > { > - snprintf (buf, sizeof (buf), "Thread %p (%s)", > - th.th_thread, thread_db_state_str (ti.ti_state)); > + snprintf (buf, sizeof (buf), "Thread %llx (%s)", > + (unsigned long long)th.th_thread, > + thread_db_state_str (ti.ti_state)); > } > > return buf; > > Modified: stable/6/lib/libthr/thread/thr_event.c > ============================================================================== > --- stable/6/lib/libthr/thread/thr_event.c Thu May 14 13:32:33 2009 (r192097) > +++ stable/6/lib/libthr/thread/thr_event.c Thu May 14 13:36:39 2009 (r192098) > @@ -42,7 +42,7 @@ void > _thr_report_creation(struct pthread *curthread, struct pthread *newthread) > { > curthread->event_buf.event = TD_CREATE; > - curthread->event_buf.th_p = (td_thrhandle_t *)newthread; > + curthread->event_buf.th_p = (uintptr_t)newthread; > curthread->event_buf.data = 0; > THR_UMTX_LOCK(curthread, &_thr_event_lock); > _thread_last_event = curthread; > @@ -55,7 +55,7 @@ void > _thr_report_death(struct pthread *curthread) > { > curthread->event_buf.event = TD_DEATH; > - curthread->event_buf.th_p = (td_thrhandle_t *)curthread; > + curthread->event_buf.th_p = (uintptr_t)curthread; > curthread->event_buf.data = 0; > THR_UMTX_LOCK(curthread, &_thr_event_lock); > _thread_last_event = curthread; > > Modified: stable/6/lib/libthread_db/Makefile > ============================================================================== > --- stable/6/lib/libthread_db/Makefile Thu May 14 13:32:33 2009 (r192097) > +++ stable/6/lib/libthread_db/Makefile Thu May 14 13:36:39 2009 (r192098) > @@ -9,7 +9,7 @@ SRCS+= libpthread_db.c libpthread_md.c > SRCS+= libc_r_db.c libc_r_md.c > SRCS+= libthr_db.c > INCS= thread_db.h > -WARNS?= 1 > +WARNS?= 6 > > CFLAGS+=-I. -I${.CURDIR} > > > Modified: stable/6/lib/libthread_db/arch/alpha/libc_r_md.c > ============================================================================== > --- stable/6/lib/libthread_db/arch/alpha/libc_r_md.c Thu May 14 13:32:33 2009 (r192097) > +++ stable/6/lib/libthread_db/arch/alpha/libc_r_md.c Thu May 14 13:36:39 2009 (r192098) > @@ -27,8 +27,7 @@ > #include > __FBSDID("$FreeBSD$"); > > -#include > -#include > +#include "libc_r_db.h" > > void > libc_r_md_getgregs(jmp_buf jb, prgregset_t r) > @@ -68,6 +67,6 @@ libc_r_md_getgregs(jmp_buf jb, prgregset > } > > void > -libc_r_md_getfpregs(jmp_buf jb, prfpregset_t *r) > +libc_r_md_getfpregs(jmp_buf jb __unused, prfpregset_t *r __unused) > { > } > > Modified: stable/6/lib/libthread_db/arch/alpha/libpthread_md.c > ============================================================================== > --- stable/6/lib/libthread_db/arch/alpha/libpthread_md.c Thu May 14 13:32:33 2009 (r192097) > +++ stable/6/lib/libthread_db/arch/alpha/libpthread_md.c Thu May 14 13:36:39 2009 (r192098) > @@ -30,23 +30,25 @@ __FBSDID("$FreeBSD$"); > #include > #include > > +#include "libpthread_db.h" > + > void > -pt_reg_to_ucontext(const struct reg *r, ucontext_t *uc) > +pt_reg_to_ucontext(const struct reg *r __unused, ucontext_t *uc __unused) > { > } > > void > -pt_ucontext_to_reg(const ucontext_t *uc, struct reg *r) > +pt_ucontext_to_reg(const ucontext_t *uc __unused, struct reg *r __unused) > { > } > > void > -pt_fpreg_to_ucontext(const struct fpreg* r, ucontext_t *uc) > +pt_fpreg_to_ucontext(const struct fpreg* r __unused, ucontext_t *uc __unused) > { > } > > void > -pt_ucontext_to_fpreg(const ucontext_t *uc, struct fpreg *r) > +pt_ucontext_to_fpreg(const ucontext_t *uc __unused, struct fpreg *r __unused) > { > } > > @@ -56,7 +58,7 @@ pt_md_init(void) > } > > int > -pt_reg_sstep(struct reg *reg, int step) > +pt_reg_sstep(struct reg *reg __unused, int step __unused) > { > return (0); > } > > Modified: stable/6/lib/libthread_db/arch/amd64/libc_r_md.c > ============================================================================== > --- stable/6/lib/libthread_db/arch/amd64/libc_r_md.c Thu May 14 13:32:33 2009 (r192097) > +++ stable/6/lib/libthread_db/arch/amd64/libc_r_md.c Thu May 14 13:36:39 2009 (r192098) > @@ -27,15 +27,14 @@ > #include > __FBSDID("$FreeBSD$"); > > -#include > -#include > +#include "libc_r_db.h" > > void > -libc_r_md_getgregs(jmp_buf jb, prgregset_t *r) > +libc_r_md_getgregs(jmp_buf jb __unused, prgregset_t r __unused) > { > } > > void > -libc_r_md_getfpregs(jmp_buf jb, prfpregset_t *r) > +libc_r_md_getfpregs(jmp_buf jb __unused, prfpregset_t *r __unused) > { > } > > Modified: stable/6/lib/libthread_db/arch/amd64/libpthread_md.c > ============================================================================== > --- stable/6/lib/libthread_db/arch/amd64/libpthread_md.c Thu May 14 13:32:33 2009 (r192097) > +++ stable/6/lib/libthread_db/arch/amd64/libpthread_md.c Thu May 14 13:36:39 2009 (r192098) > @@ -29,8 +29,11 @@ > __FBSDID("$FreeBSD$"); > > #include > +#include > #include > > +#include "libpthread_db.h" > + > void > pt_reg_to_ucontext(const struct reg *r, ucontext_t *uc) > { > > Modified: stable/6/lib/libthread_db/arch/i386/libc_r_md.c > ============================================================================== > --- stable/6/lib/libthread_db/arch/i386/libc_r_md.c Thu May 14 13:32:33 2009 (r192097) > +++ stable/6/lib/libthread_db/arch/i386/libc_r_md.c Thu May 14 13:36:39 2009 (r192098) > @@ -27,8 +27,7 @@ > #include > __FBSDID("$FreeBSD$"); > > -#include > -#include > +#include "libc_r_db.h" > > void > libc_r_md_getgregs(jmp_buf jb, prgregset_t r) > @@ -43,6 +42,6 @@ libc_r_md_getgregs(jmp_buf jb, prgregset > } > > void > -libc_r_md_getfpregs(jmp_buf jb, prfpregset_t *r) > +libc_r_md_getfpregs(jmp_buf jb __unused, prfpregset_t *r __unused) > { > } > > Modified: stable/6/lib/libthread_db/arch/i386/libpthread_md.c > ============================================================================== > --- stable/6/lib/libthread_db/arch/i386/libpthread_md.c Thu May 14 13:32:33 2009 (r192097) > +++ stable/6/lib/libthread_db/arch/i386/libpthread_md.c Thu May 14 13:36:39 2009 (r192098) > @@ -27,11 +27,10 @@ > #include > __FBSDID("$FreeBSD$"); > > -#include > #include > -#include > -#include > #include > +#include > +#include > > #include "libpthread_db.h" > > > Modified: stable/6/lib/libthread_db/arch/ia64/libc_r_md.c > ============================================================================== > --- stable/6/lib/libthread_db/arch/ia64/libc_r_md.c Thu May 14 13:32:33 2009 (r192097) > +++ stable/6/lib/libthread_db/arch/ia64/libc_r_md.c Thu May 14 13:36:39 2009 (r192098) > @@ -27,15 +27,14 @@ > #include > __FBSDID("$FreeBSD$"); > > -#include > -#include > +#include "libc_r_db.h" > > void > -libc_r_md_getgregs(jmp_buf jb, prgregset_t *r) > +libc_r_md_getgregs(jmp_buf jb __unused, prgregset_t r __unused) > { > } > > void > -libc_r_md_getfpregs(jmp_buf jb, prfpregset_t *r) > +libc_r_md_getfpregs(jmp_buf jb __unused, prfpregset_t *r __unused) > { > } > > Modified: stable/6/lib/libthread_db/arch/ia64/libpthread_md.c > ============================================================================== > --- stable/6/lib/libthread_db/arch/ia64/libpthread_md.c Thu May 14 13:32:33 2009 (r192097) > +++ stable/6/lib/libthread_db/arch/ia64/libpthread_md.c Thu May 14 13:36:39 2009 (r192098) > @@ -28,25 +28,28 @@ > __FBSDID("$FreeBSD$"); > > #include > +#include > #include > > +#include "libpthread_db.h" > + > void > -pt_reg_to_ucontext(const struct reg *r, ucontext_t *uc) > +pt_reg_to_ucontext(const struct reg *r __unused, ucontext_t *uc __unused) > { > } > > void > -pt_ucontext_to_reg(const ucontext_t *uc, struct reg *r) > +pt_ucontext_to_reg(const ucontext_t *uc __unused, struct reg *r __unused) > { > } > > void > -pt_fpreg_to_ucontext(const struct fpreg* r, ucontext_t *uc) > +pt_fpreg_to_ucontext(const struct fpreg* r __unused, ucontext_t *uc __unused) > { > } > > void > -pt_ucontext_to_fpreg(const ucontext_t *uc, struct fpreg *r) > +pt_ucontext_to_fpreg(const ucontext_t *uc __unused, struct fpreg *r __unused) > { > } > > @@ -56,7 +59,7 @@ pt_md_init(void) > } > > int > -pt_reg_sstep(struct reg *reg, int step) > +pt_reg_sstep(struct reg *reg __unused, int step __unused) > { > return (0); > } > > Modified: stable/6/lib/libthread_db/arch/sparc64/libc_r_md.c > ============================================================================== > --- stable/6/lib/libthread_db/arch/sparc64/libc_r_md.c Thu May 14 13:32:33 2009 (r192097) > +++ stable/6/lib/libthread_db/arch/sparc64/libc_r_md.c Thu May 14 13:36:39 2009 (r192098) > @@ -27,15 +27,14 @@ > #include > __FBSDID("$FreeBSD$"); > > -#include > -#include > +#include "libc_r_db.h" > > void > -libc_r_md_getgregs(jmp_buf jb, prgregset_t *r) > +libc_r_md_getgregs(jmp_buf jb __unused, prgregset_t r __unused) > { > } > > void > -libc_r_md_getfpregs(jmp_buf jb, prfpregset_t *r) > +libc_r_md_getfpregs(jmp_buf jb __unused, prfpregset_t *r __unused) > { > } > > Modified: stable/6/lib/libthread_db/arch/sparc64/libpthread_md.c > ============================================================================== > --- stable/6/lib/libthread_db/arch/sparc64/libpthread_md.c Thu May 14 13:32:33 2009 (r192097) > +++ stable/6/lib/libthread_db/arch/sparc64/libpthread_md.c Thu May 14 13:36:39 2009 (r192098) > @@ -28,25 +28,28 @@ > __FBSDID("$FreeBSD$"); > > #include > +#include > #include > > +#include "libpthread_db.h" > + > void > -pt_reg_to_ucontext(const struct reg *r, ucontext_t *uc) > +pt_reg_to_ucontext(const struct reg *r __unused, ucontext_t *uc __unused) > { > } > > void > -pt_ucontext_to_reg(const ucontext_t *uc, struct reg *r) > +pt_ucontext_to_reg(const ucontext_t *uc __unused, struct reg *r __unused) > { > } > > void > -pt_fpreg_to_ucontext(const struct fpreg* r, ucontext_t *uc) > +pt_fpreg_to_ucontext(const struct fpreg* r __unused, ucontext_t *uc __unused) > { > } > > void > -pt_ucontext_to_fpreg(const ucontext_t *uc, struct fpreg *r) > +pt_ucontext_to_fpreg(const ucontext_t *uc __unused, struct fpreg *r __unused) > { > } > > @@ -56,7 +59,7 @@ pt_md_init(void) > } > > int > -pt_reg_sstep(struct reg *reg, int step) > +pt_reg_sstep(struct reg *reg __unused, int step __unused) > { > return (0); > } > > Modified: stable/6/lib/libthread_db/libc_r_db.c > ============================================================================== > --- stable/6/lib/libthread_db/libc_r_db.c Thu May 14 13:32:33 2009 (r192097) > +++ stable/6/lib/libthread_db/libc_r_db.c Thu May 14 13:36:39 2009 (r192098) > @@ -34,11 +34,9 @@ __FBSDID("$FreeBSD$"); > #include > #include > > +#include "libc_r_db.h" > #include "thread_db_int.h" > > -void libc_r_md_getfpregs(jmp_buf jb, prfpregset_t *); > -void libc_r_md_getgregs(jmp_buf jb, prgregset_t); > - > struct td_thragent { > TD_THRAGENT_FIELDS; > struct ps_prochandle *ta_ph; > @@ -51,13 +49,14 @@ struct td_thragent { > }; > > static td_err_e > -libc_r_db_init() > +libc_r_db_init(void) > { > return (TD_OK); > } > > static td_err_e > -libc_r_db_ta_clear_event(const td_thragent_t *ta, td_thr_events_t *ev) > +libc_r_db_ta_clear_event(const td_thragent_t *ta __unused, > + td_thr_events_t *ev __unused) > { > return (0); > } > @@ -70,27 +69,28 @@ libc_r_db_ta_delete(td_thragent_t *ta) > } > > static td_err_e > -libc_r_db_ta_event_addr(const td_thragent_t *ta, td_thr_events_e ev, > - td_notify_t *n) > +libc_r_db_ta_event_addr(const td_thragent_t *ta __unused, > + td_thr_events_e ev __unused, td_notify_t *n __unused) > { > return (TD_ERR); > } > > static td_err_e > -libc_r_db_ta_event_getmsg(const td_thragent_t *ta, td_event_msg_t *msg) > +libc_r_db_ta_event_getmsg(const td_thragent_t *ta __unused, > + td_event_msg_t *msg __unused) > { > return (TD_ERR); > } > > static td_err_e > -libc_r_db_ta_map_id2thr(const td_thragent_t *ta, thread_t tid, > - td_thrhandle_t *th) > +libc_r_db_ta_map_id2thr(const td_thragent_t *ta __unused, thread_t tid __unused, > + td_thrhandle_t *th __unused) > { > return (TD_ERR); > } > > static td_err_e > -libc_r_db_ta_map_lwp2thr(const td_thragent_t *ta, lwpid_t lwpid, > +libc_r_db_ta_map_lwp2thr(const td_thragent_t *ta, lwpid_t lwpid __unused, > td_thrhandle_t *th) > { > psaddr_t addr; > @@ -100,7 +100,7 @@ libc_r_db_ta_map_lwp2thr(const td_thrage > err = ps_pread(ta->ta_ph, ta->ta_thread_initial, &addr, sizeof(addr)); > if (err != PS_OK) > return (TD_ERR); > - if (addr == NULL) > + if (addr == 0) > return (TD_NOLWP); > err = ps_pread(ta->ta_ph, ta->ta_thread_run, &th->th_thread, > sizeof(psaddr_t)); > @@ -159,14 +159,16 @@ libc_r_db_ta_new(struct ps_prochandle *p > } > > static td_err_e > -libc_r_db_ta_set_event(const td_thragent_t *ta, td_thr_events_t *ev) > +libc_r_db_ta_set_event(const td_thragent_t *ta __unused, > + td_thr_events_t *ev __unused) > { > return (0); > } > > static td_err_e > libc_r_db_ta_thr_iter(const td_thragent_t *ta, td_thr_iter_f *cb, void *data, > - td_thr_state_e state, int pri, sigset_t *mask, unsigned int flags) > + td_thr_state_e state __unused, int pri __unused, sigset_t *mask __unused, > + unsigned int flags __unused) > { > td_thrhandle_t th; > psaddr_t addr; > @@ -178,10 +180,10 @@ libc_r_db_ta_thr_iter(const td_thragent_ > sizeof(th.th_thread)); > if (err != PS_OK) > return (TD_ERR); > - while (th.th_thread != NULL) { > + while (th.th_thread != 0) { > if (cb(&th, data) != 0) > return (TD_OK); > - addr = (psaddr_t)((uintptr_t)th.th_thread + ta->ta_ofs_next); > + addr = (psaddr_t)(th.th_thread + ta->ta_ofs_next); > err = ps_pread(ta->ta_ph, addr, &th.th_thread, > sizeof(th.th_thread)); > if (err != PS_OK) > @@ -191,19 +193,21 @@ libc_r_db_ta_thr_iter(const td_thragent_ > } > > static td_err_e > -libc_r_db_thr_clear_event(const td_thrhandle_t *th, td_thr_events_t *ev) > +libc_r_db_thr_clear_event(const td_thrhandle_t *th __unused, > + td_thr_events_t *ev __unused) > { > return (0); > } > > static td_err_e > -libc_r_db_thr_event_enable(const td_thrhandle_t *th, int oo) > +libc_r_db_thr_event_enable(const td_thrhandle_t *th __unused, int oo __unused) > { > return (0); > } > > static td_err_e > -libc_r_db_thr_event_getmsg(const td_thrhandle_t *th, td_event_msg_t *msg) > +libc_r_db_thr_event_getmsg(const td_thrhandle_t *th __unused, > + td_event_msg_t *msg __unused) > { > return (TD_ERR); > } > @@ -231,7 +235,8 @@ libc_r_db_thr_get_info(const td_thrhandl > > #ifdef __i386__ > static td_err_e > -libc_r_db_thr_getxmmregs(const td_thrhandle_t *th, char *fxsave) > +libc_r_db_thr_getxmmregs(const td_thrhandle_t *th __unused, > + char *fxsave __unused) > { > return (TD_NOFPREGS); > } > @@ -288,33 +293,37 @@ libc_r_db_thr_getgregs(const td_thrhandl > } > > static td_err_e > -libc_r_db_thr_set_event(const td_thrhandle_t *th, td_thr_events_t *ev) > +libc_r_db_thr_set_event(const td_thrhandle_t *th __unused, > + td_thr_events_t *ev __unused) > { > return (0); > } > > #ifdef __i386__ > static td_err_e > -libc_r_db_thr_setxmmregs(const td_thrhandle_t *th, const char *fxsave) > +libc_r_db_thr_setxmmregs(const td_thrhandle_t *th __unused, > + const char *fxsave __unused) > { > return (TD_NOFPREGS); > } > #endif > > static td_err_e > -libc_r_db_thr_setfpregs(const td_thrhandle_t *th, const prfpregset_t *r) > +libc_r_db_thr_setfpregs(const td_thrhandle_t *th __unused, > + const prfpregset_t *r __unused) > { > return (TD_ERR); > } > > static td_err_e > -libc_r_db_thr_setgregs(const td_thrhandle_t *th, const prgregset_t r) > +libc_r_db_thr_setgregs(const td_thrhandle_t *th __unused, > + const prgregset_t r __unused) > { > return (TD_ERR); > } > > static td_err_e > -libc_r_db_thr_validate(const td_thrhandle_t *th) > +libc_r_db_thr_validate(const td_thrhandle_t *th __unused) > { > return (TD_ERR); > } > > Added: stable/6/lib/libthread_db/libc_r_db.h > ============================================================================== > --- /dev/null 00:00:00 1970 (empty, because file is newly added) > +++ stable/6/lib/libthread_db/libc_r_db.h Thu May 14 13:36:39 2009 (r192098) > @@ -0,0 +1,41 @@ > +/* > + * Copyright (c) 2008 Sandvine Incorporated > + * All rights reserved. > + * > + * This software was developed by Attilio Rao for the SVOS project under > + * contract to Sandvine Incorporated. > + * > + * Redistribution and use in source and binary forms, with or without > + * modification, are permitted provided that the following conditions > + * are met: > + * 1. Redistributions of source code must retain the above copyright > + * notice, this list of conditions and the following disclaimer. > + * 2. Redistributions in binary form must reproduce the above copyright > + * notice, this list of conditions and the following disclaimer in the > + * documentation and/or other materials provided with the distribution. > + * > + * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND > + * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE > + * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE > + * ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE > + * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL > + * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS > + * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) > + * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT > + * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY > + * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF > + * SUCH DAMAGE. > + * > + * $FreeBSD$ > + */ > + > +#ifndef _LIBC_R_DB_H_ > +#define _LIBC_R_DB_H_ > + > +#include > +#include > + > +void libc_r_md_getgregs(jmp_buf jb, prgregset_t r); > +void libc_r_md_getfpregs(jmp_buf jb, prfpregset_t *r); > + > +#endif /* _LIBC_R_DB_H_ */ > > Modified: stable/6/lib/libthread_db/libpthread_db.c > ============================================================================== > --- stable/6/lib/libthread_db/libpthread_db.c Thu May 14 13:32:33 2009 (r192097) > +++ stable/6/lib/libthread_db/libpthread_db.c Thu May 14 13:36:39 2009 (r192098) > @@ -70,7 +70,7 @@ ps2td(int c) > } > > static long > -pt_map_thread(const td_thragent_t *const_ta, psaddr_t pt, int type) > +pt_map_thread(const td_thragent_t *const_ta, psaddr_t pt, enum pt_type type) > { > td_thragent_t *ta = __DECONST(td_thragent_t *, const_ta); > struct pt_map *new; > @@ -220,7 +220,6 @@ static td_err_e > pt_ta_map_id2thr(const td_thragent_t *ta, thread_t id, td_thrhandle_t *th) > { > prgregset_t gregs; > - TAILQ_HEAD(, pthread) thread_list; > psaddr_t pt, tcb_addr; > lwpid_t lwp; > int ret; > @@ -229,28 +228,24 @@ pt_ta_map_id2thr(const td_thragent_t *ta > > if (id < 0 || id >= ta->map_len || ta->map[id].type == PT_NONE) > return (TD_NOTHR); > - ret = ps_pread(ta->ph, ta->thread_list_addr, &thread_list, > - sizeof(thread_list)); > + > + ret = thr_pread_ptr(ta, ta->thread_list_addr, &pt); > if (ret != 0) > - return (P2T(ret)); > - pt = (psaddr_t)thread_list.tqh_first; > + return (TD_ERR); > if (ta->map[id].type == PT_LWP) { > /* > * if we are referencing a lwp, make sure it was not already > * mapped to user thread. > */ > while (pt != 0) { > - ret = ps_pread(ta->ph, > - pt + ta->thread_off_tcb, > - &tcb_addr, sizeof(tcb_addr)); > + ret = thr_pread_ptr(ta, pt + ta->thread_off_tcb, > + &tcb_addr); > if (ret != 0) > - return (P2T(ret)); > - ret = ps_pread(ta->ph, > - tcb_addr + ta->thread_off_tmbx + > - offsetof(struct kse_thr_mailbox, tm_lwp), > - &lwp, sizeof(lwp)); > + return (TD_ERR); > + ret = thr_pread_int(ta, tcb_addr + ta->thread_off_tmbx + > + offsetof(struct kse_thr_mailbox, tm_lwp), &lwp); > if (ret != 0) > - return (P2T(ret)); > + return (TD_ERR); > /* > * If the lwp was already mapped to userland thread, > * we shouldn't reference it directly in future. > @@ -260,11 +255,9 @@ pt_ta_map_id2thr(const td_thragent_t *ta > return (TD_NOTHR); > } > /* get next thread */ > - ret = ps_pread(ta->ph, > - pt + ta->thread_off_next, > - &pt, sizeof(pt)); > + ret = thr_pread_ptr(ta, pt + ta->thread_off_next, &pt); > if (ret != 0) > - return (P2T(ret)); > + return (TD_ERR); > } > /* check lwp */ > ret = ps_lgetregs(ta->ph, ta->map[id].lwp, gregs); > @@ -275,17 +268,14 @@ pt_ta_map_id2thr(const td_thragent_t *ta > } > } else { > while (pt != 0 && ta->map[id].thr != pt) { > - ret = ps_pread(ta->ph, > - pt + ta->thread_off_tcb, > - &tcb_addr, sizeof(tcb_addr)); > + ret = thr_pread_ptr(ta, pt + ta->thread_off_tcb, > + &tcb_addr); > if (ret != 0) > - return (P2T(ret)); > + return (TD_ERR); > /* get next thread */ > - ret = ps_pread(ta->ph, > - pt + ta->thread_off_next, > - &pt, sizeof(pt)); > + ret = thr_pread_ptr(ta, pt + ta->thread_off_next, &pt); > if (ret != 0) > - return (P2T(ret)); > + return (TD_ERR); > } > > if (pt == 0) { > @@ -303,29 +293,24 @@ pt_ta_map_id2thr(const td_thragent_t *ta > static td_err_e > pt_ta_map_lwp2thr(const td_thragent_t *ta, lwpid_t lwp, td_thrhandle_t *th) > { > - TAILQ_HEAD(, pthread) thread_list; > - psaddr_t pt, ptr; > - lwpid_t tmp_lwp; > + psaddr_t pt, tcb_addr; > + lwpid_t lwp1; > int ret; > - > + > TDBG_FUNC(); > > - ret = ps_pread(ta->ph, ta->thread_list_addr, &thread_list, > - sizeof(thread_list)); > + ret = thr_pread_ptr(ta, ta->thread_list_addr, &pt); > if (ret != 0) > - return (P2T(ret)); > - pt = (psaddr_t)thread_list.tqh_first; > + return (TD_ERR); > while (pt != 0) { > - ret = ps_pread(ta->ph, pt + ta->thread_off_tcb, > - &ptr, sizeof(ptr)); > + ret = thr_pread_ptr(ta, pt + ta->thread_off_tcb, &tcb_addr); > if (ret != 0) > - return (P2T(ret)); > - ptr += ta->thread_off_tmbx + > - offsetof(struct kse_thr_mailbox, tm_lwp); > - ret = ps_pread(ta->ph, ptr, &tmp_lwp, sizeof(lwpid_t)); > + return (TD_ERR); > + ret = thr_pread_int(ta, tcb_addr + ta->thread_off_tmbx + > + offsetof(struct kse_thr_mailbox, tm_lwp), &lwp1); > if (ret != 0) > - return (P2T(ret)); > - if (tmp_lwp == lwp) { > + return (TD_ERR); > + if (lwp1 == lwp) { > th->th_ta = ta; > th->th_tid = pt_map_thread(ta, pt, PT_USER); > if (th->th_tid == -1) > @@ -336,28 +321,23 @@ pt_ta_map_lwp2thr(const td_thragent_t *t > } > > /* get next thread */ > - ret = ps_pread(ta->ph, > - pt + ta->thread_off_next, > - &pt, sizeof(pt)); > + ret = thr_pread_ptr(ta, pt + ta->thread_off_next, &pt); > if (ret != 0) > - return (P2T(ret)); > + return (TD_ERR); > } > > return (TD_NOTHR); > } > > static td_err_e > -pt_ta_thr_iter(const td_thragent_t *ta, > - td_thr_iter_f *callback, void *cbdata_p, > - td_thr_state_e state, int ti_pri, > - sigset_t *ti_sigmask_p, > - unsigned int ti_user_flags) > +pt_ta_thr_iter(const td_thragent_t *ta, td_thr_iter_f *callback, > + void *cbdata_p, td_thr_state_e state __unused, int ti_pri __unused, > + sigset_t *ti_sigmask_p __unused, unsigned int ti_user_flags __unused) > { > - TAILQ_HEAD(, pthread) thread_list; > td_thrhandle_t th; > psaddr_t pt; > ps_err_e pserr; > - int activated; > + int activated, ret; > > TDBG_FUNC(); > > @@ -368,11 +348,9 @@ pt_ta_thr_iter(const td_thragent_t *ta, > if (!activated) > return (TD_OK); > > - pserr = ps_pread(ta->ph, ta->thread_list_addr, &thread_list, > - sizeof(thread_list)); > - if (pserr != 0) > - return (P2T(pserr)); > - pt = (psaddr_t)thread_list.tqh_first; > + ret = thr_pread_ptr(ta, ta->thread_list_addr, &pt); > + if (ret != 0) > + return (TD_ERR); > while (pt != 0) { > th.th_ta = ta; > th.th_tid = pt_map_thread(ta, pt, PT_USER); > @@ -383,11 +361,9 @@ pt_ta_thr_iter(const td_thragent_t *ta, > if ((*callback)(&th, cbdata_p)) > return (TD_DBERR); > /* get next thread */ > - pserr = ps_pread(ta->ph, > - pt + ta->thread_off_next, &pt, > - sizeof(pt)); > - if (pserr != PS_OK) > - return (P2T(pserr)); > + ret = thr_pread_ptr(ta, pt + ta->thread_off_next, &pt); > + if (ret != 0) > + return (TD_ERR); > } > return (TD_OK); > } > @@ -395,7 +371,7 @@ pt_ta_thr_iter(const td_thragent_t *ta, > static td_err_e > pt_ta_tsd_iter(const td_thragent_t *ta, td_key_iter_f *ki, void *arg) > { > - char *keytable; > + void *keytable; > void *destructor; > int i, ret, allocated; > > @@ -411,10 +387,10 @@ pt_ta_tsd_iter(const td_thragent_t *ta, > return (P2T(ret)); > } > for (i = 0; i < ta->thread_max_keys; i++) { > - allocated = *(int *)(keytable + i * ta->thread_size_key + > - ta->thread_off_key_allocated); > - destructor = *(void **)(keytable + i * ta->thread_size_key + > - ta->thread_off_key_destructor); > + allocated = *(int *)(void *)((uintptr_t)keytable + > + i * ta->thread_size_key + ta->thread_off_key_allocated); > + destructor = *(void **)(void *)((uintptr_t)keytable + > + i * ta->thread_size_key + ta->thread_off_key_destructor); > if (allocated) { > ret = (ki)(i, destructor, arg); > if (ret != 0) { > @@ -428,28 +404,32 @@ pt_ta_tsd_iter(const td_thragent_t *ta, > } > > static td_err_e > -pt_ta_event_addr(const td_thragent_t *ta, td_event_e event, td_notify_t *ptr) > +pt_ta_event_addr(const td_thragent_t *ta __unused, td_event_e event __unused, > + td_notify_t *ptr __unused) > { > TDBG_FUNC(); > return (TD_ERR); > } > > static td_err_e > -pt_ta_set_event(const td_thragent_t *ta, td_thr_events_t *events) > +pt_ta_set_event(const td_thragent_t *ta __unused, > + td_thr_events_t *events __unused) > { > TDBG_FUNC(); > return (0); > } > > static td_err_e > -pt_ta_clear_event(const td_thragent_t *ta, td_thr_events_t *events) > +pt_ta_clear_event(const td_thragent_t *ta __unused, > + td_thr_events_t *events __unused) > { > TDBG_FUNC(); > return (0); > } > > static td_err_e > -pt_ta_event_getmsg(const td_thragent_t *ta, td_event_msg_t *msg) > +pt_ta_event_getmsg(const td_thragent_t *ta __unused, > + td_event_msg_t *msg __unused) > { > TDBG_FUNC(); > return (TD_NOMSG); > @@ -458,7 +438,7 @@ pt_ta_event_getmsg(const td_thragent_t * > static td_err_e > pt_dbsuspend(const td_thrhandle_t *th, int suspend) > { > - td_thragent_t *ta = (td_thragent_t *)th->th_ta; > + const td_thragent_t *ta = th->th_ta; > psaddr_t tcb_addr, tmbx_addr, ptr; > lwpid_t lwp; > uint32_t dflags; > @@ -952,28 +932,31 @@ pt_thr_setgregs(const td_thrhandle_t *th > } > > static td_err_e > -pt_thr_event_enable(const td_thrhandle_t *th, int en) > +pt_thr_event_enable(const td_thrhandle_t *th __unused, int en __unused) > { > TDBG_FUNC(); > return (0); > } > > static td_err_e > -pt_thr_set_event(const td_thrhandle_t *th, td_thr_events_t *setp) > +pt_thr_set_event(const td_thrhandle_t *th __unused, > + td_thr_events_t *setp __unused) > { > TDBG_FUNC(); > return (0); > } > > static td_err_e > -pt_thr_clear_event(const td_thrhandle_t *th, td_thr_events_t *setp) > +pt_thr_clear_event(const td_thrhandle_t *th __unused, > + td_thr_events_t *setp __unused) > { > TDBG_FUNC(); > return (0); > } > > static td_err_e > -pt_thr_event_getmsg(const td_thrhandle_t *th, td_event_msg_t *msg) > +pt_thr_event_getmsg(const td_thrhandle_t *th __unused, > + td_event_msg_t *msg __unused) > { > TDBG_FUNC(); > return (TD_NOMSG); > @@ -1074,17 +1057,16 @@ pt_validate(const td_thrhandle_t *th) > return (TD_OK); > } > > -td_err_e > -pt_thr_tls_get_addr(const td_thrhandle_t *th, void *_linkmap, size_t offset, > - void **address) > +static td_err_e > +pt_thr_tls_get_addr(const td_thrhandle_t *th, psaddr_t _linkmap, size_t offset, > + psaddr_t *address) > { > - char *obj_entry; > const td_thragent_t *ta = th->th_ta; > - psaddr_t tcb_addr, *dtv_addr; > + psaddr_t dtv_addr, obj_entry, tcb_addr; > int tls_index, ret; > > /* linkmap is a member of Obj_Entry */ > - obj_entry = (char *)_linkmap - ta->thread_off_linkmap; > + obj_entry = _linkmap - ta->thread_off_linkmap; > > /* get tlsindex of the object file */ > ret = ps_pread(ta->ph, > @@ -1106,8 +1088,8 @@ pt_thr_tls_get_addr(const td_thrhandle_t > if (ret != 0) > return (P2T(ret)); > /* now get the object's tls block base address */ > - ret = ps_pread(ta->ph, &dtv_addr[tls_index+1], address, > - sizeof(*address)); > + ret = ps_pread(ta->ph, dtv_addr + sizeof (void *) * (tls_index + 1), > + address, sizeof(*address)); > if (ret != 0) > return (P2T(ret)); > > > Modified: stable/6/lib/libthread_db/libpthread_db.h > ============================================================================== > --- stable/6/lib/libthread_db/libpthread_db.h Thu May 14 13:32:33 2009 (r192097) > +++ stable/6/lib/libthread_db/libpthread_db.h Thu May 14 13:36:39 2009 (r192098) > @@ -34,13 +34,14 @@ > > #include "thread_db_int.h" > > -struct pt_map { > - enum { > - PT_NONE, > - PT_USER, > - PT_LWP > - } type; > +enum pt_type { > + PT_NONE, > + PT_USER, > + PT_LWP > +}; > > +struct pt_map { > + enum pt_type type; > union { > lwpid_t lwp; > psaddr_t thr; > > Modified: stable/6/lib/libthread_db/libthr_db.c > ============================================================================== > --- stable/6/lib/libthread_db/libthr_db.c Thu May 14 13:32:33 2009 (r192097) > +++ stable/6/lib/libthread_db/libthr_db.c Thu May 14 13:36:39 2009 (r192098) > @@ -201,36 +201,30 @@ pt_ta_delete(td_thragent_t *ta) > static td_err_e > pt_ta_map_id2thr(const td_thragent_t *ta, thread_t id, td_thrhandle_t *th) > { > - TAILQ_HEAD(, pthread) thread_list; > psaddr_t pt; > - long lwp; > + int32_t lwp; > int ret; > > TDBG_FUNC(); > > if (id == 0) > return (TD_NOTHR); > - ret = ps_pread(ta->ph, ta->thread_list_addr, &thread_list, > - sizeof(thread_list)); > + ret = thr_pread_ptr(ta, ta->thread_list_addr, &pt); > if (ret != 0) > - return (P2T(ret)); > + return (TD_ERR); > /* Iterate through thread list to find pthread */ > - pt = (psaddr_t)thread_list.tqh_first; > - while (pt != NULL) { > - ret = ps_pread(ta->ph, pt + ta->thread_off_tid, > - &lwp, sizeof(lwp)); > + while (pt != 0) { > + ret = thr_pread_int(ta, pt + ta->thread_off_tid, &lwp); > if (ret != 0) > - return (P2T(ret)); > + return (TD_ERR); > if (lwp == id) > break; > /* get next thread */ > - ret = ps_pread(ta->ph, > - pt + ta->thread_off_next, > - &pt, sizeof(pt)); > + ret = thr_pread_ptr(ta, pt + ta->thread_off_next, &pt); > if (ret != 0) > - return (P2T(ret)); > + return (TD_ERR); > } > - if (pt == NULL) > + if (pt == 0) > return (TD_NOTHR); > th->th_ta = ta; > th->th_tid = id; > @@ -245,30 +239,24 @@ pt_ta_map_lwp2thr(const td_thragent_t *t > } > > static td_err_e > -pt_ta_thr_iter(const td_thragent_t *ta, > - td_thr_iter_f *callback, void *cbdata_p, > - td_thr_state_e state, int ti_pri, > - sigset_t *ti_sigmask_p, > - unsigned int ti_user_flags) > +pt_ta_thr_iter(const td_thragent_t *ta, td_thr_iter_f *callback, > + void *cbdata_p, td_thr_state_e state __unused, int ti_pri __unused, > + sigset_t *ti_sigmask_p __unused, unsigned int ti_user_flags __unused) > { > - TAILQ_HEAD(, pthread) thread_list; > td_thrhandle_t th; > psaddr_t pt; > - long lwp; > + int32_t lwp; > int ret; > > TDBG_FUNC(); > > - ret = ps_pread(ta->ph, ta->thread_list_addr, &thread_list, > - sizeof(thread_list)); > + ret = thr_pread_ptr(ta, ta->thread_list_addr, &pt); > if (ret != 0) > - return (P2T(ret)); > > *** DIFF OUTPUT TRUNCATED AT 1000 LINES *** > From owner-svn-src-stable-6@FreeBSD.ORG Mon May 18 14:05:33 2009 Return-Path: Delivered-To: svn-src-stable-6@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 16DD21065673; Mon, 18 May 2009 14:05:33 +0000 (UTC) (envelope-from asmrookie@gmail.com) Received: from mail-fx0-f216.google.com (mail-fx0-f216.google.com [209.85.220.216]) by mx1.freebsd.org (Postfix) with ESMTP id D40118FC0C; Mon, 18 May 2009 14:05:31 +0000 (UTC) (envelope-from asmrookie@gmail.com) Received: by fxm12 with SMTP id 12so3253032fxm.43 for ; Mon, 18 May 2009 07:05:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:sender:received:in-reply-to :references:date:x-google-sender-auth:message-id:subject:from:to:cc :content-type:content-transfer-encoding; bh=cRdYSi5ycKWVTdXkTRjhWz0saibalX4pkWlK35G9qDg=; b=a7anTKg0bW0xJqFAvmLN2hRg2BAZ6IgDS6Xw2G155TKqKGI8r4upwl2V3uzNfT9Gr2 2Sd+1BlaHBOgBlOWCkcVRw1Q8cKMcEr5EyPGMHOUrsU5em4+MWN32LpMuTtYVcMyKQa7 SsjVgpD0MHCNJpLxWNi8vE2PE3SIQ1fuY3Ygk= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type :content-transfer-encoding; b=N8xdkk7cqNyQqoQejllkcTw6YN1NmRtM4D0oAN1oAMp5j0H+kPzEPeJJR81xRpEUOW ooNYuZfT+GbZu6CErIGiHc1QS86uDDI+mtADRScHP0RrHPaJt3iJj/ia0v/f3BQDrJxv JoM/sQgOFNTFbBifFii1z93Bib5KfxTKb+tf8= MIME-Version: 1.0 Sender: asmrookie@gmail.com Received: by 10.223.126.10 with SMTP id a10mr4368595fas.17.1242655530846; Mon, 18 May 2009 07:05:30 -0700 (PDT) In-Reply-To: References: <200905141336.n4EDadT4092402@svn.freebsd.org> Date: Mon, 18 May 2009 16:05:28 +0200 X-Google-Sender-Auth: 99f50dd720a3b761 Message-ID: <3bbf2fe10905180705k4cb91677l2231746b1c6f78d2@mail.gmail.com> From: Attilio Rao To: Robert Watson Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Cc: svn-src-stable@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, svn-src-stable-6@freebsd.org Subject: Re: svn commit: r192098 - in stable/6: gnu/usr.bin/gdb/libgdb lib/libthr/thread lib/libthread_db lib/libthread_db/arch/alpha lib/libthread_db/arch/amd64 lib/libthread_db/arch/i386 lib/libthread_db/arch... X-BeenThere: svn-src-stable-6@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for only the 6-stable src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 18 May 2009 14:05:33 -0000 2009/5/18 Robert Watson : > On Thu, 14 May 2009, Attilio Rao wrote: > >> Author: attilio >> Date: Thu May 14 13:36:39 2009 >> New Revision: 192098 >> URL: http://svn.freebsd.org/changeset/base/192098 >> >> Log: >> =C2=A0MFC libthread_db modifies until r181651: >> =C2=A0- Introduce and use new functions pthread_{read, write}_* >> =C2=A0- Move psaddr_t from void * to uintptr_t >> =C2=A0- Fix some ABI mismatches >> =C2=A0- Introduce WARN 6 for compilation > > The 6-STABLE amd64 build has been broken for four days now. =C2=A0Please = fix, or > back out this change. Yes, sorry, I just saw the report. I'm not sure what happened as this part was supposed to be tested, but in any case, I'm going to fix. Thanks, Attilio --=20 Peace can only be achieved by understanding - A. Einstein From owner-svn-src-stable-6@FreeBSD.ORG Mon May 18 14:46:38 2009 Return-Path: Delivered-To: svn-src-stable-6@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 6C3B4106567B; Mon, 18 May 2009 14:46:38 +0000 (UTC) (envelope-from rwatson@FreeBSD.org) Received: from cyrus.watson.org (cyrus.watson.org [65.122.17.42]) by mx1.freebsd.org (Postfix) with ESMTP id 370948FC15; Mon, 18 May 2009 14:46:38 +0000 (UTC) (envelope-from rwatson@FreeBSD.org) Received: from fledge.watson.org (fledge.watson.org [65.122.17.41]) by cyrus.watson.org (Postfix) with ESMTPS id DDCD046B17; Mon, 18 May 2009 10:46:37 -0400 (EDT) Date: Mon, 18 May 2009 15:46:37 +0100 (BST) From: Robert Watson X-X-Sender: robert@fledge.watson.org To: Attilio Rao In-Reply-To: <3bbf2fe10905180705k4cb91677l2231746b1c6f78d2@mail.gmail.com> Message-ID: References: <200905141336.n4EDadT4092402@svn.freebsd.org> <3bbf2fe10905180705k4cb91677l2231746b1c6f78d2@mail.gmail.com> User-Agent: Alpine 2.00 (BSF 1167 2008-08-23) MIME-Version: 1.0 Content-Type: MULTIPART/MIXED; BOUNDARY="621616949-1212259834-1242657997=:24449" Cc: svn-src-stable@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, svn-src-stable-6@freebsd.org Subject: Re: svn commit: r192098 - in stable/6: gnu/usr.bin/gdb/libgdb lib/libthr/thread lib/libthread_db lib/libthread_db/arch/alpha lib/libthread_db/arch/amd64 lib/libthread_db/arch/i386 lib/libthread_db/arch... X-BeenThere: svn-src-stable-6@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for only the 6-stable src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 18 May 2009 14:46:39 -0000 This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. --621616949-1212259834-1242657997=:24449 Content-Type: TEXT/PLAIN; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8BIT On Mon, 18 May 2009, Attilio Rao wrote: >>>  MFC libthread_db modifies until r181651: >>>  - Introduce and use new functions pthread_{read, write}_* >>>  - Move psaddr_t from void * to uintptr_t >>>  - Fix some ABI mismatches >>>  - Introduce WARN 6 for compilation >> >> The 6-STABLE amd64 build has been broken for four days now.  Please fix, or >> back out this change. > > Yes, sorry, I just saw the report. I'm not sure what happened as this part > was supposed to be tested, but in any case, I'm going to fix. As a general rule, developers should not commit to stable/, releng/, or release/ branches without being subscribed to the freebsd-stable mailing list -- not just to catch tinderbox results, but also to field reports of problems, feedback on changes, etc. Robert N M Watson Computer Laboratory University of Cambridge --621616949-1212259834-1242657997=:24449-- From owner-svn-src-stable-6@FreeBSD.ORG Mon May 18 18:04:43 2009 Return-Path: Delivered-To: svn-src-stable-6@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 3898E106566C; Mon, 18 May 2009 18:04:43 +0000 (UTC) (envelope-from attilio@FreeBSD.org) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:4f8:fff6::2c]) by mx1.freebsd.org (Postfix) with ESMTP id 25CCD8FC14; Mon, 18 May 2009 18:04:43 +0000 (UTC) (envelope-from attilio@FreeBSD.org) Received: from svn.freebsd.org (localhost [127.0.0.1]) by svn.freebsd.org (8.14.3/8.14.3) with ESMTP id n4II4hST070394; Mon, 18 May 2009 18:04:43 GMT (envelope-from attilio@svn.freebsd.org) Received: (from attilio@localhost) by svn.freebsd.org (8.14.3/8.14.3/Submit) id n4II4hxl070393; Mon, 18 May 2009 18:04:43 GMT (envelope-from attilio@svn.freebsd.org) Message-Id: <200905181804.n4II4hxl070393@svn.freebsd.org> From: Attilio Rao Date: Mon, 18 May 2009 18:04:43 +0000 (UTC) To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-6@freebsd.org X-SVN-Group: stable-6 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Cc: Subject: svn commit: r192320 - stable/6/lib/libthread_db/arch/amd64 X-BeenThere: svn-src-stable-6@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for only the 6-stable src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 18 May 2009 18:04:43 -0000 Author: attilio Date: Mon May 18 18:04:42 2009 New Revision: 192320 URL: http://svn.freebsd.org/changeset/base/192320 Log: Fix the world breakage, on amd64, introduced by the last libthread_db commit. Reported by: many Sponsored by: Sandvine Incorporated Pointy hat to: attilio Modified: stable/6/lib/libthread_db/arch/amd64/libpthread_md.c Modified: stable/6/lib/libthread_db/arch/amd64/libpthread_md.c ============================================================================== --- stable/6/lib/libthread_db/arch/amd64/libpthread_md.c Mon May 18 17:57:33 2009 (r192319) +++ stable/6/lib/libthread_db/arch/amd64/libpthread_md.c Mon May 18 18:04:42 2009 (r192320) @@ -29,6 +29,7 @@ __FBSDID("$FreeBSD$"); #include +#include #include #include From owner-svn-src-stable-6@FreeBSD.ORG Mon May 18 18:09:26 2009 Return-Path: Delivered-To: svn-src-stable-6@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id E6FBE1065673; Mon, 18 May 2009 18:09:26 +0000 (UTC) (envelope-from asmrookie@gmail.com) Received: from mail-bw0-f213.google.com (mail-bw0-f213.google.com [209.85.218.213]) by mx1.freebsd.org (Postfix) with ESMTP id ABD538FC19; Mon, 18 May 2009 18:09:25 +0000 (UTC) (envelope-from asmrookie@gmail.com) Received: by bwz9 with SMTP id 9so3356737bwz.43 for ; Mon, 18 May 2009 11:09:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:sender:received:in-reply-to :references:date:x-google-sender-auth:message-id:subject:from:to:cc :content-type:content-transfer-encoding; bh=f7+X+R8ZT0q2KrKKPwTM0FtvsDeIe8V6JtzNX9koTB8=; b=DN/mLIsR1Cy77nIvPXa4qAVq7261LNrmowIbKTCCNmTUT4gEmjyxqHUQUI7XbxXtJY oCerRr0CTL95gKVHE3163w+QvVxazkYbxL2VO62uXylpe8ObdzQO4nEQf8btux/79ETO B8QIzgLMIEIU8yRWHLp8LF08YYVv43iq4QHes= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type :content-transfer-encoding; b=L0lM6ekUa7dyMSBFPPQL2pixTVYfTiKP5iKmQqDUja9khOPhu6ndm2ZuaoQP5Ac9YH 1YQ5rlPEPn8xAkf2zS3qF2cPWONit3gPt/9QZ3aIXz/NOo0heDrIlbUnml8YCQe2Xk71 quBWZl2goXSJDY7rllU51TRrY6MJoh+SQS894= MIME-Version: 1.0 Sender: asmrookie@gmail.com Received: by 10.223.126.66 with SMTP id b2mr4535302fas.18.1242670164592; Mon, 18 May 2009 11:09:24 -0700 (PDT) In-Reply-To: References: <200905141336.n4EDadT4092402@svn.freebsd.org> <3bbf2fe10905180705k4cb91677l2231746b1c6f78d2@mail.gmail.com> Date: Mon, 18 May 2009 20:09:24 +0200 X-Google-Sender-Auth: c88b020f3fa74730 Message-ID: <3bbf2fe10905181109p1d1cf471i694115707aed4322@mail.gmail.com> From: Attilio Rao To: Robert Watson Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Cc: svn-src-stable@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, svn-src-stable-6@freebsd.org Subject: Re: svn commit: r192098 - in stable/6: gnu/usr.bin/gdb/libgdb lib/libthr/thread lib/libthread_db lib/libthread_db/arch/alpha lib/libthread_db/arch/amd64 lib/libthread_db/arch/i386 lib/libthread_db/arch... X-BeenThere: svn-src-stable-6@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for only the 6-stable src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 18 May 2009 18:09:27 -0000 2009/5/18 Robert Watson : > > On Mon, 18 May 2009, Attilio Rao wrote: > >>>> =C2=A0MFC libthread_db modifies until r181651: >>>> =C2=A0- Introduce and use new functions pthread_{read, write}_* >>>> =C2=A0- Move psaddr_t from void * to uintptr_t >>>> =C2=A0- Fix some ABI mismatches >>>> =C2=A0- Introduce WARN 6 for compilation >>> >>> The 6-STABLE amd64 build has been broken for four days now. =C2=A0Pleas= e fix, >>> or back out this change. >> >> Yes, sorry, I just saw the report. I'm not sure what happened as this pa= rt >> was supposed to be tested, but in any case, I'm going to fix. > > As a general rule, developers should not commit to stable/, releng/, or > release/ branches without being subscribed to the freebsd-stable mailing > list -- not just to catch tinderbox results, but also to field reports of > problems, feedback on changes, etc. Code fixed, sorry for the mis-service and the late response. Attilio --=20 Peace can only be achieved by understanding - A. Einstein From owner-svn-src-stable-6@FreeBSD.ORG Mon May 18 21:58:58 2009 Return-Path: Delivered-To: svn-src-stable-6@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 54F911065676; Mon, 18 May 2009 21:58:58 +0000 (UTC) (envelope-from jhb@FreeBSD.org) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:4f8:fff6::2c]) by mx1.freebsd.org (Postfix) with ESMTP id 42AC08FC2C; Mon, 18 May 2009 21:58:58 +0000 (UTC) (envelope-from jhb@FreeBSD.org) Received: from svn.freebsd.org (localhost [127.0.0.1]) by svn.freebsd.org (8.14.3/8.14.3) with ESMTP id n4ILww3W076122; Mon, 18 May 2009 21:58:58 GMT (envelope-from jhb@svn.freebsd.org) Received: (from jhb@localhost) by svn.freebsd.org (8.14.3/8.14.3/Submit) id n4ILwwLI076121; Mon, 18 May 2009 21:58:58 GMT (envelope-from jhb@svn.freebsd.org) Message-Id: <200905182158.n4ILwwLI076121@svn.freebsd.org> From: John Baldwin Date: Mon, 18 May 2009 21:58:58 +0000 (UTC) To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-6@freebsd.org X-SVN-Group: stable-6 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Cc: Subject: svn commit: r192346 - in stable/6/sys: . contrib/pf dev/cxgb net X-BeenThere: svn-src-stable-6@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for only the 6-stable src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 18 May 2009 21:58:59 -0000 Author: jhb Date: Mon May 18 21:58:57 2009 New Revision: 192346 URL: http://svn.freebsd.org/changeset/base/192346 Log: MFC: Remove dependency of vlan.ko on miibus.ko. Modified: stable/6/sys/ (props changed) stable/6/sys/contrib/pf/ (props changed) stable/6/sys/dev/cxgb/ (props changed) stable/6/sys/net/if_vlan.c Modified: stable/6/sys/net/if_vlan.c ============================================================================== --- stable/6/sys/net/if_vlan.c Mon May 18 21:58:21 2009 (r192345) +++ stable/6/sys/net/if_vlan.c Mon May 18 21:58:57 2009 (r192346) @@ -315,7 +315,6 @@ static moduledata_t vlan_mod = { }; DECLARE_MODULE(if_vlan, vlan_mod, SI_SUB_PSEUDO, SI_ORDER_ANY); -MODULE_DEPEND(if_vlan, miibus, 1, 1, 1); static struct ifnet * vlan_clone_match_ethertag(struct if_clone *ifc, const char *name, int *tag) From owner-svn-src-stable-6@FreeBSD.ORG Tue May 19 19:27:07 2009 Return-Path: Delivered-To: svn-src-stable-6@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id E9099106566B; Tue, 19 May 2009 19:27:07 +0000 (UTC) (envelope-from scottl@FreeBSD.org) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:4f8:fff6::2c]) by mx1.freebsd.org (Postfix) with ESMTP id D67C48FC19; Tue, 19 May 2009 19:27:07 +0000 (UTC) (envelope-from scottl@FreeBSD.org) Received: from svn.freebsd.org (localhost [127.0.0.1]) by svn.freebsd.org (8.14.3/8.14.3) with ESMTP id n4JJR7xs007698; Tue, 19 May 2009 19:27:07 GMT (envelope-from scottl@svn.freebsd.org) Received: (from scottl@localhost) by svn.freebsd.org (8.14.3/8.14.3/Submit) id n4JJR73L007697; Tue, 19 May 2009 19:27:07 GMT (envelope-from scottl@svn.freebsd.org) Message-Id: <200905191927.n4JJR73L007697@svn.freebsd.org> From: Scott Long Date: Tue, 19 May 2009 19:27:07 +0000 (UTC) To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-6@freebsd.org X-SVN-Group: stable-6 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Cc: Subject: svn commit: r192405 - in stable/6/sys: . contrib/pf dev/ciss dev/cxgb X-BeenThere: svn-src-stable-6@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for only the 6-stable src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 19 May 2009 19:27:08 -0000 Author: scottl Date: Tue May 19 19:27:07 2009 New Revision: 192405 URL: http://svn.freebsd.org/changeset/base/192405 Log: Merge PCI IDs from vendor Modified: stable/6/sys/ (props changed) stable/6/sys/contrib/pf/ (props changed) stable/6/sys/dev/ciss/ciss.c stable/6/sys/dev/cxgb/ (props changed) Modified: stable/6/sys/dev/ciss/ciss.c ============================================================================== --- stable/6/sys/dev/ciss/ciss.c Tue May 19 19:21:25 2009 (r192404) +++ stable/6/sys/dev/ciss/ciss.c Tue May 19 19:27:07 2009 (r192405) @@ -287,17 +287,20 @@ static struct { 0x103C, 0x3234, CISS_BOARD_SA5, "HP Smart Array P400" }, { 0x103C, 0x3235, CISS_BOARD_SA5, "HP Smart Array P400i" }, { 0x103C, 0x3236, CISS_BOARD_SA5, "HP Smart Array" }, - { 0x103C, 0x3237, CISS_BOARD_SA5, "HP Smart Array" }, + { 0x103C, 0x3237, CISS_BOARD_SA5, "HP Smart Array E500" }, { 0x103C, 0x3238, CISS_BOARD_SA5, "HP Smart Array" }, { 0x103C, 0x3239, CISS_BOARD_SA5, "HP Smart Array" }, { 0x103C, 0x323A, CISS_BOARD_SA5, "HP Smart Array" }, { 0x103C, 0x323B, CISS_BOARD_SA5, "HP Smart Array" }, { 0x103C, 0x323C, CISS_BOARD_SA5, "HP Smart Array" }, + { 0x103C, 0x323D, CISS_BOARD_SA5, "HP Smart Array P700m" }, { 0x103C, 0x3241, CISS_BOARD_SA5, "HP Smart Array P212" }, { 0x103C, 0x3243, CISS_BOARD_SA5, "HP Smart Array P410" }, { 0x103C, 0x3245, CISS_BOARD_SA5, "HP Smart Array P410i" }, { 0x103C, 0x3247, CISS_BOARD_SA5, "HP Smart Array P411" }, { 0x103C, 0x3249, CISS_BOARD_SA5, "HP Smart Array P812" }, + { 0x103C, 0x324A, CISS_BOARD_SA5, "HP Smart Array P712m" }, + { 0x103C, 0x324B, CISS_BOARD_SA5, "HP Smart Array" }, { 0, 0, 0, NULL } }; From owner-svn-src-stable-6@FreeBSD.ORG Sat May 23 10:45:28 2009 Return-Path: Delivered-To: svn-src-stable-6@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 94B83106564A; Sat, 23 May 2009 10:45:28 +0000 (UTC) (envelope-from brueffer@FreeBSD.org) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:4f8:fff6::2c]) by mx1.freebsd.org (Postfix) with ESMTP id 820448FC15; Sat, 23 May 2009 10:45:28 +0000 (UTC) (envelope-from brueffer@FreeBSD.org) Received: from svn.freebsd.org (localhost [127.0.0.1]) by svn.freebsd.org (8.14.3/8.14.3) with ESMTP id n4NAjSil053616; Sat, 23 May 2009 10:45:28 GMT (envelope-from brueffer@svn.freebsd.org) Received: (from brueffer@localhost) by svn.freebsd.org (8.14.3/8.14.3/Submit) id n4NAjSXx053615; Sat, 23 May 2009 10:45:28 GMT (envelope-from brueffer@svn.freebsd.org) Message-Id: <200905231045.n4NAjSXx053615@svn.freebsd.org> From: Christian Brueffer Date: Sat, 23 May 2009 10:45:28 +0000 (UTC) To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-6@freebsd.org X-SVN-Group: stable-6 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Cc: Subject: svn commit: r192633 - in stable/6/lib/libc: . inet posix1e sys X-BeenThere: svn-src-stable-6@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for only the 6-stable src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 23 May 2009 10:45:29 -0000 Author: brueffer Date: Sat May 23 10:45:28 2009 New Revision: 192633 URL: http://svn.freebsd.org/changeset/base/192633 Log: MFC: r192410 Since audit(4) isn't based on posix1e, remove the commented out audit.h header, xref libbsm(3). Modified: stable/6/lib/libc/ (props changed) stable/6/lib/libc/inet/inet_net_pton.c (props changed) stable/6/lib/libc/posix1e/posix1e.3 stable/6/lib/libc/sys/ (props changed) Modified: stable/6/lib/libc/posix1e/posix1e.3 ============================================================================== --- stable/6/lib/libc/posix1e/posix1e.3 Sat May 23 10:35:27 2009 (r192632) +++ stable/6/lib/libc/posix1e/posix1e.3 Sat May 23 10:45:28 2009 (r192633) @@ -25,7 +25,7 @@ .\" .\" $FreeBSD$ .\" -.Dd January 17, 2000 +.Dd May 20, 2009 .Dt POSIX1E 3 .Os .Sh NAME @@ -36,7 +36,6 @@ .Sh SYNOPSIS .In sys/types.h .In sys/acl.h -.\" .In sys/audit.h .In sys/capability.h .In sys/mac.h .Sh DESCRIPTION @@ -104,6 +103,7 @@ structure. .Sh SEE ALSO .Xr extattr 2 , .Xr acl 3 , +.Xr libbsm 3 , .Xr mac 3 , .Xr acl 9 , .Xr extattr 9 ,