All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/4] Do some changes about kprobe
@ 2021-10-25  3:30 Tiezhu Yang
  2021-10-25  3:30 ` [PATCH 1/4] samples/kretprobes: Fix return value if register_kretprobe() failed Tiezhu Yang
                   ` (4 more replies)
  0 siblings, 5 replies; 12+ messages in thread
From: Tiezhu Yang @ 2021-10-25  3:30 UTC (permalink / raw)
  To: naveen.n.rao, anil.s.keshavamurthy, davem, mhiramat, corbet
  Cc: ananth, akpm, randy.dunlap, mathieu.desnoyers, linux-doc, linux-kernel

This patchset is based on kprobes kernel tree:
https://git.kernel.org/pub/scm/linux/kernel/git/mhiramat/linux.git/ for-next

Tiezhu Yang (4):
  samples/kretprobes: Fix return value if register_kretprobe() failed
  docs, kprobes: Remove invalid URL and add new reference
  test_kprobes: Move it from kernel/ to lib/
  MAINTAINERS: Add git tree and missing files for KPROBES

 Documentation/trace/kprobes.rst     | 2 +-
 MAINTAINERS                         | 3 +++
 kernel/Makefile                     | 1 -
 lib/Makefile                        | 1 +
 {kernel => lib}/test_kprobes.c      | 0
 samples/kprobes/kretprobe_example.c | 2 +-
 6 files changed, 6 insertions(+), 3 deletions(-)
 rename {kernel => lib}/test_kprobes.c (100%)

-- 
2.1.0


^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 1/4] samples/kretprobes: Fix return value if register_kretprobe() failed
  2021-10-25  3:30 [PATCH 0/4] Do some changes about kprobe Tiezhu Yang
@ 2021-10-25  3:30 ` Tiezhu Yang
  2021-10-25  6:15   ` Masami Hiramatsu
  2021-10-25  3:30 ` [PATCH 2/4] docs, kprobes: Remove invalid URL and add new reference Tiezhu Yang
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 12+ messages in thread
From: Tiezhu Yang @ 2021-10-25  3:30 UTC (permalink / raw)
  To: naveen.n.rao, anil.s.keshavamurthy, davem, mhiramat, corbet
  Cc: ananth, akpm, randy.dunlap, mathieu.desnoyers, linux-doc, linux-kernel

Use the actual return value instead of always -1 if register_kretprobe()
failed.

E.g. without this patch:

 # insmod samples/kprobes/kretprobe_example.ko func=no_such_func
 insmod: ERROR: could not insert module samples/kprobes/kretprobe_example.ko: Operation not permitted

With this patch:

 # insmod samples/kprobes/kretprobe_example.ko func=no_such_func
 insmod: ERROR: could not insert module samples/kprobes/kretprobe_example.ko: Unknown symbol in module

Fixes: 804defea1c02 ("Kprobes: move kprobe examples to samples/")
Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
---
 samples/kprobes/kretprobe_example.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/samples/kprobes/kretprobe_example.c b/samples/kprobes/kretprobe_example.c
index 5dc1bf3..228321e 100644
--- a/samples/kprobes/kretprobe_example.c
+++ b/samples/kprobes/kretprobe_example.c
@@ -86,7 +86,7 @@ static int __init kretprobe_init(void)
 	ret = register_kretprobe(&my_kretprobe);
 	if (ret < 0) {
 		pr_err("register_kretprobe failed, returned %d\n", ret);
-		return -1;
+		return ret;
 	}
 	pr_info("Planted return probe at %s: %p\n",
 			my_kretprobe.kp.symbol_name, my_kretprobe.kp.addr);
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 2/4] docs, kprobes: Remove invalid URL and add new reference
  2021-10-25  3:30 [PATCH 0/4] Do some changes about kprobe Tiezhu Yang
  2021-10-25  3:30 ` [PATCH 1/4] samples/kretprobes: Fix return value if register_kretprobe() failed Tiezhu Yang
@ 2021-10-25  3:30 ` Tiezhu Yang
  2021-10-25  6:29   ` Masami Hiramatsu
  2021-10-25  3:30 ` [PATCH 3/4] test_kprobes: Move it from kernel/ to lib/ Tiezhu Yang
                   ` (2 subsequent siblings)
  4 siblings, 1 reply; 12+ messages in thread
From: Tiezhu Yang @ 2021-10-25  3:30 UTC (permalink / raw)
  To: naveen.n.rao, anil.s.keshavamurthy, davem, mhiramat, corbet
  Cc: ananth, akpm, randy.dunlap, mathieu.desnoyers, linux-doc, linux-kernel

The following reference is invalid, remove it.
https://www.ibm.com/developerworks/library/l-kprobes/index.html

Add the following new reference "An introduction to KProbes":
https://lwn.net/Articles/132196/

Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
---
 Documentation/trace/kprobes.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/trace/kprobes.rst b/Documentation/trace/kprobes.rst
index 998149c..f318bce 100644
--- a/Documentation/trace/kprobes.rst
+++ b/Documentation/trace/kprobes.rst
@@ -784,6 +784,6 @@ References
 
 For additional information on Kprobes, refer to the following URLs:
 
-- https://www.ibm.com/developerworks/library/l-kprobes/index.html
+- https://lwn.net/Articles/132196/
 - https://www.kernel.org/doc/ols/2006/ols2006v2-pages-109-124.pdf
 
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 3/4] test_kprobes: Move it from kernel/ to lib/
  2021-10-25  3:30 [PATCH 0/4] Do some changes about kprobe Tiezhu Yang
  2021-10-25  3:30 ` [PATCH 1/4] samples/kretprobes: Fix return value if register_kretprobe() failed Tiezhu Yang
  2021-10-25  3:30 ` [PATCH 2/4] docs, kprobes: Remove invalid URL and add new reference Tiezhu Yang
@ 2021-10-25  3:30 ` Tiezhu Yang
  2021-10-25  6:33   ` Masami Hiramatsu
  2021-10-25  3:31 ` [PATCH 4/4] MAINTAINERS: Add git tree and missing files for KPROBES Tiezhu Yang
  2021-10-25  6:14 ` [PATCH 0/4] Do some changes about kprobe Masami Hiramatsu
  4 siblings, 1 reply; 12+ messages in thread
From: Tiezhu Yang @ 2021-10-25  3:30 UTC (permalink / raw)
  To: naveen.n.rao, anil.s.keshavamurthy, davem, mhiramat, corbet
  Cc: ananth, akpm, randy.dunlap, mathieu.desnoyers, linux-doc, linux-kernel

Since config KPROBES_SANITY_TEST is in lib/Kconfig.debug, it is better to
let test_kprobes.c in lib/, just like other similar tests found in lib/.

Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
---
 kernel/Makefile                | 1 -
 lib/Makefile                   | 1 +
 {kernel => lib}/test_kprobes.c | 0
 3 files changed, 1 insertion(+), 1 deletion(-)
 rename {kernel => lib}/test_kprobes.c (100%)

diff --git a/kernel/Makefile b/kernel/Makefile
index 4df609b..9e4d33d 100644
--- a/kernel/Makefile
+++ b/kernel/Makefile
@@ -85,7 +85,6 @@ obj-$(CONFIG_PID_NS) += pid_namespace.o
 obj-$(CONFIG_IKCONFIG) += configs.o
 obj-$(CONFIG_IKHEADERS) += kheaders.o
 obj-$(CONFIG_SMP) += stop_machine.o
-obj-$(CONFIG_KPROBES_SANITY_TEST) += test_kprobes.o
 obj-$(CONFIG_AUDIT) += audit.o auditfilter.o
 obj-$(CONFIG_AUDITSYSCALL) += auditsc.o audit_watch.o audit_fsnotify.o audit_tree.o
 obj-$(CONFIG_GCOV_KERNEL) += gcov/
diff --git a/lib/Makefile b/lib/Makefile
index 2cfd339..2c70452 100644
--- a/lib/Makefile
+++ b/lib/Makefile
@@ -100,6 +100,7 @@ obj-$(CONFIG_TEST_MEMINIT) += test_meminit.o
 obj-$(CONFIG_TEST_LOCKUP) += test_lockup.o
 obj-$(CONFIG_TEST_HMM) += test_hmm.o
 obj-$(CONFIG_TEST_FREE_PAGES) += test_free_pages.o
+obj-$(CONFIG_KPROBES_SANITY_TEST) += test_kprobes.o
 
 #
 # CFLAGS for compiling floating point code inside the kernel. x86/Makefile turns
diff --git a/kernel/test_kprobes.c b/lib/test_kprobes.c
similarity index 100%
rename from kernel/test_kprobes.c
rename to lib/test_kprobes.c
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 4/4] MAINTAINERS: Add git tree and missing files for KPROBES
  2021-10-25  3:30 [PATCH 0/4] Do some changes about kprobe Tiezhu Yang
                   ` (2 preceding siblings ...)
  2021-10-25  3:30 ` [PATCH 3/4] test_kprobes: Move it from kernel/ to lib/ Tiezhu Yang
@ 2021-10-25  3:31 ` Tiezhu Yang
  2021-10-25  6:38   ` Masami Hiramatsu
  2021-10-25  6:14 ` [PATCH 0/4] Do some changes about kprobe Masami Hiramatsu
  4 siblings, 1 reply; 12+ messages in thread
From: Tiezhu Yang @ 2021-10-25  3:31 UTC (permalink / raw)
  To: naveen.n.rao, anil.s.keshavamurthy, davem, mhiramat, corbet
  Cc: ananth, akpm, randy.dunlap, mathieu.desnoyers, linux-doc, linux-kernel

There is no git tree for KPROBES in MAINTAINERS, it is not convinent to
rebase, lib/test_kprobes.c and samples/kprobes belong to kprobe, add them.

Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
---
 MAINTAINERS | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 4372473..0e9bc60 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -10506,10 +10506,13 @@ M:	Anil S Keshavamurthy <anil.s.keshavamurthy@intel.com>
 M:	"David S. Miller" <davem@davemloft.net>
 M:	Masami Hiramatsu <mhiramat@kernel.org>
 S:	Maintained
+T:	git git://git.kernel.org/pub/scm/linux/kernel/git/mhiramat/linux.git
 F:	Documentation/trace/kprobes.rst
 F:	include/asm-generic/kprobes.h
 F:	include/linux/kprobes.h
 F:	kernel/kprobes.c
+F:	lib/test_kprobes.c
+F:	samples/kprobes
 
 KS0108 LCD CONTROLLER DRIVER
 M:	Miguel Ojeda <ojeda@kernel.org>
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH 0/4] Do some changes about kprobe
  2021-10-25  3:30 [PATCH 0/4] Do some changes about kprobe Tiezhu Yang
                   ` (3 preceding siblings ...)
  2021-10-25  3:31 ` [PATCH 4/4] MAINTAINERS: Add git tree and missing files for KPROBES Tiezhu Yang
@ 2021-10-25  6:14 ` Masami Hiramatsu
  4 siblings, 0 replies; 12+ messages in thread
From: Masami Hiramatsu @ 2021-10-25  6:14 UTC (permalink / raw)
  To: Tiezhu Yang
  Cc: naveen.n.rao, anil.s.keshavamurthy, davem, corbet, ananth, akpm,
	randy.dunlap, mathieu.desnoyers, linux-doc, linux-kernel

Hi Tiezhu,

On Mon, 25 Oct 2021 11:30:56 +0800
Tiezhu Yang <yangtiezhu@loongson.cn> wrote:

> This patchset is based on kprobes kernel tree:
> https://git.kernel.org/pub/scm/linux/kernel/git/mhiramat/linux.git/ for-next

Sorry for confusion, that is not the kernel branch for kprobes. Currently
the kprobes is maintained under the Steve's tracing tree.

Anyway, some of your patch should be merged. Let me pick it.

Thank you,

> 
> Tiezhu Yang (4):
>   samples/kretprobes: Fix return value if register_kretprobe() failed
>   docs, kprobes: Remove invalid URL and add new reference
>   test_kprobes: Move it from kernel/ to lib/
>   MAINTAINERS: Add git tree and missing files for KPROBES
> 
>  Documentation/trace/kprobes.rst     | 2 +-
>  MAINTAINERS                         | 3 +++
>  kernel/Makefile                     | 1 -
>  lib/Makefile                        | 1 +
>  {kernel => lib}/test_kprobes.c      | 0
>  samples/kprobes/kretprobe_example.c | 2 +-
>  6 files changed, 6 insertions(+), 3 deletions(-)
>  rename {kernel => lib}/test_kprobes.c (100%)
> 
> -- 
> 2.1.0
> 


-- 
Masami Hiramatsu <mhiramat@kernel.org>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/4] samples/kretprobes: Fix return value if register_kretprobe() failed
  2021-10-25  3:30 ` [PATCH 1/4] samples/kretprobes: Fix return value if register_kretprobe() failed Tiezhu Yang
@ 2021-10-25  6:15   ` Masami Hiramatsu
  0 siblings, 0 replies; 12+ messages in thread
From: Masami Hiramatsu @ 2021-10-25  6:15 UTC (permalink / raw)
  To: Tiezhu Yang
  Cc: naveen.n.rao, anil.s.keshavamurthy, davem, corbet, ananth, akpm,
	randy.dunlap, mathieu.desnoyers, linux-doc, linux-kernel

On Mon, 25 Oct 2021 11:30:57 +0800
Tiezhu Yang <yangtiezhu@loongson.cn> wrote:

> Use the actual return value instead of always -1 if register_kretprobe()
> failed.
> 
> E.g. without this patch:
> 
>  # insmod samples/kprobes/kretprobe_example.ko func=no_such_func
>  insmod: ERROR: could not insert module samples/kprobes/kretprobe_example.ko: Operation not permitted
> 
> With this patch:
> 
>  # insmod samples/kprobes/kretprobe_example.ko func=no_such_func
>  insmod: ERROR: could not insert module samples/kprobes/kretprobe_example.ko: Unknown symbol in module
> 
> Fixes: 804defea1c02 ("Kprobes: move kprobe examples to samples/")
> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>

This looks good to me.

Acked-by: Masami Hiramatsu <mhiramat@kernel.org>

Thanks!

> ---
>  samples/kprobes/kretprobe_example.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/samples/kprobes/kretprobe_example.c b/samples/kprobes/kretprobe_example.c
> index 5dc1bf3..228321e 100644
> --- a/samples/kprobes/kretprobe_example.c
> +++ b/samples/kprobes/kretprobe_example.c
> @@ -86,7 +86,7 @@ static int __init kretprobe_init(void)
>  	ret = register_kretprobe(&my_kretprobe);
>  	if (ret < 0) {
>  		pr_err("register_kretprobe failed, returned %d\n", ret);
> -		return -1;
> +		return ret;
>  	}
>  	pr_info("Planted return probe at %s: %p\n",
>  			my_kretprobe.kp.symbol_name, my_kretprobe.kp.addr);
> -- 
> 2.1.0
> 


-- 
Masami Hiramatsu <mhiramat@kernel.org>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 2/4] docs, kprobes: Remove invalid URL and add new reference
  2021-10-25  3:30 ` [PATCH 2/4] docs, kprobes: Remove invalid URL and add new reference Tiezhu Yang
@ 2021-10-25  6:29   ` Masami Hiramatsu
  0 siblings, 0 replies; 12+ messages in thread
From: Masami Hiramatsu @ 2021-10-25  6:29 UTC (permalink / raw)
  To: Tiezhu Yang
  Cc: naveen.n.rao, anil.s.keshavamurthy, davem, corbet, ananth, akpm,
	randy.dunlap, mathieu.desnoyers, linux-doc, linux-kernel

On Mon, 25 Oct 2021 11:30:58 +0800
Tiezhu Yang <yangtiezhu@loongson.cn> wrote:

> The following reference is invalid, remove it.
> https://www.ibm.com/developerworks/library/l-kprobes/index.html
> 
> Add the following new reference "An introduction to KProbes":
> https://lwn.net/Articles/132196/

Looks good to me.

Acked-by: Masami Hiramatsu <mhiramat@kernel.org>

Thanks!

> 
> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
> ---
>  Documentation/trace/kprobes.rst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/trace/kprobes.rst b/Documentation/trace/kprobes.rst
> index 998149c..f318bce 100644
> --- a/Documentation/trace/kprobes.rst
> +++ b/Documentation/trace/kprobes.rst
> @@ -784,6 +784,6 @@ References
>  
>  For additional information on Kprobes, refer to the following URLs:
>  
> -- https://www.ibm.com/developerworks/library/l-kprobes/index.html
> +- https://lwn.net/Articles/132196/
>  - https://www.kernel.org/doc/ols/2006/ols2006v2-pages-109-124.pdf
>  
> -- 
> 2.1.0
> 


-- 
Masami Hiramatsu <mhiramat@kernel.org>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 3/4] test_kprobes: Move it from kernel/ to lib/
  2021-10-25  3:30 ` [PATCH 3/4] test_kprobes: Move it from kernel/ to lib/ Tiezhu Yang
@ 2021-10-25  6:33   ` Masami Hiramatsu
  0 siblings, 0 replies; 12+ messages in thread
From: Masami Hiramatsu @ 2021-10-25  6:33 UTC (permalink / raw)
  To: Tiezhu Yang
  Cc: naveen.n.rao, anil.s.keshavamurthy, davem, corbet, ananth, akpm,
	randy.dunlap, mathieu.desnoyers, linux-doc, linux-kernel

On Mon, 25 Oct 2021 11:30:59 +0800
Tiezhu Yang <yangtiezhu@loongson.cn> wrote:

> Since config KPROBES_SANITY_TEST is in lib/Kconfig.debug, it is better to
> let test_kprobes.c in lib/, just like other similar tests found in lib/.

This is also good to me. It may be a good timing to move this under the
lib/ because there is KUnit too.

Acked-by: Masami Hiramatsu <mhiramat@kernel.org>

Thank you,

> 
> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
> ---
>  kernel/Makefile                | 1 -
>  lib/Makefile                   | 1 +
>  {kernel => lib}/test_kprobes.c | 0
>  3 files changed, 1 insertion(+), 1 deletion(-)
>  rename {kernel => lib}/test_kprobes.c (100%)
> 
> diff --git a/kernel/Makefile b/kernel/Makefile
> index 4df609b..9e4d33d 100644
> --- a/kernel/Makefile
> +++ b/kernel/Makefile
> @@ -85,7 +85,6 @@ obj-$(CONFIG_PID_NS) += pid_namespace.o
>  obj-$(CONFIG_IKCONFIG) += configs.o
>  obj-$(CONFIG_IKHEADERS) += kheaders.o
>  obj-$(CONFIG_SMP) += stop_machine.o
> -obj-$(CONFIG_KPROBES_SANITY_TEST) += test_kprobes.o
>  obj-$(CONFIG_AUDIT) += audit.o auditfilter.o
>  obj-$(CONFIG_AUDITSYSCALL) += auditsc.o audit_watch.o audit_fsnotify.o audit_tree.o
>  obj-$(CONFIG_GCOV_KERNEL) += gcov/
> diff --git a/lib/Makefile b/lib/Makefile
> index 2cfd339..2c70452 100644
> --- a/lib/Makefile
> +++ b/lib/Makefile
> @@ -100,6 +100,7 @@ obj-$(CONFIG_TEST_MEMINIT) += test_meminit.o
>  obj-$(CONFIG_TEST_LOCKUP) += test_lockup.o
>  obj-$(CONFIG_TEST_HMM) += test_hmm.o
>  obj-$(CONFIG_TEST_FREE_PAGES) += test_free_pages.o
> +obj-$(CONFIG_KPROBES_SANITY_TEST) += test_kprobes.o
>  
>  #
>  # CFLAGS for compiling floating point code inside the kernel. x86/Makefile turns
> diff --git a/kernel/test_kprobes.c b/lib/test_kprobes.c
> similarity index 100%
> rename from kernel/test_kprobes.c
> rename to lib/test_kprobes.c
> -- 
> 2.1.0
> 


-- 
Masami Hiramatsu <mhiramat@kernel.org>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 4/4] MAINTAINERS: Add git tree and missing files for KPROBES
  2021-10-25  3:31 ` [PATCH 4/4] MAINTAINERS: Add git tree and missing files for KPROBES Tiezhu Yang
@ 2021-10-25  6:38   ` Masami Hiramatsu
  2021-10-25  8:31     ` Tiezhu Yang
  0 siblings, 1 reply; 12+ messages in thread
From: Masami Hiramatsu @ 2021-10-25  6:38 UTC (permalink / raw)
  To: Tiezhu Yang
  Cc: naveen.n.rao, anil.s.keshavamurthy, davem, corbet, ananth, akpm,
	randy.dunlap, mathieu.desnoyers, linux-doc, linux-kernel,
	Steven Rostedt

On Mon, 25 Oct 2021 11:31:00 +0800
Tiezhu Yang <yangtiezhu@loongson.cn> wrote:

> There is no git tree for KPROBES in MAINTAINERS, it is not convinent to
> rebase, lib/test_kprobes.c and samples/kprobes belong to kprobe, add them.
> 
> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
> ---
>  MAINTAINERS | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 4372473..0e9bc60 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -10506,10 +10506,13 @@ M:	Anil S Keshavamurthy <anil.s.keshavamurthy@intel.com>
>  M:	"David S. Miller" <davem@davemloft.net>
>  M:	Masami Hiramatsu <mhiramat@kernel.org>
>  S:	Maintained
> +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/mhiramat/linux.git

NACK. As I said, this branch was prepared when I discussed with tip maintainer 
on kretprobe stackfix series but now it has been maintained on Steve's tracing
tree.

Steve, should we put your tree here?

Anyway, I will pick your [1/4]-[3/4].

Thank you,

>  F:	Documentation/trace/kprobes.rst
>  F:	include/asm-generic/kprobes.h
>  F:	include/linux/kprobes.h
>  F:	kernel/kprobes.c
> +F:	lib/test_kprobes.c
> +F:	samples/kprobes
>  
>  KS0108 LCD CONTROLLER DRIVER
>  M:	Miguel Ojeda <ojeda@kernel.org>
> -- 
> 2.1.0
> 


-- 
Masami Hiramatsu <mhiramat@kernel.org>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 4/4] MAINTAINERS: Add git tree and missing files for KPROBES
  2021-10-25  6:38   ` Masami Hiramatsu
@ 2021-10-25  8:31     ` Tiezhu Yang
  2021-10-25 14:48       ` Steven Rostedt
  0 siblings, 1 reply; 12+ messages in thread
From: Tiezhu Yang @ 2021-10-25  8:31 UTC (permalink / raw)
  To: Masami Hiramatsu
  Cc: naveen.n.rao, anil.s.keshavamurthy, davem, corbet, ananth, akpm,
	randy.dunlap, mathieu.desnoyers, linux-doc, linux-kernel,
	Steven Rostedt

On 10/25/2021 02:38 PM, Masami Hiramatsu wrote:
> On Mon, 25 Oct 2021 11:31:00 +0800
> Tiezhu Yang <yangtiezhu@loongson.cn> wrote:
>
>> There is no git tree for KPROBES in MAINTAINERS, it is not convinent to
>> rebase, lib/test_kprobes.c and samples/kprobes belong to kprobe, add them.
>>
>> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
>> ---
>>   MAINTAINERS | 3 +++
>>   1 file changed, 3 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 4372473..0e9bc60 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -10506,10 +10506,13 @@ M:	Anil S Keshavamurthy <anil.s.keshavamurthy@intel.com>
>>   M:	"David S. Miller" <davem@davemloft.net>
>>   M:	Masami Hiramatsu <mhiramat@kernel.org>
>>   S:	Maintained
>> +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/mhiramat/linux.git
> NACK. As I said, this branch was prepared when I discussed with tip maintainer
> on kretprobe stackfix series but now it has been maintained on Steve's tracing
> tree.
>
> Steve, should we put your tree here?

If yes, should I update patch #4 and then send a v2 version of this 
patch set?

@@ -10505,11 +10505,16 @@ M:    Naveen N. Rao <naveen.n.rao@linux.ibm.com>
  M:     Anil S Keshavamurthy <anil.s.keshavamurthy@intel.com>
  M:     "David S. Miller" <davem@davemloft.net>
  M:     Masami Hiramatsu <mhiramat@kernel.org>
+L:     linux-trace-devel@vger.kernel.org
  S:     Maintained
+Q:     https://patchwork.kernel.org/project/linux-trace-devel/list/
+T:     git 
git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace.git
  F:     Documentation/trace/kprobes.rst
  F:     include/asm-generic/kprobes.h
  F:     include/linux/kprobes.h
  F:     kernel/kprobes.c
+F:     lib/test_kprobes.c
+F:     samples/kprobes

By the way, it seems that we should also update the TRACING git tree [1]?
If yes, should I send the following change as a new patch #5 in the v2 
version
or do the following change in the above patch #4?

@@ -19065,8 +19070,10 @@ F:     drivers/char/tpm/
  TRACING
  M:     Steven Rostedt <rostedt@goodmis.org>
  M:     Ingo Molnar <mingo@redhat.com>
+L:     linux-trace-devel@vger.kernel.org
  S:     Maintained
-T:     git git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git 
perf/core
+Q:     https://patchwork.kernel.org/project/linux-trace-devel/list/
+T:     git 
git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace.git
  F:     Documentation/trace/ftrace.rst
  F:     arch/*/*/*/ftrace.h
  F:     arch/*/kernel/ftrace.c

  [1] 
https://lore.kernel.org/lkml/20211019091344.65629198@gandalf.local.home/

  Thanks,
  Tiezhu

>
> Anyway, I will pick your [1/4]-[3/4].
>
> Thank you,
>
>>   F:	Documentation/trace/kprobes.rst
>>   F:	include/asm-generic/kprobes.h
>>   F:	include/linux/kprobes.h
>>   F:	kernel/kprobes.c
>> +F:	lib/test_kprobes.c
>> +F:	samples/kprobes
>>   
>>   KS0108 LCD CONTROLLER DRIVER
>>   M:	Miguel Ojeda <ojeda@kernel.org>
>> -- 
>> 2.1.0
>>
>


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 4/4] MAINTAINERS: Add git tree and missing files for KPROBES
  2021-10-25  8:31     ` Tiezhu Yang
@ 2021-10-25 14:48       ` Steven Rostedt
  0 siblings, 0 replies; 12+ messages in thread
From: Steven Rostedt @ 2021-10-25 14:48 UTC (permalink / raw)
  To: Tiezhu Yang
  Cc: Masami Hiramatsu, naveen.n.rao, anil.s.keshavamurthy, davem,
	corbet, ananth, akpm, randy.dunlap, mathieu.desnoyers, linux-doc,
	linux-kernel

On Mon, 25 Oct 2021 16:31:07 +0800
Tiezhu Yang <yangtiezhu@loongson.cn> wrote:

> On 10/25/2021 02:38 PM, Masami Hiramatsu wrote:
> > On Mon, 25 Oct 2021 11:31:00 +0800
> > Tiezhu Yang <yangtiezhu@loongson.cn> wrote:
> >  
> >> There is no git tree for KPROBES in MAINTAINERS, it is not convinent to
> >> rebase, lib/test_kprobes.c and samples/kprobes belong to kprobe, add them.
> >>
> >> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
> >> ---
> >>   MAINTAINERS | 3 +++
> >>   1 file changed, 3 insertions(+)
> >>
> >> diff --git a/MAINTAINERS b/MAINTAINERS
> >> index 4372473..0e9bc60 100644
> >> --- a/MAINTAINERS
> >> +++ b/MAINTAINERS
> >> @@ -10506,10 +10506,13 @@ M:	Anil S Keshavamurthy <anil.s.keshavamurthy@intel.com>
> >>   M:	"David S. Miller" <davem@davemloft.net>
> >>   M:	Masami Hiramatsu <mhiramat@kernel.org>
> >>   S:	Maintained
> >> +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/mhiramat/linux.git  
> > NACK. As I said, this branch was prepared when I discussed with tip maintainer
> > on kretprobe stackfix series but now it has been maintained on Steve's tracing
> > tree.
> >
> > Steve, should we put your tree here?  
> 

I'm fine if kprobes goes through my tree.

> If yes, should I update patch #4 and then send a v2 version of this 
> patch set?
> 
> @@ -10505,11 +10505,16 @@ M:    Naveen N. Rao <naveen.n.rao@linux.ibm.com>
>   M:     Anil S Keshavamurthy <anil.s.keshavamurthy@intel.com>
>   M:     "David S. Miller" <davem@davemloft.net>
>   M:     Masami Hiramatsu <mhiramat@kernel.org>
> +L:     linux-trace-devel@vger.kernel.org

Please do not add the above mailing list. That's more for tracing tools
like trace-cmd, kernelshark and the libtracefs/libtraceevent libraries.
Only API changes (additions) should go to that list. Not internal updates.

>   S:     Maintained
> +Q:     https://patchwork.kernel.org/project/linux-trace-devel/list/

And this too is for the tracing tools, not the kernel. The patches status
on that patchwork do not get updated by changes to the kernel. Only changes
to the libraries and tool git trees will update those patches.

> +T:     git 
> git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace.git
>   F:     Documentation/trace/kprobes.rst
>   F:     include/asm-generic/kprobes.h
>   F:     include/linux/kprobes.h
>   F:     kernel/kprobes.c
> +F:     lib/test_kprobes.c
> +F:     samples/kprobes
> 
> By the way, it seems that we should also update the TRACING git tree [1]?
> If yes, should I send the following change as a new patch #5 in the v2 
> version
> or do the following change in the above patch #4?
> 
> @@ -19065,8 +19070,10 @@ F:     drivers/char/tpm/
>   TRACING
>   M:     Steven Rostedt <rostedt@goodmis.org>
>   M:     Ingo Molnar <mingo@redhat.com>
> +L:     linux-trace-devel@vger.kernel.org

Again, don't add that list.

>   S:     Maintained
> -T:     git git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git 
> perf/core
> +Q:     https://patchwork.kernel.org/project/linux-trace-devel/list/

Nor the patchwork.

> +T:     git 
> git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace.git
>   F:     Documentation/trace/ftrace.rst
>   F:     arch/*/*/*/ftrace.h
>   F:     arch/*/kernel/ftrace.c
> 

-- Steve

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2021-10-25 14:49 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-25  3:30 [PATCH 0/4] Do some changes about kprobe Tiezhu Yang
2021-10-25  3:30 ` [PATCH 1/4] samples/kretprobes: Fix return value if register_kretprobe() failed Tiezhu Yang
2021-10-25  6:15   ` Masami Hiramatsu
2021-10-25  3:30 ` [PATCH 2/4] docs, kprobes: Remove invalid URL and add new reference Tiezhu Yang
2021-10-25  6:29   ` Masami Hiramatsu
2021-10-25  3:30 ` [PATCH 3/4] test_kprobes: Move it from kernel/ to lib/ Tiezhu Yang
2021-10-25  6:33   ` Masami Hiramatsu
2021-10-25  3:31 ` [PATCH 4/4] MAINTAINERS: Add git tree and missing files for KPROBES Tiezhu Yang
2021-10-25  6:38   ` Masami Hiramatsu
2021-10-25  8:31     ` Tiezhu Yang
2021-10-25 14:48       ` Steven Rostedt
2021-10-25  6:14 ` [PATCH 0/4] Do some changes about kprobe Masami Hiramatsu

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.