Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Yao Qi <yao@codesourcery.com>
To: <gdb-patches@sourceware.org>
Subject: [PATCH v2 0/5] mips hardware watchpoint support in gdbserver
Date: Sat, 29 Jun 2013 03:11:00 -0000	[thread overview]
Message-ID: <1372475427-24862-1-git-send-email-yao@codesourcery.com> (raw)
In-Reply-To: <1369881867-11372-1-git-send-email-yao@codesourcery.com>

Here is the V2 of this patch series which is to post Jie and Dan's work
to support mips hardware watchpoint in gdbserver.  There are several
changes compared with V1:

 - Keep these watchpoint-related structures as the fall back when
PTRACE_GET_WATCH_REGS is not defined.
 - Re-org the patch series as Maciej suggested, split the function
renaming patch out of the function moving patch.
 - Share 'enum target_hw_bp_type' in GDB and GDBserver, so that a new
file common/break-common.h is created.
 - Some typos and code format issues are fixed.
 - Rename 'debug register' to 'watch register' in comments and code.

Patch 1/5 is to share 'enum target_hw_bp_type' and patch 2/5 is to
include asm/ptrace.h in mips-linux-nat.c, but keep watchpoint related
structures as a fall back.  Patch 3/5 is to refactor mips-linux-nat.c,
rename functions and update comments, etc.  Patch 4/5 is to move mips h/w
wathcpoint stuff to common, and Patch 5/5 is to support h/w watchpoint
in gdbserver for MIPS.

The whole series is tested on a MIPS board with only one hardware
watchpoint register.  I pick this patch up during the test, because
it fixes an internal error, so this patch series depend on it.

  [RFC] Catch exception after stepped over watchpoint.
  http://sourceware.org/ml/gdb-patches/2013-06/msg00866.html

*** BLURB HERE ***

Yao Qi (5):
  Share 'enum target_hw_bp_type' in GDB and GDBserver.
  Include asm/ptrace.h in mips-linux-nat.c
  Refactor in mips-linux-nat.c
  Move mips hardware watchpoint stuff to common/
  MIPS GDBserver watchpoint

 gdb/Makefile.in                |    6 +-
 gdb/NEWS                       |    3 +
 gdb/breakpoint.h               |   10 +-
 gdb/common/break-common.h      |   30 +++
 gdb/common/mips-linux-watch.c  |  352 ++++++++++++++++++++++++++++++++
 gdb/common/mips-linux-watch.h  |  129 ++++++++++++
 gdb/config/mips/linux.mh       |    2 +-
 gdb/gdbserver/Makefile.in      |    7 +-
 gdb/gdbserver/configure.srv    |    1 +
 gdb/gdbserver/i386-low.c       |    9 +-
 gdb/gdbserver/linux-mips-low.c |  366 +++++++++++++++++++++++++++++++++
 gdb/mips-linux-nat.c           |  439 +++------------------------------------
 12 files changed, 929 insertions(+), 425 deletions(-)
 create mode 100644 gdb/common/break-common.h
 create mode 100644 gdb/common/mips-linux-watch.c
 create mode 100644 gdb/common/mips-linux-watch.h

-- 
1.7.7.6


  parent reply	other threads:[~2013-06-29  3:11 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-30  2:44 [PATCH 0/3] " Yao Qi
2013-05-30  2:44 ` [PATCH 1/3] Include asm/ptrace.h in mips-linux-nat.c Yao Qi
2013-06-13 17:49   ` Maciej W. Rozycki
2013-06-14  6:53     ` Yao Qi
2013-06-14 12:53       ` Maciej W. Rozycki
2013-06-20 19:40         ` Pedro Alves
2013-06-20 20:45           ` Maciej W. Rozycki
2013-06-21 14:58             ` Pedro Alves
2013-06-17 16:04     ` Maciej W. Rozycki
2013-05-30  2:44 ` [PATCH 3/3] MIPS h/w watchpoint in GDBserver Yao Qi
2013-06-13  8:20   ` Yao Qi
2013-06-13 13:09     ` Eli Zaretskii
2013-06-13 16:56     ` Pedro Alves
2013-06-19 22:22     ` Maciej W. Rozycki
2013-06-21 15:00       ` Pedro Alves
2013-05-30  2:44 ` [PATCH 2/3] Move mips hardware watchpoint stuff to common/ Yao Qi
2013-06-13  4:12   ` Yao Qi
2013-06-19 22:05     ` Maciej W. Rozycki
2013-06-20 14:21       ` Yao Qi
2013-06-20 15:27         ` Maciej W. Rozycki
2013-06-20 17:50           ` Joel Brobecker
2013-06-21  8:03             ` Maciej W. Rozycki
2013-06-21 15:55               ` Joel Brobecker
2013-05-30 12:29 ` [PATCH 0/3] mips hardware watchpoint support in gdbserver Maciej W. Rozycki
2013-05-30 18:06 ` Pedro Alves
2013-05-30 18:08   ` Pedro Alves
2013-06-29  3:11 ` Yao Qi [this message]
2013-06-29  3:11   ` [PATCH 1/5] Share 'enum target_hw_bp_type' in GDB and GDBserver Yao Qi
2013-07-24  0:26     ` Maciej W. Rozycki
2013-07-24 14:04       ` Tom Tromey
2013-07-28  0:41         ` Yao Qi
2013-06-29  3:11   ` [PATCH 5/5] MIPS GDBserver watchpoint Yao Qi
2013-06-29 15:20     ` Eli Zaretskii
2013-07-24  0:35     ` Maciej W. Rozycki
2013-07-25  0:17       ` Yao Qi
2013-07-25 21:20         ` Maciej W. Rozycki
2013-07-28  0:49           ` Yao Qi
2013-07-24 18:11     ` Pedro Alves
2013-06-29  3:11   ` [PATCH 2/5] Include asm/ptrace.h in mips-linux-nat.c Yao Qi
2013-07-24  0:26     ` Maciej W. Rozycki
2013-07-28  0:43       ` Yao Qi
2013-06-29  3:11   ` [PATCH 3/5] Refactor " Yao Qi
2013-07-24  0:27     ` Maciej W. Rozycki
2013-07-28  0:44       ` Yao Qi
2013-06-29  8:01   ` [PATCH 4/5] Move mips hardware watchpoint stuff to common/ Yao Qi
2013-07-24  0:31     ` Maciej W. Rozycki
2013-07-24  2:08       ` Yao Qi
2013-07-24 18:09         ` Pedro Alves
2013-07-25  0:07       ` Yao Qi
2013-07-25 21:17         ` Maciej W. Rozycki
2013-07-28  0:47           ` Yao Qi
2013-07-22  1:11   ` [PATCH v2 0/5] mips hardware watchpoint support in gdbserver Yao Qi

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=1372475427-24862-1-git-send-email-yao@codesourcery.com \
    --to=yao@codesourcery.com \
    --cc=gdb-patches@sourceware.org \
    /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