* [PATCH] ARM: mcpm: Make all mcpm functions notrace
@ 2013-12-02 9:29 Jon Medhurst (Tixy)
2013-12-02 17:25 ` Dave Martin
0 siblings, 1 reply; 4+ messages in thread
From: Jon Medhurst (Tixy) @ 2013-12-02 9:29 UTC (permalink / raw)
To: linux-arm-kernel
From: Dave Martin <dave.martin@linaro.org>
The functions in mcpm_entry.c are mostly intended for use during
scary cache and coherency disabling sequences, or do other things
which confuse trace ... like powering a CPU down and not
returning. Similarly for the backend code.
For simplicity, this patch just makes whole files notrace.
There should be more than enough traceable points on the paths to
these functions, but we can be more fine-grained later if there is
a need for it.
Signed-off-by: Dave Martin <dave.martin@linaro.org>
Also added spc.o to the list of files as it contains functions used by
MCPM code which have comments comments like: "might be used in code
paths where normal cacheable locks are not working"
Signed-off-by: Jon Medhurst <tixy@linaro.org>
---
arch/arm/common/Makefile | 1 +
arch/arm/mach-vexpress/Makefile | 3 +++
2 files changed, 4 insertions(+)
diff --git a/arch/arm/common/Makefile b/arch/arm/common/Makefile
index 4bdc416..70b1eff 100644
--- a/arch/arm/common/Makefile
+++ b/arch/arm/common/Makefile
@@ -13,6 +13,7 @@ obj-$(CONFIG_SHARP_SCOOP) += scoop.o
obj-$(CONFIG_PCI_HOST_ITE8152) += it8152.o
obj-$(CONFIG_ARM_TIMER_SP804) += timer-sp.o
obj-$(CONFIG_MCPM) += mcpm_head.o mcpm_entry.o mcpm_platsmp.o vlock.o
+CFLAGS_REMOVE_mcpm_entry.o = -pg
AFLAGS_mcpm_head.o := -march=armv7-a
AFLAGS_vlock.o := -march=armv7-a
obj-$(CONFIG_TI_PRIV_EDMA) += edma.o
diff --git a/arch/arm/mach-vexpress/Makefile b/arch/arm/mach-vexpress/Makefile
index 0997e0b..fc649bc 100644
--- a/arch/arm/mach-vexpress/Makefile
+++ b/arch/arm/mach-vexpress/Makefile
@@ -8,8 +8,11 @@ obj-y := v2m.o
obj-$(CONFIG_ARCH_VEXPRESS_CA9X4) += ct-ca9x4.o
obj-$(CONFIG_ARCH_VEXPRESS_DCSCB) += dcscb.o dcscb_setup.o
CFLAGS_dcscb.o += -march=armv7-a
+CFLAGS_REMOVE_dcscb.o = -pg
obj-$(CONFIG_ARCH_VEXPRESS_SPC) += spc.o
+CFLAGS_REMOVE_spc.o = -pg
obj-$(CONFIG_ARCH_VEXPRESS_TC2_PM) += tc2_pm.o
CFLAGS_tc2_pm.o += -march=armv7-a
+CFLAGS_REMOVE_tc2_pm.o = -pg
obj-$(CONFIG_SMP) += platsmp.o
obj-$(CONFIG_HOTPLUG_CPU) += hotplug.o
--
1.7.10.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH] ARM: mcpm: Make all mcpm functions notrace
2013-12-02 9:29 [PATCH] ARM: mcpm: Make all mcpm functions notrace Jon Medhurst (Tixy)
@ 2013-12-02 17:25 ` Dave Martin
2013-12-04 8:42 ` Tarek Dakhran
0 siblings, 1 reply; 4+ messages in thread
From: Dave Martin @ 2013-12-02 17:25 UTC (permalink / raw)
To: linux-arm-kernel
Vyacheslav, Tarek, can you add the corresponding change in your Exynos
Dual Cluster support code please?
There's no need for it to be a separate patch, you can simply add
the CFLAGS_REMOVE_ definitions alongside the other Makefile additions
for edcs. If you want to be more fine-grained, you could mark
individual functions notrace instead -- but that may just add effort
and not be very useful.
Cheers
---Dave
On Mon, Dec 02, 2013 at 09:29:07AM +0000, Jon Medhurst (Tixy) wrote:
> From: Dave Martin <dave.martin@linaro.org>
>
> The functions in mcpm_entry.c are mostly intended for use during
> scary cache and coherency disabling sequences, or do other things
> which confuse trace ... like powering a CPU down and not
> returning. Similarly for the backend code.
>
> For simplicity, this patch just makes whole files notrace.
> There should be more than enough traceable points on the paths to
> these functions, but we can be more fine-grained later if there is
> a need for it.
>
> Signed-off-by: Dave Martin <dave.martin@linaro.org>
>
> Also added spc.o to the list of files as it contains functions used by
> MCPM code which have comments comments like: "might be used in code
> paths where normal cacheable locks are not working"
>
> Signed-off-by: Jon Medhurst <tixy@linaro.org>
> ---
> arch/arm/common/Makefile | 1 +
> arch/arm/mach-vexpress/Makefile | 3 +++
> 2 files changed, 4 insertions(+)
>
> diff --git a/arch/arm/common/Makefile b/arch/arm/common/Makefile
> index 4bdc416..70b1eff 100644
> --- a/arch/arm/common/Makefile
> +++ b/arch/arm/common/Makefile
> @@ -13,6 +13,7 @@ obj-$(CONFIG_SHARP_SCOOP) += scoop.o
> obj-$(CONFIG_PCI_HOST_ITE8152) += it8152.o
> obj-$(CONFIG_ARM_TIMER_SP804) += timer-sp.o
> obj-$(CONFIG_MCPM) += mcpm_head.o mcpm_entry.o mcpm_platsmp.o vlock.o
> +CFLAGS_REMOVE_mcpm_entry.o = -pg
> AFLAGS_mcpm_head.o := -march=armv7-a
> AFLAGS_vlock.o := -march=armv7-a
> obj-$(CONFIG_TI_PRIV_EDMA) += edma.o
> diff --git a/arch/arm/mach-vexpress/Makefile b/arch/arm/mach-vexpress/Makefile
> index 0997e0b..fc649bc 100644
> --- a/arch/arm/mach-vexpress/Makefile
> +++ b/arch/arm/mach-vexpress/Makefile
> @@ -8,8 +8,11 @@ obj-y := v2m.o
> obj-$(CONFIG_ARCH_VEXPRESS_CA9X4) += ct-ca9x4.o
> obj-$(CONFIG_ARCH_VEXPRESS_DCSCB) += dcscb.o dcscb_setup.o
> CFLAGS_dcscb.o += -march=armv7-a
> +CFLAGS_REMOVE_dcscb.o = -pg
> obj-$(CONFIG_ARCH_VEXPRESS_SPC) += spc.o
> +CFLAGS_REMOVE_spc.o = -pg
> obj-$(CONFIG_ARCH_VEXPRESS_TC2_PM) += tc2_pm.o
> CFLAGS_tc2_pm.o += -march=armv7-a
> +CFLAGS_REMOVE_tc2_pm.o = -pg
> obj-$(CONFIG_SMP) += platsmp.o
> obj-$(CONFIG_HOTPLUG_CPU) += hotplug.o
> --
> 1.7.10.4
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] ARM: mcpm: Make all mcpm functions notrace
2013-12-02 17:25 ` Dave Martin
@ 2013-12-04 8:42 ` Tarek Dakhran
2013-12-04 12:48 ` Dave Martin
0 siblings, 1 reply; 4+ messages in thread
From: Tarek Dakhran @ 2013-12-04 8:42 UTC (permalink / raw)
To: linux-arm-kernel
Hi, Dave!
On 12/02/2013 09:25 PM, Dave Martin wrote:
> Vyacheslav, Tarek, can you add the corresponding change in your Exynos
> Dual Cluster support code please?
>
> There's no need for it to be a separate patch, you can simply add
> the CFLAGS_REMOVE_ definitions alongside the other Makefile additions
> for edcs. If you want to be more fine-grained, you could mark
> individual functions notrace instead -- but that may just add effort
> and not be very useful.
>
> Cheers
> ---Dave
>
> On Mon, Dec 02, 2013 at 09:29:07AM +0000, Jon Medhurst (Tixy) wrote:
>> From: Dave Martin <dave.martin@linaro.org>
>>
>> The functions in mcpm_entry.c are mostly intended for use during
>> scary cache and coherency disabling sequences, or do other things
>> which confuse trace ... like powering a CPU down and not
>> returning. Similarly for the backend code.
>>
>> For simplicity, this patch just makes whole files notrace.
>> There should be more than enough traceable points on the paths to
>> these functions, but we can be more fine-grained later if there is
>> a need for it.
>>
>> Signed-off-by: Dave Martin <dave.martin@linaro.org>
>>
>> Also added spc.o to the list of files as it contains functions used by
>> MCPM code which have comments comments like: "might be used in code
>> paths where normal cacheable locks are not working"
>>
>> Signed-off-by: Jon Medhurst <tixy@linaro.org>
>> ---
>> arch/arm/common/Makefile | 1 +
>> arch/arm/mach-vexpress/Makefile | 3 +++
>> 2 files changed, 4 insertions(+)
>>
>> diff --git a/arch/arm/common/Makefile b/arch/arm/common/Makefile
>> index 4bdc416..70b1eff 100644
>> --- a/arch/arm/common/Makefile
>> +++ b/arch/arm/common/Makefile
>> @@ -13,6 +13,7 @@ obj-$(CONFIG_SHARP_SCOOP) += scoop.o
>> obj-$(CONFIG_PCI_HOST_ITE8152) += it8152.o
>> obj-$(CONFIG_ARM_TIMER_SP804) += timer-sp.o
>> obj-$(CONFIG_MCPM) += mcpm_head.o mcpm_entry.o mcpm_platsmp.o vlock.o
>> +CFLAGS_REMOVE_mcpm_entry.o = -pg
>> AFLAGS_mcpm_head.o := -march=armv7-a
>> AFLAGS_vlock.o := -march=armv7-a
>> obj-$(CONFIG_TI_PRIV_EDMA) += edma.o
>> diff --git a/arch/arm/mach-vexpress/Makefile b/arch/arm/mach-vexpress/Makefile
>> index 0997e0b..fc649bc 100644
>> --- a/arch/arm/mach-vexpress/Makefile
>> +++ b/arch/arm/mach-vexpress/Makefile
>> @@ -8,8 +8,11 @@ obj-y := v2m.o
>> obj-$(CONFIG_ARCH_VEXPRESS_CA9X4) += ct-ca9x4.o
>> obj-$(CONFIG_ARCH_VEXPRESS_DCSCB) += dcscb.o dcscb_setup.o
>> CFLAGS_dcscb.o += -march=armv7-a
>> +CFLAGS_REMOVE_dcscb.o = -pg
>> obj-$(CONFIG_ARCH_VEXPRESS_SPC) += spc.o
>> +CFLAGS_REMOVE_spc.o = -pg
>> obj-$(CONFIG_ARCH_VEXPRESS_TC2_PM) += tc2_pm.o
>> CFLAGS_tc2_pm.o += -march=armv7-a
>> +CFLAGS_REMOVE_tc2_pm.o = -pg
>> obj-$(CONFIG_SMP) += platsmp.o
>> obj-$(CONFIG_HOTPLUG_CPU) += hotplug.o
>> --
>> 1.7.10.4
>>
>>
Will be done in next version.
Best regards,
Tarek Dakhran.
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] ARM: mcpm: Make all mcpm functions notrace
2013-12-04 8:42 ` Tarek Dakhran
@ 2013-12-04 12:48 ` Dave Martin
0 siblings, 0 replies; 4+ messages in thread
From: Dave Martin @ 2013-12-04 12:48 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Dec 04, 2013 at 12:42:23PM +0400, Tarek Dakhran wrote:
> Hi, Dave!
>
> On 12/02/2013 09:25 PM, Dave Martin wrote:
> >Vyacheslav, Tarek, can you add the corresponding change in your Exynos
> >Dual Cluster support code please?
> >
> >There's no need for it to be a separate patch, you can simply add
> >the CFLAGS_REMOVE_ definitions alongside the other Makefile additions
> >for edcs. If you want to be more fine-grained, you could mark
> >individual functions notrace instead -- but that may just add effort
> >and not be very useful.
> >
[...]
>
> Will be done in next version.
Thanks
---Dave
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-12-04 12:48 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-02 9:29 [PATCH] ARM: mcpm: Make all mcpm functions notrace Jon Medhurst (Tixy)
2013-12-02 17:25 ` Dave Martin
2013-12-04 8:42 ` Tarek Dakhran
2013-12-04 12:48 ` Dave Martin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).