Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1451452

Re: [PATCH 1/2] mm: page_alloc.c: Add tracepoints for slowpath

Show all headers | View raw

On Wed, 27 Jul 2016 14:16:22 -0400
Rik van Riel <riel@surriel.com> wrote:


> As for the function tracer, I wish I had known
> about that!

The kernel (and just the tracing infrastructure) is too big to know
everything that is there.

> 
> That looks like it should provide the info that
> Janani needs to write her memory allocation latency
> tracing script/tool.
> 
> As her Outreachy mentor, I should probably apologize
> for potentially having sent her down the wrong path
> with tracepoints, and I hope it has been an
> educational trip at least :)
> 

No, it was a perfect example of how we work, and I don't see this as a
wrong path. It's a good learning tool because that patch is exactly
what someone wanting to do a specific task will probably do as their
first attempt. There should be no shame in sending out a patch and have
feedback on another way to accomplish the same thing that doesn't
impact the system as much.

As stated above, the kernel is too big to know everything that needs to
be done. Thus, kernel development is really about trial and error. Send
out what works for you, and then take feedback from those that know
their system better than you to make your patch better.

That's how the workflow should happen on a daily basis. And is exactly
how I operate. There's lots of patches that I send out to other
maintainers that end up being something complete different because I
don't know their systems as well as they do.

-- Steve

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 0/2] New tracepoints for slowpath and memory compaction Janani Ravichandran <janani.rvchndrn@gmail.com> - 2016-07-27 16:50 +0200
  [PATCH 1/2] mm: page_alloc.c: Add tracepoints for slowpath Janani Ravichandran <janani.rvchndrn@gmail.com> - 2016-07-27 16:50 +0200
    Re: [PATCH 1/2] mm: page_alloc.c: Add tracepoints for slowpath Steven Rostedt <rostedt@goodmis.org> - 2016-07-27 17:30 +0200
      Re: [PATCH 1/2] mm: page_alloc.c: Add tracepoints for slowpath Dave Hansen <dave.hansen@intel.com> - 2016-07-27 19:30 +0200
        Re: [PATCH 1/2] mm: page_alloc.c: Add tracepoints for slowpath Steven Rostedt <rostedt@goodmis.org> - 2016-07-27 19:40 +0200
      Re: [PATCH 1/2] mm: page_alloc.c: Add tracepoints for slowpath Janani Ravichandran <janani.rvchndrn@gmail.com> - 2016-07-28 22:20 +0200
        Re: [PATCH 1/2] mm: page_alloc.c: Add tracepoints for slowpath Steven Rostedt <rostedt@goodmis.org> - 2016-08-04 17:30 +0200
          Re: [PATCH 1/2] mm: page_alloc.c: Add tracepoints for slowpath Janani Ravichandran <janani.rvchndrn@gmail.com> - 2016-08-05 18:20 +0200
            Re: [PATCH 1/2] mm: page_alloc.c: Add tracepoints for slowpath Steven Rostedt <rostedt@goodmis.org> - 2016-08-05 18:40 +0200
              Re: [PATCH 1/2] mm: page_alloc.c: Add tracepoints for slowpath Janani Ravichandran <janani.rvchndrn@gmail.com> - 2016-08-07 12:40 +0200
    Re: [PATCH 1/2] mm: page_alloc.c: Add tracepoints for slowpath Michal Hocko <mhocko@kernel.org> - 2016-07-27 18:40 +0200
      Re: [PATCH 1/2] mm: page_alloc.c: Add tracepoints for slowpath Rik van Riel <riel@surriel.com> - 2016-07-27 20:20 +0200
        Re: [PATCH 1/2] mm: page_alloc.c: Add tracepoints for slowpath Steven Rostedt <rostedt@goodmis.org> - 2016-07-27 20:30 +0200
        Re: [PATCH 1/2] mm: page_alloc.c: Add tracepoints for slowpath Michal Hocko <mhocko@kernel.org> - 2016-07-27 20:50 +0200
          Re: [PATCH 1/2] mm: page_alloc.c: Add tracepoints for slowpath Rik van Riel <riel@surriel.com> - 2016-07-27 21:00 +0200
  [PATCH 2/2] mm: compaction.c: Add/Modify direct compaction  tracepoints Janani Ravichandran <janani.rvchndrn@gmail.com> - 2016-07-27 17:00 +0200
    Re: [PATCH 2/2] mm: compaction.c: Add/Modify direct compaction  tracepoints Steven Rostedt <rostedt@goodmis.org> - 2016-07-27 17:30 +0200
    Re: [PATCH 2/2] mm: compaction.c: Add/Modify direct compaction  tracepoints Vlastimil Babka <vbabka@suse.cz> - 2016-08-01 15:30 +0200
      Re: [PATCH 2/2] mm: compaction.c: Add/Modify direct compaction tracepoints Janani Ravichandran <janani.rvchndrn@gmail.com> - 2016-08-07 14:40 +0200

csiph-web