2016-10-05 22:57:11

by Babu Moger

[permalink] [raw]
Subject: Re: [PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc

Dave, Gentle reminder to review this patch. Thanks

On 9/30/2016 12:19 AM, David Miller wrote:
> From: Babu Moger <[email protected]>
> 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 <[email protected]>
>>> 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.


2016-10-06 04:27:45

by Sam Ravnborg

[permalink] [raw]
Subject: Re: [PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc

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