On 2022-07-18 00:34, Christoph Hellwig wrote: > Ensure that all private data is only freed once all accesses are done. > > Signed-off-by: Christoph Hellwig <hch@xxxxxx> > --- > drivers/md/md.c | 17 ++++++++++++----- > 1 file changed, 12 insertions(+), 5 deletions(-) > > diff --git a/drivers/md/md.c b/drivers/md/md.c > index 64c7c24d267bc..1e658d5060842 100644 > --- a/drivers/md/md.c > +++ b/drivers/md/md.c > @@ -5602,11 +5602,6 @@ static void md_free(struct kobject *ko) > del_gendisk(mddev->gendisk); > blk_cleanup_disk(mddev->gendisk); > } Looks like, with the previous patch, we can now remove the conditional on gendisk. Other than that it looks good: Reviewed-by: Logan Gunthorpe <logang@xxxxxxxxxxxx> Logan