All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Daney <ddaney@avtrex.com>
To: David VomLehn <dvomlehn@cisco.com>
Cc: linux-mips@linux-mips.org
Subject: Re: Patch 0/6] MIPS: Hardware watch register support for gdb (version 5).
Date: Tue, 23 Sep 2008 12:02:35 -0700	[thread overview]
Message-ID: <48D93D4B.1010707@avtrex.com> (raw)
In-Reply-To: <48D93A23.3020302@cisco.com>

David VomLehn wrote:
> David Daney wrote:
> 
>> To follow is my fifth pass at MIPS watch register support.
> 
> I'd like to add a feature request, understanding fully that the response 
> may very well be, "interesting idea, now show us a patch"--can we have 
> an interface that would allow kernel-space allocation of watchpoint 
> registers?
> 
> The rationale is that we have found it quite useful to have kernel and 
> driver code set watchpoints for debugging purposes. I would not expect 
> that kernel space code could grab watchpoint registers already in use by 
> ptrace, and that ptrace would be free to allocate all watchpoint 
> registers not in use for kernel space purposes, i.e. there would be no 
> watchpoint registers permanently allocated for kernel space usage.
> 

The current patch has an artificial limit of 4 registers that it would use.

I think adding this feature would be possible, but would rather make it 
part of a follow-on patch.

You would limit the number of registers available to ptrace and then 
handle the others specially in the do_watch handler.

David Daney

      reply	other threads:[~2008-09-23 19:03 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-09-23  7:02 Patch 0/6] MIPS: Hardware watch register support for gdb (version 5) David Daney
2008-09-23  7:04 ` [Patch 1/6] MIPS: Add HARDWARE_WATCHPOINTS configure option David Daney
2008-09-23  7:05 ` [Patch 2/6] MIPS: Add HARDWARE_WATCHPOINTS definitions and support code David Daney
2008-09-23  7:07 ` [Patch 3/6] MIPS: Probe watch registers and report configuration David Daney
2008-09-23  7:08 ` [Patch 4/6] MIPS: Watch exception handling for HARDWARE_WATCHPOINTS David Daney
2008-09-23  7:09 ` [Patch 5/6] MIPS: Scheduler support " David Daney
2008-09-23  7:11 ` [Patch 6/6] MIPS: Ptrace " David Daney
2008-09-23 18:49 ` Patch 0/6] MIPS: Hardware watch register support for gdb (version 5) David VomLehn
2008-09-23 19:02   ` David Daney [this message]

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=48D93D4B.1010707@avtrex.com \
    --to=ddaney@avtrex.com \
    --cc=dvomlehn@cisco.com \
    --cc=linux-mips@linux-mips.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.