On Thu, Apr 03, 2025 at 01:33:12PM -0500, Terry Bowman wrote:
> From: Nathan Fontenot <nathan.fontenot@xxxxxxx>
>
> Add a release_Sam_region_adjustable() interface to allow for
> removing SOFT RESERVE memory resources. This extracts out the code
> to remove a mem region into a common __release_mem_region_adjustable()
> routine, this routine takes additional parameters of an IORES
> descriptor type to add checks for IORES_DESC_* and a flag to check
> for IORESOURCE_BUSY to control it's behavior.
>
> The existing release_mem_region_adjustable() is a front end to the
> common code and a new release_srmem_region_adjustable() is added to
> release SOFT RESERVE resources.
...
> +void release_mem_region_adjustable(resource_size_t start, resource_size_t size)
> +{
> + return __release_mem_region_adjustable(start, size,
You have still room on the previous line for the parameters.
> + true, IORES_DESC_NONE);
Return on void?! Interesting... What do you want to do here?
> +}
> +EXPORT_SYMBOL(release_mem_region_adjustable);
> +#endif
> +
> +#ifdef CONFIG_CXL_REGION
> +void release_srmem_region_adjustable(resource_size_t start,
> + resource_size_t size)
This can be put on a single line.
> +{
> + return __release_mem_region_adjustable(start, size,
> + false, IORES_DESC_SOFT_RESERVED);
Same comments as per above function.
> +}
> +EXPORT_SYMBOL(release_srmem_region_adjustable);
> +#endif
--
With Best Regards,
Andy Shevchenko
