Re: [PATCH 04/62] dax/bus.c: Fix a locking bug

[dan.j.williams]

Bart Van Assche wrote:
> From: Bart Van Assche <bvanassche@xxxxxxx>
> 
> Only unlock dax_dev_rwsem if it has been locked. This locking bug was
> detected by the Clang thread-safety analyzer.
> 
> Cc: Dan Williams <dan.j.williams@xxxxxxxxx>
> Cc: Vishal Verma <vishal.l.verma@xxxxxxxxx>
> Cc: Dave Jiang <dave.jiang@xxxxxxxxx>
> Cc: Alison Schofield <alison.schofield@xxxxxxxxx>
> Cc: nvdimm@xxxxxxxxxxxxxxx
> Cc: linux-cxl@xxxxxxxxxxxxxxx
> Fixes: c05ae9d85b47 ("dax/bus.c: replace driver-core lock usage by a local rwsem")
> Signed-off-by: Bart Van Assche <bvanassche@xxxxxxx>

Looks good,

Reviewed-by: Dan Williams <dan.j.williams@xxxxxxxxx>