All of lore.kernel.org
 help / color / mirror / Atom feed
From: will.deacon@arm.com (Will Deacon)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/2] Fix ptrace software breakpoints
Date: Tue, 27 Jul 2010 10:43:42 +0100	[thread overview]
Message-ID: <002401cb2d70$33206820$99613860$@deacon@arm.com> (raw)
In-Reply-To: <4C4E36CE.3080108@ti.com>

Hi Rob,

> > Will Deacon (2):
> >    ARM: errata: TLBIASIDIS and TLBIMVAIS operations can broadcast a
> >      faulty ASID
> >    ARM: flush_ptrace_access: invalidate all I-caches
> >
> >   arch/arm/Kconfig                |   12 ++++++++++++
> >   arch/arm/include/asm/tlbflush.h |    8 ++++++++
> >   arch/arm/mm/flush.c             |    4 ++--
> >   3 files changed, 22 insertions(+), 2 deletions(-)
> >
> 
> Tested on omap4430 (cortex-a9 SMP).. with this patchset I have yet to
> see it fail, in regular use nor in the test case I wrote to show the
> problem (git://github.com/robclark/gdb-test.git).  Without these
> patches, I'd see SIGILL and missed breakpoints 90+% of the time in
> regular use of gdb (basically gdb was unusable unless I booted with
> nosmp or disabled cache), and 100% of the time in my gdb-test.
> 
> Tested-by: Rob Clark<rob@ti.com>

Thanks for testing this. I've seen the same results on my Versatile
Express board running Ubuntu Lucid.

Russell - can I submit these to the patch system please or do you have
any outstanding concerns?

Cheers,

Will

 

      reply	other threads:[~2010-07-27  9:43 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-15 15:53 [PATCH 0/2] Fix ptrace software breakpoints Will Deacon
2010-07-15 15:53 ` [PATCH 1/2] ARM: errata: TLBIASIDIS and TLBIMVAIS operations can broadcast a faulty ASID Will Deacon
2010-07-15 15:53   ` [PATCH 2/2] ARM: flush_ptrace_access: invalidate all I-caches Will Deacon
2010-07-15 16:32     ` Russell King - ARM Linux
2010-07-15 16:43       ` Will Deacon
2010-07-15 16:54         ` Catalin Marinas
2010-07-19 16:23         ` Will Deacon
2010-08-04 11:24       ` Will Deacon
2010-07-15 16:21   ` [PATCH 1/2] ARM: errata: TLBIASIDIS and TLBIMVAIS operations can broadcast a faulty ASID Shilimkar, Santosh
2010-07-15 16:32     ` Will Deacon
2010-07-15 16:43       ` Shilimkar, Santosh
2010-07-16  4:15 ` [PATCH 0/2] Fix ptrace software breakpoints Rob Clark
2010-07-20 17:23   ` Will Deacon
2010-07-27  1:30 ` Rob Clark
2010-07-27  9:43   ` Will Deacon [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='002401cb2d70$33206820$99613860$@deacon@arm.com' \
    --to=will.deacon@arm.com \
    --cc=linux-arm-kernel@lists.infradead.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.