Blame SOURCES/gdb-rhbz795424-bitpos-22of25.patch

190f2a
From FEDORA_PATCHES Mon Sep 17 00:00:00 2001
190f2a
From: Fedora GDB patches <invalid@email.com>
190f2a
Date: Fri, 27 Oct 2017 21:07:50 +0200
190f2a
Subject: gdb-rhbz795424-bitpos-22of25.patch
190f2a
190f2a
;; Fix `GDB cannot access struct member whose offset is larger than 256MB'
190f2a
;; (RH BZ 795424).
190f2a
;;=push
190f2a
190f2a
http://sourceware.org/ml/gdb-patches/2012-09/msg00629.html
190f2a
190f2a
--MP_/6HRlH6vpyqtSy4CYyMrX6b2
190f2a
Content-Type: text/plain; charset=US-ASCII
190f2a
Content-Transfer-Encoding: 7bit
190f2a
Content-Disposition: inline
190f2a
190f2a
Hi,
190f2a
190f2a
This is part three of the bitpos expansion change.  Some architectures
190f2a
allow arbitrary length watchpoints and combined with the fact that type
190f2a
lengths could be large enough, we need LONGEST for watchpoint lengths.
190f2a
It is architecture dependent however, whether the LONGEST is needed or
190f2a
not.  This patch updates the signatures of watchpoint insertion and
190f2a
removal functions of all architectures (to comply with the function
190f2a
signatures in the callback struct), but expands types only in
190f2a
architectures that need it.  Tested on Fedora 16 x86_64.
190f2a
190f2a
Regards,
190f2a
Siddhesh
190f2a
--MP_/6HRlH6vpyqtSy4CYyMrX6b2
190f2a
Content-Type: text/plain
190f2a
Content-Transfer-Encoding: quoted-printable
190f2a
Content-Disposition: attachment; filename=ChangeLog-wp
190f2a
190f2a
gdb/ChangeLog:
190f2a
190f2a
	* arm-linux-nat.c (arm_linux_insert_watchpoint): Expand
190f2a
	parameter LEN to LONGEST.
190f2a
	(arm_linux_remove_watchpoint): Likewise.
190f2a
	(arm_linux_watchpoint_addr_within_range): Expand parameter
190f2a
	LENGTH to LONGEST.
190f2a
	* i386-nat.c (i386_insert_watchpoint): Expand parameter LEN to
190f2a
	LONGEST.
190f2a
	(i386_remove_watchpoint): Likewise.
190f2a
	* ia64-linux-nat.c (ia64_linux_insert_watchpoint): Likewise.
190f2a
	(ia64_linux_remove_watchpoint): Likewise.
190f2a
	* inf-ttrace.c (inf_ttrace_insert_watchpoint): Likewise.
190f2a
	Expand NUM_PAGES, PAGE to LONGEST.
190f2a
	(inf_ttrace_remove_watchpoint): Likewise.
190f2a
	* mips-linux-nat.c (mips_linux_insert_watchpoint): Expand
190f2a
	parameter LEN to LONGEST.
190f2a
	(mips_linux_remove_watchpoint): Likewise.
190f2a
	* nto-procfs.c (procfs_remove_hw_watchpoint): Likewise.
190f2a
	(procfs_insert_hw_watchpoint): Likewise.
190f2a
	* ppc-linux-nat.c (calculate_dvc): Likewise.  Expand I,
190f2a
	NUM_BYTE_ENABLE to LONGEST.
190f2a
	(check_condition): Expand parameter LEN to point to LONGEST.
190f2a
	(ppc_linux_can_accel_watchpoint_condition): Expand parameter
190f2a
	LEN to LONGEST.
190f2a
	(create_watchpoint_request): Likewise.
190f2a
	(ppc_linux_insert_watchpoint): Likewise.
190f2a
	(ppc_linux_remove_watchpoint): Likewise.
190f2a
	(ppc_linux_watchpoint_addr_within_range): Expand parameter
190f2a
	LENGTH to LONGEST.
190f2a
	* procfs.c (proc_set_watchpoint): Expand parameter LEN to
190f2a
	LONGEST.
190f2a
	(procfs_set_watchpoint): Likewise.
190f2a
	(procfs_insert_watchpoint): Likewise.
190f2a
	(procfs_remove_watchpoint): Likewise.
190f2a
	* remote-m32r-sdi.c (m32r_insert_watchpoint): Likewise.  Use
190f2a
	plongest to format print LEN.
190f2a
	(m32r_remove_watchpoint): Likewise.
190f2a
	* remote-mips.c (mips_insert_watchpoint): Expand parameter LEN
190f2a
	to LONGEST.
190f2a
	(mips_remove_watchpoint): Likewise.
190f2a
	* remote.c (remote_insert_watchpoint): Likewise.
190f2a
	Use phex_nz to format print LEN.
190f2a
	(remote_remove_watchpoint): Likewise.
190f2a
	(remote_watchpoint_addr_within_range): Expand parameter LENGTH
190f2a
	to LONGEST.
190f2a
	* s390-linux-nat.c (s390_insert_watchpoint): Expand parameter LEN to
190f2a
	LONGEST.
190f2a
	(s390_remove_watchpoint): Likewise.
190f2a
	* target.c (update_current_target): Expand parameter LEN for
190f2a
	callbacks to TO_INSERT_WATCHPOINT, TO_REMOVE_WATCHPOINT,
190f2a
	TO_CAN_ACCEL_WATCHPOINT_CONDITION, to LONGEST.
190f2a
	(default_watchpoint_addr_within_range): Expand parameter
190f2a
	LENGTH to LONGEST.
190f2a
	(debug_to_can_accel_watchpoint_condition): Expand parameter LEN
190f2a
	to LONGEST.  Use plongest to format print LEN.
190f2a
	(debug_to_watchpoint_addr_within_range): Expand parameter LENGTH
190f2a
	to LONGEST.  Use plongest to format print LENGTH.
190f2a
	(debug_to_insert_watchpoint): Expand parameter LEN to LONGEST.
190f2a
	Use plongest to format print LEN.
190f2a
	(debug_to_remove_watchpoint): Likewise.
190f2a
	* target.h (struct target_ops): Expand parameter LEN of
190f2a
	TO_REMOVE_WATCHPOINT, TO_INSERT_WATCHPOINT,
190f2a
	TO_WATCHPOINT_ADDR_WITHIN_RANGE and
190f2a
	TO_CAN_ACCEL_WATCHPOINT_CONDITION to LONGEST.
190f2a
190f2a
--MP_/6HRlH6vpyqtSy4CYyMrX6b2
190f2a
Content-Type: text/x-patch
190f2a
Content-Transfer-Encoding: 7bit
190f2a
Content-Disposition: attachment; filename=bitpos-wp.patch
190f2a
190f2a
diff --git a/gdb/aarch64-linux-nat.c b/gdb/aarch64-linux-nat.c
190f2a
--- a/gdb/aarch64-linux-nat.c
190f2a
+++ b/gdb/aarch64-linux-nat.c
190f2a
@@ -63,14 +63,14 @@ public:
190f2a
   int can_use_hw_breakpoint (enum bptype, int, int) override;
190f2a
   int insert_hw_breakpoint (struct gdbarch *, struct bp_target_info *) override;
190f2a
   int remove_hw_breakpoint (struct gdbarch *, struct bp_target_info *) override;
190f2a
-  int region_ok_for_hw_watchpoint (CORE_ADDR, int) override;
190f2a
+  int region_ok_for_hw_watchpoint (CORE_ADDR, LONGEST) override;
190f2a
   int insert_watchpoint (CORE_ADDR, int, enum target_hw_bp_type,
190f2a
 			 struct expression *) override;
190f2a
   int remove_watchpoint (CORE_ADDR, int, enum target_hw_bp_type,
190f2a
 			 struct expression *) override;
190f2a
   bool stopped_by_watchpoint () override;
190f2a
   bool stopped_data_address (CORE_ADDR *) override;
190f2a
-  bool watchpoint_addr_within_range (CORE_ADDR, CORE_ADDR, int) override;
190f2a
+  bool watchpoint_addr_within_range (CORE_ADDR, CORE_ADDR, LONGEST) override;
190f2a
 
190f2a
   int can_do_single_step () override;
190f2a
 
190f2a
@@ -887,7 +887,8 @@ aarch64_linux_nat_target::stopped_by_watchpoint ()
190f2a
 
190f2a
 bool
190f2a
 aarch64_linux_nat_target::watchpoint_addr_within_range (CORE_ADDR addr,
190f2a
-							CORE_ADDR start, int length)
190f2a
+							CORE_ADDR start,
190f2a
+							LONGEST length)
190f2a
 {
190f2a
   return start <= addr && start + length - 1 >= addr;
190f2a
 }
190f2a
diff --git a/gdb/arm-linux-nat.c b/gdb/arm-linux-nat.c
190f2a
--- a/gdb/arm-linux-nat.c
190f2a
+++ b/gdb/arm-linux-nat.c
190f2a
@@ -80,7 +80,7 @@ public:
190f2a
 
190f2a
   int remove_hw_breakpoint (struct gdbarch *, struct bp_target_info *) override;
190f2a
 
190f2a
-  int region_ok_for_hw_watchpoint (CORE_ADDR, int) override;
190f2a
+  int region_ok_for_hw_watchpoint (CORE_ADDR, LONGEST) override;
190f2a
 
190f2a
   int insert_watchpoint (CORE_ADDR, int, enum target_hw_bp_type,
190f2a
 			 struct expression *) override;
190f2a
@@ -91,7 +91,7 @@ public:
190f2a
 
190f2a
   bool stopped_data_address (CORE_ADDR *) override;
190f2a
 
190f2a
-  bool watchpoint_addr_within_range (CORE_ADDR, CORE_ADDR, int) override;
190f2a
+  bool watchpoint_addr_within_range (CORE_ADDR, CORE_ADDR, LONGEST) override;
190f2a
 
190f2a
   const struct target_desc *read_description () override;
190f2a
 
190f2a
@@ -1207,7 +1207,7 @@ arm_linux_nat_target::stopped_by_watchpoint ()
190f2a
 bool
190f2a
 arm_linux_nat_target::watchpoint_addr_within_range (CORE_ADDR addr,
190f2a
 						    CORE_ADDR start,
190f2a
-						    int length)
190f2a
+						    LONGEST length)
190f2a
 {
190f2a
   return start <= addr && start + length - 1 >= addr;
190f2a
 }
190f2a
diff --git a/gdb/nat/aarch64-linux-hw-point.c b/gdb/nat/aarch64-linux-hw-point.c
190f2a
--- a/gdb/nat/aarch64-linux-hw-point.c
190f2a
+++ b/gdb/nat/aarch64-linux-hw-point.c
190f2a
@@ -137,7 +137,7 @@ aarch64_point_encode_ctrl_reg (enum target_hw_bp_type type, int offset, int len)
190f2a
    Return 0 for any non-compliant ADDR and/or LEN; return 1 otherwise.  */
190f2a
 
190f2a
 static int
190f2a
-aarch64_point_is_aligned (int is_watchpoint, CORE_ADDR addr, int len)
190f2a
+aarch64_point_is_aligned (int is_watchpoint, CORE_ADDR addr, LONGEST len)
190f2a
 {
190f2a
   unsigned int alignment = 0;
190f2a
 
190f2a
@@ -212,9 +212,10 @@ aarch64_point_is_aligned (int is_watchpoint, CORE_ADDR addr, int len)
190f2a
    an address within the latter. */
190f2a
 
190f2a
 static void
190f2a
-aarch64_align_watchpoint (CORE_ADDR addr, int len, CORE_ADDR *aligned_addr_p,
190f2a
+aarch64_align_watchpoint (CORE_ADDR addr, LONGEST len,
190f2a
+			  CORE_ADDR *aligned_addr_p,
190f2a
 			  int *aligned_offset_p, int *aligned_len_p,
190f2a
-			  CORE_ADDR *next_addr_p, int *next_len_p,
190f2a
+			  CORE_ADDR *next_addr_p, LONGEST *next_len_p,
190f2a
 			  CORE_ADDR *next_addr_orig_p)
190f2a
 {
190f2a
   int aligned_len;
190f2a
@@ -611,7 +612,7 @@ aarch64_handle_aligned_watchpoint (enum target_hw_bp_type type,
190f2a
 
190f2a
 static int
190f2a
 aarch64_handle_unaligned_watchpoint (enum target_hw_bp_type type,
190f2a
-				     CORE_ADDR addr, int len, int is_insert,
190f2a
+				     CORE_ADDR addr, LONGEST len, int is_insert,
190f2a
 				     struct aarch64_debug_reg_state *state)
190f2a
 {
190f2a
   CORE_ADDR addr_orig = addr;
190f2a
@@ -641,12 +642,12 @@ aarch64_handle_unaligned_watchpoint (enum target_hw_bp_type type,
190f2a
 		      "                                "
190f2a
 		      "addr_orig: %s\n"
190f2a
 		      "                                "
190f2a
-		      "next_addr: %s,    next_len: %d\n"
190f2a
+		      "next_addr: %s,    next_len: %s\n"
190f2a
 		      "                           "
190f2a
 		      "addr_orig_next: %s\n",
190f2a
 		      is_insert, core_addr_to_string_nz (aligned_addr),
190f2a
 		      aligned_len, core_addr_to_string_nz (addr_orig),
190f2a
-		      core_addr_to_string_nz (addr), len,
190f2a
+		      core_addr_to_string_nz (addr), plongest (len),
190f2a
 		      core_addr_to_string_nz (addr_orig_next));
190f2a
 
190f2a
       addr_orig = addr_orig_next;
190f2a
@@ -660,7 +661,7 @@ aarch64_handle_unaligned_watchpoint (enum target_hw_bp_type type,
190f2a
 
190f2a
 int
190f2a
 aarch64_handle_watchpoint (enum target_hw_bp_type type, CORE_ADDR addr,
190f2a
-			   int len, int is_insert,
190f2a
+			   LONGEST len, int is_insert,
190f2a
 			   struct aarch64_debug_reg_state *state)
190f2a
 {
190f2a
   if (aarch64_point_is_aligned (1 /* is_watchpoint */ , addr, len))
190f2a
@@ -722,14 +723,14 @@ aarch64_linux_set_debug_regs (struct aarch64_debug_reg_state *state,
190f2a
 void
190f2a
 aarch64_show_debug_reg_state (struct aarch64_debug_reg_state *state,
190f2a
 			      const char *func, CORE_ADDR addr,
190f2a
-			      int len, enum target_hw_bp_type type)
190f2a
+			      LONGEST len, enum target_hw_bp_type type)
190f2a
 {
190f2a
   int i;
190f2a
 
190f2a
   debug_printf ("%s", func);
190f2a
   if (addr || len)
190f2a
-    debug_printf (" (addr=0x%08lx, len=%d, type=%s)",
190f2a
-		  (unsigned long) addr, len,
190f2a
+    debug_printf (" (addr=0x%08lx, len=%s, type=%s)",
190f2a
+		  (unsigned long) addr, plongest (len),
190f2a
 		  type == hw_write ? "hw-write-watchpoint"
190f2a
 		  : (type == hw_read ? "hw-read-watchpoint"
190f2a
 		     : (type == hw_access ? "hw-access-watchpoint"
190f2a
@@ -812,7 +813,7 @@ aarch64_linux_get_debug_reg_capacity (int tid)
190f2a
    ADDR and whose length is LEN in bytes.  */
190f2a
 
190f2a
 int
190f2a
-aarch64_linux_region_ok_for_watchpoint (CORE_ADDR addr, int len)
190f2a
+aarch64_linux_region_ok_for_watchpoint (CORE_ADDR addr, LONGEST len)
190f2a
 {
190f2a
   CORE_ADDR aligned_addr;
190f2a
 
190f2a
diff --git a/gdb/nat/aarch64-linux-hw-point.h b/gdb/nat/aarch64-linux-hw-point.h
190f2a
--- a/gdb/nat/aarch64-linux-hw-point.h
190f2a
+++ b/gdb/nat/aarch64-linux-hw-point.h
190f2a
@@ -176,7 +176,7 @@ int aarch64_handle_breakpoint (enum target_hw_bp_type type, CORE_ADDR addr,
190f2a
 			       int len, int is_insert,
190f2a
 			       struct aarch64_debug_reg_state *state);
190f2a
 int aarch64_handle_watchpoint (enum target_hw_bp_type type, CORE_ADDR addr,
190f2a
-			       int len, int is_insert,
190f2a
+			       LONGEST len, int is_insert,
190f2a
 			       struct aarch64_debug_reg_state *state);
190f2a
 
190f2a
 void aarch64_linux_set_debug_regs (struct aarch64_debug_reg_state *state,
190f2a
@@ -184,12 +184,12 @@ void aarch64_linux_set_debug_regs (struct aarch64_debug_reg_state *state,
190f2a
 
190f2a
 void aarch64_show_debug_reg_state (struct aarch64_debug_reg_state *state,
190f2a
 				   const char *func, CORE_ADDR addr,
190f2a
-				   int len, enum target_hw_bp_type type);
190f2a
+				   LONGEST len, enum target_hw_bp_type type);
190f2a
 
190f2a
 void aarch64_linux_get_debug_reg_capacity (int tid);
190f2a
 
190f2a
 struct aarch64_debug_reg_state *aarch64_get_debug_reg_state (pid_t pid);
190f2a
 
190f2a
-int aarch64_linux_region_ok_for_watchpoint (CORE_ADDR addr, int len);
190f2a
+int aarch64_linux_region_ok_for_watchpoint (CORE_ADDR addr, LONGEST len);
190f2a
 
190f2a
 #endif /* AARCH64_LINUX_HW_POINT_H */
190f2a
diff --git a/gdb/ppc-linux-nat.c b/gdb/ppc-linux-nat.c
190f2a
--- a/gdb/ppc-linux-nat.c
190f2a
+++ b/gdb/ppc-linux-nat.c
190f2a
@@ -282,7 +282,7 @@ struct ppc_linux_nat_target final : public linux_nat_target
190f2a
   int remove_hw_breakpoint (struct gdbarch *, struct bp_target_info *)
190f2a
     override;
190f2a
 
190f2a
-  int region_ok_for_hw_watchpoint (CORE_ADDR, int) override;
190f2a
+  int region_ok_for_hw_watchpoint (CORE_ADDR, LONGEST) override;
190f2a
 
190f2a
   int insert_watchpoint (CORE_ADDR, int, enum target_hw_bp_type,
190f2a
 			 struct expression *) override;
190f2a
@@ -300,9 +300,9 @@ struct ppc_linux_nat_target final : public linux_nat_target
190f2a
 
190f2a
   bool stopped_data_address (CORE_ADDR *) override;
190f2a
 
190f2a
-  bool watchpoint_addr_within_range (CORE_ADDR, CORE_ADDR, int) override;
190f2a
+  bool watchpoint_addr_within_range (CORE_ADDR, CORE_ADDR, LONGEST) override;
190f2a
 
190f2a
-  bool can_accel_watchpoint_condition (CORE_ADDR, int, int, struct expression *)
190f2a
+  bool can_accel_watchpoint_condition (CORE_ADDR, LONGEST, int, struct expression *)
190f2a
     override;
190f2a
 
190f2a
   int masked_watch_num_registers (CORE_ADDR, CORE_ADDR) override;
190f2a
@@ -1659,11 +1659,11 @@ can_use_watchpoint_cond_accel (void)
190f2a
    CONDITION_VALUE will hold the value which should be put in the
190f2a
    DVC register.  */
190f2a
 static void
190f2a
-calculate_dvc (CORE_ADDR addr, int len, CORE_ADDR data_value,
190f2a
+calculate_dvc (CORE_ADDR addr, LONGEST len, CORE_ADDR data_value,
190f2a
 	       uint32_t *condition_mode, uint64_t *condition_value)
190f2a
 {
190f2a
-  int i, num_byte_enable, align_offset, num_bytes_off_dvc,
190f2a
-      rightmost_enabled_byte;
190f2a
+  LONGEST i, num_byte_enable;
190f2a
+  int align_offset, num_bytes_off_dvc, rightmost_enabled_byte;
190f2a
   CORE_ADDR addr_end_data, addr_end_dvc;
190f2a
 
190f2a
   /* The DVC register compares bytes within fixed-length windows which
190f2a
@@ -1751,7 +1751,7 @@ num_memory_accesses (const std::vector<value_ref_ptr> &chain)
190f2a
    of the constant.  */
190f2a
 static int
190f2a
 check_condition (CORE_ADDR watch_addr, struct expression *cond,
190f2a
-		 CORE_ADDR *data_value, int *len)
190f2a
+		 CORE_ADDR *data_value, LONGEST *len)
190f2a
 {
190f2a
   int pc = 1, num_accesses_left, num_accesses_right;
190f2a
   struct value *left_val, *right_val;
190f2a
@@ -1802,7 +1802,8 @@ check_condition (CORE_ADDR watch_addr, struct expression *cond,
190f2a
    the condition expression, thus only triggering the watchpoint when it is
190f2a
    true.  */
190f2a
 bool
190f2a
-ppc_linux_nat_target::can_accel_watchpoint_condition (CORE_ADDR addr, int len,
190f2a
+ppc_linux_nat_target::can_accel_watchpoint_condition (CORE_ADDR addr,
190f2a
+						      LONGEST len,
190f2a
 						      int rw,
190f2a
 						      struct expression *cond)
190f2a
 {
190f2a
@@ -1820,7 +1821,7 @@ ppc_linux_nat_target::can_accel_watchpoint_condition (CORE_ADDR addr, int len,
190f2a
 
190f2a
 static void
190f2a
 create_watchpoint_request (struct ppc_hw_breakpoint *p, CORE_ADDR addr,
190f2a
-			   int len, enum target_hw_bp_type type,
190f2a
+			   LONGEST len, enum target_hw_bp_type type,
190f2a
 			   struct expression *cond, int insert)
190f2a
 {
190f2a
   if (len == 1
190f2a
@@ -2086,7 +2087,7 @@ ppc_linux_nat_target::stopped_by_watchpoint ()
190f2a
 bool
190f2a
 ppc_linux_nat_target::watchpoint_addr_within_range (CORE_ADDR addr,
190f2a
 						    CORE_ADDR start,
190f2a
-						    int length)
190f2a
+						    LONGEST length)
190f2a
 {
190f2a
   int mask;
190f2a
 
190f2a
diff --git a/gdb/procfs.c b/gdb/procfs.c
190f2a
--- a/gdb/procfs.c
190f2a
+++ b/gdb/procfs.c
190f2a
@@ -1561,7 +1561,7 @@ procfs_address_to_host_pointer (CORE_ADDR addr)
190f2a
 }
190f2a
 
190f2a
 static int
190f2a
-proc_set_watchpoint (procinfo *pi, CORE_ADDR addr, int len, int wflags)
190f2a
+proc_set_watchpoint (procinfo *pi, CORE_ADDR addr, LONGEST len, int wflags)
190f2a
 {
190f2a
   struct {
190f2a
     procfs_ctl_t cmd;
190f2a
@@ -3214,7 +3214,7 @@ procfs_target::pid_to_str (ptid_t ptid)
190f2a
 /* Insert a watchpoint.  */
190f2a
 
190f2a
 static int
190f2a
-procfs_set_watchpoint (ptid_t ptid, CORE_ADDR addr, int len, int rwflag,
190f2a
+procfs_set_watchpoint (ptid_t ptid, CORE_ADDR addr, LONGEST len, int rwflag,
190f2a
 		       int after)
190f2a
 {
190f2a
   int       pflags = 0;
190f2a
diff --git a/gdb/remote.c b/gdb/remote.c
190f2a
--- a/gdb/remote.c
190f2a
+++ b/gdb/remote.c
190f2a
@@ -454,7 +454,7 @@ public:
190f2a
 
190f2a
   bool stopped_data_address (CORE_ADDR *) override;
190f2a
 
190f2a
-  bool watchpoint_addr_within_range (CORE_ADDR, CORE_ADDR, int) override;
190f2a
+  bool watchpoint_addr_within_range (CORE_ADDR, CORE_ADDR, LONGEST) override;
190f2a
 
190f2a
   int can_use_hw_breakpoint (enum bptype, int, int) override;
190f2a
 
190f2a
@@ -10342,7 +10342,7 @@ remote_target::insert_watchpoint (CORE_ADDR addr, int len,
190f2a
   p = strchr (rs->buf, '\0');
190f2a
   addr = remote_address_masked (addr);
190f2a
   p += hexnumstr (p, (ULONGEST) addr);
190f2a
-  xsnprintf (p, endbuf - p, ",%x", len);
190f2a
+  xsnprintf (p, endbuf - p, ",%s", phex_nz (len, sizeof (len)));
190f2a
 
190f2a
   putpkt (rs->buf);
190f2a
   getpkt (&rs->buf, &rs->buf_size, 0);
190f2a
@@ -10362,7 +10362,7 @@ remote_target::insert_watchpoint (CORE_ADDR addr, int len,
190f2a
 
190f2a
 bool
190f2a
 remote_target::watchpoint_addr_within_range (CORE_ADDR addr,
190f2a
-					     CORE_ADDR start, int length)
190f2a
+					     CORE_ADDR start, LONGEST length)
190f2a
 {
190f2a
   CORE_ADDR diff = remote_address_masked (addr - start);
190f2a
 
190f2a
@@ -10391,7 +10391,7 @@ remote_target::remove_watchpoint (CORE_ADDR addr, int len,
190f2a
   p = strchr (rs->buf, '\0');
190f2a
   addr = remote_address_masked (addr);
190f2a
   p += hexnumstr (p, (ULONGEST) addr);
190f2a
-  xsnprintf (p, endbuf - p, ",%x", len);
190f2a
+  xsnprintf (p, endbuf - p, ",%s", phex_nz (len, sizeof (len)));
190f2a
   putpkt (rs->buf);
190f2a
   getpkt (&rs->buf, &rs->buf_size, 0);
190f2a
 
190f2a
diff --git a/gdb/s390-linux-nat.c b/gdb/s390-linux-nat.c
190f2a
--- a/gdb/s390-linux-nat.c
190f2a
+++ b/gdb/s390-linux-nat.c
190f2a
@@ -121,7 +121,7 @@ public:
190f2a
     override;
190f2a
   int remove_hw_breakpoint (struct gdbarch *, struct bp_target_info *)
190f2a
     override;
190f2a
-  int region_ok_for_hw_watchpoint (CORE_ADDR, int) override;
190f2a
+  int region_ok_for_hw_watchpoint (CORE_ADDR, LONGEST) override;
190f2a
   bool have_continuable_watchpoint () override { return true; }
190f2a
   bool stopped_by_watchpoint () override;
190f2a
   int insert_watchpoint (CORE_ADDR, int, enum target_hw_bp_type,
190f2a
diff --git a/gdb/target-delegates.c b/gdb/target-delegates.c
190f2a
--- a/gdb/target-delegates.c
190f2a
+++ b/gdb/target-delegates.c
190f2a
@@ -38,9 +38,9 @@ struct dummy_target : public target_ops
190f2a
   int have_steppable_watchpoint () override;
190f2a
   bool have_continuable_watchpoint () override;
190f2a
   bool stopped_data_address (CORE_ADDR *arg0) override;
190f2a
-  bool watchpoint_addr_within_range (CORE_ADDR arg0, CORE_ADDR arg1, int arg2) override;
190f2a
+  bool watchpoint_addr_within_range (CORE_ADDR arg0, CORE_ADDR arg1, LONGEST arg2) override;
190f2a
   int region_ok_for_hw_watchpoint (CORE_ADDR arg0, LONGEST arg1) override;
190f2a
-  bool can_accel_watchpoint_condition (CORE_ADDR arg0, int arg1, int arg2, struct expression *arg3) override;
190f2a
+  bool can_accel_watchpoint_condition (CORE_ADDR arg0, LONGEST arg1, int arg2, struct expression *arg3) override;
190f2a
   int masked_watch_num_registers (CORE_ADDR arg0, CORE_ADDR arg1) override;
190f2a
   int can_do_single_step () override;
190f2a
   bool supports_terminal_ours () override;
190f2a
@@ -206,9 +206,9 @@ struct debug_target : public target_ops
190f2a
   int have_steppable_watchpoint () override;
190f2a
   bool have_continuable_watchpoint () override;
190f2a
   bool stopped_data_address (CORE_ADDR *arg0) override;
190f2a
-  bool watchpoint_addr_within_range (CORE_ADDR arg0, CORE_ADDR arg1, int arg2) override;
190f2a
+  bool watchpoint_addr_within_range (CORE_ADDR arg0, CORE_ADDR arg1, LONGEST arg2) override;
190f2a
   int region_ok_for_hw_watchpoint (CORE_ADDR arg0, LONGEST arg1) override;
190f2a
-  bool can_accel_watchpoint_condition (CORE_ADDR arg0, int arg1, int arg2, struct expression *arg3) override;
190f2a
+  bool can_accel_watchpoint_condition (CORE_ADDR arg0, LONGEST arg1, int arg2, struct expression *arg3) override;
190f2a
   int masked_watch_num_registers (CORE_ADDR arg0, CORE_ADDR arg1) override;
190f2a
   int can_do_single_step () override;
190f2a
   bool supports_terminal_ours () override;
190f2a
@@ -1068,19 +1068,19 @@ debug_target::stopped_data_address (CORE_ADDR *arg0)
190f2a
 }
190f2a
 
190f2a
 bool
190f2a
-target_ops::watchpoint_addr_within_range (CORE_ADDR arg0, CORE_ADDR arg1, int arg2)
190f2a
+target_ops::watchpoint_addr_within_range (CORE_ADDR arg0, CORE_ADDR arg1, LONGEST arg2)
190f2a
 {
190f2a
   return this->beneath ()->watchpoint_addr_within_range (arg0, arg1, arg2);
190f2a
 }
190f2a
 
190f2a
 bool
190f2a
-dummy_target::watchpoint_addr_within_range (CORE_ADDR arg0, CORE_ADDR arg1, int arg2)
190f2a
+dummy_target::watchpoint_addr_within_range (CORE_ADDR arg0, CORE_ADDR arg1, LONGEST arg2)
190f2a
 {
190f2a
   return default_watchpoint_addr_within_range (this, arg0, arg1, arg2);
190f2a
 }
190f2a
 
190f2a
 bool
190f2a
-debug_target::watchpoint_addr_within_range (CORE_ADDR arg0, CORE_ADDR arg1, int arg2)
190f2a
+debug_target::watchpoint_addr_within_range (CORE_ADDR arg0, CORE_ADDR arg1, LONGEST arg2)
190f2a
 {
190f2a
   bool result;
190f2a
   fprintf_unfiltered (gdb_stdlog, "-> %s->watchpoint_addr_within_range (...)\n", this->beneath ()->shortname ());
190f2a
@@ -1090,7 +1090,7 @@ debug_target::watchpoint_addr_within_range (CORE_ADDR arg0, CORE_ADDR arg1, int
190f2a
   fputs_unfiltered (", ", gdb_stdlog);
190f2a
   target_debug_print_CORE_ADDR (arg1);
190f2a
   fputs_unfiltered (", ", gdb_stdlog);
190f2a
-  target_debug_print_int (arg2);
190f2a
+  target_debug_print_LONGEST (arg2);
190f2a
   fputs_unfiltered (") = ", gdb_stdlog);
190f2a
   target_debug_print_bool (result);
190f2a
   fputs_unfiltered ("\n", gdb_stdlog);
190f2a
@@ -1126,19 +1126,19 @@ debug_target::region_ok_for_hw_watchpoint (CORE_ADDR arg0, LONGEST arg1)
190f2a
 }
190f2a
 
190f2a
 bool
190f2a
-target_ops::can_accel_watchpoint_condition (CORE_ADDR arg0, int arg1, int arg2, struct expression *arg3)
190f2a
+target_ops::can_accel_watchpoint_condition (CORE_ADDR arg0, LONGEST arg1, int arg2, struct expression *arg3)
190f2a
 {
190f2a
   return this->beneath ()->can_accel_watchpoint_condition (arg0, arg1, arg2, arg3);
190f2a
 }
190f2a
 
190f2a
 bool
190f2a
-dummy_target::can_accel_watchpoint_condition (CORE_ADDR arg0, int arg1, int arg2, struct expression *arg3)
190f2a
+dummy_target::can_accel_watchpoint_condition (CORE_ADDR arg0, LONGEST arg1, int arg2, struct expression *arg3)
190f2a
 {
190f2a
   return false;
190f2a
 }
190f2a
 
190f2a
 bool
190f2a
-debug_target::can_accel_watchpoint_condition (CORE_ADDR arg0, int arg1, int arg2, struct expression *arg3)
190f2a
+debug_target::can_accel_watchpoint_condition (CORE_ADDR arg0, LONGEST arg1, int arg2, struct expression *arg3)
190f2a
 {
190f2a
   bool result;
190f2a
   fprintf_unfiltered (gdb_stdlog, "-> %s->can_accel_watchpoint_condition (...)\n", this->beneath ()->shortname ());
190f2a
@@ -1146,7 +1146,7 @@ debug_target::can_accel_watchpoint_condition (CORE_ADDR arg0, int arg1, int arg2
190f2a
   fprintf_unfiltered (gdb_stdlog, "<- %s->can_accel_watchpoint_condition (", this->beneath ()->shortname ());
190f2a
   target_debug_print_CORE_ADDR (arg0);
190f2a
   fputs_unfiltered (", ", gdb_stdlog);
190f2a
-  target_debug_print_int (arg1);
190f2a
+  target_debug_print_LONGEST (arg1);
190f2a
   fputs_unfiltered (", ", gdb_stdlog);
190f2a
   target_debug_print_int (arg2);
190f2a
   fputs_unfiltered (", ", gdb_stdlog);
190f2a
diff --git a/gdb/target.c b/gdb/target.c
190f2a
--- a/gdb/target.c
190f2a
+++ b/gdb/target.c
190f2a
@@ -56,7 +56,7 @@ static void generic_tls_error (void) ATTRIBUTE_NORETURN;
190f2a
 static void default_terminal_info (struct target_ops *, const char *, int);
190f2a
 
190f2a
 static int default_watchpoint_addr_within_range (struct target_ops *,
190f2a
-						 CORE_ADDR, CORE_ADDR, int);
190f2a
+						 CORE_ADDR, CORE_ADDR, LONGEST);
190f2a
 
190f2a
 static int default_region_ok_for_hw_watchpoint (struct target_ops *,
190f2a
 						CORE_ADDR, LONGEST);
190f2a
@@ -3189,7 +3189,7 @@ default_region_ok_for_hw_watchpoint (struct target_ops *self,
190f2a
 static int
190f2a
 default_watchpoint_addr_within_range (struct target_ops *target,
190f2a
 				      CORE_ADDR addr,
190f2a
-				      CORE_ADDR start, int length)
190f2a
+				      CORE_ADDR start, LONGEST length)
190f2a
 {
190f2a
   return addr >= start && addr < start + length;
190f2a
 }
190f2a
diff --git a/gdb/target.h b/gdb/target.h
190f2a
--- a/gdb/target.h
190f2a
+++ b/gdb/target.h
190f2a
@@ -557,7 +557,7 @@ struct target_ops
190f2a
       TARGET_DEFAULT_RETURN (false);
190f2a
     virtual bool stopped_data_address (CORE_ADDR *)
190f2a
       TARGET_DEFAULT_RETURN (false);
190f2a
-    virtual bool watchpoint_addr_within_range (CORE_ADDR, CORE_ADDR, int)
190f2a
+    virtual bool watchpoint_addr_within_range (CORE_ADDR, CORE_ADDR, LONGEST)
190f2a
       TARGET_DEFAULT_FUNC (default_watchpoint_addr_within_range);
190f2a
 
190f2a
     /* Documentation of this routine is provided with the corresponding
190f2a
@@ -565,7 +565,7 @@ struct target_ops
190f2a
     virtual int region_ok_for_hw_watchpoint (CORE_ADDR, LONGEST)
190f2a
       TARGET_DEFAULT_FUNC (default_region_ok_for_hw_watchpoint);
190f2a
 
190f2a
-    virtual bool can_accel_watchpoint_condition (CORE_ADDR, int, int,
190f2a
+    virtual bool can_accel_watchpoint_condition (CORE_ADDR, LONGEST, int,
190f2a
 						 struct expression *)
190f2a
       TARGET_DEFAULT_RETURN (false);
190f2a
     virtual int masked_watch_num_registers (CORE_ADDR, CORE_ADDR)