Re: [PATCH v2 0/3] ppc32/kprobe: Fix a bug for kprobe stwu r1

2012-01-10 Thread tiejun.chen
Tiejun Chen wrote:
 Changes from V1:
 
 * use memcpy simply to withdraw copy_exc_stack
 * add !(regs-msr  MSR_PR)) and
   WARN_ON(test_thread_flag(TIF_EMULATE_STACK_STORE));
   to make sure we're in goot path.
 * move this migration process inside 'restore'
 * clear TIF flag atomically 

Ben,

Is this series OK?

Thanks
Tiejun

 
 Tiejun Chen (3):
   powerpc/kprobe: introduce a new thread flag
   ppc32/kprobe: complete kprobe and migrate exception frame
   ppc32/kprobe: don't emulate store when kprobe stwu r1
 
  arch/powerpc/include/asm/thread_info.h |3 ++
  arch/powerpc/kernel/entry_32.S |   35 
 
  arch/powerpc/lib/sstep.c   |   25 +-
  3 files changed, 61 insertions(+), 2 deletions(-)
 
 Tiejun
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH v2 0/3] ppc32/kprobe: Fix a bug for kprobe stwu r1

2012-01-10 Thread Benjamin Herrenschmidt
On Tue, 2012-01-10 at 17:15 +0800, tiejun.chen wrote:
 Tiejun Chen wrote:
  Changes from V1:
  
  * use memcpy simply to withdraw copy_exc_stack
  * add !(regs-msr  MSR_PR)) and
  WARN_ON(test_thread_flag(TIF_EMULATE_STACK_STORE));
to make sure we're in goot path.
  * move this migration process inside 'restore'
  * clear TIF flag atomically 
 
 Ben,
 
 Is this series OK?

Not completely sorry. I was about to send you some comments a couple of
weeks ago then had to leave urgently and forgot about it, then started
rewriting your code and didn't finish :-)

I'll give you some more feedback asap, sorry about that. It's getting
there but there's a couple of things I'd like to see done a bit
differently.

Cheers,
Ben.

 Thanks
 Tiejun
 
  
  Tiejun Chen (3):
powerpc/kprobe: introduce a new thread flag
ppc32/kprobe: complete kprobe and migrate exception frame
ppc32/kprobe: don't emulate store when kprobe stwu r1
  
   arch/powerpc/include/asm/thread_info.h |3 ++
   arch/powerpc/kernel/entry_32.S |   35 
  
   arch/powerpc/lib/sstep.c   |   25 +-
   3 files changed, 61 insertions(+), 2 deletions(-)
  
  Tiejun


___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


[PATCH v2 0/3] ppc32/kprobe: Fix a bug for kprobe stwu r1

2011-12-15 Thread Tiejun Chen
Changes from V1:

* use memcpy simply to withdraw copy_exc_stack
* add !(regs-msr  MSR_PR)) and
WARN_ON(test_thread_flag(TIF_EMULATE_STACK_STORE));
  to make sure we're in goot path.
* move this migration process inside 'restore'
* clear TIF flag atomically 

Tiejun Chen (3):
  powerpc/kprobe: introduce a new thread flag
  ppc32/kprobe: complete kprobe and migrate exception frame
  ppc32/kprobe: don't emulate store when kprobe stwu r1

 arch/powerpc/include/asm/thread_info.h |3 ++
 arch/powerpc/kernel/entry_32.S |   35 
 arch/powerpc/lib/sstep.c   |   25 +-
 3 files changed, 61 insertions(+), 2 deletions(-)

Tiejun
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev