diff mbox series

security/keys: remove request_key_conswq and keyring_search_instkey() declarations

Message ID 20220909060452.1121620-1-cuigaosheng1@huawei.com (mailing list archive)
State New
Headers show
Series security/keys: remove request_key_conswq and keyring_search_instkey() declarations | expand

Commit Message

cuigaosheng Sept. 9, 2022, 6:04 a.m. UTC
request_key_conswq has been removed since
commit 76181c134f87 ("KEYS: Make request_key() and co fundamentally
asynchronous").

keyring_search_instkey() has been removed since
commit b5f545c880a2 ("[PATCH] keys: Permit running process to
instantiate keys").

so remove the declare for them from header file.

Signed-off-by: Gaosheng Cui <cuigaosheng1@huawei.com>
---
 security/keys/internal.h | 4 ----
 1 file changed, 4 deletions(-)

Comments

Jarkko Sakkinen Sept. 12, 2022, 11:39 a.m. UTC | #1
"KEYS: Remove orphan declarations from security/keys/internal.h"

On Fri, Sep 09, 2022 at 02:04:52PM +0800, Gaosheng Cui wrote:
> request_key_conswq has been removed since
> commit 76181c134f87 ("KEYS: Make request_key() and co fundamentally
> asynchronous").
> 
> keyring_search_instkey() has been removed since
> commit b5f545c880a2 ("[PATCH] keys: Permit running process to
> instantiate keys").
> 
> so remove the declare for them from header file.
> 
> Signed-off-by: Gaosheng Cui <cuigaosheng1@huawei.com>

"
Remove the following orphan declarations from security/keys/internal.h:

1. request_key_conswq()
2. keyring_search_instkey()
"

Then add two fixes tags after that.

BR, Jarkko
cuigaosheng Sept. 13, 2022, 1:53 a.m. UTC | #2
Thanks for taking the time to review this patch, I have made a patch V2 and submitted it.

link: https://patchwork.kernel.org/project/linux-security-module/patch/20220913014613.524253-1-cuigaosheng1@huawei.com/

在 2022/9/12 19:39, Jarkko Sakkinen 写道:
> "KEYS: Remove orphan declarations from security/keys/internal.h"
>
> On Fri, Sep 09, 2022 at 02:04:52PM +0800, Gaosheng Cui wrote:
>> request_key_conswq has been removed since
>> commit 76181c134f87 ("KEYS: Make request_key() and co fundamentally
>> asynchronous").
>>
>> keyring_search_instkey() has been removed since
>> commit b5f545c880a2 ("[PATCH] keys: Permit running process to
>> instantiate keys").
>>
>> so remove the declare for them from header file.
>>
>> Signed-off-by: Gaosheng Cui <cuigaosheng1@huawei.com>
> "
> Remove the following orphan declarations from security/keys/internal.h:
>
> 1. request_key_conswq()
> 2. keyring_search_instkey()
> "
>
> Then add two fixes tags after that.
>
> BR, Jarkko
>
> .
diff mbox series

Patch

diff --git a/security/keys/internal.h b/security/keys/internal.h
index 9b9cf3b6fcbb..6a2fb45c22f8 100644
--- a/security/keys/internal.h
+++ b/security/keys/internal.h
@@ -86,7 +86,6 @@  extern struct kmem_cache *key_jar;
 extern struct rb_root key_serial_tree;
 extern spinlock_t key_serial_lock;
 extern struct mutex key_construction_mutex;
-extern wait_queue_head_t request_key_conswq;
 
 extern void key_set_index_key(struct keyring_index_key *index_key);
 extern struct key_type *key_type_lookup(const char *type);
@@ -109,9 +108,6 @@  extern void __key_link_end(struct key *keyring,
 extern key_ref_t find_key_to_update(key_ref_t keyring_ref,
 				    const struct keyring_index_key *index_key);
 
-extern struct key *keyring_search_instkey(struct key *keyring,
-					  key_serial_t target_id);
-
 extern int iterate_over_keyring(const struct key *keyring,
 				int (*func)(const struct key *key, void *data),
 				void *data);