Re: [PATCH v2 4/8] cxl/decoder: Drop pointless locking

From: Alison Schofield
Date: Fri Jun 20 2025 - 17:03:14 EST


On Wed, Jun 18, 2025 at 10:04:12PM -0700, Dan Williams wrote:
> cxl_dpa_rwsem coordinates changes to dpa allocation settings for a given
> decoder. cxl_decoder_reset() has no need for a consistent snapshot of the
> dpa settings since it is merely clearing out whatever was there previously.
>
> Otherwise, cxl_region_rwsem protects against 'reset' racing 'setup'.
>
> In preparationg for converting to rw_semaphore_acquire semantics, drop this
> locking.
>

Reviewed-by: Alison Schofield <alison.schofield@xxxxxxxxx>