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