From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5104 invoked by alias); 10 Jul 2015 09:10:03 -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 5095 invoked by uid 89); 10 Jul 2015 09:10:02 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.5 required=5.0 tests=AWL,BAYES_00,SPF_PASS autolearn=ham version=3.3.2 X-HELO: eu-smtp-delivery-143.mimecast.com Received: from eu-smtp-delivery-143.mimecast.com (HELO eu-smtp-delivery-143.mimecast.com) (207.82.80.143) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 10 Jul 2015 09:09:59 +0000 Received: from cam-owa1.Emea.Arm.com (fw-tnat.cambridge.arm.com [217.140.96.140]) by eu-smtp-1.mimecast.com with ESMTP id uk-mta-22-jGpW5sbURlWocGat1-s4qg-1; Fri, 10 Jul 2015 10:09:55 +0100 Received: from e105615-lin.cambridge.arm.com ([10.1.2.79]) by cam-owa1.Emea.Arm.com with Microsoft SMTPSVC(6.0.3790.3959); Fri, 10 Jul 2015 10:09:54 +0100 From: Pierre Langlois To: gdb-patches@sourceware.org Cc: Pierre Langlois Subject: [PATCH] [GDBserver][AArch64] Cleanup comments for each linux_target_ops method Date: Fri, 10 Jul 2015 09:10:00 -0000 Message-Id: <1436519393-26328-1-git-send-email-pierre.langlois@arm.com> X-MC-Unique: jGpW5sbURlWocGat1-s4qg-1 Content-Type: text/plain; charset=WINDOWS-1252 Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes X-SW-Source: 2015-07/txt/msg00314.txt.bz2 Hi all, This patch cleans up the comments for each linux_target_ops methods. We should mention which method each function implements but there is no need to duplicate information already mentionned in the base target_ops or linux_target_ops definitions. Thanks, Pierre gdb/gdbserver/ChangeLog: * linux-aarch64-low.c: Add comments for each linux_target_ops method. Remove comments already covered in target_ops and linux_target_ops definitions. (the_low_target): Add comments for each unimplemented method. --- gdb/gdbserver/linux-aarch64-low.c | 63 +++++++++++++++++++++++------------= ---- 1 file changed, 37 insertions(+), 26 deletions(-) diff --git a/gdb/gdbserver/linux-aarch64-low.c b/gdb/gdbserver/linux-aarch6= 4-low.c index 91afd1b..b068e55 100644 --- a/gdb/gdbserver/linux-aarch64-low.c +++ b/gdb/gdbserver/linux-aarch64-low.c @@ -189,12 +189,16 @@ struct arch_lwp_info static int aarch64_num_bp_regs; static int aarch64_num_wp_regs; =20 +/* Implementation of linux_target_ops method "cannot_store_register". */ + static int aarch64_cannot_store_register (int regno) { return regno >=3D AARCH64_NUM_REGS; } =20 +/* Implementation of linux_target_ops method "cannot_fetch_register". */ + static int aarch64_cannot_fetch_register (int regno) { @@ -255,6 +259,8 @@ aarch64_store_fpregset (struct regcache *regcache, cons= t void *buf) was originally used to debug LinuxThreads support. */ extern int debug_threads; =20 +/* Implementation of linux_target_ops method "get_pc". */ + static CORE_ADDR aarch64_get_pc (struct regcache *regcache) { @@ -266,6 +272,8 @@ aarch64_get_pc (struct regcache *regcache) return pc; } =20 +/* Implementation of linux_target_ops method "set_pc". */ + static void aarch64_set_pc (struct regcache *regcache, CORE_ADDR pc) { @@ -280,6 +288,8 @@ aarch64_set_pc (struct regcache *regcache, CORE_ADDR pc) (aarch64_default_breakpoint). */ static const gdb_byte aarch64_breakpoint[] =3D {0x00, 0x00, 0x20, 0xd4}; =20 +/* Implementation of linux_target_ops method "breakpoint_at". */ + static int aarch64_breakpoint_at (CORE_ADDR where) { @@ -925,6 +935,8 @@ aarch64_handle_watchpoint (enum target_hw_bp_type type,= CORE_ADDR addr, return aarch64_handle_unaligned_watchpoint (type, addr, len, is_insert= ); } =20 +/* Implementation of linux_target_ops method "supports_z_point_type". */ + static int aarch64_supports_z_point_type (char z_type) { @@ -941,13 +953,10 @@ aarch64_supports_z_point_type (char z_type) } } =20 -/* Insert a hardware breakpoint/watchpoint. - It actually only records the info of the to-be-inserted bp/wp; - the actual insertion will happen when threads are resumed. +/* Implementation of linux_target_ops method "insert_point". =20 - Return 0 if succeed; - Return 1 if TYPE is unsupported type; - Return -1 if an error occurs. */ + It actually only records the info of the to-be-inserted bp/wp; + the actual insertion will happen when threads are resumed. */ =20 static int aarch64_insert_point (enum raw_bkpt_type type, CORE_ADDR addr, @@ -977,13 +986,10 @@ aarch64_insert_point (enum raw_bkpt_type type, CORE_A= DDR addr, return ret; } =20 -/* Remove a hardware breakpoint/watchpoint. - It actually only records the info of the to-be-removed bp/wp, - the actual removal will be done when threads are resumed. +/* Implementation of linux_target_ops method "remove_point". =20 - Return 0 if succeed; - Return 1 if TYPE is an unsupported type; - Return -1 if an error occurs. */ + It actually only records the info of the to-be-removed bp/wp, + the actual removal will be done when threads are resumed. */ =20 static int aarch64_remove_point (enum raw_bkpt_type type, CORE_ADDR addr, @@ -1014,8 +1020,7 @@ aarch64_remove_point (enum raw_bkpt_type type, CORE_A= DDR addr, return ret; } =20 -/* Returns the address associated with the watchpoint that hit, if - any; returns 0 otherwise. */ +/* Implementation of linux_target_ops method "stopped_data_address". */ =20 static CORE_ADDR aarch64_stopped_data_address (void) @@ -1052,8 +1057,7 @@ aarch64_stopped_data_address (void) return (CORE_ADDR) 0; } =20 -/* Returns 1 if target was stopped due to a watchpoint hit, 0 - otherwise. */ +/* Implementation of linux_target_ops method "stopped_by_watchpoint". */ =20 static int aarch64_stopped_by_watchpoint (void) @@ -1087,7 +1091,7 @@ ps_get_thread_area (const struct ps_prochandle *ph, return PS_OK; } =20 -/* Called when a new process is created. */ +/* Implementation of linux_target_ops method "linux_new_process". */ =20 static struct arch_process_info * aarch64_linux_new_process (void) @@ -1099,7 +1103,7 @@ aarch64_linux_new_process (void) return info; } =20 -/* Called when a new thread is detected. */ +/* Implementation of linux_target_ops method "linux_new_thread". */ =20 static void aarch64_linux_new_thread (struct lwp_info *lwp) @@ -1115,6 +1119,8 @@ aarch64_linux_new_thread (struct lwp_info *lwp) lwp->arch_private =3D info; } =20 +/* Implementation of linux_target_ops method "linux_new_fork". */ + static void aarch64_linux_new_fork (struct process_info *parent, struct process_info *child) @@ -1142,7 +1148,8 @@ aarch64_linux_new_fork (struct process_info *parent, *child->priv->arch_private =3D *parent->priv->arch_private; } =20 -/* Called when resuming a thread. +/* Implementation of linux_target_ops method "linux_prepare_to_resume". + If the debug regs have changed, update the thread's copies. */ =20 static void @@ -1190,6 +1197,8 @@ aarch64_linux_prepare_to_resume (struct lwp_info *lwp) #define AARCH64_DEBUG_ARCH(x) (((x) >> 8) & 0xff) #define AARCH64_DEBUG_ARCH_V8 0x6 =20 +/* Implementation of linux_target_ops method "arch_setup". */ + static void aarch64_arch_setup (void) { @@ -1270,6 +1279,8 @@ static struct regs_info regs_info =3D &aarch64_regsets_info, }; =20 +/* Implementation of linux_target_ops method "regs_info". */ + static const struct regs_info * aarch64_regs_info (void) { @@ -1290,27 +1301,27 @@ struct linux_target_ops the_low_target =3D aarch64_regs_info, aarch64_cannot_fetch_register, aarch64_cannot_store_register, - NULL, + NULL, /* fetch_register */ aarch64_get_pc, aarch64_set_pc, (const unsigned char *) &aarch64_breakpoint, aarch64_breakpoint_len, - NULL, - 0, + NULL, /* breakpoint_reinsert_addr */ + 0, /* decr_pc_after_break */ aarch64_breakpoint_at, aarch64_supports_z_point_type, aarch64_insert_point, aarch64_remove_point, aarch64_stopped_by_watchpoint, aarch64_stopped_data_address, - NULL, - NULL, - NULL, + NULL, /* collect_ptrace_register */ + NULL, /* supply_ptrace_register */ + NULL, /* siginfo_fixup */ aarch64_linux_new_process, aarch64_linux_new_thread, aarch64_linux_new_fork, aarch64_linux_prepare_to_resume, - NULL, + NULL, /* process_qsupported */ aarch64_supports_tracepoints, }; =20 --=20 2.1.0