* [PATCH 0/2] Ajust lockdep static allocations
@ 2016-09-22 18:43 Babu Moger
2016-09-22 18:43 ` [PATCH 1/2] lockdep: Keep the default static allocations small Babu Moger
` (2 more replies)
0 siblings, 3 replies; 14+ messages in thread
From: Babu Moger @ 2016-09-22 18:43 UTC (permalink / raw)
To: peterz, mingo, akpm, keescook, dan.j.williams, aryabinin, tj
Cc: linux-kernel, babu.moger
These patches adjust the static allocations for lockdep
data structures used for debugging locking correctness. The current
code reserves about 4MB extra space for these data structures. Most
of the configurations do not need these many data structures. While
testing, I have not seen it go beyond 20% of already reserved entries.
$grep "lock-classes" /proc/lockdep_stats
lock-classes: 1560 [max: 8191]
Reserving even more space seems unreasonable. So, keeping the default
entries small as before the Commit 1413c0389333 ("lockdep: Increase static
allocations"). Added new CONFIG_PROVE_LOCKING_PLUS in case someone
needs more entries to debug their large configuration.
Patch 1 : Adjusts the sizes based on the new config parameter
patch 2 : Adds new config parameter
Babu Moger (2):
lockdep: Keep the default static allocations small
config: Add new CONFIG_PROVE_LOCKING_PLUS
kernel/locking/lockdep_internals.h | 14 +++++++++++---
lib/Kconfig.debug | 10 ++++++++++
2 files changed, 21 insertions(+), 3 deletions(-)
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 1/2] lockdep: Keep the default static allocations small
2016-09-22 18:43 [PATCH 0/2] Ajust lockdep static allocations Babu Moger
@ 2016-09-22 18:43 ` Babu Moger
2016-09-22 18:43 ` [PATCH 2/2] config: Add new CONFIG_PROVE_LOCKING_PLUS Babu Moger
2016-09-23 7:12 ` [PATCH 0/2] Ajust lockdep static allocations Peter Zijlstra
2 siblings, 0 replies; 14+ messages in thread
From: Babu Moger @ 2016-09-22 18:43 UTC (permalink / raw)
To: peterz, mingo, akpm, keescook, dan.j.williams, aryabinin, tj
Cc: linux-kernel, babu.moger
The Commit 1413c0389333 ("lockdep: Increase static allocations")
doubled the static allocation for lockdep. The size is unusually
high and not required for majority of the configurations. This
could cause problems to some environments with limited memory
configurations. We are already seeing issues on our sparc
configuration where kernel fails to boot when lockdep feature
is enabled. This patch keeps the default size to same as before
Commit 1413c0389333 ("lockdep: Increase static allocations").
Adding the new config parameter CONFIG_PROVE_LOCKING_PLUS in case
someone needs to enable more static space for lockdep entries,
lock chains and stack traces to debug large configurations.
Signed-off-by: Babu Moger <babu.moger@oracle.com>
---
kernel/locking/lockdep_internals.h | 14 +++++++++++---
1 files changed, 11 insertions(+), 3 deletions(-)
diff --git a/kernel/locking/lockdep_internals.h b/kernel/locking/lockdep_internals.h
index 51c4b24..47336a6 100644
--- a/kernel/locking/lockdep_internals.h
+++ b/kernel/locking/lockdep_internals.h
@@ -54,18 +54,26 @@ enum {
* table (if it's not there yet), and we check it for lock order
* conflicts and deadlocks.
*/
+#ifdef CONFIG_PROVE_LOCKING_PLUS
#define MAX_LOCKDEP_ENTRIES 32768UL
#define MAX_LOCKDEP_CHAINS_BITS 16
-#define MAX_LOCKDEP_CHAINS (1UL << MAX_LOCKDEP_CHAINS_BITS)
-
-#define MAX_LOCKDEP_CHAIN_HLOCKS (MAX_LOCKDEP_CHAINS*5)
/*
* Stack-trace: tightly packed array of stack backtrace
* addresses. Protected by the hash_lock.
*/
#define MAX_STACK_TRACE_ENTRIES 524288UL
+#else
+#define MAX_LOCKDEP_ENTRIES 16384UL
+#define MAX_LOCKDEP_CHAINS_BITS 15
+#define MAX_STACK_TRACE_ENTRIES 262144UL
+#endif
+
+#define MAX_LOCKDEP_CHAINS (1UL << MAX_LOCKDEP_CHAINS_BITS)
+
+#define MAX_LOCKDEP_CHAIN_HLOCKS (MAX_LOCKDEP_CHAINS*5)
+
extern struct list_head all_lock_classes;
extern struct lock_chain lock_chains[];
--
1.7.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 2/2] config: Add new CONFIG_PROVE_LOCKING_PLUS
2016-09-22 18:43 [PATCH 0/2] Ajust lockdep static allocations Babu Moger
2016-09-22 18:43 ` [PATCH 1/2] lockdep: Keep the default static allocations small Babu Moger
@ 2016-09-22 18:43 ` Babu Moger
2016-09-23 7:12 ` [PATCH 0/2] Ajust lockdep static allocations Peter Zijlstra
2 siblings, 0 replies; 14+ messages in thread
From: Babu Moger @ 2016-09-22 18:43 UTC (permalink / raw)
To: peterz, mingo, akpm, keescook, dan.j.williams, aryabinin, tj
Cc: linux-kernel, babu.moger
Adding the new config parameter CONFIG_PROVE_LOCKING_PLUS in case
someone needs to enable more static space for lockdep entries,
lock chains and stack traces to debug large configurations.
The default size is kept small to cover majority of the configs.
Signed-off-by: Babu Moger <babu.moger@oracle.com>
---
lib/Kconfig.debug | 10 ++++++++++
1 files changed, 10 insertions(+), 0 deletions(-)
diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
index b9cfdbf..d5d995e 100644
--- a/lib/Kconfig.debug
+++ b/lib/Kconfig.debug
@@ -1070,6 +1070,16 @@ config PROVE_LOCKING
For more details, see Documentation/locking/lockdep-design.txt.
+config PROVE_LOCKING_PLUS
+ bool "Reserve extra space for prove locking correctness"
+ depends on PROVE_LOCKING
+ default n
+ help
+ This feature reserves more space for lockdep entries, lock chains
+ and stack traces to debug large configurations. This could add
+ about additional 4MB static memory to kernel size. This is not
+ suitable for embedded or other limited memory configurations.
+
config LOCKDEP
bool
depends on DEBUG_KERNEL && TRACE_IRQFLAGS_SUPPORT && STACKTRACE_SUPPORT && LOCKDEP_SUPPORT
--
1.7.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH 0/2] Ajust lockdep static allocations
2016-09-22 18:43 [PATCH 0/2] Ajust lockdep static allocations Babu Moger
2016-09-22 18:43 ` [PATCH 1/2] lockdep: Keep the default static allocations small Babu Moger
2016-09-22 18:43 ` [PATCH 2/2] config: Add new CONFIG_PROVE_LOCKING_PLUS Babu Moger
@ 2016-09-23 7:12 ` Peter Zijlstra
2016-09-23 14:04 ` Babu Moger
2 siblings, 1 reply; 14+ messages in thread
From: Peter Zijlstra @ 2016-09-23 7:12 UTC (permalink / raw)
To: Babu Moger
Cc: mingo, akpm, keescook, dan.j.williams, aryabinin, tj, linux-kernel
On Thu, Sep 22, 2016 at 11:43:34AM -0700, Babu Moger wrote:
> These patches adjust the static allocations for lockdep
> data structures used for debugging locking correctness. The current
> code reserves about 4MB extra space for these data structures. Most
> of the configurations do not need these many data structures. While
> testing, I have not seen it go beyond 20% of already reserved entries.
>
> $grep "lock-classes" /proc/lockdep_stats
> lock-classes: 1560 [max: 8191]
>
> Reserving even more space seems unreasonable. So, keeping the default
> entries small as before the Commit 1413c0389333 ("lockdep: Increase static
> allocations"). Added new CONFIG_PROVE_LOCKING_PLUS in case someone
> needs more entries to debug their large configuration.
Why make this more complicated? There's absolutely no upside to this
change as far as I can see.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 0/2] Ajust lockdep static allocations
2016-09-23 7:12 ` [PATCH 0/2] Ajust lockdep static allocations Peter Zijlstra
@ 2016-09-23 14:04 ` Babu Moger
2016-09-23 14:34 ` Peter Zijlstra
0 siblings, 1 reply; 14+ messages in thread
From: Babu Moger @ 2016-09-23 14:04 UTC (permalink / raw)
To: Peter Zijlstra
Cc: mingo, akpm, keescook, dan.j.williams, aryabinin, tj, linux-kernel
On 9/23/2016 2:12 AM, Peter Zijlstra wrote:
> On Thu, Sep 22, 2016 at 11:43:34AM -0700, Babu Moger wrote:
>> These patches adjust the static allocations for lockdep
>> data structures used for debugging locking correctness. The current
>> code reserves about 4MB extra space for these data structures. Most
>> of the configurations do not need these many data structures. While
>> testing, I have not seen it go beyond 20% of already reserved entries.
>>
>> $grep "lock-classes" /proc/lockdep_stats
>> lock-classes: 1560 [max: 8191]
>>
>> Reserving even more space seems unreasonable. So, keeping the default
>> entries small as before the Commit 1413c0389333 ("lockdep: Increase static
>> allocations"). Added new CONFIG_PROVE_LOCKING_PLUS in case someone
>> needs more entries to debug their large configuration.
> Why make this more complicated? There's absolutely no upside to this
> change as far as I can see.
Peter, What do you mean? Revert the commit 1413c038933? Right now,
I cannot boot my setup after enabling lockdep. How do you think we can
handle this?
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 0/2] Ajust lockdep static allocations
2016-09-23 14:04 ` Babu Moger
@ 2016-09-23 14:34 ` Peter Zijlstra
2016-09-23 14:50 ` Babu Moger
0 siblings, 1 reply; 14+ messages in thread
From: Peter Zijlstra @ 2016-09-23 14:34 UTC (permalink / raw)
To: Babu Moger
Cc: mingo, akpm, keescook, dan.j.williams, aryabinin, tj, linux-kernel
On Fri, Sep 23, 2016 at 09:04:42AM -0500, Babu Moger wrote:
> On 9/23/2016 2:12 AM, Peter Zijlstra wrote:
> >On Thu, Sep 22, 2016 at 11:43:34AM -0700, Babu Moger wrote:
> >>These patches adjust the static allocations for lockdep
> >>data structures used for debugging locking correctness. The current
> >>code reserves about 4MB extra space for these data structures. Most
> >>of the configurations do not need these many data structures. While
> >>testing, I have not seen it go beyond 20% of already reserved entries.
> >>
> >>$grep "lock-classes" /proc/lockdep_stats
> >> lock-classes: 1560 [max: 8191]
> >>
> >>Reserving even more space seems unreasonable. So, keeping the default
> >>entries small as before the Commit 1413c0389333 ("lockdep: Increase static
> >>allocations"). Added new CONFIG_PROVE_LOCKING_PLUS in case someone
> >>needs more entries to debug their large configuration.
> >Why make this more complicated? There's absolutely no upside to this
> >change as far as I can see.
> Peter, What do you mean?
I mean I see no point to the patches you send.
> Revert the commit 1413c038933?
Nah, why would I?
> Right now, I cannot boot my setup after enabling lockdep. How do you
> think we can handle this?
Why can't you boot? You have that little memories? 4MB doesn't seem like
a worthwhile amount of memory.
Also, you didn't say. This seems a somewhat crucial point.
In any case, maybe invert this, add make it depend on CONFIG_BASE_SMALL,
since this really only matters for really dinky systems.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 0/2] Ajust lockdep static allocations
2016-09-23 14:34 ` Peter Zijlstra
@ 2016-09-23 14:50 ` Babu Moger
2016-09-23 15:04 ` Peter Zijlstra
0 siblings, 1 reply; 14+ messages in thread
From: Babu Moger @ 2016-09-23 14:50 UTC (permalink / raw)
To: Peter Zijlstra
Cc: mingo, akpm, keescook, dan.j.williams, aryabinin, tj, linux-kernel
On 9/23/2016 9:34 AM, Peter Zijlstra wrote:
> On Fri, Sep 23, 2016 at 09:04:42AM -0500, Babu Moger wrote:
>> On 9/23/2016 2:12 AM, Peter Zijlstra wrote:
>>> On Thu, Sep 22, 2016 at 11:43:34AM -0700, Babu Moger wrote:
>>>> These patches adjust the static allocations for lockdep
>>>> data structures used for debugging locking correctness. The current
>>>> code reserves about 4MB extra space for these data structures. Most
>>>> of the configurations do not need these many data structures. While
>>>> testing, I have not seen it go beyond 20% of already reserved entries.
>>>>
>>>> $grep "lock-classes" /proc/lockdep_stats
>>>> lock-classes: 1560 [max: 8191]
>>>>
>>>> Reserving even more space seems unreasonable. So, keeping the default
>>>> entries small as before the Commit 1413c0389333 ("lockdep: Increase static
>>>> allocations"). Added new CONFIG_PROVE_LOCKING_PLUS in case someone
>>>> needs more entries to debug their large configuration.
>>> Why make this more complicated? There's absolutely no upside to this
>>> change as far as I can see.
>> Peter, What do you mean?
> I mean I see no point to the patches you send.
>
>> Revert the commit 1413c038933?
> Nah, why would I?
>
>> Right now, I cannot boot my setup after enabling lockdep. How do you
>> think we can handle this?
> Why can't you boot? You have that little memories? 4MB doesn't seem like
> a worthwhile amount of memory.
>
> Also, you didn't say. This seems a somewhat crucial point.
Correct, We can't boot with lockdep. Sorry I did not make that
clear. We have a limit on static size of the kernel.
>
> In any case, maybe invert this, add make it depend on CONFIG_BASE_SMALL,
> since this really only matters for really dinky systems.
Sure. Will use CONFIG_BASE_SMALL and re-post the patches. Thanks
>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 0/2] Ajust lockdep static allocations
2016-09-23 14:50 ` Babu Moger
@ 2016-09-23 15:04 ` Peter Zijlstra
2016-09-23 15:15 ` Babu Moger
0 siblings, 1 reply; 14+ messages in thread
From: Peter Zijlstra @ 2016-09-23 15:04 UTC (permalink / raw)
To: Babu Moger
Cc: mingo, akpm, keescook, dan.j.williams, aryabinin, tj, linux-kernel
On Fri, Sep 23, 2016 at 09:50:52AM -0500, Babu Moger wrote:
> >Why can't you boot? You have that little memories? 4MB doesn't seem like
> >a worthwhile amount of memory.
> >
> >Also, you didn't say. This seems a somewhat crucial point.
>
> Correct, We can't boot with lockdep. Sorry I did not make that clear.
> We have a limit on static size of the kernel.
This stuff should be in .bss not .data. It should not affect the static
size at all. Or am I misunderstanding things?
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 0/2] Ajust lockdep static allocations
2016-09-23 15:04 ` Peter Zijlstra
@ 2016-09-23 15:15 ` Babu Moger
2016-09-23 15:40 ` Peter Zijlstra
0 siblings, 1 reply; 14+ messages in thread
From: Babu Moger @ 2016-09-23 15:15 UTC (permalink / raw)
To: Peter Zijlstra
Cc: mingo, akpm, keescook, dan.j.williams, aryabinin, tj, linux-kernel
On 9/23/2016 10:04 AM, Peter Zijlstra wrote:
> On Fri, Sep 23, 2016 at 09:50:52AM -0500, Babu Moger wrote:
>>> Why can't you boot? You have that little memories? 4MB doesn't seem like
>>> a worthwhile amount of memory.
>>>
>>> Also, you didn't say. This seems a somewhat crucial point.
>> Correct, We can't boot with lockdep. Sorry I did not make that clear.
>> We have a limit on static size of the kernel.
> This stuff should be in .bss not .data. It should not affect the static
> size at all. Or am I misunderstanding things?
Here it is.
$ ./scripts/bloat-o-meter vmlinux.lockdep.small vmlinux.lockdep.big
add/remove: 0/0 grow/shrink: 5/0 up/down: 4653056/0 (4653056)
function old new delta
stack_trace 2097152 4194304 +2097152
lock_chains 1048576 2097152 +1048576
list_entries 1048576 2097152 +1048576
chain_hlocks 327680 655360 +327680
chainhash_table 131072 262144 +131072
Total: Before=21046200, After=25699256, chg 22.000000%
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 0/2] Ajust lockdep static allocations
2016-09-23 15:15 ` Babu Moger
@ 2016-09-23 15:40 ` Peter Zijlstra
2016-09-23 19:57 ` Babu Moger
0 siblings, 1 reply; 14+ messages in thread
From: Peter Zijlstra @ 2016-09-23 15:40 UTC (permalink / raw)
To: Babu Moger
Cc: mingo, akpm, keescook, dan.j.williams, aryabinin, tj, linux-kernel
On Fri, Sep 23, 2016 at 10:15:46AM -0500, Babu Moger wrote:
> >> Correct, We can't boot with lockdep. Sorry I did not make that clear.
> >>We have a limit on static size of the kernel.
> >This stuff should be in .bss not .data. It should not affect the static
> >size at all. Or am I misunderstanding things?
> Here it is.
> $ ./scripts/bloat-o-meter vmlinux.lockdep.small vmlinux.lockdep.big
What does bloat-o-meter have to do with things? The static image size is
not dependent on .bss, right?
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 0/2] Ajust lockdep static allocations
2016-09-23 15:40 ` Peter Zijlstra
@ 2016-09-23 19:57 ` Babu Moger
2016-09-23 20:08 ` Rob Gardner
2016-09-23 20:17 ` Peter Zijlstra
0 siblings, 2 replies; 14+ messages in thread
From: Babu Moger @ 2016-09-23 19:57 UTC (permalink / raw)
To: Peter Zijlstra
Cc: mingo, akpm, keescook, dan.j.williams, aryabinin, tj,
linux-kernel, Rob Gardner, davem
On 9/23/2016 10:40 AM, Peter Zijlstra wrote:
> On Fri, Sep 23, 2016 at 10:15:46AM -0500, Babu Moger wrote:
>
>>>> Correct, We can't boot with lockdep. Sorry I did not make that clear.
>>>> We have a limit on static size of the kernel.
>>> This stuff should be in .bss not .data. It should not affect the static
>>> size at all. Or am I misunderstanding things?
>> Here it is.
>> $ ./scripts/bloat-o-meter vmlinux.lockdep.small vmlinux.lockdep.big
> What does bloat-o-meter have to do with things? The static image size is
> not dependent on .bss, right?
Peter,
We checked again. Yes, It goes in .bss section. But in sparc we have
to fit .text, .data,
.bss in 7 permanent TLBs(that is totally 28MB). It was fine so far.
But the commit
1413c0389333 ("lockdep: Increase static allocations") added extra 4MB
which makes
it go beyond 28MB. That is causing system boot up problems in sparc.
Yes. We know it.
This is a limitation. Changing this limit in our hardware is a much
bigger change which
we cannot address right away. So, we are trying to come up with a
solution which can
work for all. I will re-post the patches with CONFIG_BASE_SMALL
option if there is no
objections.
CCing David Miller and Rob Gardner. They might be able to explain more
if you
have any more questions. Here is the discussion thread if you guys
want to look at history.
https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1237642.html
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 0/2] Ajust lockdep static allocations
2016-09-23 19:57 ` Babu Moger
@ 2016-09-23 20:08 ` Rob Gardner
2016-09-23 20:17 ` Peter Zijlstra
1 sibling, 0 replies; 14+ messages in thread
From: Rob Gardner @ 2016-09-23 20:08 UTC (permalink / raw)
To: Babu Moger, Peter Zijlstra
Cc: mingo, akpm, keescook, dan.j.williams, aryabinin, tj,
linux-kernel, davem
On 09/23/2016 01:57 PM, Babu Moger wrote:
>
> On 9/23/2016 10:40 AM, Peter Zijlstra wrote:
>> On Fri, Sep 23, 2016 at 10:15:46AM -0500, Babu Moger wrote:
>>
>>>>> Correct, We can't boot with lockdep. Sorry I did not make
>>>>> that clear.
>>>>> We have a limit on static size of the kernel.
>>>> This stuff should be in .bss not .data. It should not affect the
>>>> static
>>>> size at all. Or am I misunderstanding things?
>>> Here it is.
>>> $ ./scripts/bloat-o-meter vmlinux.lockdep.small vmlinux.lockdep.big
>> What does bloat-o-meter have to do with things? The static image size is
>> not dependent on .bss, right?
>
> Peter,
> We checked again. Yes, It goes in .bss section. But in sparc we have
> to fit .text, .data,
> .bss in 7 permanent TLBs(that is totally 28MB). It was fine so far.
> But the commit
> 1413c0389333 ("lockdep: Increase static allocations") added extra 4MB
> which makes
> it go beyond 28MB. That is causing system boot up problems in sparc.
> Yes. We know it.
> This is a limitation. Changing this limit in our hardware is a much
> bigger change which
> we cannot address right away. So, we are trying to come up with a
> solution which can
> work for all. I will re-post the patches with CONFIG_BASE_SMALL
> option if there is no
> objections.
>
> CCing David Miller and Rob Gardner. They might be able to explain
> more if you
> have any more questions. Here is the discussion thread if you guys
> want to look at history.
> https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1237642.html
>
>
>
Yes, perhaps I can help clarify the problem Babu is seeing. It is true
that stuff in bss doesn't increase the static size of the file that
contains the kernel. But it does increase the kernel's memory footprint.
And as the system is booting, all the kernel's code, data, and bss, must
have locked translations in the TLB so that we don't get TLB misses on
kernel code and data. Current sparc chips have 8 TLB entries available
that may be locked down, and with a 4mb page size, this gives a maximum
of 32mb of memory that can be covered. One of these is used for kexec (I
think), so that leaves 28mb. It sounds to me like Babu is saying that
the change in question has increased the size of bss data so this limit
is exceeded, thus causing boot problems, and he proposes to somewhat
reduce the added space to alleviate this problem.
And also as Babu says, changing to a larger page size is very tricky.
Not only do different sparc cpus support a different set of h/w page
sizes, the effects of changing this are quite far reaching and would
affect a lot of code.
Rob Gardner
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 0/2] Ajust lockdep static allocations
2016-09-23 19:57 ` Babu Moger
2016-09-23 20:08 ` Rob Gardner
@ 2016-09-23 20:17 ` Peter Zijlstra
2016-09-23 20:30 ` Babu Moger
1 sibling, 1 reply; 14+ messages in thread
From: Peter Zijlstra @ 2016-09-23 20:17 UTC (permalink / raw)
To: Babu Moger
Cc: mingo, akpm, keescook, dan.j.williams, aryabinin, tj,
linux-kernel, Rob Gardner, davem
On Fri, Sep 23, 2016 at 02:57:39PM -0500, Babu Moger wrote:
> We checked again. Yes, It goes in .bss section. But in sparc we have
> to fit .text, .data, .bss in 7 permanent TLBs(that is totally 28MB).
> It was fine so far. But the commit 1413c0389333 ("lockdep: Increase
> static allocations") added extra 4MB which makes it go beyond 28MB.
> That is causing system boot up problems in sparc.
*sigh*, why didn't you start with that :/
> Yes. We know it. This is a limitation. Changing this limit in our
> hardware is a much bigger change which we cannot address right away.
> So, we are trying to come up with a solution which can work for all. I
> will re-post the patches with CONFIG_BASE_SMALL option if there is no
> objections.
OK, so double check BASE_SMALL doesn't imply other things you cannot
live with, Sparc64 isn't a dinky system. If BASE_SMALL works for you
then good, otherwise do a PROVE_LOCKING_SMALL symbol that is not user
selectable and have SPARC select that. Use the invisible Help for that
symbol to explain all this again.
> CCing David Miller and Rob Gardner. They might be able to explain
> more if you have any more questions.
Nah, I think I remember enough of how the Sparc MMU works to see reason.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 0/2] Ajust lockdep static allocations
2016-09-23 20:17 ` Peter Zijlstra
@ 2016-09-23 20:30 ` Babu Moger
0 siblings, 0 replies; 14+ messages in thread
From: Babu Moger @ 2016-09-23 20:30 UTC (permalink / raw)
To: Peter Zijlstra
Cc: mingo, akpm, keescook, dan.j.williams, aryabinin, tj,
linux-kernel, Rob Gardner, davem
On 9/23/2016 3:17 PM, Peter Zijlstra wrote:
> On Fri, Sep 23, 2016 at 02:57:39PM -0500, Babu Moger wrote:
>
>> We checked again. Yes, It goes in .bss section. But in sparc we have
>> to fit .text, .data, .bss in 7 permanent TLBs(that is totally 28MB).
>> It was fine so far. But the commit 1413c0389333 ("lockdep: Increase
>> static allocations") added extra 4MB which makes it go beyond 28MB.
>> That is causing system boot up problems in sparc.
> *sigh*, why didn't you start with that :/
>
>> Yes. We know it. This is a limitation. Changing this limit in our
>> hardware is a much bigger change which we cannot address right away.
>> So, we are trying to come up with a solution which can work for all. I
>> will re-post the patches with CONFIG_BASE_SMALL option if there is no
>> objections.
> OK, so double check BASE_SMALL doesn't imply other things you cannot
> live with, Sparc64 isn't a dinky system. If BASE_SMALL works for you
> then good, otherwise do a PROVE_LOCKING_SMALL symbol that is not user
> selectable and have SPARC select that. Use the invisible Help for that
> symbol to explain all this again.
Thanks. Will work on it.
>
>> CCing David Miller and Rob Gardner. They might be able to explain
>> more if you have any more questions.
> Nah, I think I remember enough of how the Sparc MMU works to see reason.
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2016-09-23 20:31 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-09-22 18:43 [PATCH 0/2] Ajust lockdep static allocations Babu Moger
2016-09-22 18:43 ` [PATCH 1/2] lockdep: Keep the default static allocations small Babu Moger
2016-09-22 18:43 ` [PATCH 2/2] config: Add new CONFIG_PROVE_LOCKING_PLUS Babu Moger
2016-09-23 7:12 ` [PATCH 0/2] Ajust lockdep static allocations Peter Zijlstra
2016-09-23 14:04 ` Babu Moger
2016-09-23 14:34 ` Peter Zijlstra
2016-09-23 14:50 ` Babu Moger
2016-09-23 15:04 ` Peter Zijlstra
2016-09-23 15:15 ` Babu Moger
2016-09-23 15:40 ` Peter Zijlstra
2016-09-23 19:57 ` Babu Moger
2016-09-23 20:08 ` Rob Gardner
2016-09-23 20:17 ` Peter Zijlstra
2016-09-23 20:30 ` Babu Moger
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.