Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1492119 > unrolled thread
| Started by | Babu Moger <babu.moger@oracle.com> |
|---|---|
| First post | 2016-09-27 21:40 +0200 |
| Last post | 2016-09-28 23:00 +0200 |
| Articles | 7 — 4 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
[PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc Babu Moger <babu.moger@oracle.com> - 2016-09-27 21:40 +0200
Re: [PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc Peter Zijlstra <peterz@infradead.org> - 2016-09-28 10:40 +0200
Re: [PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc Babu Moger <babu.moger@oracle.com> - 2016-09-29 16:00 +0200
Re: [PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc David Miller <davem@davemloft.net> - 2016-09-30 07:20 +0200
Re: [PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc Babu Moger <babu.moger@oracle.com> - 2016-10-06 01:00 +0200
Re: [PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc Sam Ravnborg <sam@ravnborg.org> - 2016-10-06 06:30 +0200
Re: [PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc Sam Ravnborg <sam@ravnborg.org> - 2016-09-28 23:00 +0200
| From | Babu Moger <babu.moger@oracle.com> |
|---|---|
| Date | 2016-09-27 21:40 +0200 |
| Subject | [PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc |
| Message-ID | <sm6oq-58G-19@gated-at.bofh.it> |
This new config parameter limits the space used for "Lock debugging: prove locking correctness" by about 4MB. The current sparc systems have the limitation of 32MB size for kernel size including .text, .data and .bss sections. With PROVE_LOCKING feature, the kernel size could grow beyond this limit and causing system boot-up issues. With this option, kernel limits the size of the entries of lock_chains, stack_trace etc., so that kernel fits in required size limit. This is not visible to user and only used for sparc. Signed-off-by: Babu Moger <babu.moger@oracle.com> --- arch/sparc/Kconfig | 1 + lib/Kconfig.debug | 3 +++ 2 files changed, 4 insertions(+), 0 deletions(-) diff --git a/arch/sparc/Kconfig b/arch/sparc/Kconfig index 59b0960..8da321c 100644 --- a/arch/sparc/Kconfig +++ b/arch/sparc/Kconfig @@ -44,6 +44,7 @@ config SPARC select ARCH_HAS_SG_CHAIN select CPU_NO_EFFICIENT_FFS select HAVE_ARCH_HARDENED_USERCOPY + select PROVE_LOCKING_SMALL if PROVE_LOCKING config SPARC32 def_bool !64BIT diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug index cab7405..597e589 100644 --- a/lib/Kconfig.debug +++ b/lib/Kconfig.debug @@ -1084,6 +1084,9 @@ config PROVE_LOCKING For more details, see Documentation/locking/lockdep-design.txt. +config PROVE_LOCKING_SMALL + bool + config LOCKDEP bool depends on DEBUG_KERNEL && TRACE_IRQFLAGS_SUPPORT && STACKTRACE_SUPPORT && LOCKDEP_SUPPORT -- 1.7.1
[toc] | [next] | [standalone]
| From | Peter Zijlstra <peterz@infradead.org> |
|---|---|
| Date | 2016-09-28 10:40 +0200 |
| Subject | Re: [PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc |
| Message-ID | <smizg-4mV-23@gated-at.bofh.it> |
| In reply to | #1492119 |
On Tue, Sep 27, 2016 at 12:33:27PM -0700, Babu Moger wrote: > This new config parameter limits the space used for "Lock debugging: > prove locking correctness" by about 4MB. The current sparc systems have > the limitation of 32MB size for kernel size including .text, .data and > .bss sections. With PROVE_LOCKING feature, the kernel size could grow > beyond this limit and causing system boot-up issues. With this option, > kernel limits the size of the entries of lock_chains, stack_trace etc., > so that kernel fits in required size limit. This is not visible to user > and only used for sparc. > > Signed-off-by: Babu Moger <babu.moger@oracle.com> You forgot to Cc Dave, and since you're touching sparc I need an Ack from him before I can queue this. Dave? > --- > arch/sparc/Kconfig | 1 + > lib/Kconfig.debug | 3 +++ > 2 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/arch/sparc/Kconfig b/arch/sparc/Kconfig > index 59b0960..8da321c 100644 > --- a/arch/sparc/Kconfig > +++ b/arch/sparc/Kconfig > @@ -44,6 +44,7 @@ config SPARC > select ARCH_HAS_SG_CHAIN > select CPU_NO_EFFICIENT_FFS > select HAVE_ARCH_HARDENED_USERCOPY > + select PROVE_LOCKING_SMALL if PROVE_LOCKING > > config SPARC32 > def_bool !64BIT > diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug > index cab7405..597e589 100644 > --- a/lib/Kconfig.debug > +++ b/lib/Kconfig.debug > @@ -1084,6 +1084,9 @@ config PROVE_LOCKING > > For more details, see Documentation/locking/lockdep-design.txt. > > +config PROVE_LOCKING_SMALL > + bool > + > config LOCKDEP > bool > depends on DEBUG_KERNEL && TRACE_IRQFLAGS_SUPPORT && STACKTRACE_SUPPORT && LOCKDEP_SUPPORT > -- > 1.7.1 >
[toc] | [prev] | [next] | [standalone]
| From | Babu Moger <babu.moger@oracle.com> |
|---|---|
| Date | 2016-09-29 16:00 +0200 |
| Subject | Re: [PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc |
| Message-ID | <smK2y-4NO-5@gated-at.bofh.it> |
| In reply to | #1492337 |
On 9/28/2016 3:39 AM, Peter Zijlstra wrote: > On Tue, Sep 27, 2016 at 12:33:27PM -0700, Babu Moger wrote: >> This new config parameter limits the space used for "Lock debugging: >> prove locking correctness" by about 4MB. The current sparc systems have >> the limitation of 32MB size for kernel size including .text, .data and >> .bss sections. With PROVE_LOCKING feature, the kernel size could grow >> beyond this limit and causing system boot-up issues. With this option, >> kernel limits the size of the entries of lock_chains, stack_trace etc., >> so that kernel fits in required size limit. This is not visible to user >> and only used for sparc. >> >> Signed-off-by: Babu Moger <babu.moger@oracle.com> > You forgot to Cc Dave, and since you're touching sparc I need an Ack > from him before I can queue this. Dave, Can you please take a look at the patch. Please ack it if it looks good. Thanks > > Dave? > >> --- >> arch/sparc/Kconfig | 1 + >> lib/Kconfig.debug | 3 +++ >> 2 files changed, 4 insertions(+), 0 deletions(-) >> >> diff --git a/arch/sparc/Kconfig b/arch/sparc/Kconfig >> index 59b0960..8da321c 100644 >> --- a/arch/sparc/Kconfig >> +++ b/arch/sparc/Kconfig >> @@ -44,6 +44,7 @@ config SPARC >> select ARCH_HAS_SG_CHAIN >> select CPU_NO_EFFICIENT_FFS >> select HAVE_ARCH_HARDENED_USERCOPY >> + select PROVE_LOCKING_SMALL if PROVE_LOCKING >> >> config SPARC32 >> def_bool !64BIT >> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug >> index cab7405..597e589 100644 >> --- a/lib/Kconfig.debug >> +++ b/lib/Kconfig.debug >> @@ -1084,6 +1084,9 @@ config PROVE_LOCKING >> >> For more details, see Documentation/locking/lockdep-design.txt. >> >> +config PROVE_LOCKING_SMALL >> + bool >> + >> config LOCKDEP >> bool >> depends on DEBUG_KERNEL && TRACE_IRQFLAGS_SUPPORT && STACKTRACE_SUPPORT && LOCKDEP_SUPPORT >> -- >> 1.7.1 >>
[toc] | [prev] | [next] | [standalone]
| From | David Miller <davem@davemloft.net> |
|---|---|
| Date | 2016-09-30 07:20 +0200 |
| Subject | Re: [PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc |
| Message-ID | <smYoN-5Xl-1@gated-at.bofh.it> |
| In reply to | #1493476 |
From: Babu Moger <babu.moger@oracle.com> Date: Thu, 29 Sep 2016 08:53:24 -0500 > > On 9/28/2016 3:39 AM, Peter Zijlstra wrote: >> On Tue, Sep 27, 2016 at 12:33:27PM -0700, Babu Moger wrote: >>> This new config parameter limits the space used for "Lock debugging: >>> prove locking correctness" by about 4MB. The current sparc systems >>> have >>> the limitation of 32MB size for kernel size including .text, .data and >>> .bss sections. With PROVE_LOCKING feature, the kernel size could grow >>> beyond this limit and causing system boot-up issues. With this option, >>> kernel limits the size of the entries of lock_chains, stack_trace >>> etc., >>> so that kernel fits in required size limit. This is not visible to >>> user >>> and only used for sparc. >>> >>> Signed-off-by: Babu Moger <babu.moger@oracle.com> >> You forgot to Cc Dave, and since you're touching sparc I need an Ack >> from him before I can queue this. > Dave, Can you please take a look at the patch. Please ack it if it > looks good. I am travelling and will look at it when I get a chance.
[toc] | [prev] | [next] | [standalone]
| From | Babu Moger <babu.moger@oracle.com> |
|---|---|
| Date | 2016-10-06 01:00 +0200 |
| Subject | Re: [PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc |
| Message-ID | <sp3kl-8nD-9@gated-at.bofh.it> |
| In reply to | #1493884 |
Dave, Gentle reminder to review this patch. Thanks On 9/30/2016 12:19 AM, David Miller wrote: > From: Babu Moger <babu.moger@oracle.com> > Date: Thu, 29 Sep 2016 08:53:24 -0500 > >> On 9/28/2016 3:39 AM, Peter Zijlstra wrote: >>> On Tue, Sep 27, 2016 at 12:33:27PM -0700, Babu Moger wrote: >>>> This new config parameter limits the space used for "Lock debugging: >>>> prove locking correctness" by about 4MB. The current sparc systems >>>> have >>>> the limitation of 32MB size for kernel size including .text, .data and >>>> .bss sections. With PROVE_LOCKING feature, the kernel size could grow >>>> beyond this limit and causing system boot-up issues. With this option, >>>> kernel limits the size of the entries of lock_chains, stack_trace >>>> etc., >>>> so that kernel fits in required size limit. This is not visible to >>>> user >>>> and only used for sparc. >>>> >>>> Signed-off-by: Babu Moger <babu.moger@oracle.com> >>> You forgot to Cc Dave, and since you're touching sparc I need an Ack >>> from him before I can queue this. >> Dave, Can you please take a look at the patch. Please ack it if it >> looks good. > I am travelling and will look at it when I get a chance.
[toc] | [prev] | [next] | [standalone]
| From | Sam Ravnborg <sam@ravnborg.org> |
|---|---|
| Date | 2016-10-06 06:30 +0200 |
| Subject | Re: [PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc |
| Message-ID | <sp8tI-3wt-3@gated-at.bofh.it> |
| In reply to | #1496113 |
On Wed, Oct 05, 2016 at 05:56:16PM -0500, Babu Moger wrote: > Dave, Gentle reminder to review this patch. Thanks No need to ping Dave. It is in patchwork and he will review the patch eventually. Sam
[toc] | [prev] | [next] | [standalone]
| From | Sam Ravnborg <sam@ravnborg.org> |
|---|---|
| Date | 2016-09-28 23:00 +0200 |
| Subject | Re: [PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc |
| Message-ID | <smu7n-3aV-13@gated-at.bofh.it> |
| In reply to | #1492119 |
On Tue, Sep 27, 2016 at 12:33:27PM -0700, Babu Moger wrote: > This new config parameter limits the space used for "Lock debugging: > prove locking correctness" by about 4MB. The current sparc systems have > the limitation of 32MB size for kernel size including .text, .data and > .bss sections. With PROVE_LOCKING feature, the kernel size could grow > beyond this limit and causing system boot-up issues. With this option, > kernel limits the size of the entries of lock_chains, stack_trace etc., > so that kernel fits in required size limit. This is not visible to user > and only used for sparc. > > Signed-off-by: Babu Moger <babu.moger@oracle.com> Acked-by: Sam Ravnborg <sam@ravnborg.org>
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web