diff options
| author | Christoph Hellwig <hch@lst.de> | 2024-06-13 11:48:16 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2024-09-08 08:56:38 +0300 |
| commit | 72de6c05742f383405581eda4fa2544559110edf (patch) | |
| tree | ec30935d35427850e52cf4ae27656a51b0f01800 | |
| parent | dc537a72f64890d883d24ae4ac58733fc5bc523d (diff) | |
| download | linux-72de6c05742f383405581eda4fa2544559110edf.tar.xz | |
block: remove the blk_flush_integrity call in blk_integrity_unregister
[ Upstream commit e8bc14d116aeac8f0f133ec8d249acf4e0658da7 ]
Now that there are no indirect calls for PI processing there is no
way to dereference a NULL pointer here. Additionally drivers now always
freeze the queue (or in case of stacking drivers use their internal
equivalent) around changing the integrity profile.
This is effectively a revert of commit 3df49967f6f1 ("block: flush the
integrity workqueue in blk_integrity_unregister").
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Martin K. Petersen <martin.petersen@oracle.com>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Link: https://lore.kernel.org/r/20240613084839.1044015-7-hch@lst.de
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Sasha Levin <sashal@kernel.org>
| -rw-r--r-- | block/blk-integrity.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/block/blk-integrity.c b/block/blk-integrity.c index ccbeb6dfa87a..8dd8a0126274 100644 --- a/block/blk-integrity.c +++ b/block/blk-integrity.c @@ -397,8 +397,6 @@ void blk_integrity_unregister(struct gendisk *disk) if (!bi->profile) return; - /* ensure all bios are off the integrity workqueue */ - blk_flush_integrity(); blk_queue_flag_clear(QUEUE_FLAG_STABLE_WRITES, disk->queue); memset(bi, 0, sizeof(*bi)); } |