We're hold "spin_lock(&ent->fs_fcheck->fc_lock)" so the allocation has
to be GFP_ATOMIC.

I changed the sizeof() because otherwise the line goes over the 80
character limit and also the new way is prefered kernel style.

Fixes: e467fe5da718 ('ocfs2: sysfile interfaces for online file check')
Signed-off-by: Dan Carpenter <dan.carpen...@oracle.com>

diff --git a/fs/ocfs2/filecheck.c b/fs/ocfs2/filecheck.c
index 3332af1..9613663 100644
--- a/fs/ocfs2/filecheck.c
+++ b/fs/ocfs2/filecheck.c
@@ -544,7 +544,7 @@ static ssize_t ocfs2_filecheck_store(struct kobject *kobj,
                        BUG_ON(!ocfs2_filecheck_erase_entry(ent));
                }
 
-               entry = kmalloc(sizeof(struct ocfs2_filecheck_entry), GFP_NOFS);
+               entry = kmalloc(sizeof(*entry), GFP_ATOMIC);
                if (entry) {
                        entry->fe_ino = args.fa_ino;
                        entry->fe_type = args.fa_type;

_______________________________________________
Ocfs2-devel mailing list
Ocfs2-devel@oss.oracle.com
https://oss.oracle.com/mailman/listinfo/ocfs2-devel

Reply via email to