[PATCH 1/6] cxlflash: Avoid mutex when destroying context

Manoj Kumar manoj at linux.vnet.ibm.com
Thu Aug 18 09:22:38 AEST 2016

Acked-by: Manoj N. Kumar <manoj at linux.vnet.ibm.com>

On 8/9/2016 6:39 PM, Matthew R. Ochs wrote:
> Context information structures are protected by a mutex that is held
> when accessing/manipulating the context. When the code that manages
> these structures was authored, a decision was made to include taking
> the mutex as part of the allocation/initialization sequence and also
> handle the scenario where the mutex was already held when freeing the
> context.
> While not a problem outright, this design decision has been deemed as
> too flexible and the code should be made more rigid to avoid future bugs.
> In addition, further review of the code yields that the existing mutex
> manipulations in both of these context management paths are superfluous.
> This commit removes the obtaining of the context mutex in the context
> initialization routine and assumes the mutex is not held in the context
> free path.
> Inspired-by: Al Viro <viro at zeniv.linux.org.uk>
> Signed-off-by: Matthew R. Ochs <mrochs at linux.vnet.ibm.com>

More information about the Linuxppc-dev mailing list