diff options
| author | Christian Brauner <brauner@kernel.org> | 2023-11-30 15:49:09 +0300 |
|---|---|---|
| committer | Christian Brauner <brauner@kernel.org> | 2023-12-12 16:24:13 +0300 |
| commit | 372a34e66fb7f95124fadae9c600b231c35696a7 (patch) | |
| tree | 5edc7444222b13d09d807f9bad860534c1a1b6e5 /fs/file_table.c | |
| parent | 24fa3ae9467f49dd9698fd884f2c6b13cc8ea12d (diff) | |
| download | linux-372a34e66fb7f95124fadae9c600b231c35696a7.tar.xz | |
fs: replace f_rcuhead with f_task_work
The naming is actively misleading since we switched to
SLAB_TYPESAFE_BY_RCU. rcu_head is #define callback_head. Use
callback_head directly and rename f_rcuhead to f_task_work.
Add comments in there to explain what it's used for.
Link: https://lore.kernel.org/r/20231130-vfs-files-fixes-v1-3-e73ca6f4ea83@kernel.org
Reviewed-by: Jan Kara <jack@suse.cz>
Reviewed-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Christian Brauner <brauner@kernel.org>
Diffstat (limited to 'fs/file_table.c')
| -rw-r--r-- | fs/file_table.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/file_table.c b/fs/file_table.c index 6deac386486d..3ba764d73fc9 100644 --- a/fs/file_table.c +++ b/fs/file_table.c @@ -407,7 +407,7 @@ static void delayed_fput(struct work_struct *unused) static void ____fput(struct callback_head *work) { - __fput(container_of(work, struct file, f_rcuhead)); + __fput(container_of(work, struct file, f_task_work)); } /* @@ -438,8 +438,8 @@ void fput(struct file *file) return; } if (likely(!in_interrupt() && !(task->flags & PF_KTHREAD))) { - init_task_work(&file->f_rcuhead, ____fput); - if (!task_work_add(task, &file->f_rcuhead, TWA_RESUME)) + init_task_work(&file->f_task_work, ____fput); + if (!task_work_add(task, &file->f_task_work, TWA_RESUME)) return; /* * After this task has run exit_task_work(), |