[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <dbce90d4-17e5-15a3-4336-9efede16c772@codeaurora.org>
Date: Mon, 28 Dec 2020 10:21:31 +0530
From: Vijayanand Jitta <vjitta@...eaurora.org>
To: Alexander Potapenko <glider@...gle.com>
Cc: Minchan Kim <minchan@...nel.org>,
Vincenzo Frascino <vincenzo.frascino@....com>,
dan.j.williams@...el.com, broonie@...nel.org,
Masami Hiramatsu <mhiramat@...nel.org>,
LKML <linux-kernel@...r.kernel.org>,
Andrew Morton <akpm@...ux-foundation.org>,
Andrey Konovalov <andreyknvl@...gle.com>, qcai@...hat.com,
ylal@...eaurora.org, vinmenon@...eaurora.org,
kasan-dev <kasan-dev@...glegroups.com>
Subject: Re: [PATCH v3] lib: stackdepot: Add support to configure
STACK_HASH_SIZE
On 12/23/2020 8:10 PM, Alexander Potapenko wrote:
>>
>> Michan, We would still need config option so that we can reduce the
>> memory consumption on low ram devices using config.
>>
>> Alex, On this,
>> "I also suppose device vendors may prefer setting a fixed (maybe
>> non-default) hash size for low-memory devices rather than letting the
>> admins increase it."
>> I see kernel param swiotlb does similar thing i.e; '0' to disable and
>> set a value to configure size.
>>
>> I am fine with either of the approaches,
>>
>> 1. I can split this patch into two
>> i) A bool variable to enable/disable stack depot.
>> ii) A config for the size.
>
> I still believe this is a more appropriate solution.
>
> Thanks in advance!
>
Thanks, Will work on a patch with above approach.
Thanks,
Vijay
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a
member of Code Aurora Forum, hosted by The Linux Foundation
Powered by blists - more mailing lists