diff options
| author | Jan Kara <jack@suse.cz> | 2024-01-09 11:00:46 +0100 |
|---|---|---|
| committer | Jan Kara <jack@suse.cz> | 2024-01-23 19:21:10 +0100 |
| commit | f6766303c0f2ffbcc640ef2701984a81a274aff2 (patch) | |
| tree | d1f49cb568e44723cae272ae141fc601d0f69bcc | |
| parent | quota: Remove BUG_ON from dqget() (diff) | |
| download | linux-f6766303c0f2ffbcc640ef2701984a81a274aff2.tar.xz linux-f6766303c0f2ffbcc640ef2701984a81a274aff2.zip | |
udf: Remove GFP_NOFS from dir iteration code
Directory iteration code was using GFP_NOFS allocations in two places.
However the code is called only under inode->i_rwsem which is generally
safe wrt reclaim. So we can do the allocations with GFP_KERNEL instead.
Signed-off-by: Jan Kara <jack@suse.cz>
| -rw-r--r-- | fs/udf/dir.c | 2 | ||||
| -rw-r--r-- | fs/udf/namei.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/fs/udf/dir.c b/fs/udf/dir.c index f6533f93851b..f94f45fe2c91 100644 --- a/fs/udf/dir.c +++ b/fs/udf/dir.c @@ -67,7 +67,7 @@ static int udf_readdir(struct file *file, struct dir_context *ctx) pos_valid = true; } - fname = kmalloc(UDF_NAME_LEN, GFP_NOFS); + fname = kmalloc(UDF_NAME_LEN, GFP_KERNEL); if (!fname) { ret = -ENOMEM; goto out; diff --git a/fs/udf/namei.c b/fs/udf/namei.c index 1bb6ed948927..1f14a0621a91 100644 --- a/fs/udf/namei.c +++ b/fs/udf/namei.c @@ -59,7 +59,7 @@ static int udf_fiiter_find_entry(struct inode *dir, const struct qstr *child, child->name[0] == '.' && child->name[1] == '.'; int ret; - fname = kmalloc(UDF_NAME_LEN, GFP_NOFS); + fname = kmalloc(UDF_NAME_LEN, GFP_KERNEL); if (!fname) return -ENOMEM; |