Commit e958f49b authored by Julia Lawall's avatar Julia Lawall Committed by Greg Kroah-Hartman

lustre: llite: Replace uses of OBD_{ALLOC,FREE}_LARGE

Replace uses of OBD_ALLOC_LARGE by libcfs_kvzalloc and OBD_FREE_LARGE by
kvfree.

The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)

// <smpl>
@@
expression ptr,size;
@@

- OBD_ALLOC_LARGE(ptr,size)
+ ptr = libcfs_kvzalloc(size, GFP_NOFS)

@@
expression ptr,size;
@@

- OBD_FREE_LARGE(ptr, size);
+ kvfree(ptr);
// </smpl>
Signed-off-by: default avatarJulia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent a7503434
...@@ -1548,7 +1548,7 @@ static long ll_dir_ioctl(struct file *file, unsigned int cmd, unsigned long arg) ...@@ -1548,7 +1548,7 @@ static long ll_dir_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
if (rc) if (rc)
return rc; return rc;
OBD_ALLOC_LARGE(lmm, lmmsize); lmm = libcfs_kvzalloc(lmmsize, GFP_NOFS);
if (lmm == NULL) if (lmm == NULL)
return -ENOMEM; return -ENOMEM;
if (copy_from_user(lmm, lum, lmmsize)) { if (copy_from_user(lmm, lum, lmmsize)) {
...@@ -1601,7 +1601,7 @@ static long ll_dir_ioctl(struct file *file, unsigned int cmd, unsigned long arg) ...@@ -1601,7 +1601,7 @@ static long ll_dir_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
free_lsm: free_lsm:
obd_free_memmd(sbi->ll_dt_exp, &lsm); obd_free_memmd(sbi->ll_dt_exp, &lsm);
free_lmm: free_lmm:
OBD_FREE_LARGE(lmm, lmmsize); kvfree(lmm);
return rc; return rc;
} }
case OBD_IOC_LLOG_CATINFO: { case OBD_IOC_LLOG_CATINFO: {
...@@ -1767,13 +1767,13 @@ static long ll_dir_ioctl(struct file *file, unsigned int cmd, unsigned long arg) ...@@ -1767,13 +1767,13 @@ static long ll_dir_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
if (totalsize >= MDS_MAXREQSIZE / 3) if (totalsize >= MDS_MAXREQSIZE / 3)
return -E2BIG; return -E2BIG;
OBD_ALLOC_LARGE(hur, totalsize); hur = libcfs_kvzalloc(totalsize, GFP_NOFS);
if (hur == NULL) if (hur == NULL)
return -ENOMEM; return -ENOMEM;
/* Copy the whole struct */ /* Copy the whole struct */
if (copy_from_user(hur, (void *)arg, totalsize)) { if (copy_from_user(hur, (void *)arg, totalsize)) {
OBD_FREE_LARGE(hur, totalsize); kvfree(hur);
return -EFAULT; return -EFAULT;
} }
...@@ -1800,7 +1800,7 @@ static long ll_dir_ioctl(struct file *file, unsigned int cmd, unsigned long arg) ...@@ -1800,7 +1800,7 @@ static long ll_dir_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
hur, NULL); hur, NULL);
} }
OBD_FREE_LARGE(hur, totalsize); kvfree(hur);
return rc; return rc;
} }
......
...@@ -1282,7 +1282,7 @@ static int ll_lov_recreate(struct inode *inode, struct ost_id *oi, u32 ost_idx) ...@@ -1282,7 +1282,7 @@ static int ll_lov_recreate(struct inode *inode, struct ost_id *oi, u32 ost_idx)
lsm_size = sizeof(*lsm) + (sizeof(struct lov_oinfo) * lsm_size = sizeof(*lsm) + (sizeof(struct lov_oinfo) *
(lsm->lsm_stripe_count)); (lsm->lsm_stripe_count));
OBD_ALLOC_LARGE(lsm2, lsm_size); lsm2 = libcfs_kvzalloc(lsm_size, GFP_NOFS);
if (lsm2 == NULL) { if (lsm2 == NULL) {
rc = -ENOMEM; rc = -ENOMEM;
goto out; goto out;
...@@ -1300,7 +1300,7 @@ static int ll_lov_recreate(struct inode *inode, struct ost_id *oi, u32 ost_idx) ...@@ -1300,7 +1300,7 @@ static int ll_lov_recreate(struct inode *inode, struct ost_id *oi, u32 ost_idx)
rc = obd_create(NULL, exp, oa, &lsm2, &oti); rc = obd_create(NULL, exp, oa, &lsm2, &oti);
ll_inode_size_unlock(inode); ll_inode_size_unlock(inode);
OBD_FREE_LARGE(lsm2, lsm_size); kvfree(lsm2);
goto out; goto out;
out: out:
ccc_inode_lsm_put(inode, lsm); ccc_inode_lsm_put(inode, lsm);
...@@ -1477,12 +1477,12 @@ static int ll_lov_setea(struct inode *inode, struct file *file, ...@@ -1477,12 +1477,12 @@ static int ll_lov_setea(struct inode *inode, struct file *file,
if (!capable(CFS_CAP_SYS_ADMIN)) if (!capable(CFS_CAP_SYS_ADMIN))
return -EPERM; return -EPERM;
OBD_ALLOC_LARGE(lump, lum_size); lump = libcfs_kvzalloc(lum_size, GFP_NOFS);
if (lump == NULL) if (lump == NULL)
return -ENOMEM; return -ENOMEM;
if (copy_from_user(lump, (struct lov_user_md *)arg, lum_size)) { if (copy_from_user(lump, (struct lov_user_md *)arg, lum_size)) {
OBD_FREE_LARGE(lump, lum_size); kvfree(lump);
return -EFAULT; return -EFAULT;
} }
...@@ -1490,7 +1490,7 @@ static int ll_lov_setea(struct inode *inode, struct file *file, ...@@ -1490,7 +1490,7 @@ static int ll_lov_setea(struct inode *inode, struct file *file,
lum_size); lum_size);
cl_lov_delay_create_clear(&file->f_flags); cl_lov_delay_create_clear(&file->f_flags);
OBD_FREE_LARGE(lump, lum_size); kvfree(lump);
return rc; return rc;
} }
...@@ -1802,7 +1802,7 @@ static int ll_ioctl_fiemap(struct inode *inode, unsigned long arg) ...@@ -1802,7 +1802,7 @@ static int ll_ioctl_fiemap(struct inode *inode, unsigned long arg)
num_bytes = sizeof(*fiemap_s) + (extent_count * num_bytes = sizeof(*fiemap_s) + (extent_count *
sizeof(struct ll_fiemap_extent)); sizeof(struct ll_fiemap_extent));
OBD_ALLOC_LARGE(fiemap_s, num_bytes); fiemap_s = libcfs_kvzalloc(num_bytes, GFP_NOFS);
if (fiemap_s == NULL) if (fiemap_s == NULL)
return -ENOMEM; return -ENOMEM;
...@@ -1839,7 +1839,7 @@ static int ll_ioctl_fiemap(struct inode *inode, unsigned long arg) ...@@ -1839,7 +1839,7 @@ static int ll_ioctl_fiemap(struct inode *inode, unsigned long arg)
rc = -EFAULT; rc = -EFAULT;
error: error:
OBD_FREE_LARGE(fiemap_s, num_bytes); kvfree(fiemap_s);
return rc; return rc;
} }
...@@ -3055,7 +3055,7 @@ static int ll_fiemap(struct inode *inode, struct fiemap_extent_info *fieinfo, ...@@ -3055,7 +3055,7 @@ static int ll_fiemap(struct inode *inode, struct fiemap_extent_info *fieinfo,
num_bytes = sizeof(*fiemap) + (extent_count * num_bytes = sizeof(*fiemap) + (extent_count *
sizeof(struct ll_fiemap_extent)); sizeof(struct ll_fiemap_extent));
OBD_ALLOC_LARGE(fiemap, num_bytes); fiemap = libcfs_kvzalloc(num_bytes, GFP_NOFS);
if (fiemap == NULL) if (fiemap == NULL)
return -ENOMEM; return -ENOMEM;
...@@ -3077,7 +3077,7 @@ static int ll_fiemap(struct inode *inode, struct fiemap_extent_info *fieinfo, ...@@ -3077,7 +3077,7 @@ static int ll_fiemap(struct inode *inode, struct fiemap_extent_info *fieinfo,
fiemap->fm_mapped_extents * fiemap->fm_mapped_extents *
sizeof(struct ll_fiemap_extent)); sizeof(struct ll_fiemap_extent));
OBD_FREE_LARGE(fiemap, num_bytes); kvfree(fiemap);
return rc; return rc;
} }
...@@ -3366,7 +3366,7 @@ static int ll_layout_fetch(struct inode *inode, struct ldlm_lock *lock) ...@@ -3366,7 +3366,7 @@ static int ll_layout_fetch(struct inode *inode, struct ldlm_lock *lock)
goto out; goto out;
} }
OBD_ALLOC_LARGE(lvbdata, lmmsize); lvbdata = libcfs_kvzalloc(lmmsize, GFP_NOFS);
if (lvbdata == NULL) { if (lvbdata == NULL) {
rc = -ENOMEM; rc = -ENOMEM;
goto out; goto out;
...@@ -3375,7 +3375,7 @@ static int ll_layout_fetch(struct inode *inode, struct ldlm_lock *lock) ...@@ -3375,7 +3375,7 @@ static int ll_layout_fetch(struct inode *inode, struct ldlm_lock *lock)
memcpy(lvbdata, lmm, lmmsize); memcpy(lvbdata, lmm, lmmsize);
lock_res_and_lock(lock); lock_res_and_lock(lock);
if (lock->l_lvb_data != NULL) if (lock->l_lvb_data != NULL)
OBD_FREE_LARGE(lock->l_lvb_data, lock->l_lvb_len); kvfree(lock->l_lvb_data);
lock->l_lvb_data = lvbdata; lock->l_lvb_data = lvbdata;
lock->l_lvb_len = lmmsize; lock->l_lvb_len = lmmsize;
......
...@@ -200,12 +200,12 @@ static inline int ll_get_user_pages(int rw, unsigned long user_addr, ...@@ -200,12 +200,12 @@ static inline int ll_get_user_pages(int rw, unsigned long user_addr,
*max_pages = (user_addr + size + PAGE_CACHE_SIZE - 1) >> PAGE_CACHE_SHIFT; *max_pages = (user_addr + size + PAGE_CACHE_SIZE - 1) >> PAGE_CACHE_SHIFT;
*max_pages -= user_addr >> PAGE_CACHE_SHIFT; *max_pages -= user_addr >> PAGE_CACHE_SHIFT;
OBD_ALLOC_LARGE(*pages, *max_pages * sizeof(**pages)); *pages = libcfs_kvzalloc(*max_pages * sizeof(**pages), GFP_NOFS);
if (*pages) { if (*pages) {
result = get_user_pages_fast(user_addr, *max_pages, result = get_user_pages_fast(user_addr, *max_pages,
(rw == READ), *pages); (rw == READ), *pages);
if (unlikely(result <= 0)) if (unlikely(result <= 0))
OBD_FREE_LARGE(*pages, *max_pages * sizeof(**pages)); kvfree(*pages);
} }
return result; return result;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment