* [PATCH] ring-buffer: remove obsolete comment for free_buffer_page()
@ 2023-03-15 14:24 Vlastimil Babka
2023-03-15 15:51 ` Mukesh Ojha
0 siblings, 1 reply; 5+ messages in thread
From: Vlastimil Babka @ 2023-03-15 14:24 UTC (permalink / raw)
To: Steven Rostedt, Masami Hiramatsu, Ingo Molnar
Cc: linux-trace-kernel, linux-kernel, patches, Vlastimil Babka,
Mike Rapoport
The comment refers to mm/slob.o which is being removed. It comes from
commit ed56829cb319 ("ring_buffer: reset buffer page when freeing") and
according to Steven the borrowed code was a page mapcount and mapping
reset, which was later removed by commit e4c2ce82ca27 ("ring_buffer:
allocate buffer page pointer"). Thus the comment is not accurate anyway,
remove it.
Reported-by: Mike Rapoport <mike.rapoport@gmail.com>
Suggested-by: Steven Rostedt (Google) <rostedt@goodmis.org>
Fixes: e4c2ce82ca27 ("ring_buffer: allocate buffer page pointer")
Signed-off-by: Vlastimil Babka <vbabka@suse.cz>
---
kernel/trace/ring_buffer.c | 4 ----
1 file changed, 4 deletions(-)
diff --git a/kernel/trace/ring_buffer.c b/kernel/trace/ring_buffer.c
index af50d931b020..c6f47b6cfd5f 100644
--- a/kernel/trace/ring_buffer.c
+++ b/kernel/trace/ring_buffer.c
@@ -354,10 +354,6 @@ static void rb_init_page(struct buffer_data_page *bpage)
local_set(&bpage->commit, 0);
}
-/*
- * Also stolen from mm/slob.c. Thanks to Mathieu Desnoyers for pointing
- * this issue out.
- */
static void free_buffer_page(struct buffer_page *bpage)
{
free_page((unsigned long)bpage->page);
--
2.39.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] ring-buffer: remove obsolete comment for free_buffer_page()
2023-03-15 14:24 [PATCH] ring-buffer: remove obsolete comment for free_buffer_page() Vlastimil Babka
@ 2023-03-15 15:51 ` Mukesh Ojha
2023-03-15 15:53 ` Mukesh Ojha
0 siblings, 1 reply; 5+ messages in thread
From: Mukesh Ojha @ 2023-03-15 15:51 UTC (permalink / raw)
To: Vlastimil Babka, Steven Rostedt, Masami Hiramatsu, Ingo Molnar
Cc: linux-trace-kernel, linux-kernel, patches, Mike Rapoport
On 3/15/2023 7:54 PM, Vlastimil Babka wrote:
> The comment refers to mm/slob.o which is being removed. It comes from
> commit ed56829cb319 ("ring_buffer: reset buffer page when freeing") and
> according to Steven the borrowed code was a page mapcount and mapping
> reset, which was later removed by commit e4c2ce82ca27 ("ring_buffer:
> allocate buffer page pointer"). Thus the comment is not accurate anyway,
> remove it.
>
> Reported-by: Mike Rapoport <mike.rapoport@gmail.com>
> Suggested-by: Steven Rostedt (Google) <rostedt@goodmis.org>
> Fixes: e4c2ce82ca27 ("ring_buffer: allocate buffer page pointer")
> Signed-off-by: Vlastimil Babka <vbabka@suse.cz>
> ---
> kernel/trace/ring_buffer.c | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/kernel/trace/ring_buffer.c b/kernel/trace/ring_buffer.c
> index af50d931b020..c6f47b6cfd5f 100644
> --- a/kernel/trace/ring_buffer.c
> +++ b/kernel/trace/ring_buffer.c
> @@ -354,10 +354,6 @@ static void rb_init_page(struct buffer_data_page *bpage)
> local_set(&bpage->commit, 0);
> }
>
> -/*
> - * Also stolen from mm/slob.c. Thanks to Mathieu Desnoyers for pointing
> - * this issue out.
> - */
Thanks for the clean up.
Reviewed-by: Mukesh Ojha <quic_mojha@quicinc.com>
-- Mukesh
> static void free_buffer_page(struct buffer_page *bpage)
> {
> free_page((unsigned long)bpage->page);
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ring-buffer: remove obsolete comment for free_buffer_page()
2023-03-15 15:51 ` Mukesh Ojha
@ 2023-03-15 15:53 ` Mukesh Ojha
2023-03-16 8:29 ` Vlastimil Babka
0 siblings, 1 reply; 5+ messages in thread
From: Mukesh Ojha @ 2023-03-15 15:53 UTC (permalink / raw)
To: Vlastimil Babka, Steven Rostedt, Masami Hiramatsu, Ingo Molnar
Cc: linux-trace-kernel, linux-kernel, patches, Mike Rapoport
On 3/15/2023 9:21 PM, Mukesh Ojha wrote:
>
>
> On 3/15/2023 7:54 PM, Vlastimil Babka wrote:
>> The comment refers to mm/slob.o which is being removed. It comes from
nit: mm/slob.c
-- Mukesh
>
>
>> commit ed56829cb319 ("ring_buffer: reset buffer page when freeing") and
>> according to Steven the borrowed code was a page mapcount and mapping
>> reset, which was later removed by commit e4c2ce82ca27 ("ring_buffer:
>> allocate buffer page pointer"). Thus the comment is not accurate anyway,
>> remove it.
>>
>> Reported-by: Mike Rapoport <mike.rapoport@gmail.com>
>> Suggested-by: Steven Rostedt (Google) <rostedt@goodmis.org>
>> Fixes: e4c2ce82ca27 ("ring_buffer: allocate buffer page pointer")
>> Signed-off-by: Vlastimil Babka <vbabka@suse.cz>
>> ---
>> kernel/trace/ring_buffer.c | 4 ----
>> 1 file changed, 4 deletions(-)
>>
>> diff --git a/kernel/trace/ring_buffer.c b/kernel/trace/ring_buffer.c
>> index af50d931b020..c6f47b6cfd5f 100644
>> --- a/kernel/trace/ring_buffer.c
>> +++ b/kernel/trace/ring_buffer.c
>> @@ -354,10 +354,6 @@ static void rb_init_page(struct buffer_data_page
>> *bpage)
>> local_set(&bpage->commit, 0);
>> }
>> -/*
>> - * Also stolen from mm/slob.c. Thanks to Mathieu Desnoyers for pointing
>> - * this issue out.
>> - */
>
> Thanks for the clean up.
>
> Reviewed-by: Mukesh Ojha <quic_mojha@quicinc.com>
>
> -- Mukesh
>
>> static void free_buffer_page(struct buffer_page *bpage)
>> {
>> free_page((unsigned long)bpage->page);
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ring-buffer: remove obsolete comment for free_buffer_page()
2023-03-15 15:53 ` Mukesh Ojha
@ 2023-03-16 8:29 ` Vlastimil Babka
2023-03-16 13:57 ` Steven Rostedt
0 siblings, 1 reply; 5+ messages in thread
From: Vlastimil Babka @ 2023-03-16 8:29 UTC (permalink / raw)
To: Mukesh Ojha, Steven Rostedt, Masami Hiramatsu, Ingo Molnar
Cc: linux-trace-kernel, linux-kernel, patches, Mike Rapoport
On 3/15/23 16:53, Mukesh Ojha wrote:
>
>
> On 3/15/2023 9:21 PM, Mukesh Ojha wrote:
>>
>>
>> On 3/15/2023 7:54 PM, Vlastimil Babka wrote:
>>> The comment refers to mm/slob.o which is being removed. It comes from
>
> nit: mm/slob.c
Thanks, Steven can you fix that up while picking the patch, or need v2?
> -- Mukesh
>
>>
>>
>>> commit ed56829cb319 ("ring_buffer: reset buffer page when freeing") and
>>> according to Steven the borrowed code was a page mapcount and mapping
>>> reset, which was later removed by commit e4c2ce82ca27 ("ring_buffer:
>>> allocate buffer page pointer"). Thus the comment is not accurate anyway,
>>> remove it.
>>>
>>> Reported-by: Mike Rapoport <mike.rapoport@gmail.com>
>>> Suggested-by: Steven Rostedt (Google) <rostedt@goodmis.org>
>>> Fixes: e4c2ce82ca27 ("ring_buffer: allocate buffer page pointer")
>>> Signed-off-by: Vlastimil Babka <vbabka@suse.cz>
>>> ---
>>> kernel/trace/ring_buffer.c | 4 ----
>>> 1 file changed, 4 deletions(-)
>>>
>>> diff --git a/kernel/trace/ring_buffer.c b/kernel/trace/ring_buffer.c
>>> index af50d931b020..c6f47b6cfd5f 100644
>>> --- a/kernel/trace/ring_buffer.c
>>> +++ b/kernel/trace/ring_buffer.c
>>> @@ -354,10 +354,6 @@ static void rb_init_page(struct buffer_data_page
>>> *bpage)
>>> local_set(&bpage->commit, 0);
>>> }
>>> -/*
>>> - * Also stolen from mm/slob.c. Thanks to Mathieu Desnoyers for pointing
>>> - * this issue out.
>>> - */
>>
>> Thanks for the clean up.
>>
>> Reviewed-by: Mukesh Ojha <quic_mojha@quicinc.com>
Thanks.
>>
>> -- Mukesh
>>
>>> static void free_buffer_page(struct buffer_page *bpage)
>>> {
>>> free_page((unsigned long)bpage->page);
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ring-buffer: remove obsolete comment for free_buffer_page()
2023-03-16 8:29 ` Vlastimil Babka
@ 2023-03-16 13:57 ` Steven Rostedt
0 siblings, 0 replies; 5+ messages in thread
From: Steven Rostedt @ 2023-03-16 13:57 UTC (permalink / raw)
To: Vlastimil Babka
Cc: Mukesh Ojha, Masami Hiramatsu, Ingo Molnar, linux-trace-kernel,
linux-kernel, patches, Mike Rapoport
On Thu, 16 Mar 2023 09:29:30 +0100
Vlastimil Babka <vbabka@suse.cz> wrote:
> On 3/15/23 16:53, Mukesh Ojha wrote:
> >
> >
> > On 3/15/2023 9:21 PM, Mukesh Ojha wrote:
> >>
> >>
> >> On 3/15/2023 7:54 PM, Vlastimil Babka wrote:
> >>> The comment refers to mm/slob.o which is being removed. It comes from
> >
> > nit: mm/slob.c
>
> Thanks, Steven can you fix that up while picking the patch, or need v2?
I'll fix it up.
-- Steve
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-03-16 13:57 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-03-15 14:24 [PATCH] ring-buffer: remove obsolete comment for free_buffer_page() Vlastimil Babka
2023-03-15 15:51 ` Mukesh Ojha
2023-03-15 15:53 ` Mukesh Ojha
2023-03-16 8:29 ` Vlastimil Babka
2023-03-16 13:57 ` Steven Rostedt
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).