[prev in list] [next in list] [prev in thread] [next in thread]
List: linux-m68k
Subject: Re: [PATCH 13/26] dm: convert to blk_alloc_disk/blk_cleanup_disk
From: Christoph Hellwig <hch () lst ! de>
Date: 2021-05-24 7:25:57
Message-ID: 20210524072557.GE23890 () lst ! de
[Download RAW message or body]
On Sun, May 23, 2021 at 10:10:34AM +0200, Hannes Reinecke wrote:
> Can't these conditionals be merged into a single 'if (md->disk)'?
> Eg like:
>
> if (md->disk) {
> spin_lock(&_minor_lock);
> md->disk->private_data = NULL;
> spin_unlock(&_minor_lock);
> del_gendisk(md->disk);
> dm_queue_destroy_keyslot_manager(md->queue);
> blk_cleanup_disk(md->queue);
> }
>
> We're now always allocating 'md->disk' and 'md->queue' together,
> so how can we end up in a situation where one is set without the other?
I guess we could do that, not sure it is worth the churn, though.
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic