Messages in this thread | | | Subject | Re: [PATCH v6 0/6] perf report/script: Support percent and multiple range in --time option | From | "Jin, Yao" <> | Date | Fri, 10 Nov 2017 21:18:29 +0800 |
| |
On 11/6/2017 8:21 PM, Jin Yao wrote: > v6: > --- > 1. Create a new option "--timestamp-boundary" in perf record. > > Currently '--buildid-all' is not enabled by default. So the walking > on all samples is the default operation. There is no big overhead > to calculate the timestamp boundary in process_sample_event handler > once we already go through all samples. So the timestamp boundary > calculation is enabled by default when '--buildid-all' is not enabled. > > While if '--buildid-all' is enabled, we creates a new option > "--timestamp-boundary" for user to decide if it enables the > timestamp boundary calculation. > > Impacted patch: > --------------- > perf record: Get the first sample time and last sample time > > 2. Fix the merge issue with the latest perf/core branch. > No functional changes. > > Impacted patch: > --------------- > perf report: support time percent and multiple time ranges > perf script: support time percent and multiple time ranges >
Hi Arnaldo,
Is the v6 acceptable or I need to continue improving?
I'd like to continue improving the patch if it's needed.
Thanks Jin Yao
| |