diff options
| author | Christophe JAILLET <christophe.jaillet@wanadoo.fr> | 2024-06-08 16:34:20 +0200 |
|---|---|---|
| committer | Christian Brauner <brauner@kernel.org> | 2024-06-25 11:15:47 +0200 |
| commit | 7f07ee5a23a5ed3da7ed3574913aada6f4143a22 (patch) | |
| tree | 05335d824c786743a134dd195c3369834a1bf81c | |
| parent | be4edd1642ee205ed7bbf66edc0453b1be1fb8d7 (diff) | |
| download | linux-7f07ee5a23a5ed3da7ed3574913aada6f4143a22.tar.gz linux-7f07ee5a23a5ed3da7ed3574913aada6f4143a22.tar.bz2 linux-7f07ee5a23a5ed3da7ed3574913aada6f4143a22.zip | |
proc: Remove usage of the deprecated ida_simple_xx() API
ida_alloc() and ida_free() should be preferred to the deprecated
ida_simple_get() and ida_simple_remove().
Note that the upper limit of ida_simple_get() is exclusive, but the one of
ida_alloc_max() is inclusive. So a -1 has been added when needed.
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Link: https://lore.kernel.org/r/ae10003feb87d240163d0854de95f09e1f00be7d.1717855701.git.christophe.jaillet@wanadoo.fr
Reviewed-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Signed-off-by: Christian Brauner <brauner@kernel.org>
| -rw-r--r-- | fs/proc/generic.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/proc/generic.c b/fs/proc/generic.c index 775ce0bcf08c..c02f1e63f82d 100644 --- a/fs/proc/generic.c +++ b/fs/proc/generic.c @@ -202,8 +202,8 @@ int proc_alloc_inum(unsigned int *inum) { int i; - i = ida_simple_get(&proc_inum_ida, 0, UINT_MAX - PROC_DYNAMIC_FIRST + 1, - GFP_KERNEL); + i = ida_alloc_max(&proc_inum_ida, UINT_MAX - PROC_DYNAMIC_FIRST, + GFP_KERNEL); if (i < 0) return i; @@ -213,7 +213,7 @@ int proc_alloc_inum(unsigned int *inum) void proc_free_inum(unsigned int inum) { - ida_simple_remove(&proc_inum_ida, inum - PROC_DYNAMIC_FIRST); + ida_free(&proc_inum_ida, inum - PROC_DYNAMIC_FIRST); } static int proc_misc_d_revalidate(struct dentry *dentry, unsigned int flags) |