On Fri, Feb 24, 2023, Colton Lewis wrote: > Oliver Upton <oliver.upton@xxxxxxxxx> writes: > > > > These functions were originally part of my patch to introduce latency > > > measurements into dirty_log_perf_test. [1] Sean Christopherson > > > suggested lifting these functions into their own patch in generic code > > > so they can be used by any test. [2] Ricardo Koller suggested the > > > addition of the MEASURE macro to more easily time individual > > > statements. [3] > > > > [1] https://lore.kernel.org/kvm/20221115173258.2530923-1-coltonlewis@xxxxxxxxxx/ > > > [2] https://lore.kernel.org/kvm/Y8gfOP5CMXK60AtH@xxxxxxxxxx/ > > > [3] https://lore.kernel.org/kvm/Y8cIdxp5k8HivVAe@xxxxxxxxxx/ > > > This patch doesn't make a great deal of sense outside of [1]. Can you > > send this as part of your larger series next time around? > > I copied the wrong email link where Sean suggested this should be > generic code in a separate patch. I may have been mistaken in thinking > he meant to upstream it separately. > > https://lore.kernel.org/kvm/Y8gjG6gG5UR6T3Yg@xxxxxxxxxx/ > > But yes, I would prefer to keep this as part of the larger series. Yeah, I just meant put it into a separate patch, but keep it in the series.
