* [PATCH] docs, kprobes: Fix the wrong location of Kprobes
@ 2022-09-16 9:55 Tiezhu Yang
2022-09-26 18:22 ` Steven Rostedt
0 siblings, 1 reply; 5+ messages in thread
From: Tiezhu Yang @ 2022-09-16 9:55 UTC (permalink / raw)
To: Steven Rostedt, Masami Hiramatsu, Jonathan Corbet; +Cc: linux-doc, linux-kernel
After commit 22471e1313f2 ("kconfig: use a menu in arch/Kconfig to reduce
clutter"), the location of Kprobes is under "General architecture-dependent
options" rather than "General setup".
Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
---
Documentation/trace/kprobes.rst | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Documentation/trace/kprobes.rst b/Documentation/trace/kprobes.rst
index f318bce..48cf778 100644
--- a/Documentation/trace/kprobes.rst
+++ b/Documentation/trace/kprobes.rst
@@ -328,8 +328,8 @@ Configuring Kprobes
===================
When configuring the kernel using make menuconfig/xconfig/oldconfig,
-ensure that CONFIG_KPROBES is set to "y". Under "General setup", look
-for "Kprobes".
+ensure that CONFIG_KPROBES is set to "y", look for "Kprobes" under
+"General architecture-dependent options".
So that you can load and unload Kprobes-based instrumentation modules,
make sure "Loadable module support" (CONFIG_MODULES) and "Module
--
2.1.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] docs, kprobes: Fix the wrong location of Kprobes
2022-09-16 9:55 [PATCH] docs, kprobes: Fix the wrong location of Kprobes Tiezhu Yang
@ 2022-09-26 18:22 ` Steven Rostedt
2022-09-29 10:01 ` Tiezhu Yang
2022-09-29 19:09 ` Jonathan Corbet
0 siblings, 2 replies; 5+ messages in thread
From: Steven Rostedt @ 2022-09-26 18:22 UTC (permalink / raw)
To: Tiezhu Yang; +Cc: Masami Hiramatsu, Jonathan Corbet, linux-doc, linux-kernel
On Fri, 16 Sep 2022 17:55:06 +0800
Tiezhu Yang <yangtiezhu@loongson.cn> wrote:
> After commit 22471e1313f2 ("kconfig: use a menu in arch/Kconfig to reduce
> clutter"), the location of Kprobes is under "General architecture-dependent
> options" rather than "General setup".
>
Probably add a "Fixes:" tag for the above mentioned commit.
Anyway, Acked-by: Steven Rostedt (Google) <rostedt@goodmis.org>
-- Steve
> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
> ---
> Documentation/trace/kprobes.rst | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/trace/kprobes.rst b/Documentation/trace/kprobes.rst
> index f318bce..48cf778 100644
> --- a/Documentation/trace/kprobes.rst
> +++ b/Documentation/trace/kprobes.rst
> @@ -328,8 +328,8 @@ Configuring Kprobes
> ===================
>
> When configuring the kernel using make menuconfig/xconfig/oldconfig,
> -ensure that CONFIG_KPROBES is set to "y". Under "General setup", look
> -for "Kprobes".
> +ensure that CONFIG_KPROBES is set to "y", look for "Kprobes" under
> +"General architecture-dependent options".
>
> So that you can load and unload Kprobes-based instrumentation modules,
> make sure "Loadable module support" (CONFIG_MODULES) and "Module
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] docs, kprobes: Fix the wrong location of Kprobes
2022-09-26 18:22 ` Steven Rostedt
@ 2022-09-29 10:01 ` Tiezhu Yang
2022-09-29 13:24 ` Steven Rostedt
2022-09-29 19:09 ` Jonathan Corbet
1 sibling, 1 reply; 5+ messages in thread
From: Tiezhu Yang @ 2022-09-29 10:01 UTC (permalink / raw)
To: Steven Rostedt; +Cc: Masami Hiramatsu, Jonathan Corbet, linux-doc, linux-kernel
On 09/27/2022 02:22 AM, Steven Rostedt wrote:
> On Fri, 16 Sep 2022 17:55:06 +0800
> Tiezhu Yang <yangtiezhu@loongson.cn> wrote:
>
>> After commit 22471e1313f2 ("kconfig: use a menu in arch/Kconfig to reduce
>> clutter"), the location of Kprobes is under "General architecture-dependent
>> options" rather than "General setup".
>>
>
> Probably add a "Fixes:" tag for the above mentioned commit.
>
> Anyway, Acked-by: Steven Rostedt (Google) <rostedt@goodmis.org>
Thank you.
Do you know which tree this patch will go through?
Is it necessary to send v2 with "Fixes:" tag?
Thanks,
Tiezhu
>
> -- Steve
>
>
>> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
>> ---
>> Documentation/trace/kprobes.rst | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/Documentation/trace/kprobes.rst b/Documentation/trace/kprobes.rst
>> index f318bce..48cf778 100644
>> --- a/Documentation/trace/kprobes.rst
>> +++ b/Documentation/trace/kprobes.rst
>> @@ -328,8 +328,8 @@ Configuring Kprobes
>> ===================
>>
>> When configuring the kernel using make menuconfig/xconfig/oldconfig,
>> -ensure that CONFIG_KPROBES is set to "y". Under "General setup", look
>> -for "Kprobes".
>> +ensure that CONFIG_KPROBES is set to "y", look for "Kprobes" under
>> +"General architecture-dependent options".
>>
>> So that you can load and unload Kprobes-based instrumentation modules,
>> make sure "Loadable module support" (CONFIG_MODULES) and "Module
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] docs, kprobes: Fix the wrong location of Kprobes
2022-09-29 10:01 ` Tiezhu Yang
@ 2022-09-29 13:24 ` Steven Rostedt
0 siblings, 0 replies; 5+ messages in thread
From: Steven Rostedt @ 2022-09-29 13:24 UTC (permalink / raw)
To: Tiezhu Yang; +Cc: Masami Hiramatsu, Jonathan Corbet, linux-doc, linux-kernel
On Thu, 29 Sep 2022 18:01:01 +0800
Tiezhu Yang <yangtiezhu@loongson.cn> wrote:
> On 09/27/2022 02:22 AM, Steven Rostedt wrote:
> > On Fri, 16 Sep 2022 17:55:06 +0800
> > Tiezhu Yang <yangtiezhu@loongson.cn> wrote:
> >
> >> After commit 22471e1313f2 ("kconfig: use a menu in arch/Kconfig to reduce
> >> clutter"), the location of Kprobes is under "General architecture-dependent
> >> options" rather than "General setup".
> >>
> >
> > Probably add a "Fixes:" tag for the above mentioned commit.
> >
> > Anyway, Acked-by: Steven Rostedt (Google) <rostedt@goodmis.org>
>
>
> Thank you.
>
> Do you know which tree this patch will go through?
I believe that Jon can take it through his documentation tree.
> Is it necessary to send v2 with "Fixes:" tag?
No, Jon could add it.
Thanks,
-- Steve
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] docs, kprobes: Fix the wrong location of Kprobes
2022-09-26 18:22 ` Steven Rostedt
2022-09-29 10:01 ` Tiezhu Yang
@ 2022-09-29 19:09 ` Jonathan Corbet
1 sibling, 0 replies; 5+ messages in thread
From: Jonathan Corbet @ 2022-09-29 19:09 UTC (permalink / raw)
To: Steven Rostedt, Tiezhu Yang; +Cc: Masami Hiramatsu, linux-doc, linux-kernel
Steven Rostedt <rostedt@goodmis.org> writes:
> On Fri, 16 Sep 2022 17:55:06 +0800
> Tiezhu Yang <yangtiezhu@loongson.cn> wrote:
>
>> After commit 22471e1313f2 ("kconfig: use a menu in arch/Kconfig to reduce
>> clutter"), the location of Kprobes is under "General architecture-dependent
>> options" rather than "General setup".
>>
>
> Probably add a "Fixes:" tag for the above mentioned commit.
>
> Anyway, Acked-by: Steven Rostedt (Google) <rostedt@goodmis.org>
When you embed a tag like that, b4 doesn't pick it up anymore...but I
was amending the commit to add the Fixes tag so I put it in too...:)
Applied, thanks.
jon
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-09-29 19:09 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-09-16 9:55 [PATCH] docs, kprobes: Fix the wrong location of Kprobes Tiezhu Yang
2022-09-26 18:22 ` Steven Rostedt
2022-09-29 10:01 ` Tiezhu Yang
2022-09-29 13:24 ` Steven Rostedt
2022-09-29 19:09 ` Jonathan Corbet
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).