Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Babu Moger
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Babu Moger
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Peter Zijlstra
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Peter Zijlstra
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Rob Gardner
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Rob Gardner
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Babu Moger
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Babu Moger
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Peter Zijlstra
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Peter Zijlstra
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Babu Moger
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Babu Moger
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Peter Zijlstra
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Peter Zijlstra
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Babu Moger
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Babu Moger
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Peter Zijlstra
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Peter Zijlstra
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Babu Moger
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Babu Moger
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Peter Zijlstra
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

Re: [PATCH 0/2] Ajust lockdep static allocations

2016-09-23 Thread Peter Zijlstra
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

[PATCH 0/2] Ajust lockdep static allocations

2016-09-22 Thread 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

[PATCH 0/2] Ajust lockdep static allocations

2016-09-22 Thread 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