On 06/25/2015 11:48 AM, Daniel Axtens wrote: > Hi Anshuman, > > Thanks for your continued work on this. > > Given that the series is now at version 9 and is 13 patches long, I > wonder if it might be worth splitting it up.
Splitting it up completely or just keeping all the generic fixes and cleanups at the beginning of the series would be sufficient. Anyways I am willing to send them out separately if that helps. > > I'd suggest: > > - Patch 1 could be sent individually as it's a bug fix. Not with the generic cleanup group as proposed below ? > > - Separating out a series of simple cleanups would make the actual > changes in your patch set easier to understand. Patches 2, 3 and 5 are > obvious candidates. Agreed. Just that adding the first patch here will prevent a three way split of the entire series. > > - It looks like the changes in patch 6 aren't used by any of the > following patches. It might be worth separating that out or just > dropping it entirely. I guess you are talking about patch 7 "powerpc, perf: Re organize PMU branch filter processing on POWER8". Patch 6 is getting used later on. > > > That would give you a series with just: > 4 powerpc, perf: Restore privilege level filter support for BHRB > 7 powerpc, perf: Re organize PMU branch filter processing on POWER8 > 8 powerpc, perf: Change the name of HW PMU branch filter tracking variable > 9 powerpc, lib: Add new branch analysis support functions > 10 powerpc, perf: Enable SW filtering in branch stack sampling framework > 11 powerpc, perf: Change POWER8 PMU configuration to work with SW filters > 12 powerpc, perf: Enable privilege mode SW branch filters > 13 selftests, powerpc: Add test for BHRB branch filters (HW & SW) > > That might make it easier for you to start getting the ground work in, > and make it easier for others to understand what you're trying to do. Sure, agreed. Here are the two set of patches after the proposed split. Patches are in the reverse order though. Hope this helps. Generic cleanups and fixes --------------------------- powerpc/perf: Re organize PMU branch filter processing on POWER8 powerpc/perf: Change name & type of 'pred' in power_pmu_bhrb_read powerpc/perf: Replace last usage of get_cpu_var with this_cpu_ptr powerpc/perf: Change type of the bhrb_users variable powerpc/perf: Drop the branch sample when 'from' cannot be fetched BHRB SW branch filter ---------------------- selftests/powerpc: Add test for BHRB branch filters (HW & SW) powerpc/perf: Enable privilege mode SW branch filters powerpc/perf: Change POWER8 PMU configuration to work with SW filters powerpc/perf: Enable SW filtering in branch stack sampling framework powerpc/lib: Add new branch analysis support functions powerpc/perf: Change the name of HW PMU branch filter tracking variable powerpc/perf: Re organize BHRB processing powerpc/perf: Restore privilege level filter support for BHRB Regards Anshuman _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
