diff options
author | Kari Argillander <kari.argillander@gmail.com> | 2021-08-24 21:37:07 +0300 |
---|---|---|
committer | Konstantin Komarov <almaz.alexandrovich@paragon-software.com> | 2021-08-27 17:05:12 +0300 |
commit | 195c52bdd5d5ecfdabf5a7c6159efe299e534f84 (patch) | |
tree | 19b4150df32241152b7b8233aa7099e1e7a794c5 /fs/ntfs3/debug.h | |
parent | fa3cacf544636b2dc48cfb2f277a2071f14d66a2 (diff) |
fs/ntfs3: Do not use driver own alloc wrappers
Problem with these wrapper is that we cannot take off example GFP_NOFS
flag. It is not recomended use those in all places. Also if we change
one driver specific wrapper to kernel wrapper then it would look really
weird. People should be most familiar with kernel wrappers so let's just
use those ones.
Driver specific alloc wrapper also confuse some static analyzing tools,
good example is example kernels checkpatch tool. After we converter
these to kernel specific then warnings is showed.
Following Coccinelle script was used to automate changing.
virtual patch
@alloc depends on patch@
expression x;
expression y;
@@
(
- ntfs_malloc(x)
+ kmalloc(x, GFP_NOFS)
|
- ntfs_zalloc(x)
+ kzalloc(x, GFP_NOFS)
|
- ntfs_vmalloc(x)
+ kvmalloc(x, GFP_NOFS)
|
- ntfs_free(x)
+ kfree(x)
|
- ntfs_vfree(x)
+ kvfree(x)
|
- ntfs_memdup(x, y)
+ kmemdup(x, y, GFP_NOFS)
)
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Kari Argillander <kari.argillander@gmail.com>
Signed-off-by: Konstantin Komarov <almaz.alexandrovich@paragon-software.com>
Diffstat (limited to 'fs/ntfs3/debug.h')
-rw-r--r-- | fs/ntfs3/debug.h | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/fs/ntfs3/debug.h b/fs/ntfs3/debug.h index aff4cead9f58..c8403e876b2d 100644 --- a/fs/ntfs3/debug.h +++ b/fs/ntfs3/debug.h @@ -47,12 +47,5 @@ void ntfs_inode_printk(struct inode *inode, const char *fmt, ...) #define ntfs_inode_warn(inode, fmt, ...) \ ntfs_inode_printk(inode, KERN_WARNING fmt, ##__VA_ARGS__) -#define ntfs_malloc(s) kmalloc(s, GFP_NOFS) -#define ntfs_zalloc(s) kzalloc(s, GFP_NOFS) -#define ntfs_vmalloc(s) kvmalloc(s, GFP_KERNEL) -#define ntfs_free(p) kfree(p) -#define ntfs_vfree(p) kvfree(p) -#define ntfs_memdup(src, len) kmemdup(src, len, GFP_NOFS) - #endif /* _LINUX_NTFS3_DEBUG_H */ // clang-format on |