linux-kernel - Re: [PATCH v3 1/4] kernel/resource: Provide mem region release for SOFT RESERVES

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <93e45085-1ddf-4f99-a7c3-17670e9a2164@amd.com>
Date: Thu, 10 Apr 2025 10:07:30 -0500
From: "Bowman, Terry" <terry.bowman@....com>
To: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
 Jonathan Cameron <Jonathan.Cameron@...wei.com>
Cc: dave@...olabs.net, dave.jiang@...el.com, alison.schofield@...el.com,
 vishal.l.verma@...el.com, ira.weiny@...el.com, dan.j.williams@...el.com,
 willy@...radead.org, jack@...e.cz, rafael@...nel.org, len.brown@...el.com,
 pavel@....cz, ming.li@...omail.com, nathan.fontenot@....com,
 Smita.KoralahalliChannabasappa@....com, huang.ying.caritas@...il.com,
 yaoxt.fnst@...itsu.com, peterz@...radead.org, gregkh@...uxfoundation.org,
 quic_jjohnson@...cinc.com, ilpo.jarvinen@...ux.intel.com,
 bhelgaas@...gle.com, mika.westerberg@...ux.intel.com,
 akpm@...ux-foundation.org, gourry@...rry.net, linux-cxl@...r.kernel.org,
 linux-kernel@...r.kernel.org, nvdimm@...ts.linux.dev,
 linux-fsdevel@...r.kernel.org, linux-pm@...r.kernel.org, rrichter@....com,
 benjamin.cheatham@....com, PradeepVineshReddy.Kodamati@....com,
 lizhijian@...itsu.com
Subject: Re: [PATCH v3 1/4] kernel/resource: Provide mem region release for
 SOFT RESERVES

On 4/4/2025 8:25 AM, Andy Shevchenko wrote:
> On Fri, Apr 04, 2025 at 02:16:39PM +0100, Jonathan Cameron wrote:
>> On Thu, 3 Apr 2025 13:33:12 -0500 Terry Bowman <terry.bowman@....com> wrote:
> 
>>> Add a release_Sam_region_adjustable() interface to allow for
>>
>> Who is Sam?  (typo)
> 
> Somebody's uncle?
> 
> ...
> 
>>>  #ifdef CONFIG_MEMORY_HOTREMOVE
>>>  extern void release_mem_region_adjustable(resource_size_t, resource_size_t);
>>>  #endif
>>> +#ifdef CONFIG_CXL_REGION
>>> +extern void release_srmem_region_adjustable(resource_size_t, resource_size_t);
>> I'm not sure the srmem is obvious enough.  Maybe it's worth the long
>> name to spell it out some more.. e.g. something like
> 
> And perhaps drop 'extern' as it's not needed.
> 

Got it.

-Terry

>> extern void release_softresv_mem_region_adjustable() ?
> 
>>>  #ifdef CONFIG_MEMORY_HOTPLUG
>>>  extern void merge_system_ram_resource(struct resource *res);
>>>  #endif
>