From patchwork Tue Jun 17 14:12:44 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Gary Benson X-Patchwork-Id: 1519 Received: (qmail 27960 invoked by alias); 17 Jun 2014 14:13:29 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 27731 invoked by uid 89); 17 Jun 2014 14:13:28 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.6 required=5.0 tests=AWL, BAYES_00, RP_MATCHES_RCVD, SPF_HELO_PASS, SPF_PASS autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 17 Jun 2014 14:13:07 +0000 Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s5HED5bs006545 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK) for ; Tue, 17 Jun 2014 10:13:05 -0400 Received: from blade.nx (ovpn-116-59.ams2.redhat.com [10.36.116.59]) by int-mx09.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id s5HED4JW018271 for ; Tue, 17 Jun 2014 10:13:05 -0400 Received: from blade.nx (localhost [127.0.0.1]) by blade.nx (Postfix) with ESMTP id 009F326247C for ; Tue, 17 Jun 2014 15:13:04 +0100 (BST) From: Gary Benson To: gdb-patches@sourceware.org Subject: [PATCH 01/15] Whitespace changes Date: Tue, 17 Jun 2014 15:12:44 +0100 Message-Id: <1403014378-4349-2-git-send-email-gbenson@redhat.com> In-Reply-To: <1403014378-4349-1-git-send-email-gbenson@redhat.com> References: <1403014378-4349-1-git-send-email-gbenson@redhat.com> X-IsSubscribed: yes This commit fixes various whitespace differences between i386-nat.c and i386-low.c. gdb/ 2014-06-17 Gary Benson * i386-nat.c: Whitespace changes. gdb/gdbserver/ 2014-06-17 Gary Benson * i386-low.c: Whitespace changes. --- gdb/ChangeLog | 4 ++++ gdb/gdbserver/ChangeLog | 4 ++++ gdb/gdbserver/i386-low.c | 16 ++++++++++------ gdb/i386-nat.c | 46 +++++++++++++++++++++++++++------------------- 4 files changed, 45 insertions(+), 25 deletions(-) diff --git a/gdb/gdbserver/i386-low.c b/gdb/gdbserver/i386-low.c index de2d11a..304d6f3 100644 --- a/gdb/gdbserver/i386-low.c +++ b/gdb/gdbserver/i386-low.c @@ -115,7 +115,7 @@ } while (0) /* Set in DR7 the RW and LEN fields for the I'th debug register. */ -#define I386_DR_SET_RW_LEN(state, i,rwlen) \ +#define I386_DR_SET_RW_LEN(state, i, rwlen) \ do { \ (state)->dr_control_mirror &= \ ~(0x0f << (DR_CONTROL_SHIFT + DR_CONTROL_SIZE * (i))); \ @@ -136,7 +136,7 @@ /* Types of operations supported by i386_handle_nonaligned_watchpoint. */ typedef enum { WP_INSERT, WP_REMOVE, WP_COUNT } i386_wp_op_t; - + /* Implementation. */ /* Clear the reference counts and forget everything we knew about the @@ -446,14 +446,16 @@ i386_low_insert_watchpoint (struct i386_debug_reg_state *state, && !(TARGET_HAS_DR_LEN_8 && len == 8)) || addr % len != 0) { - retval = i386_handle_nonaligned_watchpoint (&local_state, WP_INSERT, + retval = i386_handle_nonaligned_watchpoint (&local_state, + WP_INSERT, addr, len, type); } else { unsigned len_rw = i386_length_and_rw_bits (len, type); - retval = i386_insert_aligned_watchpoint (&local_state, addr, len_rw); + retval = i386_insert_aligned_watchpoint (&local_state, + addr, len_rw); } if (retval == 0) @@ -483,14 +485,16 @@ i386_low_remove_watchpoint (struct i386_debug_reg_state *state, && !(TARGET_HAS_DR_LEN_8 && len == 8)) || addr % len != 0) { - retval = i386_handle_nonaligned_watchpoint (&local_state, WP_REMOVE, + retval = i386_handle_nonaligned_watchpoint (&local_state, + WP_REMOVE, addr, len, type); } else { unsigned len_rw = i386_length_and_rw_bits (len, type); - retval = i386_remove_aligned_watchpoint (&local_state, addr, len_rw); + retval = i386_remove_aligned_watchpoint (&local_state, + addr, len_rw); } if (retval == 0) diff --git a/gdb/i386-nat.c b/gdb/i386-nat.c index 2f80a6e..a56a116 100644 --- a/gdb/i386-nat.c +++ b/gdb/i386-nat.c @@ -39,7 +39,6 @@ struct i386_dr_low_type i386_dr_low; - /* Support for 8-byte wide hw watchpoints. */ #define TARGET_HAS_DR_LEN_8 (i386_dr_low.debug_register_length == 8) @@ -90,7 +89,7 @@ struct i386_dr_low_type i386_dr_low; later processors do that automatically. We set these flags for backwards compatibility. */ #define DR_LOCAL_SLOWDOWN (0x100) -#define DR_GLOBAL_SLOWDOWN (0x200) +#define DR_GLOBAL_SLOWDOWN (0x200) /* Fields reserved by Intel. This includes the GD (General Detect Enable) flag, which causes a debug exception to be generated when a @@ -106,7 +105,7 @@ struct i386_dr_low_type i386_dr_low; /* The I'th debug register is vacant if its Local and Global Enable bits are reset in the Debug Control register. */ -#define I386_DR_VACANT(state, i) \ +#define I386_DR_VACANT(state, i) \ (((state)->dr_control_mirror & (3 << (DR_ENABLE_SIZE * (i)))) == 0) /* Locally enable the break/watchpoint in the I'th debug register. */ @@ -322,7 +321,7 @@ i386_show_dr (struct i386_debug_reg_state *state, printf_unfiltered (" (addr=%lx, len=%d, type=%s)", /* This code is for ia32, so casting CORE_ADDR to unsigned long should be okay. */ - (unsigned long)addr, len, + (unsigned long) addr, len, type == hw_write ? "data-write" : (type == hw_read ? "data-read" : (type == hw_access ? "data-read/write" @@ -335,14 +334,14 @@ i386_show_dr (struct i386_debug_reg_state *state, printf_unfiltered ("\tCONTROL (DR7): %s STATUS (DR6): %s\n", phex (state->dr_control_mirror, 8), phex (state->dr_status_mirror, 8)); - ALL_DEBUG_REGISTERS(i) + ALL_DEBUG_REGISTERS (i) { printf_unfiltered ("\ \tDR%d: addr=0x%s, ref.count=%d DR%d: addr=0x%s, ref.count=%d\n", i, phex (state->dr_mirror[i], addr_size), state->dr_ref_count[i], i + 1, phex (state->dr_mirror[i + 1], addr_size), - state->dr_ref_count[i+1]); + state->dr_ref_count[i + 1]); i++; } } @@ -419,7 +418,7 @@ i386_insert_aligned_watchpoint (struct i386_debug_reg_state *state, /* First, look for an occupied debug register with the same address and the same RW and LEN definitions. If we find one, we can reuse it for this watchpoint as well (and save a register). */ - ALL_DEBUG_REGISTERS(i) + ALL_DEBUG_REGISTERS (i) { if (!I386_DR_VACANT (state, i) && state->dr_mirror[i] == addr @@ -431,7 +430,7 @@ i386_insert_aligned_watchpoint (struct i386_debug_reg_state *state, } /* Next, look for a vacant debug register. */ - ALL_DEBUG_REGISTERS(i) + ALL_DEBUG_REGISTERS (i) { if (I386_DR_VACANT (state, i)) break; @@ -473,7 +472,7 @@ i386_remove_aligned_watchpoint (struct i386_debug_reg_state *state, { int i, retval = -1; - ALL_DEBUG_REGISTERS(i) + ALL_DEBUG_REGISTERS (i) { if (!I386_DR_VACANT (state, i) && state->dr_mirror[i] == addr @@ -548,7 +547,7 @@ i386_handle_nonaligned_watchpoint (struct i386_debug_reg_state *state, else internal_error (__FILE__, __LINE__, _("\ Invalid value %d of operation in i386_handle_nonaligned_watchpoint.\n"), - (int)what); + (int) what); if (retval) break; } @@ -603,10 +602,14 @@ i386_insert_watchpoint (struct target_ops *self, if (type == hw_read) return 1; /* unsupported */ - if (((len != 1 && len !=2 && len !=4) && !(TARGET_HAS_DR_LEN_8 && len == 8)) + if (((len != 1 && len != 2 && len != 4) + && !(TARGET_HAS_DR_LEN_8 && len == 8)) || addr % len != 0) - retval = i386_handle_nonaligned_watchpoint (&local_state, - WP_INSERT, addr, len, type); + { + retval = i386_handle_nonaligned_watchpoint (&local_state, + WP_INSERT, + addr, len, type); + } else { unsigned len_rw = i386_length_and_rw_bits (len, type); @@ -639,10 +642,14 @@ i386_remove_watchpoint (struct target_ops *self, commit the change back to the inferior. */ struct i386_debug_reg_state local_state = *state; - if (((len != 1 && len !=2 && len !=4) && !(TARGET_HAS_DR_LEN_8 && len == 8)) + if (((len != 1 && len != 2 && len != 4) + && !(TARGET_HAS_DR_LEN_8 && len == 8)) || addr % len != 0) - retval = i386_handle_nonaligned_watchpoint (&local_state, - WP_REMOVE, addr, len, type); + { + retval = i386_handle_nonaligned_watchpoint (&local_state, + WP_REMOVE, + addr, len, type); + } else { unsigned len_rw = i386_length_and_rw_bits (len, type); @@ -673,8 +680,8 @@ i386_region_ok_for_watchpoint (struct target_ops *self, /* Compute how many aligned watchpoints we would need to cover this region. */ - nregs = i386_handle_nonaligned_watchpoint (state, - WP_COUNT, addr, len, hw_write); + nregs = i386_handle_nonaligned_watchpoint (state, WP_COUNT, + addr, len, hw_write); return nregs <= DR_NADDR ? 1 : 0; } @@ -726,7 +733,7 @@ i386_stopped_data_address (struct target_ops *ops, CORE_ADDR *addr_p) registers. */ status = i386_dr_low.get_status (); - ALL_DEBUG_REGISTERS(i) + ALL_DEBUG_REGISTERS (i) { if (!I386_DR_WATCH_HIT (status, i)) continue; @@ -750,6 +757,7 @@ i386_stopped_data_address (struct target_ops *ops, CORE_ADDR *addr_p) i386_show_dr (state, "watchpoint_hit", addr, -1, hw_write); } } + if (maint_show_dr && addr == 0) i386_show_dr (state, "stopped_data_addr", 0, 0, hw_write);