#include "lod_internal.h"
/*
- * Keep a refcount of lod->lod_osts usage to prevent racing with
+ * Keep a refcount of lod->ltd_tgts usage to prevent racing with
* addition/deletion. Any function that expects lov_tgts to remain stationary
* must take a ref.
*
* \param lod - is the lod device from which we want to grab a reference
*/
-void lod_getref(struct lod_device *lod)
+void lod_getref(struct lod_tgt_descs *ltd)
{
- down_read(&lod->lod_rw_sem);
- mutex_lock(&lod->lod_mutex);
- lod->lod_refcount++;
- mutex_unlock(&lod->lod_mutex);
+ down_read(<d->ltd_rw_sem);
+ mutex_lock(<d->ltd_mutex);
+ ltd->ltd_refcount++;
+ mutex_unlock(<d->ltd_mutex);
}
/*
*
* \param lod - is the lod device from which we release a reference
*/
-void lod_putref(struct lod_device *lod)
+void lod_putref(struct lod_device *lod, struct lod_tgt_descs *ltd)
{
- mutex_lock(&lod->lod_mutex);
- lod->lod_refcount--;
- if (lod->lod_refcount == 0 && lod->lod_death_row) {
- struct lod_ost_desc *ost_desc, *tmp;
+ mutex_lock(<d->ltd_mutex);
+ ltd->ltd_refcount--;
+ if (ltd->ltd_refcount == 0 && ltd->ltd_death_row) {
+ struct lod_tgt_desc *tgt_desc, *tmp;
int idx;
CFS_LIST_HEAD(kill);
- CDEBUG(D_CONFIG, "destroying %d lod desc\n",
- lod->lod_death_row);
+ CDEBUG(D_CONFIG, "destroying %d ltd desc\n",
+ ltd->ltd_death_row);
- cfs_foreach_bit(lod->lod_ost_bitmap, idx) {
- ost_desc = OST_TGT(lod, idx);
- LASSERT(ost_desc);
+ cfs_foreach_bit(ltd->ltd_tgt_bitmap, idx) {
+ tgt_desc = LTD_TGT(ltd, idx);
+ LASSERT(tgt_desc);
- if (!ost_desc->ltd_reap)
+ if (!tgt_desc->ltd_reap)
continue;
- cfs_list_add(&ost_desc->ltd_kill, &kill);
-
- lod_ost_pool_remove(&lod->lod_pool_info, idx);
- OST_TGT(lod, idx) = NULL;
- lod->lod_ostnr--;
- cfs_bitmap_clear(lod->lod_ost_bitmap, idx);
- if (ost_desc->ltd_active)
- lod->lod_desc.ld_active_tgt_count--;
- lod->lod_death_row--;
+ cfs_list_add(&tgt_desc->ltd_kill, &kill);
+ LTD_TGT(ltd, idx) = NULL;
+ /*FIXME: only support ost pool for now */
+ if (ltd == &lod->lod_ost_descs) {
+ lod_ost_pool_remove(&lod->lod_pool_info, idx);
+ if (tgt_desc->ltd_active)
+ lod->lod_desc.ld_active_tgt_count--;
+ }
+ ltd->ltd_tgtnr--;
+ cfs_bitmap_clear(ltd->ltd_tgt_bitmap, idx);
+ ltd->ltd_death_row--;
}
- mutex_unlock(&lod->lod_mutex);
- up_read(&lod->lod_rw_sem);
+ mutex_unlock(<d->ltd_mutex);
+ up_read(<d->ltd_rw_sem);
- cfs_list_for_each_entry_safe(ost_desc, tmp, &kill, ltd_kill) {
+ cfs_list_for_each_entry_safe(tgt_desc, tmp, &kill, ltd_kill) {
int rc;
- cfs_list_del(&ost_desc->ltd_kill);
- /* remove from QoS structures */
- rc = qos_del_tgt(lod, ost_desc);
- if (rc)
- CERROR("%s: qos_del_tgt(%s) failed: rc = %d\n",
- lod2obd(lod)->obd_name,
- obd_uuid2str(&ost_desc->ltd_uuid), rc);
- rc = obd_disconnect(ost_desc->ltd_exp);
+ cfs_list_del(&tgt_desc->ltd_kill);
+ if (ltd == &lod->lod_ost_descs) {
+ /* remove from QoS structures */
+ rc = qos_del_tgt(lod, tgt_desc);
+ if (rc)
+ CERROR("%s: qos_del_tgt(%s) failed:"
+ "rc = %d\n",
+ lod2obd(lod)->obd_name,
+ obd_uuid2str(&tgt_desc->ltd_uuid),
+ rc);
+ }
+ rc = obd_disconnect(tgt_desc->ltd_exp);
if (rc)
CERROR("%s: failed to disconnect %s: rc = %d\n",
lod2obd(lod)->obd_name,
- obd_uuid2str(&ost_desc->ltd_uuid), rc);
- OBD_FREE_PTR(ost_desc);
+ obd_uuid2str(&tgt_desc->ltd_uuid), rc);
+ OBD_FREE_PTR(tgt_desc);
}
} else {
- mutex_unlock(&lod->lod_mutex);
- up_read(&lod->lod_rw_sem);
+ mutex_unlock(<d->ltd_mutex);
+ up_read(<d->ltd_rw_sem);
}
}
-static int lod_bitmap_resize(struct lod_device *lod, __u32 newsize)
+static int ltd_bitmap_resize(struct lod_tgt_descs *ltd, __u32 newsize)
{
cfs_bitmap_t *new_bitmap, *old_bitmap = NULL;
int rc = 0;
/* grab write reference on the lod. Relocating the array requires
* exclusive access */
- down_write(&lod->lod_rw_sem);
- if (newsize <= lod->lod_osts_size)
+ down_write(<d->ltd_rw_sem);
+ if (newsize <= ltd->ltd_tgts_size)
/* someone else has already resize the array */
GOTO(out, rc = 0);
if (!new_bitmap)
GOTO(out, rc = -ENOMEM);
- if (lod->lod_osts_size > 0) {
+ if (ltd->ltd_tgts_size > 0) {
/* the bitmap already exists, we need
* to copy data from old one */
- cfs_bitmap_copy(new_bitmap, lod->lod_ost_bitmap);
- old_bitmap = lod->lod_ost_bitmap;
+ cfs_bitmap_copy(new_bitmap, ltd->ltd_tgt_bitmap);
+ old_bitmap = ltd->ltd_tgt_bitmap;
}
- lod->lod_osts_size = newsize;
- lod->lod_ost_bitmap = new_bitmap;
+ ltd->ltd_tgts_size = newsize;
+ ltd->ltd_tgt_bitmap = new_bitmap;
if (old_bitmap)
CFS_FREE_BITMAP(old_bitmap);
- CDEBUG(D_CONFIG, "ost size: %d\n", lod->lod_osts_size);
+ CDEBUG(D_CONFIG, "tgt size: %d\n", ltd->ltd_tgts_size);
EXIT;
out:
- up_write(&lod->lod_rw_sem);
+ up_write(<d->ltd_rw_sem);
return rc;
}
* \param osp - is the name of OSP device name about to be added
* \param index - is the OSP index
* \param gen - is the generation number
+ * \param tgt_index - is the group of the OSP.
+ * \param type - is the type of device (mdc or osc)
*/
int lod_add_device(const struct lu_env *env, struct lod_device *lod,
- char *osp, unsigned index, unsigned gen, int active)
+ char *osp, unsigned index, unsigned gen, int tgt_index,
+ char *type, int active)
{
struct obd_connect_data *data = NULL;
struct obd_export *exp = NULL;
struct lu_device *ldev;
struct dt_device *d;
int rc;
- struct lod_ost_desc *ost_desc;
- struct obd_uuid obd_uuid;
-
+ struct lod_tgt_desc *tgt_desc;
+ struct lod_tgt_descs *ltd;
+ struct obd_uuid obd_uuid;
ENTRY;
CDEBUG(D_CONFIG, "osp:%s idx:%d gen:%d\n", osp, index, gen);
data->ocd_version = LUSTRE_VERSION_CODE;
data->ocd_index = index;
+ if (strcmp(LUSTRE_OSC_NAME, type) == 0) {
+ data->ocd_connect_flags |= OBD_CONNECT_AT |
+ OBD_CONNECT_FULL20 |
+ OBD_CONNECT_INDEX |
+#ifdef HAVE_LRU_RESIZE_SUPPORT
+ OBD_CONNECT_LRU_RESIZE |
+#endif
+ OBD_CONNECT_MDS |
+ OBD_CONNECT_OSS_CAPA |
+ OBD_CONNECT_REQPORTAL |
+ OBD_CONNECT_SKIP_ORPHAN |
+ OBD_CONNECT_FID |
+ OBD_CONNECT_LVB_TYPE |
+ OBD_CONNECT_VERSION |
+ OBD_CONNECT_PINGLESS;
+
+ data->ocd_group = tgt_index;
+ ltd = &lod->lod_ost_descs;
+ } else {
+ struct obd_import *imp = obd->u.cli.cl_import;
+
+ data->ocd_ibits_known = MDS_INODELOCK_UPDATE;
+ data->ocd_connect_flags |= OBD_CONNECT_ACL |
+ OBD_CONNECT_MDS_CAPA |
+ OBD_CONNECT_OSS_CAPA |
+ OBD_CONNECT_IBITS |
+ OBD_CONNECT_MDS_MDS |
+ OBD_CONNECT_FID |
+ OBD_CONNECT_AT |
+ OBD_CONNECT_FULL20;
+ /* XXX set MDS-MDS flags, remove this when running this
+ * on client*/
+ data->ocd_connect_flags |= OBD_CONNECT_MDS_MDS;
+ spin_lock(&imp->imp_lock);
+ imp->imp_server_timeout = 1;
+ spin_unlock(&imp->imp_lock);
+ imp->imp_client->cli_request_portal = MDS_MDS_PORTAL;
+ CDEBUG(D_OTHER, "%s: Set 'mds' portal and timeout\n",
+ obd->obd_name);
+ ltd = &lod->lod_mdt_descs;
+ }
+
rc = obd_connect(env, &exp, obd, &obd->obd_uuid, data, NULL);
OBD_FREE_PTR(data);
if (rc) {
}
LASSERT(obd->obd_lu_dev);
- LASSERT(obd->obd_lu_dev->ld_site = lod->lod_dt_dev.dd_lu_dev.ld_site);
+ LASSERT(obd->obd_lu_dev->ld_site == lod->lod_dt_dev.dd_lu_dev.ld_site);
ldev = obd->obd_lu_dev;
d = lu2dt_dev(ldev);
/* Allocate ost descriptor and fill it */
- OBD_ALLOC_PTR(ost_desc);
- if (!ost_desc)
+ OBD_ALLOC_PTR(tgt_desc);
+ if (!tgt_desc)
GOTO(out_conn, rc = -ENOMEM);
- ost_desc->ltd_ost = d;
- ost_desc->ltd_exp = exp;
- ost_desc->ltd_uuid = obd->u.cli.cl_target_uuid;
- ost_desc->ltd_gen = gen;
- ost_desc->ltd_index = index;
- ost_desc->ltd_active = active;
+ tgt_desc->ltd_tgt = d;
+ tgt_desc->ltd_exp = exp;
+ tgt_desc->ltd_uuid = obd->u.cli.cl_target_uuid;
+ tgt_desc->ltd_gen = gen;
+ tgt_desc->ltd_index = index;
+ tgt_desc->ltd_active = active;
- lod_getref(lod);
- if (index >= lod->lod_osts_size) {
+ lod_getref(ltd);
+ if (index >= ltd->ltd_tgts_size) {
/* we have to increase the size of the lod_osts array */
__u32 newsize;
- newsize = max(lod->lod_osts_size, (__u32)2);
+ newsize = max(ltd->ltd_tgts_size, (__u32)2);
while (newsize < index + 1)
newsize = newsize << 1;
/* lod_bitmap_resize() needs lod_rw_sem
* which we hold with th reference */
- lod_putref(lod);
+ lod_putref(lod, ltd);
- rc = lod_bitmap_resize(lod, newsize);
+ rc = ltd_bitmap_resize(ltd, newsize);
if (rc)
GOTO(out_desc, rc);
- lod_getref(lod);
+ lod_getref(ltd);
}
- mutex_lock(&lod->lod_mutex);
- if (cfs_bitmap_check(lod->lod_ost_bitmap, index)) {
+ mutex_lock(<d->ltd_mutex);
+ if (cfs_bitmap_check(ltd->ltd_tgt_bitmap, index)) {
CERROR("%s: device %d is registered already\n", obd->obd_name,
index);
GOTO(out_mutex, rc = -EEXIST);
}
- if (lod->lod_ost_idx[index / OST_PTRS_PER_BLOCK] == NULL) {
- OBD_ALLOC_PTR(lod->lod_ost_idx[index / OST_PTRS_PER_BLOCK]);
- if (lod->lod_ost_idx[index / OST_PTRS_PER_BLOCK] == NULL) {
+ if (ltd->ltd_tgt_idx[index / TGT_PTRS_PER_BLOCK] == NULL) {
+ OBD_ALLOC_PTR(ltd->ltd_tgt_idx[index / TGT_PTRS_PER_BLOCK]);
+ if (ltd->ltd_tgt_idx[index / TGT_PTRS_PER_BLOCK] == NULL) {
CERROR("can't allocate index to add %s\n",
obd->obd_name);
GOTO(out_mutex, rc = -ENOMEM);
}
}
- rc = lod_ost_pool_add(&lod->lod_pool_info, index, lod->lod_osts_size);
- if (rc) {
- CERROR("%s: can't set up pool, failed with %d\n",
- obd->obd_name, rc);
- GOTO(out_mutex, rc);
- }
+ if (!strcmp(LUSTRE_OSC_NAME, type)) {
+ /* pool and qos are not supported for MDS stack yet */
+ rc = lod_ost_pool_add(&lod->lod_pool_info, index,
+ lod->lod_osts_size);
+ if (rc) {
+ CERROR("%s: can't set up pool, failed with %d\n",
+ obd->obd_name, rc);
+ GOTO(out_mutex, rc);
+ }
- rc = qos_add_tgt(lod, ost_desc);
- if (rc) {
- CERROR("%s: qos_add_tgt(%s) failed: rc = %d\n", obd->obd_name,
- obd_uuid2str(&ost_desc->ltd_uuid), rc);
- GOTO(out_pool, rc);
- }
-
- /* The new OST is now a full citizen */
- if (index >= lod->lod_desc.ld_tgt_count)
- lod->lod_desc.ld_tgt_count = index + 1;
- if (active)
- lod->lod_desc.ld_active_tgt_count++;
- OST_TGT(lod, index) = ost_desc;
- cfs_bitmap_set(lod->lod_ost_bitmap, index);
- lod->lod_ostnr++;
- mutex_unlock(&lod->lod_mutex);
- lod_putref(lod);
+ rc = qos_add_tgt(lod, tgt_desc);
+ if (rc) {
+ CERROR("%s: qos_add_tgt failed with %d\n",
+ obd->obd_name, rc);
+ GOTO(out_pool, rc);
+ }
+ /* The new OST is now a full citizen */
+ if (index >= lod->lod_desc.ld_tgt_count)
+ lod->lod_desc.ld_tgt_count = index + 1;
+ if (active)
+ lod->lod_desc.ld_active_tgt_count++;
+ }
+
+ LTD_TGT(ltd, index) = tgt_desc;
+ cfs_bitmap_set(ltd->ltd_tgt_bitmap, index);
+ ltd->ltd_tgtnr++;
+ mutex_unlock(<d->ltd_mutex);
+ lod_putref(lod, ltd);
if (lod->lod_recovery_completed)
ldev->ld_ops->ldo_recovery_complete(env, ldev);
out_pool:
lod_ost_pool_remove(&lod->lod_pool_info, index);
out_mutex:
- mutex_unlock(&lod->lod_mutex);
- lod_putref(lod);
+ mutex_unlock(<d->ltd_mutex);
+ lod_putref(lod, ltd);
out_desc:
- OBD_FREE_PTR(ost_desc);
+ OBD_FREE_PTR(tgt_desc);
out_conn:
obd_disconnect(exp);
out_free:
/*
* helper function to schedule OST removal from the device table
*/
-static void __lod_del_device(struct lod_device *lod, unsigned idx)
+static void __lod_del_device(struct lod_tgt_descs *ltd,
+ unsigned idx)
{
- LASSERT(OST_TGT(lod,idx));
- if (OST_TGT(lod,idx)->ltd_reap == 0) {
- OST_TGT(lod,idx)->ltd_reap = 1;
- lod->lod_death_row++;
+ LASSERT(LTD_TGT(ltd, idx));
+ if (LTD_TGT(ltd, idx)->ltd_reap == 0) {
+ LTD_TGT(ltd, idx)->ltd_reap = 1;
+ ltd->ltd_death_row++;
}
}
+int lod_fini_tgt(struct lod_device *lod, struct lod_tgt_descs *ltd)
+{
+ int idx;
+
+ if (ltd->ltd_tgts_size <= 0)
+ return 0;
+ lod_getref(ltd);
+ mutex_lock(<d->ltd_mutex);
+ cfs_foreach_bit(ltd->ltd_tgt_bitmap, idx)
+ __lod_del_device(ltd, idx);
+ mutex_unlock(<d->ltd_mutex);
+ lod_putref(lod, ltd);
+ CFS_FREE_BITMAP(ltd->ltd_tgt_bitmap);
+ for (idx = 0; idx < TGT_PTRS; idx++) {
+ if (ltd->ltd_tgt_idx[idx])
+ OBD_FREE_PTR(ltd->ltd_tgt_idx[idx]);
+ }
+ ltd->ltd_tgts_size = 0;
+ return 0;
+}
+
/*
* Add support for administratively disabled OST (through the MGS).
* Schedule a target for deletion. Disconnection and real removal from the
* \param gen - is the generation number, not used currently
*/
int lod_del_device(const struct lu_env *env, struct lod_device *lod,
- char *osp, unsigned idx, unsigned gen)
+ struct lod_tgt_descs *ltd, char *osp, unsigned idx,
+ unsigned gen)
{
struct obd_device *obd;
int rc = 0;
CDEBUG(D_CONFIG, "osp:%s idx:%d gen:%d\n", osp, idx, gen);
- obd = class_name2obd(osp);
+ obd_str2uuid(&uuid, osp);
+
+ obd = class_find_client_obd(&uuid, LUSTRE_OSP_NAME,
+ &lod->lod_dt_dev.dd_lu_dev.ld_obd->obd_uuid);
if (obd == NULL) {
CERROR("can't find %s device\n", osp);
RETURN(-EINVAL);
obd_str2uuid(&uuid, osp);
- lod_getref(lod);
- mutex_lock(&lod->lod_mutex);
+ lod_getref(ltd);
+ mutex_lock(<d->ltd_mutex);
/* check that the index is allocated in the bitmap */
- if (!cfs_bitmap_check(lod->lod_ost_bitmap, idx) || !OST_TGT(lod,idx)) {
+ if (!cfs_bitmap_check(ltd->ltd_tgt_bitmap, idx) ||
+ !LTD_TGT(ltd, idx)) {
CERROR("%s: device %d is not set up\n", obd->obd_name, idx);
GOTO(out, rc = -EINVAL);
}
/* check that the UUID matches */
- if (!obd_uuid_equals(&uuid, &OST_TGT(lod,idx)->ltd_uuid)) {
+ if (!obd_uuid_equals(&uuid, <D_TGT(ltd, idx)->ltd_uuid)) {
CERROR("%s: LOD target UUID %s at index %d does not match %s\n",
- obd->obd_name, obd_uuid2str(&OST_TGT(lod,idx)->ltd_uuid),
+ obd->obd_name, obd_uuid2str(<D_TGT(ltd,idx)->ltd_uuid),
idx, osp);
GOTO(out, rc = -EINVAL);
}
- __lod_del_device(lod, idx);
+ __lod_del_device(ltd, idx);
EXIT;
out:
- mutex_unlock(&lod->lod_mutex);
- lod_putref(lod);
+ mutex_unlock(<d->ltd_mutex);
+ lod_putref(lod, ltd);
return(rc);
}
struct lov_ost_data_v1 *objs;
__u32 magic;
int i, rc, lmm_size;
+ int cplen = 0;
ENTRY;
LASSERT(lo);
lmm->lmm_magic = cpu_to_le32(magic);
lmm->lmm_pattern = cpu_to_le32(LOV_PATTERN_RAID0);
- lmm->lmm_object_id = cpu_to_le64(fid_ver_oid(fid));
- lmm->lmm_object_seq = cpu_to_le64(fid_seq(fid));
+ fid_to_lmm_oi(fid, &lmm->lmm_oi);
+ lmm_oi_cpu_to_le(&lmm->lmm_oi, &lmm->lmm_oi);
lmm->lmm_stripe_size = cpu_to_le32(lo->ldo_stripe_size);
lmm->lmm_stripe_count = cpu_to_le16(lo->ldo_stripenr);
lmm->lmm_layout_gen = 0;
objs = &lmm->lmm_objects[0];
} else {
struct lov_mds_md_v3 *v3 = (struct lov_mds_md_v3 *) lmm;
- strncpy(v3->lmm_pool_name, lo->ldo_pool, LOV_MAXPOOLNAME);
+ cplen = strlcpy(v3->lmm_pool_name, lo->ldo_pool,
+ sizeof(v3->lmm_pool_name));
+ if (cplen >= sizeof(v3->lmm_pool_name))
+ RETURN(-E2BIG);
objs = &v3->lmm_objects[0];
}
for (i = 0; i < lo->ldo_stripenr; i++) {
- const struct lu_fid *fid;
+ const struct lu_fid *fid;
+ struct lod_device *lod;
+ __u32 index;
+ lod = lu2lod_dev(lo->ldo_obj.do_lu.lo_dev);
LASSERT(lo->ldo_stripe[i]);
fid = lu_object_fid(&lo->ldo_stripe[i]->do_lu);
- rc = fid_ostid_pack(fid, &info->lti_ostid);
+ rc = fid_to_ostid(fid, &info->lti_ostid);
LASSERT(rc == 0);
- LASSERT(info->lti_ostid.oi_seq == FID_SEQ_OST_MDT0);
- objs[i].l_object_id = cpu_to_le64(info->lti_ostid.oi_id);
- objs[i].l_object_seq = cpu_to_le64(info->lti_ostid.oi_seq);
+ ostid_cpu_to_le(&info->lti_ostid, &objs[i].l_ost_oi);
objs[i].l_ost_gen = cpu_to_le32(0);
- objs[i].l_ost_idx = cpu_to_le32(fid_idif_ost_idx(fid));
+ rc = lod_fld_lookup(env, lod, fid, &index, LU_SEQ_RANGE_OST);
+ if (rc < 0) {
+ CERROR("%s: Can not locate "DFID": rc = %d\n",
+ lod2obd(lod)->obd_name, PFID(fid), rc);
+ RETURN(rc);
+ }
+ objs[i].l_ost_idx = cpu_to_le32(index);
}
info->lti_buf.lb_buf = lmm;
struct dt_object *next = dt_object_child(dt);
struct lov_user_md_v3 *v3;
int rc;
+ int cplen = 0;
ENTRY;
LASSERT(S_ISDIR(dt->do_lu.lo_header->loh_attr));
v3->lmm_magic = cpu_to_le32(LOV_MAGIC_V3);
v3->lmm_pattern = cpu_to_le32(LOV_PATTERN_RAID0);
- v3->lmm_object_id = 0;
- v3->lmm_object_seq = 0;
v3->lmm_stripe_size = cpu_to_le32(lo->ldo_def_stripe_size);
v3->lmm_stripe_count = cpu_to_le16(lo->ldo_def_stripenr);
v3->lmm_stripe_offset = cpu_to_le16(lo->ldo_def_stripe_offset);
- if (lo->ldo_pool)
- strncpy(v3->lmm_pool_name, lo->ldo_pool, LOV_MAXPOOLNAME);
+ if (lo->ldo_pool) {
+ cplen = strlcpy(v3->lmm_pool_name, lo->ldo_pool,
+ sizeof(v3->lmm_pool_name));
+ if (cplen >= sizeof(v3->lmm_pool_name)) {
+ OBD_FREE_PTR(v3);
+ RETURN(-E2BIG);
+ }
+ }
info->lti_buf.lb_buf = v3;
info->lti_buf.lb_len = sizeof(*v3);
struct lod_device *md = lu2lod_dev(lo->ldo_obj.do_lu.lo_dev);
struct lu_object *o, *n;
struct lu_device *nd;
- int i, idx, rc = 0;
+ struct dt_object **stripe;
+ int stripe_len;
+ int i, idx, rc = 0;
ENTRY;
LASSERT(lo);
LASSERT(lo->ldo_stripenr > 0);
LASSERT(lo->ldo_stripe_size > 0);
- i = sizeof(struct dt_object *) * lo->ldo_stripenr;
- OBD_ALLOC(lo->ldo_stripe, i);
- if (lo->ldo_stripe == NULL)
- GOTO(out, rc = -ENOMEM);
- lo->ldo_stripes_allocated = lo->ldo_stripenr;
+ stripe_len = lo->ldo_stripenr;
+ OBD_ALLOC(stripe, sizeof(stripe[0]) * stripe_len);
+ if (stripe == NULL)
+ RETURN(-ENOMEM);
for (i = 0; i < lo->ldo_stripenr; i++) {
-
- info->lti_ostid.oi_id = le64_to_cpu(objs[i].l_object_id);
- /* XXX: support for DNE? */
- info->lti_ostid.oi_seq = le64_to_cpu(objs[i].l_object_seq);
+ ostid_le_to_cpu(&objs[i].l_ost_oi, &info->lti_ostid);
idx = le64_to_cpu(objs[i].l_ost_idx);
- fid_ostid_unpack(&info->lti_fid, &info->lti_ostid, idx);
-
+ rc = ostid_to_fid(&info->lti_fid, &info->lti_ostid, idx);
+ if (rc != 0)
+ GOTO(out, rc);
+ LASSERTF(fid_is_sane(&info->lti_fid), ""DFID" insane!\n",
+ PFID(&info->lti_fid));
/*
* XXX: assertion is left for testing, to make
* sure we never process requests till configuration
* is completed. to be changed to -EINVAL
*/
- lod_getref(md);
+ lod_getref(&md->lod_ost_descs);
LASSERT(cfs_bitmap_check(md->lod_ost_bitmap, idx));
LASSERT(OST_TGT(md,idx));
LASSERTF(OST_TGT(md,idx)->ltd_ost, "idx %d\n", idx);
nd = &OST_TGT(md,idx)->ltd_ost->dd_lu_dev;
- lod_putref(md);
+ lod_putref(md, &md->lod_ost_descs);
+ /* In the function below, .hs_keycmp resolves to
+ * u_obj_hop_keycmp() */
+ /* coverity[overrun-buffer-val] */
o = lu_object_find_at(env, nd, &info->lti_fid, NULL);
if (IS_ERR(o))
GOTO(out, rc = PTR_ERR(o));
n = lu_object_locate(o->lo_header, nd->ld_type);
LASSERT(n);
- lo->ldo_stripe[i] = container_of(n, struct dt_object, do_lu);
+ stripe[i] = container_of(n, struct dt_object, do_lu);
}
out:
+ if (rc != 0) {
+ for (i = 0; i < stripe_len; i++)
+ if (stripe[i] != NULL)
+ lu_object_put(env, &stripe[i]->do_lu);
+
+ OBD_FREE(stripe, sizeof(stripe[0]) * stripe_len);
+ } else {
+ lo->ldo_stripe = stripe;
+ lo->ldo_stripes_allocated = stripe_len;
+ }
+
RETURN(rc);
}
v3 = buf->lb_buf;
if (v3)
+ /* In the function below, .hs_keycmp resolves to
+ * pool_hashkey_keycmp() */
+ /* coverity[overrun-buffer-val] */
pool = lod_find_pool(d, v3->lmm_pool_name);
if (pool != NULL) {
void lod_fix_desc_stripe_size(__u64 *val)
{
- if (*val < PTLRPC_MAX_BRW_SIZE) {
+ if (*val < LOV_DEFAULT_STRIPE_SIZE) {
LCONSOLE_WARN("Increasing default stripe size to min %u\n",
- PTLRPC_MAX_BRW_SIZE);
- *val = PTLRPC_MAX_BRW_SIZE;
+ LOV_DEFAULT_STRIPE_SIZE);
+ *val = LOV_DEFAULT_STRIPE_SIZE;
} else if (*val & (LOV_MIN_STRIPE_SIZE - 1)) {
*val &= ~(LOV_MIN_STRIPE_SIZE - 1);
LCONSOLE_WARN("Changing default stripe size to "LPU64" (a "
if (rc)
GOTO(out_pool_info, rc);
- /* the OST array and bitmap are allocated/grown dynamically as OSTs are
- * added to the LOD, see lod_add_device() */
- lod->lod_ost_bitmap = NULL;
- lod->lod_osts_size = 0;
- lod->lod_ostnr = 0;
-
- lod->lod_death_row = 0;
- lod->lod_refcount = 0;
-
RETURN(0);
out_pool_info:
pool = cfs_list_entry(pos, struct pool_desc, pool_list);
/* free pool structs */
CDEBUG(D_INFO, "delete pool %p\n", pool);
+ /* In the function below, .hs_keycmp resolves to
+ * pool_hashkey_keycmp() */
+ /* coverity[overrun-buffer-val] */
lod_pool_del(obd, pool->pool_name);
}
- if (lod->lod_osts_size > 0) {
- int idx;
- lod_getref(lod);
- mutex_lock(&lod->lod_mutex);
- cfs_foreach_bit(lod->lod_ost_bitmap, idx)
- __lod_del_device(lod, idx);
- mutex_unlock(&lod->lod_mutex);
- lod_putref(lod);
- CFS_FREE_BITMAP(lod->lod_ost_bitmap);
- for (idx = 0; idx < OST_PTRS; idx++) {
- if (lod->lod_ost_idx[idx])
- OBD_FREE_PTR(lod->lod_ost_idx[idx]);
- }
- lod->lod_osts_size = 0;
- }
-
cfs_hash_putref(lod->lod_pools_hash_body);
lod_ost_pool_free(&(lod->lod_qos.lq_rr.lqr_pool));
lod_ost_pool_free(&lod->lod_pool_info);