Re: [PATCH] SAMPLES: kprobe_example: Make it print something on ARM.

2015-02-04 Thread long.wanglong
On 2015/2/4 11:17, Masami Hiramatsu wrote:
> (2015/02/04 11:56), Wang Long wrote:
>> This KProbes example is a little useless if it doesn't print anything.
>> For ARM print similar messages to those produced on x86 and PPC.
> 
> BTW, I guess similar update required for s390, doesn't it?
> 

yes, because i am not familiar with s390, someone else may update this example 
for s390.

Best Regards
Wang Long

>>
>> Signed-off-by: Wang Long 
> 
> Acked-by: Masami Hiramatsu 
> 
> Thank you!
> 
>> ---
>>  samples/kprobes/kprobe_example.c | 9 +
>>  1 file changed, 9 insertions(+)
>>
>> diff --git a/samples/kprobes/kprobe_example.c 
>> b/samples/kprobes/kprobe_example.c
>> index 366db1a..eba9f1a 100644
>> --- a/samples/kprobes/kprobe_example.c
>> +++ b/samples/kprobes/kprobe_example.c
>> @@ -42,6 +42,11 @@ static int handler_pre(struct kprobe *p, struct pt_regs 
>> *regs)
>>  " ex1 = 0x%lx\n",
>>  p->addr, regs->pc, regs->ex1);
>>  #endif
>> +#ifdef CONFIG_ARM
>> +printk(KERN_INFO "pre_handler: p->addr = 0x%p, ARM_pc = 0x%lx,"
>> +" ARM_cpsr = 0x%lx\n",
>> +p->addr, regs->ARM_pc, regs->ARM_cpsr);
>> +#endif
>>  
>>  /* A dump_stack() here will give a stack backtrace */
>>  return 0;
>> @@ -67,6 +72,10 @@ static void handler_post(struct kprobe *p, struct pt_regs 
>> *regs,
>>  printk(KERN_INFO "post_handler: p->addr = 0x%p, ex1 = 0x%lx\n",
>>  p->addr, regs->ex1);
>>  #endif
>> +#ifdef CONFIG_ARM
>> +printk(KERN_INFO "post_handler: p->addr = 0x%p, ARM_cpsr = 0x%lx\n",
>> +p->addr, regs->ARM_cpsr);
>> +#endif
>>  }
>>  
>>  /*
>>
> 
> 


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] SAMPLES: kprobe_example: Make it print something on ARM.

2015-02-04 Thread long.wanglong
On 2015/2/4 11:17, Masami Hiramatsu wrote:
 (2015/02/04 11:56), Wang Long wrote:
 This KProbes example is a little useless if it doesn't print anything.
 For ARM print similar messages to those produced on x86 and PPC.
 
 BTW, I guess similar update required for s390, doesn't it?
 

yes, because i am not familiar with s390, someone else may update this example 
for s390.

Best Regards
Wang Long


 Signed-off-by: Wang Long long.wangl...@huawei.com
 
 Acked-by: Masami Hiramatsu masami.hiramatsu...@hitachi.com
 
 Thank you!
 
 ---
  samples/kprobes/kprobe_example.c | 9 +
  1 file changed, 9 insertions(+)

 diff --git a/samples/kprobes/kprobe_example.c 
 b/samples/kprobes/kprobe_example.c
 index 366db1a..eba9f1a 100644
 --- a/samples/kprobes/kprobe_example.c
 +++ b/samples/kprobes/kprobe_example.c
 @@ -42,6 +42,11 @@ static int handler_pre(struct kprobe *p, struct pt_regs 
 *regs)
   ex1 = 0x%lx\n,
  p-addr, regs-pc, regs-ex1);
  #endif
 +#ifdef CONFIG_ARM
 +printk(KERN_INFO pre_handler: p-addr = 0x%p, ARM_pc = 0x%lx,
 + ARM_cpsr = 0x%lx\n,
 +p-addr, regs-ARM_pc, regs-ARM_cpsr);
 +#endif
  
  /* A dump_stack() here will give a stack backtrace */
  return 0;
 @@ -67,6 +72,10 @@ static void handler_post(struct kprobe *p, struct pt_regs 
 *regs,
  printk(KERN_INFO post_handler: p-addr = 0x%p, ex1 = 0x%lx\n,
  p-addr, regs-ex1);
  #endif
 +#ifdef CONFIG_ARM
 +printk(KERN_INFO post_handler: p-addr = 0x%p, ARM_cpsr = 0x%lx\n,
 +p-addr, regs-ARM_cpsr);
 +#endif
  }
  
  /*

 
 


--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] SAMPLES: kprobe_example: Make it print something on ARM.

2015-02-03 Thread Masami Hiramatsu
(2015/02/04 11:56), Wang Long wrote:
> This KProbes example is a little useless if it doesn't print anything.
> For ARM print similar messages to those produced on x86 and PPC.

BTW, I guess similar update required for s390, doesn't it?

> 
> Signed-off-by: Wang Long 

Acked-by: Masami Hiramatsu 

Thank you!

> ---
>  samples/kprobes/kprobe_example.c | 9 +
>  1 file changed, 9 insertions(+)
> 
> diff --git a/samples/kprobes/kprobe_example.c 
> b/samples/kprobes/kprobe_example.c
> index 366db1a..eba9f1a 100644
> --- a/samples/kprobes/kprobe_example.c
> +++ b/samples/kprobes/kprobe_example.c
> @@ -42,6 +42,11 @@ static int handler_pre(struct kprobe *p, struct pt_regs 
> *regs)
>   " ex1 = 0x%lx\n",
>   p->addr, regs->pc, regs->ex1);
>  #endif
> +#ifdef CONFIG_ARM
> + printk(KERN_INFO "pre_handler: p->addr = 0x%p, ARM_pc = 0x%lx,"
> + " ARM_cpsr = 0x%lx\n",
> + p->addr, regs->ARM_pc, regs->ARM_cpsr);
> +#endif
>  
>   /* A dump_stack() here will give a stack backtrace */
>   return 0;
> @@ -67,6 +72,10 @@ static void handler_post(struct kprobe *p, struct pt_regs 
> *regs,
>   printk(KERN_INFO "post_handler: p->addr = 0x%p, ex1 = 0x%lx\n",
>   p->addr, regs->ex1);
>  #endif
> +#ifdef CONFIG_ARM
> + printk(KERN_INFO "post_handler: p->addr = 0x%p, ARM_cpsr = 0x%lx\n",
> + p->addr, regs->ARM_cpsr);
> +#endif
>  }
>  
>  /*
> 


-- 
Masami HIRAMATSU
Software Platform Research Dept. Linux Technology Research Center
Hitachi, Ltd., Yokohama Research Laboratory
E-mail: masami.hiramatsu...@hitachi.com


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] SAMPLES: kprobe_example: Make it print something on ARM.

2015-02-03 Thread Wang Long
This KProbes example is a little useless if it doesn't print anything.
For ARM print similar messages to those produced on x86 and PPC.

Signed-off-by: Wang Long 
---
 samples/kprobes/kprobe_example.c | 9 +
 1 file changed, 9 insertions(+)

diff --git a/samples/kprobes/kprobe_example.c b/samples/kprobes/kprobe_example.c
index 366db1a..eba9f1a 100644
--- a/samples/kprobes/kprobe_example.c
+++ b/samples/kprobes/kprobe_example.c
@@ -42,6 +42,11 @@ static int handler_pre(struct kprobe *p, struct pt_regs 
*regs)
" ex1 = 0x%lx\n",
p->addr, regs->pc, regs->ex1);
 #endif
+#ifdef CONFIG_ARM
+   printk(KERN_INFO "pre_handler: p->addr = 0x%p, ARM_pc = 0x%lx,"
+   " ARM_cpsr = 0x%lx\n",
+   p->addr, regs->ARM_pc, regs->ARM_cpsr);
+#endif
 
/* A dump_stack() here will give a stack backtrace */
return 0;
@@ -67,6 +72,10 @@ static void handler_post(struct kprobe *p, struct pt_regs 
*regs,
printk(KERN_INFO "post_handler: p->addr = 0x%p, ex1 = 0x%lx\n",
p->addr, regs->ex1);
 #endif
+#ifdef CONFIG_ARM
+   printk(KERN_INFO "post_handler: p->addr = 0x%p, ARM_cpsr = 0x%lx\n",
+   p->addr, regs->ARM_cpsr);
+#endif
 }
 
 /*
-- 
1.8.3.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] SAMPLES: kprobe_example: Make it print something on ARM.

2015-02-03 Thread Wang Long
This KProbes example is a little useless if it doesn't print anything.
For ARM print similar messages to those produced on x86 and PPC.

Signed-off-by: Wang Long long.wangl...@huawei.com
---
 samples/kprobes/kprobe_example.c | 9 +
 1 file changed, 9 insertions(+)

diff --git a/samples/kprobes/kprobe_example.c b/samples/kprobes/kprobe_example.c
index 366db1a..eba9f1a 100644
--- a/samples/kprobes/kprobe_example.c
+++ b/samples/kprobes/kprobe_example.c
@@ -42,6 +42,11 @@ static int handler_pre(struct kprobe *p, struct pt_regs 
*regs)
 ex1 = 0x%lx\n,
p-addr, regs-pc, regs-ex1);
 #endif
+#ifdef CONFIG_ARM
+   printk(KERN_INFO pre_handler: p-addr = 0x%p, ARM_pc = 0x%lx,
+ARM_cpsr = 0x%lx\n,
+   p-addr, regs-ARM_pc, regs-ARM_cpsr);
+#endif
 
/* A dump_stack() here will give a stack backtrace */
return 0;
@@ -67,6 +72,10 @@ static void handler_post(struct kprobe *p, struct pt_regs 
*regs,
printk(KERN_INFO post_handler: p-addr = 0x%p, ex1 = 0x%lx\n,
p-addr, regs-ex1);
 #endif
+#ifdef CONFIG_ARM
+   printk(KERN_INFO post_handler: p-addr = 0x%p, ARM_cpsr = 0x%lx\n,
+   p-addr, regs-ARM_cpsr);
+#endif
 }
 
 /*
-- 
1.8.3.4

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] SAMPLES: kprobe_example: Make it print something on ARM.

2015-02-03 Thread Masami Hiramatsu
(2015/02/04 11:56), Wang Long wrote:
 This KProbes example is a little useless if it doesn't print anything.
 For ARM print similar messages to those produced on x86 and PPC.

BTW, I guess similar update required for s390, doesn't it?

 
 Signed-off-by: Wang Long long.wangl...@huawei.com

Acked-by: Masami Hiramatsu masami.hiramatsu...@hitachi.com

Thank you!

 ---
  samples/kprobes/kprobe_example.c | 9 +
  1 file changed, 9 insertions(+)
 
 diff --git a/samples/kprobes/kprobe_example.c 
 b/samples/kprobes/kprobe_example.c
 index 366db1a..eba9f1a 100644
 --- a/samples/kprobes/kprobe_example.c
 +++ b/samples/kprobes/kprobe_example.c
 @@ -42,6 +42,11 @@ static int handler_pre(struct kprobe *p, struct pt_regs 
 *regs)
ex1 = 0x%lx\n,
   p-addr, regs-pc, regs-ex1);
  #endif
 +#ifdef CONFIG_ARM
 + printk(KERN_INFO pre_handler: p-addr = 0x%p, ARM_pc = 0x%lx,
 +  ARM_cpsr = 0x%lx\n,
 + p-addr, regs-ARM_pc, regs-ARM_cpsr);
 +#endif
  
   /* A dump_stack() here will give a stack backtrace */
   return 0;
 @@ -67,6 +72,10 @@ static void handler_post(struct kprobe *p, struct pt_regs 
 *regs,
   printk(KERN_INFO post_handler: p-addr = 0x%p, ex1 = 0x%lx\n,
   p-addr, regs-ex1);
  #endif
 +#ifdef CONFIG_ARM
 + printk(KERN_INFO post_handler: p-addr = 0x%p, ARM_cpsr = 0x%lx\n,
 + p-addr, regs-ARM_cpsr);
 +#endif
  }
  
  /*
 


-- 
Masami HIRAMATSU
Software Platform Research Dept. Linux Technology Research Center
Hitachi, Ltd., Yokohama Research Laboratory
E-mail: masami.hiramatsu...@hitachi.com


--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/