From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 74068 invoked by alias); 27 Apr 2015 19:17:02 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 74013 invoked by uid 89); 27 Apr 2015 19:17:02 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.6 required=5.0 tests=AWL,BAYES_00,SPF_HELO_PASS,SPF_SOFTFAIL autolearn=no version=3.3.2 X-HELO: bigwig.baldwin.cx Received: from bigwig.baldwin.cx (HELO bigwig.baldwin.cx) (96.47.65.170) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (CAMELLIA256-SHA encrypted) ESMTPS; Mon, 27 Apr 2015 19:17:01 +0000 Received: from ralph.baldwin.cx (pool-173-54-116-245.nwrknj.fios.verizon.net [173.54.116.245]) by bigwig.baldwin.cx (Postfix) with ESMTPSA id 77724B923; Mon, 27 Apr 2015 15:16:57 -0400 (EDT) From: John Baldwin To: gdb-patches@sourceware.org Cc: Pedro Alves Subject: Re: [PATCH 1/3] Add fbsd_nat_add_target. Date: Mon, 27 Apr 2015 19:27:00 -0000 Message-ID: <2013405.YhOVhnvfYq@ralph.baldwin.cx> User-Agent: KMail/4.14.2 (FreeBSD/10.1-STABLE; KDE/4.14.2; amd64; ; ) In-Reply-To: <553E899A.9070105@redhat.com> References: <4032488.W8nPzteMFC@ralph.baldwin.cx> <553E899A.9070105@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-IsSubscribed: yes X-SW-Source: 2015-04/txt/msg01014.txt.bz2 On Monday, April 27, 2015 08:10:18 PM Pedro Alves wrote: > On 04/26/2015 02:24 AM, John Baldwin wrote: > > Add a wrapper for add_target in fbsd-nat.c to override target operations > > common to all native FreeBSD targets. > > > > gdb/ChangeLog: > > > > * fbsd-nat.c (fbsd_pid_to_exec_file): Mark static. > > (fbsd_find_memory_regions): Mark static. > > (fbsd_nat_add_target): New function. > > * fbsd-nat.h: Export fbsd_nat_add_target and remove prototypes for > > fbsd_pid_to_exec_file and fbsd_find_memory_regions. > > * amd64fbsd-nat.c (_initialize_amd64fbsd_nat): Use fbsd_nat_add_target. > > * i386fbsd-nat.c (_initialize_i386fbsd_nat): Likewise. > > * ppcfbsd-nat.c (_initialize_ppcfbsd_nat): Likewise. > > * sparc64fbsd-nat.c (_initialize_sparc64fbsd_nat): Likewise. > > OOC, any reason you didn't instead do it like: > > struct target_ops * > fbsd_nat_target (void) > { > struct target_ops *t = inf_ptrace_target (); > > t->to_pid_to_exec_file = fbsd_pid_to_exec_file; > t->to_find_memory_regions = fbsd_find_memory_regions; > return t; > } > > and then use fbsd_nat_target instead of inf_ptrace_target > directly? > > This maps a little better to a C++ world. > > linux-nat.c does it the way you did as it keeps a separate > linux_ops target instance around. I was probably just using linux-nat.c as a reference. One thing that confuses me about the linux-nat target is that it keeps linux_ops around so that it can call the original methods that it overrides, and yet for a few methods it also uses a local 'super_foo' variable to call an original method. I think that those are both doing the same thing, but perhaps there is some subtlety I'm missing? I do use a 'super_wait' to call ptrace's wait method in the second patch in this series, so I could certainly change this to return a target rather than modifying an existing one if that is preferred. -- John Baldwin