Re: [PATCH 03/10] md: implement ->free_disk
From: Logan Gunthorpe <logang@deltatee.com>
Date: 2022-07-18 20:02:17
Also in:
linux-block
On 2022-07-18 00:34, Christoph Hellwig wrote:
quoted hunk ↗ jump to hunk
Ensure that all private data is only freed once all accesses are done. Signed-off-by: Christoph Hellwig <hch@lst.de> --- 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@deltatee.com> Logan