Re: [PATCH 6/4] arm64: Move to generic sched_clock infrastructure
On 05/01/2013 05:11 AM, Catalin Marinas wrote: > On Wed, May 01, 2013 at 01:54:35AM +0100, Stephen Boyd wrote: >> Use the generic sched_clock infrastructure instead of rolling our >> own. This has the added benefit of fixing suspend/resume as >> outlined in 6a4dae5 (ARM: 7565/1: sched: stop sched_clock() >> during suspend, 2012-10-23) and correcting the timestamps when >> the hardware returns a value instead of 0 upon the first read. >> >> Signed-off-by: Stephen Boyd > > Looks ok. > > Acked-by: Catalin Marinas I built and ran this change and dependencies on top of Catalin's soc-armv8-model branch [1] and was able to verify that it fixed the printk timestamp jump. 1. http://git.kernel.org/cgit/linux/kernel/git/cmarinas/linux-aarch64.git/log/?h=soc-armv8-model Tested-by: Christopher Covington -- Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 6/4] arm64: Move to generic sched_clock infrastructure
On Wed, May 01, 2013 at 01:54:35AM +0100, Stephen Boyd wrote: > Use the generic sched_clock infrastructure instead of rolling our > own. This has the added benefit of fixing suspend/resume as > outlined in 6a4dae5 (ARM: 7565/1: sched: stop sched_clock() > during suspend, 2012-10-23) and correcting the timestamps when > the hardware returns a value instead of 0 upon the first read. > > Signed-off-by: Stephen Boyd Looks ok. Acked-by: Catalin Marinas -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 6/4] arm64: Move to generic sched_clock infrastructure
On Wed, May 01, 2013 at 01:54:35AM +0100, Stephen Boyd wrote: Use the generic sched_clock infrastructure instead of rolling our own. This has the added benefit of fixing suspend/resume as outlined in 6a4dae5 (ARM: 7565/1: sched: stop sched_clock() during suspend, 2012-10-23) and correcting the timestamps when the hardware returns a value instead of 0 upon the first read. Signed-off-by: Stephen Boyd sb...@codeaurora.org Looks ok. Acked-by: Catalin Marinas catalin.mari...@arm.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 6/4] arm64: Move to generic sched_clock infrastructure
On 05/01/2013 05:11 AM, Catalin Marinas wrote: On Wed, May 01, 2013 at 01:54:35AM +0100, Stephen Boyd wrote: Use the generic sched_clock infrastructure instead of rolling our own. This has the added benefit of fixing suspend/resume as outlined in 6a4dae5 (ARM: 7565/1: sched: stop sched_clock() during suspend, 2012-10-23) and correcting the timestamps when the hardware returns a value instead of 0 upon the first read. Signed-off-by: Stephen Boyd sb...@codeaurora.org Looks ok. Acked-by: Catalin Marinas catalin.mari...@arm.com I built and ran this change and dependencies on top of Catalin's soc-armv8-model branch [1] and was able to verify that it fixed the printk timestamp jump. 1. http://git.kernel.org/cgit/linux/kernel/git/cmarinas/linux-aarch64.git/log/?h=soc-armv8-model Tested-by: Christopher Covington c...@codeaurora.org -- Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/