[PATCH v2 0/7] Add basic trace events for vmap/vmalloc (v2)

Uladzislau Rezki (Sony) posted 7 patches 3 years, 5 months ago
MAINTAINERS                    |  12 +++-
include/trace/events/vmalloc.h | 123 +++++++++++++++++++++++++++++++++
mm/vmalloc.c                   |  20 +++++-
3 files changed, 151 insertions(+), 4 deletions(-)
create mode 100644 include/trace/events/vmalloc.h
[PATCH v2 0/7] Add basic trace events for vmap/vmalloc (v2)
Posted by Uladzislau Rezki (Sony) 3 years, 5 months ago
Description is here https://lore.kernel.org/linux-mm/20221017160233.16582-1-urezki@gmail.com/

Changelog v1 -> v2:
- Fix a compilation error for the ARCH=um SUBARCH=i386

Uladzislau Rezki (Sony) (7):
  mm: vmalloc: Add alloc_vmap_area trace event
  mm: vmalloc: Add purge_vmap_area_lazy trace event
  mm: vmalloc: Add free_vmap_area_noflush trace event
  mm: vmalloc: Use trace_alloc_vmap_area event
  mm: vmalloc: Use trace_purge_vmap_area_lazy event
  mm: vmalloc: Use trace_free_vmap_area_noflush event
  vmalloc: Add reviewers for vmalloc code

 MAINTAINERS                    |  12 +++-
 include/trace/events/vmalloc.h | 123 +++++++++++++++++++++++++++++++++
 mm/vmalloc.c                   |  20 +++++-
 3 files changed, 151 insertions(+), 4 deletions(-)
 create mode 100644 include/trace/events/vmalloc.h

-- 
2.30.2
Re: [PATCH v2 0/7] Add basic trace events for vmap/vmalloc (v2)
Posted by Steven Rostedt 3 years, 5 months ago
On Tue, 18 Oct 2022 20:10:46 +0200
"Uladzislau Rezki (Sony)" <urezki@gmail.com> wrote:

> Description is here https://lore.kernel.org/linux-mm/20221017160233.16582-1-urezki@gmail.com/

You should always copy the description in each version. It's annoying to
have to click a link from a console email client.

Anyway, it's pretty basic trace events.

For POV of tracing (not the usage of it)

Reviewed-by: Steven Rostedt (Google) <rostedt@goodmis.org>

for the entire series.

-- Steve

> 
> Changelog v1 -> v2:
> - Fix a compilation error for the ARCH=um SUBARCH=i386
> 
> Uladzislau Rezki (Sony) (7):
>   mm: vmalloc: Add alloc_vmap_area trace event
>   mm: vmalloc: Add purge_vmap_area_lazy trace event
>   mm: vmalloc: Add free_vmap_area_noflush trace event
>   mm: vmalloc: Use trace_alloc_vmap_area event
>   mm: vmalloc: Use trace_purge_vmap_area_lazy event
>   mm: vmalloc: Use trace_free_vmap_area_noflush event
>   vmalloc: Add reviewers for vmalloc code
> 
>  MAINTAINERS                    |  12 +++-
>  include/trace/events/vmalloc.h | 123 +++++++++++++++++++++++++++++++++
>  mm/vmalloc.c                   |  20 +++++-
>  3 files changed, 151 insertions(+), 4 deletions(-)
>  create mode 100644 include/trace/events/vmalloc.h
>
Re: [PATCH v2 0/7] Add basic trace events for vmap/vmalloc (v2)
Posted by Steven Rostedt 3 years, 5 months ago
On Tue, 18 Oct 2022 14:23:02 -0400
Steven Rostedt <rostedt@goodmis.org> wrote:

> > Description is here https://lore.kernel.org/linux-mm/20221017160233.16582-1-urezki@gmail.com/  
> 
> You should always copy the description in each version. It's annoying to
> have to click a link from a console email client.

Although, what I do is, include the previous version when talking about
differences in this version. That way the link to the old series can be
easily found.

> Changelog v1 -> v2: (https://lore.kernel.org/linux-mm/20221017160233.16582-1-urezki@gmail.com/)

-- Steve
Re: [PATCH v2 0/7] Add basic trace events for vmap/vmalloc (v2)
Posted by Uladzislau Rezki 3 years, 5 months ago
On Tue, Oct 18, 2022 at 02:25:09PM -0400, Steven Rostedt wrote:
> On Tue, 18 Oct 2022 14:23:02 -0400
> Steven Rostedt <rostedt@goodmis.org> wrote:
> 
> > > Description is here https://lore.kernel.org/linux-mm/20221017160233.16582-1-urezki@gmail.com/  
> > 
> > You should always copy the description in each version. It's annoying to
> > have to click a link from a console email client.
> 
> Although, what I do is, include the previous version when talking about
> differences in this version. That way the link to the old series can be
> easily found.
> 
OK. Thanks for the review!

--
Uladzislau Rezki