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

lustre: lov: 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 8cc3792a
...@@ -95,7 +95,7 @@ struct lov_stripe_md *lsm_alloc_plain(__u16 stripe_count, int *size) ...@@ -95,7 +95,7 @@ struct lov_stripe_md *lsm_alloc_plain(__u16 stripe_count, int *size)
oinfo_ptrs_size = sizeof(struct lov_oinfo *) * stripe_count; oinfo_ptrs_size = sizeof(struct lov_oinfo *) * stripe_count;
*size = sizeof(struct lov_stripe_md) + oinfo_ptrs_size; *size = sizeof(struct lov_stripe_md) + oinfo_ptrs_size;
OBD_ALLOC_LARGE(lsm, *size); lsm = libcfs_kvzalloc(*size, GFP_NOFS);
if (!lsm) if (!lsm)
return NULL; return NULL;
...@@ -111,7 +111,7 @@ struct lov_stripe_md *lsm_alloc_plain(__u16 stripe_count, int *size) ...@@ -111,7 +111,7 @@ struct lov_stripe_md *lsm_alloc_plain(__u16 stripe_count, int *size)
err: err:
while (--i >= 0) while (--i >= 0)
OBD_SLAB_FREE(lsm->lsm_oinfo[i], lov_oinfo_slab, sizeof(*loi)); OBD_SLAB_FREE(lsm->lsm_oinfo[i], lov_oinfo_slab, sizeof(*loi));
OBD_FREE_LARGE(lsm, *size); kvfree(lsm);
return NULL; return NULL;
} }
...@@ -123,8 +123,7 @@ void lsm_free_plain(struct lov_stripe_md *lsm) ...@@ -123,8 +123,7 @@ void lsm_free_plain(struct lov_stripe_md *lsm)
for (i = 0; i < stripe_count; i++) for (i = 0; i < stripe_count; i++)
OBD_SLAB_FREE(lsm->lsm_oinfo[i], lov_oinfo_slab, OBD_SLAB_FREE(lsm->lsm_oinfo[i], lov_oinfo_slab,
sizeof(struct lov_oinfo)); sizeof(struct lov_oinfo));
OBD_FREE_LARGE(lsm, sizeof(struct lov_stripe_md) + kvfree(lsm);
stripe_count * sizeof(struct lov_oinfo *));
} }
static void lsm_unpackmd_common(struct lov_stripe_md *lsm, static void lsm_unpackmd_common(struct lov_stripe_md *lsm,
......
...@@ -286,8 +286,10 @@ static int lov_io_subio_init(const struct lu_env *env, struct lov_io *lio, ...@@ -286,8 +286,10 @@ static int lov_io_subio_init(const struct lu_env *env, struct lov_io *lio,
* Need to be optimized, we can't afford to allocate a piece of memory * Need to be optimized, we can't afford to allocate a piece of memory
* when writing a page. -jay * when writing a page. -jay
*/ */
OBD_ALLOC_LARGE(lio->lis_subs, lio->lis_subs =
lsm->lsm_stripe_count * sizeof(lio->lis_subs[0])); libcfs_kvzalloc(lsm->lsm_stripe_count *
sizeof(lio->lis_subs[0]),
GFP_NOFS);
if (lio->lis_subs != NULL) { if (lio->lis_subs != NULL) {
lio->lis_nr_subios = lio->lis_stripe_count; lio->lis_nr_subios = lio->lis_stripe_count;
lio->lis_single_subio_index = -1; lio->lis_single_subio_index = -1;
...@@ -360,8 +362,7 @@ static void lov_io_fini(const struct lu_env *env, const struct cl_io_slice *ios) ...@@ -360,8 +362,7 @@ static void lov_io_fini(const struct lu_env *env, const struct cl_io_slice *ios)
if (lio->lis_subs != NULL) { if (lio->lis_subs != NULL) {
for (i = 0; i < lio->lis_nr_subios; i++) for (i = 0; i < lio->lis_nr_subios; i++)
lov_io_sub_fini(env, lio, &lio->lis_subs[i]); lov_io_sub_fini(env, lio, &lio->lis_subs[i]);
OBD_FREE_LARGE(lio->lis_subs, kvfree(lio->lis_subs);
lio->lis_nr_subios * sizeof(lio->lis_subs[0]));
lio->lis_nr_subios = 0; lio->lis_nr_subios = 0;
} }
...@@ -604,8 +605,10 @@ static int lov_io_submit(const struct lu_env *env, ...@@ -604,8 +605,10 @@ static int lov_io_submit(const struct lu_env *env,
LASSERT(lio->lis_subs != NULL); LASSERT(lio->lis_subs != NULL);
if (alloc) { if (alloc) {
OBD_ALLOC_LARGE(stripes_qin, stripes_qin =
sizeof(*stripes_qin) * lio->lis_nr_subios); libcfs_kvzalloc(sizeof(*stripes_qin) *
lio->lis_nr_subios,
GFP_NOFS);
if (stripes_qin == NULL) if (stripes_qin == NULL)
return -ENOMEM; return -ENOMEM;
...@@ -658,8 +661,7 @@ static int lov_io_submit(const struct lu_env *env, ...@@ -658,8 +661,7 @@ static int lov_io_submit(const struct lu_env *env,
} }
if (alloc) { if (alloc) {
OBD_FREE_LARGE(stripes_qin, kvfree(stripes_qin);
sizeof(*stripes_qin) * lio->lis_nr_subios);
} else { } else {
int i; int i;
......
...@@ -314,7 +314,7 @@ static int lov_lock_sub_init(const struct lu_env *env, ...@@ -314,7 +314,7 @@ static int lov_lock_sub_init(const struct lu_env *env,
nr++; nr++;
} }
LASSERT(nr > 0); LASSERT(nr > 0);
OBD_ALLOC_LARGE(lck->lls_sub, nr * sizeof(lck->lls_sub[0])); lck->lls_sub = libcfs_kvzalloc(nr * sizeof(lck->lls_sub[0]), GFP_NOFS);
if (lck->lls_sub == NULL) if (lck->lls_sub == NULL)
return -ENOMEM; return -ENOMEM;
...@@ -441,8 +441,7 @@ static void lov_lock_fini(const struct lu_env *env, ...@@ -441,8 +441,7 @@ static void lov_lock_fini(const struct lu_env *env,
* a reference on its parent. * a reference on its parent.
*/ */
LASSERT(lck->lls_sub[i].sub_lock == NULL); LASSERT(lck->lls_sub[i].sub_lock == NULL);
OBD_FREE_LARGE(lck->lls_sub, kvfree(lck->lls_sub);
lck->lls_nr * sizeof(lck->lls_sub[0]));
} }
OBD_SLAB_FREE_PTR(lck, lov_lock_kmem); OBD_SLAB_FREE_PTR(lck, lov_lock_kmem);
} }
......
...@@ -1737,7 +1737,7 @@ static int lov_fiemap(struct lov_obd *lov, __u32 keylen, void *key, ...@@ -1737,7 +1737,7 @@ static int lov_fiemap(struct lov_obd *lov, __u32 keylen, void *key,
if (fiemap_count_to_size(fm_key->fiemap.fm_extent_count) < buffer_size) if (fiemap_count_to_size(fm_key->fiemap.fm_extent_count) < buffer_size)
buffer_size = fiemap_count_to_size(fm_key->fiemap.fm_extent_count); buffer_size = fiemap_count_to_size(fm_key->fiemap.fm_extent_count);
OBD_ALLOC_LARGE(fm_local, buffer_size); fm_local = libcfs_kvzalloc(buffer_size, GFP_NOFS);
if (fm_local == NULL) { if (fm_local == NULL) {
rc = -ENOMEM; rc = -ENOMEM;
goto out; goto out;
...@@ -1943,7 +1943,7 @@ static int lov_fiemap(struct lov_obd *lov, __u32 keylen, void *key, ...@@ -1943,7 +1943,7 @@ static int lov_fiemap(struct lov_obd *lov, __u32 keylen, void *key,
fiemap->fm_mapped_extents = current_extent; fiemap->fm_mapped_extents = current_extent;
out: out:
OBD_FREE_LARGE(fm_local, buffer_size); kvfree(fm_local);
return rc; return rc;
} }
......
...@@ -218,7 +218,8 @@ static int lov_init_raid0(const struct lu_env *env, ...@@ -218,7 +218,8 @@ static int lov_init_raid0(const struct lu_env *env,
r0->lo_nr = lsm->lsm_stripe_count; r0->lo_nr = lsm->lsm_stripe_count;
LASSERT(r0->lo_nr <= lov_targets_nr(dev)); LASSERT(r0->lo_nr <= lov_targets_nr(dev));
OBD_ALLOC_LARGE(r0->lo_sub, r0->lo_nr * sizeof(r0->lo_sub[0])); r0->lo_sub = libcfs_kvzalloc(r0->lo_nr * sizeof(r0->lo_sub[0]),
GFP_NOFS);
if (r0->lo_sub != NULL) { if (r0->lo_sub != NULL) {
result = 0; result = 0;
subconf->coc_inode = conf->coc_inode; subconf->coc_inode = conf->coc_inode;
...@@ -375,7 +376,7 @@ static void lov_fini_raid0(const struct lu_env *env, struct lov_object *lov, ...@@ -375,7 +376,7 @@ static void lov_fini_raid0(const struct lu_env *env, struct lov_object *lov,
struct lov_layout_raid0 *r0 = &state->raid0; struct lov_layout_raid0 *r0 = &state->raid0;
if (r0->lo_sub != NULL) { if (r0->lo_sub != NULL) {
OBD_FREE_LARGE(r0->lo_sub, r0->lo_nr * sizeof(r0->lo_sub[0])); kvfree(r0->lo_sub);
r0->lo_sub = NULL; r0->lo_sub = NULL;
} }
......
...@@ -192,13 +192,13 @@ int lov_packmd(struct obd_export *exp, struct lov_mds_md **lmmp, ...@@ -192,13 +192,13 @@ int lov_packmd(struct obd_export *exp, struct lov_mds_md **lmmp,
if (*lmmp && !lsm) { if (*lmmp && !lsm) {
stripe_count = le16_to_cpu((*lmmp)->lmm_stripe_count); stripe_count = le16_to_cpu((*lmmp)->lmm_stripe_count);
lmm_size = lov_mds_md_size(stripe_count, lmm_magic); lmm_size = lov_mds_md_size(stripe_count, lmm_magic);
OBD_FREE_LARGE(*lmmp, lmm_size); kvfree(*lmmp);
*lmmp = NULL; *lmmp = NULL;
return 0; return 0;
} }
if (!*lmmp) { if (!*lmmp) {
OBD_ALLOC_LARGE(*lmmp, lmm_size); *lmmp = libcfs_kvzalloc(lmm_size, GFP_NOFS);
if (!*lmmp) if (!*lmmp)
return -ENOMEM; return -ENOMEM;
} }
...@@ -285,7 +285,7 @@ static int lov_verify_lmm(void *lmm, int lmm_bytes, __u16 *stripe_count) ...@@ -285,7 +285,7 @@ static int lov_verify_lmm(void *lmm, int lmm_bytes, __u16 *stripe_count)
CERROR("bad disk LOV MAGIC: 0x%08X; dumping LMM (size=%d):\n", CERROR("bad disk LOV MAGIC: 0x%08X; dumping LMM (size=%d):\n",
le32_to_cpu(*(__u32 *)lmm), lmm_bytes); le32_to_cpu(*(__u32 *)lmm), lmm_bytes);
sz = lmm_bytes * 2 + 1; sz = lmm_bytes * 2 + 1;
OBD_ALLOC_LARGE(buffer, sz); buffer = libcfs_kvzalloc(sz, GFP_NOFS);
if (buffer != NULL) { if (buffer != NULL) {
int i; int i;
...@@ -293,7 +293,7 @@ static int lov_verify_lmm(void *lmm, int lmm_bytes, __u16 *stripe_count) ...@@ -293,7 +293,7 @@ static int lov_verify_lmm(void *lmm, int lmm_bytes, __u16 *stripe_count)
sprintf(buffer+2*i, "%.2X", ((char *)lmm)[i]); sprintf(buffer+2*i, "%.2X", ((char *)lmm)[i]);
buffer[sz - 1] = '\0'; buffer[sz - 1] = '\0';
CERROR("%s\n", buffer); CERROR("%s\n", buffer);
OBD_FREE_LARGE(buffer, sz); kvfree(buffer);
} }
return -EINVAL; return -EINVAL;
} }
......
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