On Mon, Oct 23, 2023 at 6:09 AM Marc Zyngier <maz@xxxxxxxxxx> wrote: > > On Fri, 20 Oct 2023 22:40:40 +0100, > Raghavendra Rao Ananta <rananta@xxxxxxxxxx> wrote: > > > > Hello, > > > > The goal of this series is to allow userspace to limit the number > > of PMU event counters on the vCPU. We need this to support migration > > across systems that implement different numbers of counters. > > [...] > > I've gone through the initial patches, and stopped before the tests > (which I usually can't be bothered to review anyway). > > The comments I have a relatively minor and could be applied as fixes > on top if Oliver can be convinced to do so. Note that patch #4 has an > attribution issue. > > > base-commit: 0a3a1665cbc59ee8d6326aa6c0b4a8d1cd67dda3 > > maz@valley-girl:~/hot-poop/arm-platforms$ git describe 0a3a1665cbc59ee8d6326aa6c0b4a8d1cd67dda3 > fatal: 0a3a1665cbc59ee8d6326aa6c0b4a8d1cd67dda3 is neither a commit nor blob > > Can you please make an effort to base your postings on a known, stable > commit? A tagged -rc would be best. but certainly not a random commit. > I usually do base on a known -rc. But this series needed a couple of series from kvmarm/next (mentioned in the original patch), and hence I rebased on top of them. How do you suggest I handle this in the future? Rebase to a known -rc on mainline, apply the required series, and then my series on top? Thank you. Raghavendra
