Re: [PATCH 6/9] trace2:experiment: clear_ce_flags_1
- From
Elijah Newren <newren@gmail.com>
- Date
- Aug 24, 2019, 00:08 UTC
- Message-ID
- <CABPp-BGJv1QJ0zeDx33My9X+GDQC1F-s8W=L1Qt-__YPj=ObCg@mail.gmail.com>
- In-Reply-To
- <86f12dc77dd94c1b2c6294842d29d8712f79e3d7.1566313865.git.gitgitgadget@gmail.com>
On Tue, Aug 20, 2019 at 8:12 AM Jeff Hostetler via GitGitGadget <gitgitgadget@gmail.com> wrote:
> > From: Jeff Hostetler <jeffhost@microsoft.com>
Can the commit summary be turned into English?
> The clear_ce_flags_1 method is used by many types of calls to > unpack_trees(). Add trace2 regions around the method, including > some flag information, so we can get granular performance data > during experiments.
It might be nice to have some words in the cover letter about why this patch is included in this series instead of being a separate submission. I'm not familiar with the trace2 stuff yet; this looks probably useful, but the commit message makes it sound like something general rather than specific to this series.
> Signed-off-by: Jeff Hostetler <jeffhost@microsoft.com> > Signed-off-by: Derrick Stolee <dstolee@microsoft.com> > ---
<snip>