* [PATCH] blk-crypto: remove blk_crypto_unregister()
@ 2021-11-24 1:37 Eric Biggers
2021-11-24 7:43 ` Christoph Hellwig
2021-11-24 15:34 ` Jens Axboe
0 siblings, 2 replies; 3+ messages in thread
From: Eric Biggers @ 2021-11-24 1:37 UTC (permalink / raw)
To: linux-block
From: Eric Biggers <ebiggers@google.com>
This function is trivial and is only used in one place. Having this
function is misleading because it implies that blk_crypto_register()
needs to be paired with blk_crypto_unregister(), which is not the case.
Just set disk->queue->crypto_profile to NULL directly.
Signed-off-by: Eric Biggers <ebiggers@google.com>
---
block/blk-crypto-profile.c | 5 -----
block/blk-integrity.c | 2 +-
include/linux/blkdev.h | 4 ----
3 files changed, 1 insertion(+), 10 deletions(-)
diff --git a/block/blk-crypto-profile.c b/block/blk-crypto-profile.c
index 605ba0626a5c0..96c511967386d 100644
--- a/block/blk-crypto-profile.c
+++ b/block/blk-crypto-profile.c
@@ -463,11 +463,6 @@ bool blk_crypto_register(struct blk_crypto_profile *profile,
}
EXPORT_SYMBOL_GPL(blk_crypto_register);
-void blk_crypto_unregister(struct request_queue *q)
-{
- q->crypto_profile = NULL;
-}
-
/**
* blk_crypto_intersect_capabilities() - restrict supported crypto capabilities
* by child device
diff --git a/block/blk-integrity.c b/block/blk-integrity.c
index d670d54e5f7ac..69eed260a8239 100644
--- a/block/blk-integrity.c
+++ b/block/blk-integrity.c
@@ -411,7 +411,7 @@ void blk_integrity_register(struct gendisk *disk, struct blk_integrity *template
#ifdef CONFIG_BLK_INLINE_ENCRYPTION
if (disk->queue->crypto_profile) {
pr_warn("blk-integrity: Integrity and hardware inline encryption are not supported together. Disabling hardware inline encryption.\n");
- blk_crypto_unregister(disk->queue);
+ disk->queue->crypto_profile = NULL;
}
#endif
}
diff --git a/include/linux/blkdev.h b/include/linux/blkdev.h
index bd4370baccca3..26cad06ed9f8d 100644
--- a/include/linux/blkdev.h
+++ b/include/linux/blkdev.h
@@ -1171,8 +1171,6 @@ int kblockd_mod_delayed_work_on(int cpu, struct delayed_work *dwork, unsigned lo
bool blk_crypto_register(struct blk_crypto_profile *profile,
struct request_queue *q);
-void blk_crypto_unregister(struct request_queue *q);
-
#else /* CONFIG_BLK_INLINE_ENCRYPTION */
static inline bool blk_crypto_register(struct blk_crypto_profile *profile,
@@ -1181,8 +1179,6 @@ static inline bool blk_crypto_register(struct blk_crypto_profile *profile,
return true;
}
-static inline void blk_crypto_unregister(struct request_queue *q) { }
-
#endif /* CONFIG_BLK_INLINE_ENCRYPTION */
enum blk_unique_id {
base-commit: 136057256686de39cc3a07c2e39ef6bc43003ff6
--
2.34.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] blk-crypto: remove blk_crypto_unregister()
2021-11-24 1:37 [PATCH] blk-crypto: remove blk_crypto_unregister() Eric Biggers
@ 2021-11-24 7:43 ` Christoph Hellwig
2021-11-24 15:34 ` Jens Axboe
1 sibling, 0 replies; 3+ messages in thread
From: Christoph Hellwig @ 2021-11-24 7:43 UTC (permalink / raw)
To: Eric Biggers; +Cc: linux-block
Looks good,
Reviewed-by: Christoph Hellwig <hch@lst.de>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] blk-crypto: remove blk_crypto_unregister()
2021-11-24 1:37 [PATCH] blk-crypto: remove blk_crypto_unregister() Eric Biggers
2021-11-24 7:43 ` Christoph Hellwig
@ 2021-11-24 15:34 ` Jens Axboe
1 sibling, 0 replies; 3+ messages in thread
From: Jens Axboe @ 2021-11-24 15:34 UTC (permalink / raw)
To: linux-block, Eric Biggers
On Tue, 23 Nov 2021 17:37:33 -0800, Eric Biggers wrote:
> From: Eric Biggers <ebiggers@google.com>
>
> This function is trivial and is only used in one place. Having this
> function is misleading because it implies that blk_crypto_register()
> needs to be paired with blk_crypto_unregister(), which is not the case.
> Just set disk->queue->crypto_profile to NULL directly.
>
> [...]
Applied, thanks!
[1/1] blk-crypto: remove blk_crypto_unregister()
commit: f08fd5d1f9d146428809a7cd61d557df6af56f7b
Best regards,
--
Jens Axboe
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-11-24 15:35 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-24 1:37 [PATCH] blk-crypto: remove blk_crypto_unregister() Eric Biggers
2021-11-24 7:43 ` Christoph Hellwig
2021-11-24 15:34 ` Jens Axboe
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.