* [PATCH] config: arm: omap2: remove PROVE_LOCKING from defconfig
[not found] <CGME20181009153624eucas1p2ad30aea33ad1c869f11bd9ca1156d9af@eucas1p2.samsung.com>
@ 2018-10-09 15:36 ` Lukasz Luba
2018-11-08 16:59 ` Tony Lindgren
0 siblings, 1 reply; 4+ messages in thread
From: Lukasz Luba @ 2018-10-09 15:36 UTC (permalink / raw)
To: linux-arm-kernel, linux-kernel; +Cc: l.luba, b.zolnierkie, tony, linux-omap
PROVE_LOCKING enables LOCKDEP, which causes big overhead on cache and
bus transactions.
On some ARM big.LITTLE architecutres (Exynos 5433) the overhead is really big.
The overhead can be measures using hackbench which will speed up
by x3 times (11sec -> 3.4sec).
When you check transaction on cache or buses, the results are way higher
than normal for the same hackbench test:
L1d cache invalidations: 26mln vs 4mln
L2u cache invalidations: 42mln vs 12mln
bus cyc/access: 30cyc/access vs. 20cyc/access
context switch is x3 times cheaper
Enable this option only when you have some locking issue to investigate.
Signed-off-by: Lukasz Luba <l.luba@partner.samsung.com>
---
arch/arm/configs/omap2plus_defconfig | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig
index 6491419..1bbfd76 100644
--- a/arch/arm/configs/omap2plus_defconfig
+++ b/arch/arm/configs/omap2plus_defconfig
@@ -536,7 +536,6 @@ CONFIG_DEBUG_INFO_SPLIT=y
CONFIG_DEBUG_INFO_DWARF4=y
CONFIG_MAGIC_SYSRQ=y
CONFIG_SCHEDSTATS=y
-CONFIG_PROVE_LOCKING=y
# CONFIG_DEBUG_BUGVERBOSE is not set
CONFIG_SECURITY=y
CONFIG_CRYPTO_MICHAEL_MIC=y
--
2.7.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] config: arm: omap2: remove PROVE_LOCKING from defconfig
2018-10-09 15:36 ` [PATCH] config: arm: omap2: remove PROVE_LOCKING from defconfig Lukasz Luba
@ 2018-11-08 16:59 ` Tony Lindgren
2018-11-13 10:22 ` Lukasz Luba
0 siblings, 1 reply; 4+ messages in thread
From: Tony Lindgren @ 2018-11-08 16:59 UTC (permalink / raw)
To: Lukasz Luba; +Cc: linux-arm-kernel, linux-kernel, b.zolnierkie, linux-omap
Hi,
* Lukasz Luba <l.luba@partner.samsung.com> [181009 08:36]:
> PROVE_LOCKING enables LOCKDEP, which causes big overhead on cache and
> bus transactions.
>
> On some ARM big.LITTLE architecutres (Exynos 5433) the overhead is really big.
> The overhead can be measures using hackbench which will speed up
> by x3 times (11sec -> 3.4sec).
> When you check transaction on cache or buses, the results are way higher
> than normal for the same hackbench test:
> L1d cache invalidations: 26mln vs 4mln
> L2u cache invalidations: 42mln vs 12mln
> bus cyc/access: 30cyc/access vs. 20cyc/access
> context switch is x3 times cheaper
>
> Enable this option only when you have some locking issue to investigate.
I'm all for this, but this disables also other less intrusive
debug options. It used to be that we'd get locking issues merged
into drivers and I think that's how it originally got enabled.
So we should take a look which ones we can or want to keep.
Or just disable CONFIG_DEBUG completely.
For distros, that's multi_v7_defconfig nowadays for most part
so I think most people using omap2plus_defconfig are developers
working on various devices.
Regards,
Tony
> Signed-off-by: Lukasz Luba <l.luba@partner.samsung.com>
> ---
> arch/arm/configs/omap2plus_defconfig | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig
> index 6491419..1bbfd76 100644
> --- a/arch/arm/configs/omap2plus_defconfig
> +++ b/arch/arm/configs/omap2plus_defconfig
> @@ -536,7 +536,6 @@ CONFIG_DEBUG_INFO_SPLIT=y
> CONFIG_DEBUG_INFO_DWARF4=y
> CONFIG_MAGIC_SYSRQ=y
> CONFIG_SCHEDSTATS=y
> -CONFIG_PROVE_LOCKING=y
> # CONFIG_DEBUG_BUGVERBOSE is not set
> CONFIG_SECURITY=y
> CONFIG_CRYPTO_MICHAEL_MIC=y
> --
> 2.7.4
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] config: arm: omap2: remove PROVE_LOCKING from defconfig
2018-11-08 16:59 ` Tony Lindgren
@ 2018-11-13 10:22 ` Lukasz Luba
2019-02-04 20:26 ` Tony Lindgren
0 siblings, 1 reply; 4+ messages in thread
From: Lukasz Luba @ 2018-11-13 10:22 UTC (permalink / raw)
To: Tony Lindgren; +Cc: linux-arm-kernel, linux-kernel, b.zolnierkie, linux-omap
Hi Tony,
On 11/8/18 5:59 PM, Tony Lindgren wrote:
> Hi,
>
> * Lukasz Luba <l.luba@partner.samsung.com> [181009 08:36]:
>> PROVE_LOCKING enables LOCKDEP, which causes big overhead on cache and
>> bus transactions.
>>
>> On some ARM big.LITTLE architecutres (Exynos 5433) the overhead is really big.
>> The overhead can be measures using hackbench which will speed up
>> by x3 times (11sec -> 3.4sec).
>> When you check transaction on cache or buses, the results are way higher
>> than normal for the same hackbench test:
>> L1d cache invalidations: 26mln vs 4mln
>> L2u cache invalidations: 42mln vs 12mln
>> bus cyc/access: 30cyc/access vs. 20cyc/access
>> context switch is x3 times cheaper
>>
>> Enable this option only when you have some locking issue to investigate.
>
> I'm all for this, but this disables also other less intrusive
> debug options. It used to be that we'd get locking issues merged
> into drivers and I think that's how it originally got enabled.
That's common reason for a few defconfigs in mainline.
>
> So we should take a look which ones we can or want to keep.
> Or just disable CONFIG_DEBUG completely.
Ideally, the CI should try a few 'types' of configs,
generated based on the 'defconfig'. Some 'smart' script might
cut the debug options for an image for performance regression tests.
>
> For distros, that's multi_v7_defconfig nowadays for most part
> so I think most people using omap2plus_defconfig are developers
> working on various devices.
>
> Regards,
>
> Tony
Regards,
Lukasz
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] config: arm: omap2: remove PROVE_LOCKING from defconfig
2018-11-13 10:22 ` Lukasz Luba
@ 2019-02-04 20:26 ` Tony Lindgren
0 siblings, 0 replies; 4+ messages in thread
From: Tony Lindgren @ 2019-02-04 20:26 UTC (permalink / raw)
To: Lukasz Luba; +Cc: linux-arm-kernel, linux-kernel, b.zolnierkie, linux-omap
* Lukasz Luba <l.luba@partner.samsung.com> [181113 10:22]:
> Hi Tony,
>
> On 11/8/18 5:59 PM, Tony Lindgren wrote:
> > Hi,
> >
> > * Lukasz Luba <l.luba@partner.samsung.com> [181009 08:36]:
> >> PROVE_LOCKING enables LOCKDEP, which causes big overhead on cache and
> >> bus transactions.
> >>
> >> On some ARM big.LITTLE architecutres (Exynos 5433) the overhead is really big.
> >> The overhead can be measures using hackbench which will speed up
> >> by x3 times (11sec -> 3.4sec).
> >> When you check transaction on cache or buses, the results are way higher
> >> than normal for the same hackbench test:
> >> L1d cache invalidations: 26mln vs 4mln
> >> L2u cache invalidations: 42mln vs 12mln
> >> bus cyc/access: 30cyc/access vs. 20cyc/access
> >> context switch is x3 times cheaper
> >>
> >> Enable this option only when you have some locking issue to investigate.
> >
> > I'm all for this, but this disables also other less intrusive
> > debug options. It used to be that we'd get locking issues merged
> > into drivers and I think that's how it originally got enabled.
> That's common reason for a few defconfigs in mainline.
Actually looks like this does not change the DEBUG
related options.. I must have done something wrong
earlier, sorry. So applying into omap-for-v5.1/defconfig.
> > So we should take a look which ones we can or want to keep.
> > Or just disable CONFIG_DEBUG completely.
> Ideally, the CI should try a few 'types' of configs,
> generated based on the 'defconfig'. Some 'smart' script might
> cut the debug options for an image for performance regression tests.
> >
> > For distros, that's multi_v7_defconfig nowadays for most part
> > so I think most people using omap2plus_defconfig are developers
> > working on various devices.
Yes I think that's the case.
Regards,
Tony
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-02-04 20:26 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <CGME20181009153624eucas1p2ad30aea33ad1c869f11bd9ca1156d9af@eucas1p2.samsung.com>
2018-10-09 15:36 ` [PATCH] config: arm: omap2: remove PROVE_LOCKING from defconfig Lukasz Luba
2018-11-08 16:59 ` Tony Lindgren
2018-11-13 10:22 ` Lukasz Luba
2019-02-04 20:26 ` Tony Lindgren
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).