On Mon, Oct 14, 2013 at 02:53:07PM +0100, Mark Brown wrote:
On Mon, Oct 14, 2013 at 02:21:22PM +0100, Liviu.Dudau@arm.com wrote:
I'm not really comfortable with this. Given the fact that we haven't tested on AArch64 at all, I'm a bit reluctant to ACK this.
We've got test coverage on ARMv8 so if there's a substantial problem we should catch it though adding tracepoints should be very low risk given how widespread their use in the kernel is - is there some particular reason to be concerned about fagility?
What about this for a change?
I'd rather avoid the ifdefs if we can - it'd make the change harder to upstream (though of course that'll need the other tracepoints adding for aarch64 too).
Hi Mark,
I'll leave it in your capable hands to decide which path you take.
If you need it for your patch:
Acked-by: Liviu Dudau Liviu.Dudau@arm.com
Regards, Liviu
Arm mailing list Arm@lists.linaro.org http://lists.linaro.org/mailman/listinfo/arm