From: "karthikeyan.s" <informkarthik@gmail.com>
To: Matthew Gretton-Dann <matthew.gretton-dann@arm.com>
Cc: Will Deacon <will.deacon@arm.com>, gdb@sourceware.org
Subject: Re: [HELP]Can GDB for ARM set hard breakpoint when native debugging?
Date: Thu, 03 Jun 2010 11:39:00 -0000 [thread overview]
Message-ID: <AANLkTimkDkrZdBqyJhYsAzSfAXf4gHKiGSBM_yN41Ykf@mail.gmail.com> (raw)
In-Reply-To: <1275405490.22316.12.camel@e102319-lin.cambridge.arm.com>
Hi Will,
Just in case you are going to re-use most of the code for 2.6.35.
In hw_breakpoint.c, the handlers do not release the locks on wtpt/bkpt
not matched.
Should rather be:
if (val != (unpredictable & ~0x3)) {
rcu_read_unlock();
continue;
}
Otherwise I get a "schedule while atomic" bug.
-Karthik
On Tue, Jun 1, 2010 at 8:48 PM, Matthew Gretton-Dann
<matthew.gretton-dann@arm.com> wrote:
> On Tue, 2010-06-01 at 20:35 +0530, karthikeyan.s wrote:
>> Hi,
>> > I've had a go at implementing a backend for hardware watchpoints and breakpoints in the Kernel:
>> > http://lists.infradead.org/pipermail/linux-arm-kernel/2010-March/01117
>> > 0.html
>> Thanks.
>>
>> >I'm hoping to submit a newer version of this patchset against 2.6.35.
>> >I'll add you on CC when I post it to the list. The main changes will be:
>> Thanks again! Looking forward to it.
>>
>> Going through your patch and will be trying it soon. At first sight,
>> seems the gdb changes aren't there in that link. Can you point me to
>> the gdb changes?
>
> We don't have any gdb patches in a releasable state at this moment in
> time. When the ptrace interface settles down in the 2.6.35 Linux
> kernel, and the updated kernel patch has been submitted, we will submit
> patches to take advantage of hardware breakpoints to gdb.
>
> Thanks,
>
> Matt
>
> --
> Matthew Gretton-Dann
> Principal Engineer - PDSW Tools
> ARM Ltd
>
>
--
---
S. Karthikeyan | +919980814745
---
next prev parent reply other threads:[~2010-06-03 11:39 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-01 15:06 karthikeyan.s
2010-06-01 15:18 ` Matthew Gretton-Dann
2010-06-03 11:39 ` karthikeyan.s [this message]
2010-06-03 11:54 ` Will Deacon
[not found] ` <-3629479195726626766@unknownmsgid>
2010-06-10 14:40 ` karthikeyan.s
2010-06-10 15:03 ` Will Deacon
[not found] ` <-2775908164685299161@unknownmsgid>
2010-06-13 13:00 ` karthikeyan.s
-- strict thread matches above, loose matches on Subject: below --
2010-05-28 6:00 xingxing pan
2010-05-28 6:22 ` xingxing pan
2010-05-28 9:28 ` Matthew Gretton-Dann
[not found] ` <113ACA888B71994BB56E5CF3704953486BAA6A18D8@BGMAIL02.nvidia.com>
2010-06-01 9:06 ` Will Deacon
2010-05-28 11:20 ` Simon Richter
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=AANLkTimkDkrZdBqyJhYsAzSfAXf4gHKiGSBM_yN41Ykf@mail.gmail.com \
--to=informkarthik@gmail.com \
--cc=gdb@sourceware.org \
--cc=matthew.gretton-dann@arm.com \
--cc=will.deacon@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