public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: Don Breazeal <donb@codesourcery.com>
To: <gdb-patches@sourceware.org>, <vidyapraveen@arm.com>
Subject: [commit/obv][PATCH] Fix build errors on arm, aarch64, and mips.
Date: Thu, 14 May 2015 20:25:00 -0000	[thread overview]
Message-ID: <1431635113-30805-1-git-send-email-donb@codesourcery.com> (raw)

Committed as obvious:

Fix build errors introduced by
https://sourceware.org/ml/gdb-patches/2015-05/msg00281.html, which
didn't account for the change of the name of the struct process_info
field 'private' to 'priv' made in
https://sourceware.org/ml/gdb-patches/2015-02/msg00829.html.

gdb/gdbserver/ChangeLog:
2015-05-14  Don Breazeal  <donb@codesourcery.com>

	* linux-aarch64-low.c (aarch64_linux_new_fork): Change reference
	to process_info.private to process_info.priv.
	* linux-arm-low.c (arm_new_fork): Likewise.
	* linux-mips-low.c (mips_linux_new_fork): Likewise.

---
 gdb/gdbserver/linux-aarch64-low.c | 10 +++++-----
 gdb/gdbserver/linux-arm-low.c     | 12 ++++++------
 gdb/gdbserver/linux-mips-low.c    | 12 ++++++------
 3 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/gdb/gdbserver/linux-aarch64-low.c b/gdb/gdbserver/linux-aarch64-low.c
index 7e6e9ac..043458d 100644
--- a/gdb/gdbserver/linux-aarch64-low.c
+++ b/gdb/gdbserver/linux-aarch64-low.c
@@ -1140,10 +1140,10 @@ aarch64_linux_new_fork (struct process_info *parent,
 			struct process_info *child)
 {
   /* These are allocated by linux_add_process.  */
-  gdb_assert (parent->private != NULL
-	      && parent->private->arch_private != NULL);
-  gdb_assert (child->private != NULL
-	      && child->private->arch_private != NULL);
+  gdb_assert (parent->priv != NULL
+	      && parent->priv->arch_private != NULL);
+  gdb_assert (child->priv != NULL
+	      && child->priv->arch_private != NULL);
 
   /* Linux kernel before 2.6.33 commit
      72f674d203cd230426437cdcf7dd6f681dad8b0d
@@ -1159,7 +1159,7 @@ aarch64_linux_new_fork (struct process_info *parent,
      in the end before detaching the forked off process, thus making
      this compatible with older Linux kernels too.  */
 
-  *child->private->arch_private = *parent->private->arch_private;
+  *child->priv->arch_private = *parent->priv->arch_private;
 }
 
 /* Called when resuming a thread.
diff --git a/gdb/gdbserver/linux-arm-low.c b/gdb/gdbserver/linux-arm-low.c
index 3420dea..dd77b48 100644
--- a/gdb/gdbserver/linux-arm-low.c
+++ b/gdb/gdbserver/linux-arm-low.c
@@ -720,17 +720,17 @@ arm_new_thread (struct lwp_info *lwp)
 static void
 arm_new_fork (struct process_info *parent, struct process_info *child)
 {
-  struct arch_process_info *parent_proc_info = parent->private->arch_private;
-  struct arch_process_info *child_proc_info = child->private->arch_private;
+  struct arch_process_info *parent_proc_info = parent->priv->arch_private;
+  struct arch_process_info *child_proc_info = child->priv->arch_private;
   struct lwp_info *child_lwp;
   struct arch_lwp_info *child_lwp_info;
   int i;
 
   /* These are allocated by linux_add_process.  */
-  gdb_assert (parent->private != NULL
-	      && parent->private->arch_private != NULL);
-  gdb_assert (child->private != NULL
-	      && child->private->arch_private != NULL);
+  gdb_assert (parent->priv != NULL
+	      && parent->priv->arch_private != NULL);
+  gdb_assert (child->priv != NULL
+	      && child->priv->arch_private != NULL);
 
   /* Linux kernel before 2.6.33 commit
      72f674d203cd230426437cdcf7dd6f681dad8b0d
diff --git a/gdb/gdbserver/linux-mips-low.c b/gdb/gdbserver/linux-mips-low.c
index 4601ad0..1695c4c 100644
--- a/gdb/gdbserver/linux-mips-low.c
+++ b/gdb/gdbserver/linux-mips-low.c
@@ -376,10 +376,10 @@ mips_linux_new_fork (struct process_info *parent,
   struct mips_watchpoint *wp;
 
   /* These are allocated by linux_add_process.  */
-  gdb_assert (parent->private != NULL
-	      && parent->private->arch_private != NULL);
-  gdb_assert (child->private != NULL
-	      && child->private->arch_private != NULL);
+  gdb_assert (parent->priv != NULL
+	      && parent->priv->arch_private != NULL);
+  gdb_assert (child->priv != NULL
+	      && child->priv->arch_private != NULL);
 
   /* Linux kernel before 2.6.33 commit
      72f674d203cd230426437cdcf7dd6f681dad8b0d
@@ -395,8 +395,8 @@ mips_linux_new_fork (struct process_info *parent,
      in the end before detaching the forked off process, thus making
      this compatible with older Linux kernels too.  */
 
-  parent_private = parent->private->arch_private;
-  child_private = child->private->arch_private;
+  parent_private = parent->priv->arch_private;
+  child_private = child->priv->arch_private;
 
   child_private->watch_readback_valid = parent_private->watch_readback_valid;
   child_private->watch_readback = parent_private->watch_readback;
-- 
1.8.1.1

                 reply	other threads:[~2015-05-14 20:25 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1431635113-30805-1-git-send-email-donb@codesourcery.com \
    --to=donb@codesourcery.com \
    --cc=gdb-patches@sourceware.org \
    --cc=vidyapraveen@arm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).