diff options
author | Namjae Jeon <namjae.jeon@samsung.com> | 2021-04-13 13:06:30 +0900 |
---|---|---|
committer | Steve French <stfrench@microsoft.com> | 2021-05-10 19:15:39 -0500 |
commit | d40012a83f87f47967ad0b3c346179c7e5339ae7 (patch) | |
tree | db996f57ce865ef62f03d8ff1ab1b4d5f94970a7 /fs/cifsd/ksmbd_work.c | |
parent | 1920bb1f8022202530eeae3e488d6f5156799faf (diff) | |
download | lwn-d40012a83f87f47967ad0b3c346179c7e5339ae7.tar.gz lwn-d40012a83f87f47967ad0b3c346179c7e5339ae7.zip |
cifsd: declare ida statically
Matthew pointed out that embedding struct ida into the struct is
better than having a pointer to it.
This patch initialise it statically using DEFINE_IDA() or ida_init()
and remove ksmbd_ida_alloc/free().
Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs/cifsd/ksmbd_work.c')
-rw-r--r-- | fs/cifsd/ksmbd_work.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/cifsd/ksmbd_work.c b/fs/cifsd/ksmbd_work.c index 33ee52c1829f..eb8c8a34acab 100644 --- a/fs/cifsd/ksmbd_work.c +++ b/fs/cifsd/ksmbd_work.c @@ -53,7 +53,7 @@ void ksmbd_free_work_struct(struct ksmbd_work *work) kfree(work->tr_buf); kvfree(work->request_buf); if (work->async_id) - ksmbd_release_id(work->conn->async_ida, work->async_id); + ksmbd_release_id(&work->conn->async_ida, work->async_id); kmem_cache_free(work_cache, work); } |