Whamcloud - gitweb
LU-13974 llog: check stale osp object
[fs/lustre-release.git] / lustre / obdclass / llog_osd.c
index f5b54d9..92d7186 100644 (file)
  *
  * You should have received a copy of the GNU General Public License
  * version 2 along with this program; If not, see
- * http://www.sun.com/software/products/lustre/docs/GPLv2.pdf
- *
- * Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara,
- * CA 95054 USA or visit www.sun.com if you need additional information or
- * have any questions.
+ * http://www.gnu.org/licenses/gpl-2.0.html
  *
  * GPL HEADER END
  */
 /*
- * Copyright (c) 2009, 2010, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2003, 2010, Oracle and/or its affiliates. All rights reserved.
  * Use is subject to license terms.
  *
- * Copyright (c) 2012, Intel Corporation.
+ * Copyright (c) 2012, 2017, Intel Corporation.
  */
 /*
  * This file is part of Lustre, http://www.lustre.org/
  * Lustre is a trademark of Sun Microsystems, Inc.
+ */
+/*
+ * lustre/obdclass/llog_osd.c
+ *
+ * Low level llog routines on top of OSD API
  *
- * lustre/obdclass/llog_osd.c - low level llog routines on top of OSD API
+ * This file provides set of methods for llog operations on top of
+ * dt_device. It contains all supported llog_operations interfaces and
+ * supplimental functions.
  *
  * Author: Alexey Zhuravlev <alexey.zhuravlev@intel.com>
  * Author: Mikhail Pershin <mike.pershin@intel.com>
 
 #define DEBUG_SUBSYSTEM S_LOG
 
-#ifndef EXPORT_SYMTAB
-#define EXPORT_SYMTAB
-#endif
+#include <linux/delay.h>
 
+#include <dt_object.h>
+#include <llog_swab.h>
+#include <lustre_fid.h>
 #include <obd.h>
 #include <obd_class.h>
-#include <lustre_fid.h>
-#include <dt_object.h>
 
 #include "llog_internal.h"
 #include "local_storage.h"
 
-/*
- * - multi-chunks or big-declaration approach
- * - use unique sequence instead of llog sb tracking unique ids
- * - re-use existing environment
- * - named llog support (can be used for testing only at the present)
- * - llog_origin_connect() work with OSD API
+/**
+ * Implementation of the llog_operations::lop_declare_create
+ *
+ * This function is a wrapper over local_storage API function
+ * local_object_declare_create().
+ *
+ * \param[in] env      execution environment
+ * \param[in] los      local_storage for bottom storage device
+ * \param[in] o                dt_object to create
+ * \param[in] th       current transaction handle
+ *
+ * \retval             0 on successful declaration of the new object
+ * \retval             negative error if declaration was failed
  */
-
 static int llog_osd_declare_new_object(const struct lu_env *env,
                                       struct local_oid_storage *los,
                                       struct dt_object *o,
@@ -76,6 +84,20 @@ static int llog_osd_declare_new_object(const struct lu_env *env,
                                           &lgi->lgi_dof, th);
 }
 
+/**
+ * Implementation of the llog_operations::lop_create
+ *
+ * This function is a wrapper over local_storage API function
+ * local_object_create().
+ *
+ * \param[in] env      execution environment
+ * \param[in] los      local_storage for bottom storage device
+ * \param[in] o                dt_object to create
+ * \param[in] th       current transaction handle
+ *
+ * \retval             0 on successful creation of the new object
+ * \retval             negative error if creation was failed
+ */
 static int llog_osd_create_new_object(const struct lu_env *env,
                                      struct local_oid_storage *los,
                                      struct dt_object *o,
@@ -91,10 +113,56 @@ static int llog_osd_create_new_object(const struct lu_env *env,
                                   &lgi->lgi_dof, th);
 }
 
+/**
+ * Implementation of the llog_operations::lop_exist
+ *
+ * This function checks that llog exists on storage.
+ *
+ * \param[in] handle   llog handle of the current llog
+ *
+ * \retval             true if llog object exists and is not just destroyed
+ * \retval             false if llog doesn't exist or just destroyed
+ */
+static int llog_osd_exist(struct llog_handle *handle)
+{
+       LASSERT(handle->lgh_obj);
+       return dt_object_exists(handle->lgh_obj) && !handle->lgh_destroyed;
+}
+
+static void *rec_tail(struct llog_rec_hdr *rec)
+{
+       return (void *)((char *)rec + rec->lrh_len -
+                       sizeof(struct llog_rec_tail));
+}
+
+/**
+ * Write a padding record to the llog
+ *
+ * This function writes a padding record to the end of llog. That may
+ * be needed if llog contains records of variable size, e.g. config logs
+ * or changelogs.
+ * The padding record just aligns llog to the llog chunk_size boundary if
+ * the current record doesn't fit in the remaining space.
+ *
+ * It allocates full length to avoid two separate writes for header and tail.
+ * Such 2-steps scheme needs extra protection and complex error handling.
+ *
+ * \param[in]     env  execution environment
+ * \param[in]     o    dt_object to create
+ * \param[in,out] off  pointer to the padding start offset
+ * \param[in]     len  padding length
+ * \param[in]     index        index of the padding record in a llog
+ * \param[in]     th   current transaction handle
+ *
+ * \retval             0 on successful padding write
+ * \retval             negative error if write failed
+ */
 static int llog_osd_pad(const struct lu_env *env, struct dt_object *o,
                        loff_t *off, int len, int index, struct thandle *th)
 {
        struct llog_thread_info *lgi = llog_info(env);
+       struct llog_rec_hdr     *rec;
+       struct llog_rec_tail    *tail;
        int                      rc;
 
        ENTRY;
@@ -103,114 +171,58 @@ static int llog_osd_pad(const struct lu_env *env, struct dt_object *o,
        LASSERT(off);
        LASSERT(len >= LLOG_MIN_REC_SIZE && (len & 0x7) == 0);
 
-       lgi->lgi_tail.lrt_len = lgi->lgi_lrh.lrh_len = len;
-       lgi->lgi_tail.lrt_index = lgi->lgi_lrh.lrh_index = index;
-       lgi->lgi_lrh.lrh_type = LLOG_PAD_MAGIC;
-
-       lgi->lgi_buf.lb_buf = &lgi->lgi_lrh;
-       lgi->lgi_buf.lb_len = sizeof(lgi->lgi_lrh);
-       dt_write_lock(env, o, 0);
-       rc = dt_record_write(env, o, &lgi->lgi_buf, off, th);
-       if (rc) {
-               CERROR("%s: error writing padding record: rc = %d\n",
-                      o->do_lu.lo_dev->ld_obd->obd_name, rc);
-               GOTO(out, rc);
-       }
+       OBD_ALLOC(rec, len);
+       if (rec == NULL)
+               RETURN(-ENOMEM);
 
-       lgi->lgi_buf.lb_buf = &lgi->lgi_tail;
-       lgi->lgi_buf.lb_len = sizeof(lgi->lgi_tail);
-       *off += len - sizeof(lgi->lgi_lrh) - sizeof(lgi->lgi_tail);
-       rc = dt_record_write(env, o, &lgi->lgi_buf, off, th);
-       if (rc)
-               CERROR("%s: error writing padding record: rc = %d\n",
-                      o->do_lu.lo_dev->ld_obd->obd_name, rc);
-out:
-       dt_write_unlock(env, o);
-       RETURN(rc);
-}
-
-static int llog_osd_write_blob(const struct lu_env *env, struct dt_object *o,
-                              struct llog_rec_hdr *rec, void *buf,
-                              loff_t *off, struct thandle *th)
-{
-       struct llog_thread_info *lgi = llog_info(env);
-       int                      buflen = rec->lrh_len;
-       int                      rc;
-
-       ENTRY;
-
-       LASSERT(env);
-       LASSERT(o);
-
-       if (buflen == 0)
-               CWARN("0-length record\n");
+       rec->lrh_len = len;
+       rec->lrh_index = index;
+       rec->lrh_type = LLOG_PAD_MAGIC;
 
-       CDEBUG(D_OTHER, "write blob with type %x, buf %p/%u at off %llu\n",
-              rec->lrh_type, buf, buflen, *off);
+       tail = rec_tail(rec);
+       tail->lrt_len = len;
+       tail->lrt_index = index;
 
-       if (!buf) {
-               lgi->lgi_buf.lb_len = buflen;
-               lgi->lgi_buf.lb_buf = rec;
-               rc = dt_record_write(env, o, &lgi->lgi_buf, off, th);
-               if (rc)
-                       CERROR("%s: error writing log record: rc = %d\n",
-                              o->do_lu.lo_dev->ld_obd->obd_name, rc);
-               RETURN(rc);
-       }
-
-       /* the buf case */
-       /* protect the following 3 writes from concurrent read */
-       dt_write_lock(env, o, 0);
-       rec->lrh_len = sizeof(*rec) + buflen + sizeof(lgi->lgi_tail);
-       lgi->lgi_buf.lb_len = sizeof(*rec);
        lgi->lgi_buf.lb_buf = rec;
-       rc = dt_record_write(env, o, &lgi->lgi_buf, off, th);
-       if (rc) {
-               CERROR("%s: error writing log hdr: rc = %d\n",
-                      o->do_lu.lo_dev->ld_obd->obd_name, rc);
-               GOTO(out, rc);
-       }
-
-       lgi->lgi_buf.lb_len = buflen;
-       lgi->lgi_buf.lb_buf = buf;
-       rc = dt_record_write(env, o, &lgi->lgi_buf, off, th);
-       if (rc) {
-               CERROR("%s: error writing log buffer: rc = %d\n",
-                      o->do_lu.lo_dev->ld_obd->obd_name,  rc);
-               GOTO(out, rc);
-       }
-
-       lgi->lgi_tail.lrt_len = rec->lrh_len;
-       lgi->lgi_tail.lrt_index = rec->lrh_index;
-       lgi->lgi_buf.lb_len = sizeof(lgi->lgi_tail);
-       lgi->lgi_buf.lb_buf = &lgi->lgi_tail;
+       lgi->lgi_buf.lb_len = len;
        rc = dt_record_write(env, o, &lgi->lgi_buf, off, th);
        if (rc)
-               CERROR("%s: error writing log tail: rc = %d\n",
+               CERROR("%s: error writing padding record: rc = %d\n",
                       o->do_lu.lo_dev->ld_obd->obd_name, rc);
-out:
-       dt_write_unlock(env, o);
+
+       OBD_FREE(rec, len);
        RETURN(rc);
 }
 
+/**
+ * Implementation of the llog_operations::lop_read_header
+ *
+ * This function reads the current llog header from the bottom storage
+ * device.
+ *
+ * \param[in] env      execution environment
+ * \param[in] handle   llog handle of the current llog
+ *
+ * \retval             0 on successful header read
+ * \retval             negative error if read failed
+ */
 static int llog_osd_read_header(const struct lu_env *env,
                                struct llog_handle *handle)
 {
        struct llog_rec_hdr     *llh_hdr;
        struct dt_object        *o;
        struct llog_thread_info *lgi;
+       enum llog_flag           flags;
        int                      rc;
 
        ENTRY;
 
-       LASSERT(sizeof(*handle->lgh_hdr) == LLOG_CHUNK_SIZE);
-
        o = handle->lgh_obj;
        LASSERT(o);
 
        lgi = llog_info(env);
 
-       rc = dt_attr_get(env, o, &lgi->lgi_attr, NULL);
+       rc = dt_attr_get(env, o, &lgi->lgi_attr);
        if (rc)
                RETURN(rc);
 
@@ -221,19 +233,25 @@ static int llog_osd_read_header(const struct lu_env *env,
                RETURN(LLOG_EEMPTY);
        }
 
+       flags = handle->lgh_hdr->llh_flags;
+
        lgi->lgi_off = 0;
        lgi->lgi_buf.lb_buf = handle->lgh_hdr;
-       lgi->lgi_buf.lb_len = LLOG_CHUNK_SIZE;
-
-       rc = dt_record_read(env, o, &lgi->lgi_buf, &lgi->lgi_off);
-       if (rc) {
-               CERROR("%s: error reading log header from "DFID": rc = %d\n",
+       lgi->lgi_buf.lb_len = handle->lgh_hdr_size;
+       rc = dt_read(env, o, &lgi->lgi_buf, &lgi->lgi_off);
+       llh_hdr = &handle->lgh_hdr->llh_hdr;
+       if (rc < sizeof(*llh_hdr) || rc < llh_hdr->lrh_len) {
+               CERROR("%s: error reading "DFID" log header size %d: rc = %d\n",
                       o->do_lu.lo_dev->ld_obd->obd_name,
-                      PFID(lu_object_fid(&o->do_lu)), rc);
+                      PFID(lu_object_fid(&o->do_lu)), rc < 0 ? 0 : rc,
+                      -EFAULT);
+
+               if (rc >= 0)
+                       rc = -EFAULT;
+
                RETURN(rc);
        }
 
-       llh_hdr = &handle->lgh_hdr->llh_hdr;
        if (LLOG_REC_HDR_NEEDS_SWABBING(llh_hdr))
                lustre_swab_llog_hdr(handle->lgh_hdr);
 
@@ -244,28 +262,61 @@ static int llog_osd_read_header(const struct lu_env *env,
                       PFID(lu_object_fid(&o->do_lu)),
                       llh_hdr->lrh_type, LLOG_HDR_MAGIC);
                RETURN(-EIO);
-       } else if (llh_hdr->lrh_len != LLOG_CHUNK_SIZE) {
+       } else if (llh_hdr->lrh_len < LLOG_MIN_CHUNK_SIZE ||
+                  llh_hdr->lrh_len > handle->lgh_hdr_size) {
                CERROR("%s: incorrectly sized log %s "DFID" header: "
-                      "%#x (expected %#x)\n"
+                      "%#x (expected at least %#x)\n"
                       "you may need to re-run lconf --write_conf.\n",
                       o->do_lu.lo_dev->ld_obd->obd_name,
                       handle->lgh_name ? handle->lgh_name : "",
                       PFID(lu_object_fid(&o->do_lu)),
-                      llh_hdr->lrh_len, LLOG_CHUNK_SIZE);
+                      llh_hdr->lrh_len, LLOG_MIN_CHUNK_SIZE);
+               RETURN(-EIO);
+       } else if (LLOG_HDR_TAIL(handle->lgh_hdr)->lrt_index >
+                  LLOG_HDR_BITMAP_SIZE(handle->lgh_hdr) ||
+                  LLOG_HDR_TAIL(handle->lgh_hdr)->lrt_len !=
+                       llh_hdr->lrh_len) {
+               CERROR("%s: incorrectly sized log %s "DFID" tailer: "
+                      "%#x : rc = %d\n",
+                      o->do_lu.lo_dev->ld_obd->obd_name,
+                      handle->lgh_name ? handle->lgh_name : "",
+                      PFID(lu_object_fid(&o->do_lu)),
+                      LLOG_HDR_TAIL(handle->lgh_hdr)->lrt_len, -EIO);
                RETURN(-EIO);
        }
 
-       handle->lgh_last_idx = handle->lgh_hdr->llh_tail.lrt_index;
+       handle->lgh_hdr->llh_flags |= (flags & LLOG_F_EXT_MASK);
+       handle->lgh_last_idx = LLOG_HDR_TAIL(handle->lgh_hdr)->lrt_index;
 
        RETURN(0);
 }
 
+/**
+ * Implementation of the llog_operations::lop_declare_write
+ *
+ * This function declares the new record write.
+ *
+ * \param[in] env      execution environment
+ * \param[in] loghandle        llog handle of the current llog
+ * \param[in] rec      llog record header. This is a real header of the full
+ *                     llog record to write. This is the beginning of buffer
+ *                     to write, the length of buffer is stored in
+ *                     \a rec::lrh_len
+ * \param[in] idx      index of the llog record. If \a idx == -1 then this is
+ *                     append case, otherwise \a idx is the index of record
+ *                     to modify
+ * \param[in] th       current transaction handle
+ *
+ * \retval             0 on successful declaration
+ * \retval             negative error if declaration failed
+ */
 static int llog_osd_declare_write_rec(const struct lu_env *env,
                                      struct llog_handle *loghandle,
                                      struct llog_rec_hdr *rec,
                                      int idx, struct thandle *th)
 {
        struct llog_thread_info *lgi = llog_info(env);
+       __u32                   chunk_size;
        struct dt_object        *o;
        int                      rc;
 
@@ -274,39 +325,61 @@ static int llog_osd_declare_write_rec(const struct lu_env *env,
        LASSERT(env);
        LASSERT(th);
        LASSERT(loghandle);
+       LASSERT(rec);
+       LASSERT(rec->lrh_len <= loghandle->lgh_ctxt->loc_chunk_size);
 
        o = loghandle->lgh_obj;
        LASSERT(o);
 
+       chunk_size = loghandle->lgh_ctxt->loc_chunk_size;
+       lgi->lgi_buf.lb_len = chunk_size;
+       lgi->lgi_buf.lb_buf = NULL;
        /* each time we update header */
-       rc = dt_declare_record_write(env, o, sizeof(struct llog_log_hdr), 0,
+       rc = dt_declare_record_write(env, o, &lgi->lgi_buf, 0,
                                     th);
        if (rc || idx == 0) /* if error or just header */
                RETURN(rc);
 
-       if (dt_object_exists(o)) {
-               rc = dt_attr_get(env, o, &lgi->lgi_attr, BYPASS_CAPA);
-               lgi->lgi_off = lgi->lgi_attr.la_size;
-               LASSERT(ergo(rc == 0, lgi->lgi_attr.la_valid & LA_SIZE));
-               if (rc)
-                       RETURN(rc);
-       } else {
-               lgi->lgi_off = 0;
-       }
-
+       /**
+        * the pad record can be inserted so take into account double
+        * record size
+        */
+       lgi->lgi_buf.lb_len = chunk_size * 2;
+       lgi->lgi_buf.lb_buf = NULL;
        /* XXX: implement declared window or multi-chunks approach */
-       rc = dt_declare_record_write(env, o, 32 * 1024, lgi->lgi_off, th);
+       rc = dt_declare_record_write(env, o, &lgi->lgi_buf, -1, th);
 
        RETURN(rc);
 }
 
-/* returns negative in on error; 0 if success && reccookie == 0; 1 otherwise */
-/* appends if idx == -1, otherwise overwrites record idx. */
+/**
+ * Implementation of the llog_operations::lop_write
+ *
+ * This function writes the new record in the llog or modify the existed one.
+ *
+ * \param[in]  env             execution environment
+ * \param[in]  loghandle       llog handle of the current llog
+ * \param[in]  rec             llog record header. This is a real header of
+ *                             the full llog record to write. This is
+ *                             the beginning of buffer to write, the length
+ *                             of buffer is stored in \a rec::lrh_len
+ * \param[in,out] reccookie    pointer to the cookie to return back if needed.
+ *                             It is used for further cancel of this llog
+ *                             record.
+ * \param[in]  idx             index of the llog record. If \a idx == -1 then
+ *                             this is append case, otherwise \a idx is
+ *                             the index of record to modify
+ * \param[in]  th              current transaction handle
+ *
+ * \retval                     0 on successful write && \a reccookie == NULL
+ *                             1 on successful write && \a reccookie != NULL
+ * \retval                     negative error if write failed
+ */
 static int llog_osd_write_rec(const struct lu_env *env,
                              struct llog_handle *loghandle,
                              struct llog_rec_hdr *rec,
-                             struct llog_cookie *reccookie, int cookiecount,
-                             void *buf, int idx, struct thandle *th)
+                             struct llog_cookie *reccookie,
+                             int idx, struct thandle *th)
 {
        struct llog_thread_info *lgi = llog_info(env);
        struct llog_log_hdr     *llh;
@@ -314,82 +387,149 @@ static int llog_osd_write_rec(const struct lu_env *env,
        int                      index, rc;
        struct llog_rec_tail    *lrt;
        struct dt_object        *o;
+       __u32                   chunk_size;
        size_t                   left;
-
+       __u32                   orig_last_idx;
        ENTRY;
 
-       LASSERT(env);
        llh = loghandle->lgh_hdr;
-       LASSERT(llh);
        o = loghandle->lgh_obj;
-       LASSERT(o);
-       LASSERT(th);
 
+       chunk_size = llh->llh_hdr.lrh_len;
        CDEBUG(D_OTHER, "new record %x to "DFID"\n",
               rec->lrh_type, PFID(lu_object_fid(&o->do_lu)));
 
-       /* record length should not bigger than LLOG_CHUNK_SIZE */
-       if (buf)
-               rc = (reclen > LLOG_CHUNK_SIZE - sizeof(struct llog_rec_hdr) -
-                     sizeof(struct llog_rec_tail)) ? -E2BIG : 0;
-       else
-               rc = (reclen > LLOG_CHUNK_SIZE) ? -E2BIG : 0;
-       if (rc)
-               RETURN(rc);
+       if (!llog_osd_exist(loghandle))
+               RETURN(-ENOENT);
 
-       rc = dt_attr_get(env, o, &lgi->lgi_attr, NULL);
-       if (rc)
-               RETURN(rc);
+       /* record length should not bigger than  */
+       if (reclen > loghandle->lgh_hdr->llh_hdr.lrh_len)
+               RETURN(-E2BIG);
 
-       if (buf)
-               /* write_blob adds header and tail to lrh_len. */
-               reclen = sizeof(*rec) + rec->lrh_len +
-                        sizeof(struct llog_rec_tail);
+       /* sanity check for fixed-records llog */
+       if (idx != LLOG_HEADER_IDX && (llh->llh_flags & LLOG_F_IS_FIXSIZE)) {
+               LASSERT(llh->llh_size != 0);
+               LASSERT(llh->llh_size == reclen);
+       }
 
-       if (idx != -1) {
-               /* no header: only allowed to insert record 1 */
-               if (idx != 1 && lgi->lgi_attr.la_size == 0)
-                       LBUG();
+       /* return error if osp object is stale */
+       if (idx != LLOG_HEADER_IDX && dt_object_stale(o))
+               RETURN(-ESTALE);
+       rc = dt_attr_get(env, o, &lgi->lgi_attr);
+       if (rc)
+               RETURN(rc);
 
-               if (idx && llh->llh_size && llh->llh_size != rec->lrh_len)
-                       RETURN(-EINVAL);
+       /**
+        * The modification case.
+        * If idx set then the record with that index must be modified.
+        * There are three cases possible:
+        * 1) the common case is the llog header update (idx == 0)
+        * 2) the llog record modification during llog process.
+        *    This is indicated by the \a loghandle::lgh_cur_idx > 0.
+        *    In that case the \a loghandle::lgh_cur_offset
+        * 3) otherwise this is assumed that llog consist of records of
+        *    fixed size, i.e. catalog. The llog header must has llh_size
+        *    field equal to record size. The record offset is calculated
+        *    just by /a idx value
+        *
+        * During modification we don't need extra header update because
+        * the bitmap and record count are not changed. The record header
+        * and tail remains the same too.
+        */
+       if (idx != LLOG_NEXT_IDX) {
+               /* llog can be empty only when first record is being written */
+               LASSERT(ergo(idx > 0, lgi->lgi_attr.la_size > 0));
 
-               if (!ext2_test_bit(idx, llh->llh_bitmap))
+               if (!test_bit_le(idx, LLOG_HDR_BITMAP(llh))) {
                        CERROR("%s: modify unset record %u\n",
                               o->do_lu.lo_dev->ld_obd->obd_name, idx);
-               if (idx != rec->lrh_index)
-                       CERROR("%s: index mismatch %d %u\n",
+                       RETURN(-ENOENT);
+               }
+
+               if (idx != rec->lrh_index) {
+                       CERROR("%s: modify index mismatch %d %u\n",
                               o->do_lu.lo_dev->ld_obd->obd_name, idx,
                               rec->lrh_index);
+                       RETURN(-EFAULT);
+               }
 
-               lgi->lgi_off = 0;
-               rc = llog_osd_write_blob(env, o, &llh->llh_hdr, NULL,
-                                        &lgi->lgi_off, th);
-               /* we are done if we only write the header or on error */
-               if (rc || idx == 0)
-                       RETURN(rc);
+               if (idx == LLOG_HEADER_IDX) {
+                       /* llog header update */
+                       __u32   *bitmap = LLOG_HDR_BITMAP(llh);
+
+                       lgi->lgi_off = 0;
+
+                       /* If it does not indicate the bitmap index
+                        * (reccookie == NULL), then it means update
+                        * the whole update header. Otherwise only
+                        * update header and bits needs to be updated,
+                        * and in DNE cases, it will signaficantly
+                        * shrink the RPC size.
+                        * see distribute_txn_cancel_records()*/
+                       if (reccookie == NULL) {
+                               lgi->lgi_buf.lb_len = reclen;
+                               lgi->lgi_buf.lb_buf = rec;
+                               rc = dt_record_write(env, o, &lgi->lgi_buf,
+                                                    &lgi->lgi_off, th);
+                               RETURN(rc);
+                       }
 
-               if (buf) {
-                       /* We assume that caller has set lgh_cur_* */
-                       lgi->lgi_off = loghandle->lgh_cur_offset;
-                       CDEBUG(D_OTHER,
-                              "modify record "LPX64": idx:%d/%u/%d, len:%u "
-                              "offset %llu\n",
-                              loghandle->lgh_id.lgl_oid, idx, rec->lrh_index,
-                              loghandle->lgh_cur_idx, rec->lrh_len,
-                              (long long)(lgi->lgi_off - sizeof(*llh)));
-                       if (rec->lrh_index != loghandle->lgh_cur_idx) {
-                               CERROR("%s: modify idx mismatch %u/%d\n",
+                       /* update the header */
+                       lgi->lgi_buf.lb_len = llh->llh_bitmap_offset;
+                       lgi->lgi_buf.lb_buf = llh;
+                       rc = dt_record_write(env, o, &lgi->lgi_buf,
+                                            &lgi->lgi_off, th);
+                       if (rc != 0)
+                               RETURN(rc);
+
+                       /* update the bitmap */
+                       index = reccookie->lgc_index;
+                       lgi->lgi_off = llh->llh_bitmap_offset +
+                                     (index / (sizeof(*bitmap) * 8)) *
+                                                       sizeof(*bitmap);
+                       lgi->lgi_buf.lb_len = sizeof(*bitmap);
+                       lgi->lgi_buf.lb_buf =
+                                       &bitmap[index/(sizeof(*bitmap)*8)];
+                       rc = dt_record_write(env, o, &lgi->lgi_buf,
+                                            &lgi->lgi_off, th);
+
+                       RETURN(rc);
+               } else if (llh->llh_flags & LLOG_F_IS_FIXSIZE) {
+                       lgi->lgi_off = llh->llh_hdr.lrh_len +
+                                      (idx - 1) * reclen;
+               } else if (reccookie != NULL && reccookie->lgc_index > 0) {
+                       /**
+                        * The lgc_offset can be used only if index is
+                        * the same.
+                        */
+                       if (idx != reccookie->lgc_index) {
+                               CERROR("%s: modify index mismatch %d %d\n",
                                       o->do_lu.lo_dev->ld_obd->obd_name, idx,
-                                      loghandle->lgh_cur_idx);
+                                      reccookie->lgc_index);
                                RETURN(-EFAULT);
                        }
+
+                       lgi->lgi_off = reccookie->lgc_offset;
+                       CDEBUG(D_OTHER, "modify record "DFID": idx:%u, "
+                              "len:%u offset %llu\n",
+                              PFID(&loghandle->lgh_id.lgl_oi.oi_fid), idx,
+                              rec->lrh_len, (long long)lgi->lgi_off);
                } else {
-                       /* Assumes constant lrh_len */
-                       lgi->lgi_off = sizeof(*llh) + (idx - 1) * reclen;
+                       /* This can be result of lgh_cur_idx is not set during
+                        * llog processing or llh_size is not set to proper
+                        * record size for fixed records llog. Therefore it is
+                        * impossible to get record offset. */
+                       CERROR("%s: can't get record offset, idx:%d, "
+                              "len:%u.\n", o->do_lu.lo_dev->ld_obd->obd_name,
+                              idx, rec->lrh_len);
+                       RETURN(-EFAULT);
                }
 
-               rc = llog_osd_write_blob(env, o, rec, buf, &lgi->lgi_off, th);
+               /* update only data, header and tail remain the same */
+               lgi->lgi_off += sizeof(struct llog_rec_hdr);
+               lgi->lgi_buf.lb_len = REC_DATA_LEN(rec);
+               lgi->lgi_buf.lb_buf = REC_DATA(rec);
+               rc = dt_record_write(env, o, &lgi->lgi_buf, &lgi->lgi_off, th);
                if (rc == 0 && reccookie) {
                        reccookie->lgc_lgl = loghandle->lgh_id;
                        reccookie->lgc_index = idx;
@@ -398,16 +538,40 @@ static int llog_osd_write_rec(const struct lu_env *env,
                RETURN(rc);
        }
 
-       /* Make sure that records don't cross a chunk boundary, so we can
+       /**
+        * The append case.
+        * The most common case of using llog. The new index is assigned to
+        * the new record, new bit is set in llog bitmap and llog count is
+        * incremented.
+        *
+        * Make sure that records don't cross a chunk boundary, so we can
         * process them page-at-a-time if needed.  If it will cross a chunk
         * boundary, write in a fake (but referenced) entry to pad the chunk.
-        *
-        * We know that llog_current_log() will return a loghandle that is
-        * big enough to hold reclen, so all we care about is padding here.
         */
+
+
+       /* simulate ENOSPC when new plain llog is being added to the
+        * catalog */
+       if (OBD_FAIL_CHECK(OBD_FAIL_MDS_LLOG_CREATE_FAILED2) &&
+           llh->llh_flags & LLOG_F_IS_CAT)
+               RETURN(-ENOSPC);
+
        LASSERT(lgi->lgi_attr.la_valid & LA_SIZE);
+       orig_last_idx = loghandle->lgh_last_idx;
        lgi->lgi_off = lgi->lgi_attr.la_size;
-       left = LLOG_CHUNK_SIZE - (lgi->lgi_off & (LLOG_CHUNK_SIZE - 1));
+
+       if (loghandle->lgh_max_size > 0 &&
+           lgi->lgi_off >= loghandle->lgh_max_size) {
+               CDEBUG(D_OTHER, "llog is getting too large (%u > %u) at %u "
+                      DFID"\n", (unsigned)lgi->lgi_off,
+                      loghandle->lgh_max_size, (int)loghandle->lgh_last_idx,
+                      PFID(&loghandle->lgh_id.lgl_oi.oi_fid));
+               /* this is to signal that this llog is full */
+               loghandle->lgh_last_idx = LLOG_HDR_BITMAP_SIZE(llh) - 1;
+               RETURN(-ENOSPC);
+       }
+
+       left = chunk_size - (lgi->lgi_off & (chunk_size - 1));
        /* NOTE: padding is a record, but no bit is set */
        if (left != 0 && left != reclen &&
            left < (reclen + LLOG_MIN_REC_SIZE)) {
@@ -415,55 +579,138 @@ static int llog_osd_write_rec(const struct lu_env *env,
                rc = llog_osd_pad(env, o, &lgi->lgi_off, left, index, th);
                if (rc)
                        RETURN(rc);
-               loghandle->lgh_last_idx++; /*for pad rec*/
+
+               loghandle->lgh_last_idx++; /* for pad rec */
+       }
+       /* if it's the last idx in log file, then return -ENOSPC
+        * or wrap around if a catalog */
+       if (llog_is_full(loghandle) ||
+           unlikely(llh->llh_flags & LLOG_F_IS_CAT &&
+                    OBD_FAIL_PRECHECK(OBD_FAIL_CAT_RECORDS) &&
+                    loghandle->lgh_last_idx >= cfs_fail_val)) {
+               if (llh->llh_flags & LLOG_F_IS_CAT)
+                       loghandle->lgh_last_idx = 0;
+               else
+                       RETURN(-ENOSPC);
        }
-       /* if it's the last idx in log file, then return -ENOSPC */
-       if (loghandle->lgh_last_idx >= LLOG_BITMAP_SIZE(llh) - 1)
-               RETURN(-ENOSPC);
 
+       down_write(&loghandle->lgh_last_sem);
+       /* increment the last_idx along with llh_tail index, they should
+        * be equal for a llog lifetime */
        loghandle->lgh_last_idx++;
        index = loghandle->lgh_last_idx;
-       LASSERT(index < LLOG_BITMAP_SIZE(llh));
+       LLOG_HDR_TAIL(llh)->lrt_index = index;
+       /**
+        * NB: the caller should make sure only 1 process access
+        * the lgh_last_idx, e.g. append should be exclusive.
+        * Otherwise it might hit the assert.
+        */
+       LASSERT(index < LLOG_HDR_BITMAP_SIZE(llh));
        rec->lrh_index = index;
-       if (buf == NULL) {
-               lrt = (struct llog_rec_tail *)((char *)rec + rec->lrh_len -
-                                              sizeof(*lrt));
-               lrt->lrt_len = rec->lrh_len;
-               lrt->lrt_index = rec->lrh_index;
-       }
-       /* The caller should make sure only 1 process access the lgh_last_idx,
-        * Otherwise it might hit the assert.*/
-       LASSERT(index < LLOG_BITMAP_SIZE(llh));
-       spin_lock(&loghandle->lgh_hdr_lock);
-       if (ext2_set_bit(index, llh->llh_bitmap)) {
-               CERROR("%s: index %u already set in log bitmap\n",
-                      o->do_lu.lo_dev->ld_obd->obd_name, index);
-               spin_unlock(&loghandle->lgh_hdr_lock);
+       lrt = rec_tail(rec);
+       lrt->lrt_len = rec->lrh_len;
+       lrt->lrt_index = rec->lrh_index;
+
+       /* the lgh_hdr_mutex protects llog header data from concurrent
+        * update/cancel, the llh_count and llh_bitmap are protected */
+       mutex_lock(&loghandle->lgh_hdr_mutex);
+       if (__test_and_set_bit_le(index, LLOG_HDR_BITMAP(llh))) {
+               CERROR("%s: index %u already set in llog bitmap "DFID"\n",
+                      o->do_lu.lo_dev->ld_obd->obd_name, index,
+                      PFID(lu_object_fid(&o->do_lu)));
+               mutex_unlock(&loghandle->lgh_hdr_mutex);
                LBUG(); /* should never happen */
        }
        llh->llh_count++;
-       spin_unlock(&loghandle->lgh_hdr_lock);
-       llh->llh_tail.lrt_index = index;
 
-       lgi->lgi_off = 0;
-       rc = llog_osd_write_blob(env, o, &llh->llh_hdr, NULL, &lgi->lgi_off,
-                                th);
-       if (rc)
-               RETURN(rc);
+       if (!(llh->llh_flags & LLOG_F_IS_FIXSIZE)) {
+               /* Update the minimum size of the llog record */
+               if (llh->llh_size == 0)
+                       llh->llh_size = reclen;
+               else if (reclen < llh->llh_size)
+                       llh->llh_size = reclen;
+       }
 
-       rc = dt_attr_get(env, o, &lgi->lgi_attr, NULL);
-       if (rc)
-               RETURN(rc);
-       LASSERT(lgi->lgi_attr.la_valid & LA_SIZE);
-       lgi->lgi_off = lgi->lgi_attr.la_size;
+       if (lgi->lgi_attr.la_size == 0) {
+               lgi->lgi_off = 0;
+               lgi->lgi_buf.lb_len = llh->llh_hdr.lrh_len;
+               lgi->lgi_buf.lb_buf = &llh->llh_hdr;
+               rc = dt_record_write(env, o, &lgi->lgi_buf, &lgi->lgi_off, th);
+               if (rc != 0)
+                       GOTO(out_unlock, rc);
+       } else {
+               __u32   *bitmap = LLOG_HDR_BITMAP(llh);
+
+               /* Note: If this is not initialization (size == 0), then do not
+                * write the whole header (8k bytes), only update header/tail
+                * and bits needs to be updated. Because this update might be
+                * part of cross-MDT operation, which needs to write these
+                * updates into the update log(32KB limit) and also pack inside
+                * the RPC (1MB limit), if we write 8K for each operation, which
+                * will cost a lot space, and keep us adding more updates to one
+                * update log.*/
+               lgi->lgi_off = 0;
+               lgi->lgi_buf.lb_len = llh->llh_bitmap_offset;
+               lgi->lgi_buf.lb_buf = &llh->llh_hdr;
+               rc = dt_record_write(env, o, &lgi->lgi_buf, &lgi->lgi_off, th);
+               if (rc != 0)
+                       GOTO(out_unlock, rc);
 
-       rc = llog_osd_write_blob(env, o, rec, buf, &lgi->lgi_off, th);
+               lgi->lgi_off = llh->llh_bitmap_offset +
+                             (index / (sizeof(*bitmap) * 8)) * sizeof(*bitmap);
+               lgi->lgi_buf.lb_len = sizeof(*bitmap);
+               lgi->lgi_buf.lb_buf = &bitmap[index/(sizeof(*bitmap)*8)];
+               rc = dt_record_write(env, o, &lgi->lgi_buf, &lgi->lgi_off, th);
+               if (rc != 0)
+                       GOTO(out_unlock, rc);
+
+               lgi->lgi_off =  (unsigned long)LLOG_HDR_TAIL(llh) -
+                               (unsigned long)llh;
+               lgi->lgi_buf.lb_len = sizeof(llh->llh_tail);
+               lgi->lgi_buf.lb_buf = LLOG_HDR_TAIL(llh);
+               rc = dt_record_write(env, o, &lgi->lgi_buf, &lgi->lgi_off, th);
+               if (rc != 0)
+                       GOTO(out_unlock, rc);
+       }
+
+out_unlock:
+       /* unlock here for remote object */
+       mutex_unlock(&loghandle->lgh_hdr_mutex);
        if (rc)
-               RETURN(rc);
+               GOTO(out, rc);
 
-       CDEBUG(D_RPCTRACE, "added record "LPX64": idx: %u, %u\n",
-              loghandle->lgh_id.lgl_oid, index, rec->lrh_len);
-       if (rc == 0 && reccookie) {
+       if (OBD_FAIL_PRECHECK(OBD_FAIL_LLOG_PROCESS_TIMEOUT) &&
+          cfs_fail_val == (unsigned int)(loghandle->lgh_id.lgl_oi.oi.oi_id &
+                                         0xFFFFFFFF)) {
+               OBD_RACE(OBD_FAIL_LLOG_PROCESS_TIMEOUT);
+               msleep(1 * MSEC_PER_SEC);
+       }
+       /* computed index can be used to determine offset for fixed-size
+        * records. This also allows to handle Catalog wrap around case */
+       if (llh->llh_flags & LLOG_F_IS_FIXSIZE) {
+               lgi->lgi_off = llh->llh_hdr.lrh_len + (index - 1) * reclen;
+       } else {
+               rc = dt_attr_get(env, o, &lgi->lgi_attr);
+               if (rc)
+                       GOTO(out, rc);
+
+               LASSERT(lgi->lgi_attr.la_valid & LA_SIZE);
+               lgi->lgi_off = max_t(__u64, lgi->lgi_attr.la_size,
+                                    lgi->lgi_off);
+       }
+
+       lgi->lgi_buf.lb_len = reclen;
+       lgi->lgi_buf.lb_buf = rec;
+       rc = dt_record_write(env, o, &lgi->lgi_buf, &lgi->lgi_off, th);
+       if (rc < 0)
+               GOTO(out, rc);
+
+       up_write(&loghandle->lgh_last_sem);
+
+       CDEBUG(D_HA, "added record "DFID".%u, %u off%llu\n",
+              PFID(lu_object_fid(&o->do_lu)), index, rec->lrh_len,
+              lgi->lgi_off);
+       if (reccookie != NULL) {
                reccookie->lgc_lgl = loghandle->lgh_id;
                reccookie->lgc_index = index;
                if ((rec->lrh_type == MDS_UNLINK_REC) ||
@@ -476,26 +723,160 @@ static int llog_osd_write_rec(const struct lu_env *env,
                rc = 1;
        }
        RETURN(rc);
+out:
+       /* cleanup llog for error case */
+       mutex_lock(&loghandle->lgh_hdr_mutex);
+       clear_bit_le(index, LLOG_HDR_BITMAP(llh));
+       llh->llh_count--;
+       mutex_unlock(&loghandle->lgh_hdr_mutex);
+
+       /* restore llog last_idx */
+       if (dt_object_remote(o)) {
+               loghandle->lgh_last_idx = orig_last_idx;
+       } else if (--loghandle->lgh_last_idx == 0 &&
+           (llh->llh_flags & LLOG_F_IS_CAT) && llh->llh_cat_idx != 0) {
+               /* catalog had just wrap-around case */
+               loghandle->lgh_last_idx = LLOG_HDR_BITMAP_SIZE(llh) - 1;
+       }
+
+       LLOG_HDR_TAIL(llh)->lrt_index = loghandle->lgh_last_idx;
+       up_write(&loghandle->lgh_last_sem);
+
+       RETURN(rc);
 }
 
-/* We can skip reading at least as many log blocks as the number of
+/**
+ * We can skip reading at least as many log blocks as the number of
  * minimum sized log records we are skipping.  If it turns out
  * that we are not far enough along the log (because the
  * actual records are larger than minimum size) we just skip
  * some more records.
+ *
+ * Note: in llog_process_thread, it will use bitmap offset as
+ * the index to locate the record, which also includs some pad
+ * records, whose record size is very small, and it also does not
+ * consider pad record when recording minimum record size (otherwise
+ * min_record size might be too small), so in some rare cases,
+ * it might skip too much record for @goal, see llog_osd_next_block().
+ *
+ * When force_mini_rec is true, it means we have to use LLOG_MIN_REC_SIZE
+ * as the min record size to skip over, usually because in the previous
+ * try, it skip too much record, see loog_osd_next(prev)_block().
+ */
+static inline void llog_skip_over(struct llog_handle *lgh, __u64 *off,
+                                 int curr, int goal, __u32 chunk_size,
+                                 bool force_mini_rec)
+{
+       struct llog_log_hdr *llh = lgh->lgh_hdr;
+
+       /* Goal should not bigger than the record count */
+       if (goal > lgh->lgh_last_idx)
+               goal = lgh->lgh_last_idx;
+
+       if (goal > curr) {
+               if (llh->llh_flags & LLOG_F_IS_FIXSIZE) {
+                       *off = chunk_size + (goal - 1) * llh->llh_size;
+               } else {
+                       __u64 min_rec_size = LLOG_MIN_REC_SIZE;
+
+                       if (llh->llh_size > 0 && !force_mini_rec)
+                               min_rec_size = llh->llh_size;
+
+                       *off = *off + (goal - curr - 1) * min_rec_size;
+               }
+       }
+       /* always align with lower chunk boundary*/
+       *off &= ~(chunk_size - 1);
+}
+
+/**
+ * Remove optional fields that the client doesn't expect.
+ * This is typically in order to ensure compatibility with older clients.
+ * It is assumed that since we exclusively remove fields, the block will be
+ * big enough to handle the remapped records. It is also assumed that records
+ * of a block have the same format (i.e.: the same features enabled).
+ *
+ * \param[in,out]    hdr          Header of the block of records to remap.
+ * \param[in,out]    last_hdr      Last header, don't read past this point.
+ * \param[in]        flags        Flags describing the fields to keep.
+ * \param[in]        extra_flags   Flags describing the extra fields to keep.
  */
-static void llog_skip_over(__u64 *off, int curr, int goal)
+static void changelog_block_trim_ext(struct llog_rec_hdr *hdr,
+                                    struct llog_rec_hdr *last_hdr,
+                                    struct llog_handle *loghandle)
 {
-       if (goal <= curr)
+       enum changelog_rec_flags flags = CLF_SUPPORTED;
+       enum changelog_rec_extra_flags extra_flags = CLFE_SUPPORTED;
+
+       if (!(loghandle->lgh_hdr->llh_flags & LLOG_F_EXT_X_XATTR))
+               extra_flags &= ~CLFE_XATTR;
+       if (!(loghandle->lgh_hdr->llh_flags & LLOG_F_EXT_X_OMODE))
+               extra_flags &= ~CLFE_OPEN;
+       if (!(loghandle->lgh_hdr->llh_flags & LLOG_F_EXT_X_NID))
+               extra_flags &= ~CLFE_NID;
+       if (!(loghandle->lgh_hdr->llh_flags & LLOG_F_EXT_X_UIDGID))
+               extra_flags &= ~CLFE_UIDGID;
+       if (!(loghandle->lgh_hdr->llh_flags & LLOG_F_EXT_EXTRA_FLAGS))
+               flags &= ~CLF_EXTRA_FLAGS;
+       if (!(loghandle->lgh_hdr->llh_flags & LLOG_F_EXT_JOBID))
+               flags &= ~CLF_JOBID;
+
+       if (flags == CLF_SUPPORTED && extra_flags == CLFE_SUPPORTED)
                return;
-       *off = (*off + (goal - curr - 1) * LLOG_MIN_REC_SIZE) &
-               ~(LLOG_CHUNK_SIZE - 1);
+
+       if (hdr->lrh_type != CHANGELOG_REC)
+               return;
+
+       do {
+               struct changelog_rec *rec = (struct changelog_rec *)(hdr + 1);
+               enum changelog_rec_extra_flags xflag = CLFE_INVALID;
+
+               if (flags & CLF_EXTRA_FLAGS &&
+                   rec->cr_flags & CLF_EXTRA_FLAGS) {
+                       xflag = changelog_rec_extra_flags(rec)->cr_extra_flags &
+                               extra_flags;
+               }
+
+               if (unlikely(hdr->lrh_len == 0)) {
+                       /* It is corruption case, we cannot know the next rec,
+                        * jump to the last one directly to avoid dead loop. */
+                       LCONSOLE(D_WARNING, "Hit invalid llog record: "
+                                "idx %u, type %u, id %u\n",
+                                hdr->lrh_index, hdr->lrh_type, hdr->lrh_id);
+                       hdr = llog_rec_hdr_next(last_hdr);
+                       if (unlikely(hdr == last_hdr))
+                               LCONSOLE(D_WARNING, "The last record crashed: "
+                                        "idx %u, type %u, id %u\n",
+                                        hdr->lrh_index, hdr->lrh_type,
+                                        hdr->lrh_id);
+                       break;
+               }
+
+               changelog_remap_rec(rec, rec->cr_flags & flags, xflag);
+               hdr = llog_rec_hdr_next(hdr);
+               /* Yield CPU to avoid soft-lockup if there are too many records
+                * to be handled. */
+               cond_resched();
+       } while ((char *)hdr <= (char *)last_hdr);
 }
 
-/* sets:
- *  - cur_offset to the furthest point read in the log file
- *  - cur_idx to the log index preceeding cur_offset
- * returns -EIO/-EINVAL on error
+/**
+ * Implementation of the llog_operations::lop_next_block
+ *
+ * This function finds the the next llog block to return which contains
+ * record with required index. It is main part of llog processing.
+ *
+ * \param[in]     env          execution environment
+ * \param[in]     loghandle    llog handle of the current llog
+ * \param[in,out] cur_idx      index preceeding cur_offset
+ * \param[in]     next_idx     target index to find
+ * \param[in,out] cur_offset   furtherst point read in the file
+ * \param[in]     buf          pointer to data buffer to fill
+ * \param[in]     len          required len to read, it is
+ *                             usually llog chunk_size.
+ *
+ * \retval                     0 on successful buffer read
+ * \retval                     negative value on error
  */
 static int llog_osd_next_block(const struct lu_env *env,
                               struct llog_handle *loghandle, int *cur_idx,
@@ -506,57 +887,57 @@ static int llog_osd_next_block(const struct lu_env *env,
        struct dt_object        *o;
        struct dt_device        *dt;
        int                      rc;
+       __u32                   chunk_size;
+       int last_idx = *cur_idx;
+       __u64 last_offset = *cur_offset;
+       bool force_mini_rec = false;
 
        ENTRY;
 
        LASSERT(env);
        LASSERT(lgi);
 
-       if (len == 0 || len & (LLOG_CHUNK_SIZE - 1))
+       chunk_size = loghandle->lgh_hdr->llh_hdr.lrh_len;
+       if (len == 0 || len & (chunk_size - 1))
                RETURN(-EINVAL);
 
-       CDEBUG(D_OTHER, "looking for log index %u (cur idx %u off "LPU64")\n",
-              next_idx, *cur_idx, *cur_offset);
-
        LASSERT(loghandle);
        LASSERT(loghandle->lgh_ctxt);
 
        o = loghandle->lgh_obj;
        LASSERT(o);
-       LASSERT(dt_object_exists(o));
+       LASSERT(llog_osd_exist(loghandle));
        dt = lu2dt_dev(o->do_lu.lo_dev);
        LASSERT(dt);
 
-       rc = dt_attr_get(env, o, &lgi->lgi_attr, BYPASS_CAPA);
+       rc = dt_attr_get(env, o, &lgi->lgi_attr);
        if (rc)
                GOTO(out, rc);
 
+       CDEBUG(D_OTHER,
+              "looking for log index %u (cur idx %u off %llu), size %llu\n",
+              next_idx, *cur_idx,
+              *cur_offset, lgi->lgi_attr.la_size);
+
        while (*cur_offset < lgi->lgi_attr.la_size) {
                struct llog_rec_hdr     *rec, *last_rec;
                struct llog_rec_tail    *tail;
 
-               llog_skip_over(cur_offset, *cur_idx, next_idx);
+               llog_skip_over(loghandle, cur_offset, *cur_idx,
+                              next_idx, chunk_size, force_mini_rec);
 
-               /* read up to next LLOG_CHUNK_SIZE block */
-               lgi->lgi_buf.lb_len = LLOG_CHUNK_SIZE -
-                                     (*cur_offset & (LLOG_CHUNK_SIZE - 1));
+               /* read up to next llog chunk_size block */
+               lgi->lgi_buf.lb_len = chunk_size -
+                                     (*cur_offset & (chunk_size - 1));
                lgi->lgi_buf.lb_buf = buf;
 
-               /* Note: read lock is not needed around la_size get above at
-                * the time of dt_attr_get(). There are only two cases that
-                * matter. Either la_size == cur_offset, in which case the
-                * entire read is skipped, or la_size > cur_offset and the loop
-                * is entered and this thread is blocked at dt_read_lock()
-                * until the write is completed. When the write completes, then
-                * the dt_read() will be done with the full length, and will
-                * get the full data.
-                */
-               dt_read_lock(env, o, 0);
                rc = dt_read(env, o, &lgi->lgi_buf, cur_offset);
-               dt_read_unlock(env, o);
                if (rc < 0) {
+                       if (rc == -EBADR && !force_mini_rec)
+                               goto retry;
+
                        CERROR("%s: can't read llog block from log "DFID
-                              " offset "LPU64": rc = %d\n",
+                              " offset %llu: rc = %d\n",
                               o->do_lu.lo_dev->ld_obd->obd_name,
                               PFID(lu_object_fid(&o->do_lu)), *cur_offset,
                               rc);
@@ -569,14 +950,20 @@ static int llog_osd_next_block(const struct lu_env *env,
                        memset(buf + rc, 0, len - rc);
                }
 
-               if (rc == 0) /* end of file, nothing to do */
+               if (rc == 0) { /* end of file, nothing to do */
+                       if (!force_mini_rec)
+                               goto retry;
                        GOTO(out, rc);
+               }
 
                if (rc < sizeof(*tail)) {
-                       CERROR("%s: invalid llog block at log id "LPU64"/%u "
-                              "offset "LPU64"\n",
+                       if (!force_mini_rec)
+                               goto retry;
+
+                       CERROR("%s: invalid llog block at log id "DFID":%x "
+                              "offset %llu\n",
                               o->do_lu.lo_dev->ld_obd->obd_name,
-                              loghandle->lgh_id.lgl_oid,
+                              PFID(&loghandle->lgh_id.lgl_oi.oi_fid),
                               loghandle->lgh_id.lgl_ogen, *cur_offset);
                        GOTO(out, rc = -EINVAL);
                }
@@ -584,46 +971,106 @@ static int llog_osd_next_block(const struct lu_env *env,
                rec = buf;
                if (LLOG_REC_HDR_NEEDS_SWABBING(rec))
                        lustre_swab_llog_rec(rec);
-
                tail = (struct llog_rec_tail *)((char *)buf + rc -
                                                sizeof(struct llog_rec_tail));
+
+               if (llog_verify_record(loghandle, rec)) {
+                       /*
+                        * the block seems corrupted. make a pad record so the
+                        * caller can skip the block and try with the next one
+                        */
+                       rec->lrh_len = rc;
+                       rec->lrh_index = next_idx;
+                       rec->lrh_type = LLOG_PAD_MAGIC;
+
+                       tail = rec_tail(rec);
+                       tail->lrt_len = rc;
+                       tail->lrt_index = next_idx;
+
+                       GOTO(out, rc = 0);
+               }
+
                /* get the last record in block */
                last_rec = (struct llog_rec_hdr *)((char *)buf + rc -
-                                                  le32_to_cpu(tail->lrt_len));
+                                                  tail->lrt_len);
 
                if (LLOG_REC_HDR_NEEDS_SWABBING(last_rec))
                        lustre_swab_llog_rec(last_rec);
-               LASSERT(last_rec->lrh_index == tail->lrt_index);
+
+               if (last_rec->lrh_index != tail->lrt_index) {
+                       CERROR("%s: invalid llog tail at log id "DFID":%x offset %llu last_rec idx %u tail idx %u lrt len %u read_size %d\n",
+                              o->do_lu.lo_dev->ld_obd->obd_name,
+                              PFID(&loghandle->lgh_id.lgl_oi.oi_fid),
+                              loghandle->lgh_id.lgl_ogen, *cur_offset,
+                              last_rec->lrh_index, tail->lrt_index,
+                              tail->lrt_len, rc);
+                       GOTO(out, rc = -EINVAL);
+               }
 
                *cur_idx = tail->lrt_index;
 
                /* this shouldn't happen */
                if (tail->lrt_index == 0) {
-                       CERROR("%s: invalid llog tail at log id "LPU64"/%u "
-                              "offset "LPU64"\n",
+                       CERROR("%s: invalid llog tail at log id "DFID":%x "
+                              "offset %llu bytes %d\n",
                               o->do_lu.lo_dev->ld_obd->obd_name,
-                              loghandle->lgh_id.lgl_oid,
-                              loghandle->lgh_id.lgl_ogen, *cur_offset);
+                              PFID(&loghandle->lgh_id.lgl_oi.oi_fid),
+                              loghandle->lgh_id.lgl_ogen, *cur_offset, rc);
                        GOTO(out, rc = -EINVAL);
                }
-               if (tail->lrt_index < next_idx)
+               if (tail->lrt_index < next_idx) {
+                       last_idx = *cur_idx;
+                       last_offset = *cur_offset;
                        continue;
+               }
 
                /* sanity check that the start of the new buffer is no farther
                 * than the record that we wanted.  This shouldn't happen. */
-               if (rec->lrh_index > next_idx) {
+               if (next_idx && rec->lrh_index > next_idx) {
+                       if (!force_mini_rec && next_idx > last_idx)
+                               goto retry;
+
                        CERROR("%s: missed desired record? %u > %u\n",
                               o->do_lu.lo_dev->ld_obd->obd_name,
                               rec->lrh_index, next_idx);
                        GOTO(out, rc = -ENOENT);
                }
+
+               /* Trim unsupported extensions for compat w/ older clients */
+               changelog_block_trim_ext(rec, last_rec, loghandle);
+
                GOTO(out, rc = 0);
+
+retry:
+               /* Note: because there are some pad records in the
+                * llog, so llog_skip_over() might skip too much
+                * records, let's try skip again with minimum record */
+               force_mini_rec = true;
+               *cur_offset = last_offset;
+               *cur_idx = last_idx;
        }
        GOTO(out, rc = -EIO);
 out:
        return rc;
 }
 
+/**
+ * Implementation of the llog_operations::lop_prev_block
+ *
+ * This function finds the llog block to return which contains
+ * record with required index but in reverse order - from end of llog
+ * to the beginning.
+ * It is main part of reverse llog processing.
+ *
+ * \param[in] env      execution environment
+ * \param[in] loghandle        llog handle of the current llog
+ * \param[in] prev_idx target index to find
+ * \param[in] buf      pointer to data buffer to fill
+ * \param[in] len      required len to read, it is llog_chunk_size usually.
+ *
+ * \retval             0 on successful buffer read
+ * \retval             negative value on error
+ */
 static int llog_osd_prev_block(const struct lu_env *env,
                               struct llog_handle *loghandle,
                               int prev_idx, void *buf, int len)
@@ -632,11 +1079,13 @@ static int llog_osd_prev_block(const struct lu_env *env,
        struct dt_object        *o;
        struct dt_device        *dt;
        loff_t                   cur_offset;
+       __u32                   chunk_size;
        int                      rc;
 
        ENTRY;
 
-       if (len == 0 || len & (LLOG_CHUNK_SIZE - 1))
+       chunk_size = loghandle->lgh_hdr->llh_hdr.lrh_len;
+       if (len == 0 || len & (chunk_size - 1))
                RETURN(-EINVAL);
 
        CDEBUG(D_OTHER, "looking for log index %u\n", prev_idx);
@@ -646,14 +1095,17 @@ static int llog_osd_prev_block(const struct lu_env *env,
 
        o = loghandle->lgh_obj;
        LASSERT(o);
-       LASSERT(dt_object_exists(o));
+       LASSERT(llog_osd_exist(loghandle));
        dt = lu2dt_dev(o->do_lu.lo_dev);
        LASSERT(dt);
 
-       cur_offset = LLOG_CHUNK_SIZE;
-       llog_skip_over(&cur_offset, 0, prev_idx);
+       /* Let's only use mini record size for previous block read
+        * for now XXX */
+       cur_offset = chunk_size;
+       llog_skip_over(loghandle, &cur_offset, 0, prev_idx,
+                      chunk_size, true);
 
-       rc = dt_attr_get(env, o, &lgi->lgi_attr, BYPASS_CAPA);
+       rc = dt_attr_get(env, o, &lgi->lgi_attr);
        if (rc)
                GOTO(out, rc);
 
@@ -663,15 +1115,10 @@ static int llog_osd_prev_block(const struct lu_env *env,
 
                lgi->lgi_buf.lb_len = len;
                lgi->lgi_buf.lb_buf = buf;
-               /* It is OK to have locking around dt_read() only, see
-                * comment in llog_osd_next_block for details
-                */
-               dt_read_lock(env, o, 0);
                rc = dt_read(env, o, &lgi->lgi_buf, &cur_offset);
-               dt_read_unlock(env, o);
                if (rc < 0) {
                        CERROR("%s: can't read llog block from log "DFID
-                              " offset "LPU64": rc = %d\n",
+                              " offset %llu: rc = %d\n",
                               o->do_lu.lo_dev->ld_obd->obd_name,
                               PFID(lu_object_fid(&o->do_lu)), cur_offset, rc);
                        GOTO(out, rc);
@@ -681,10 +1128,10 @@ static int llog_osd_prev_block(const struct lu_env *env,
                        GOTO(out, rc);
 
                if (rc < sizeof(*tail)) {
-                       CERROR("%s: invalid llog block at log id "LPU64"/%u "
-                              "offset "LPU64"\n",
+                       CERROR("%s: invalid llog block at log id "DFID":%x "
+                              "offset %llu\n",
                               o->do_lu.lo_dev->ld_obd->obd_name,
-                              loghandle->lgh_id.lgl_oid,
+                              PFID(&loghandle->lgh_id.lgl_oi.oi_fid),
                               loghandle->lgh_id.lgl_ogen, cur_offset);
                        GOTO(out, rc = -EINVAL);
                }
@@ -705,10 +1152,10 @@ static int llog_osd_prev_block(const struct lu_env *env,
 
                /* this shouldn't happen */
                if (tail->lrt_index == 0) {
-                       CERROR("%s: invalid llog tail at log id "LPU64"/%u "
-                              "offset "LPU64"\n",
+                       CERROR("%s: invalid llog tail at log id "DFID":%x "
+                              "offset %llu\n",
                               o->do_lu.lo_dev->ld_obd->obd_name,
-                              loghandle->lgh_id.lgl_oid,
+                              PFID(&loghandle->lgh_id.lgl_oi.oi_fid),
                               loghandle->lgh_id.lgl_ogen, cur_offset);
                        GOTO(out, rc = -EINVAL);
                }
@@ -723,6 +1170,10 @@ static int llog_osd_prev_block(const struct lu_env *env,
                               rec->lrh_index, prev_idx);
                        GOTO(out, rc = -ENOENT);
                }
+
+               /* Trim unsupported extensions for compat w/ older clients */
+               changelog_block_trim_ext(rec, last_rec, loghandle);
+
                GOTO(out, rc = 0);
        }
        GOTO(out, rc = -EIO);
@@ -730,8 +1181,18 @@ out:
        return rc;
 }
 
-struct dt_object *llog_osd_dir_get(const struct lu_env *env,
-                                  struct llog_ctxt *ctxt)
+/**
+ * This is helper function to get llog directory object. It is used by named
+ * llog operations to find/insert/delete llog entry from llog directory.
+ *
+ * \param[in] env      execution environment
+ * \param[in] ctxt     llog context
+ *
+ * \retval             dt_object of llog directory
+ * \retval             ERR_PTR of negative value on error
+ */
+static struct dt_object *llog_osd_dir_get(const struct lu_env *env,
+                                         struct llog_ctxt *ctxt)
 {
        struct dt_device        *dt;
        struct dt_thread_info   *dti = dt_info(env);
@@ -744,6 +1205,11 @@ struct dt_object *llog_osd_dir_get(const struct lu_env *env,
                if (rc)
                        return ERR_PTR(rc);
                dir = dt_locate(env, dt, &dti->dti_fid);
+
+               if (!IS_ERR(dir) && !dt_try_as_dir(env, dir)) {
+                       dt_object_put(env, dir);
+                       return ERR_PTR(-ENOTDIR);
+               }
        } else {
                lu_object_get(&ctxt->loc_dir->do_lu);
                dir = ctxt->loc_dir;
@@ -752,6 +1218,27 @@ struct dt_object *llog_osd_dir_get(const struct lu_env *env,
        return dir;
 }
 
+/**
+ * Implementation of the llog_operations::lop_open
+ *
+ * This function opens the llog by its logid or by name, it may open also
+ * non existent llog and assing then new id to it.
+ * The llog_open/llog_close pair works similar to lu_object_find/put,
+ * the object may not exist prior open. The result of open is just dt_object
+ * in the llog header.
+ *
+ * \param[in] env              execution environment
+ * \param[in] handle           llog handle of the current llog
+ * \param[in] logid            logid of llog to open (nameless llog)
+ * \param[in] name             name of llog to open (named llog)
+ * \param[in] open_param
+ *                             LLOG_OPEN_NEW - new llog, may not exist
+ *                             LLOG_OPEN_EXIST - old llog, must exist
+ *
+ * \retval                     0 on successful open, llog_handle::lgh_obj
+ *                             contains the dt_object of the llog.
+ * \retval                     negative value on error
+ */
 static int llog_osd_open(const struct lu_env *env, struct llog_handle *handle,
                         struct llog_logid *logid, char *name,
                         enum llog_open_param open_param)
@@ -761,8 +1248,9 @@ static int llog_osd_open(const struct lu_env *env, struct llog_handle *handle,
        struct dt_object                *o;
        struct dt_device                *dt;
        struct ls_device                *ls;
-       struct local_oid_storage        *los;
+       struct local_oid_storage        *los = NULL;
        int                              rc = 0;
+       bool new_id = false;
 
        ENTRY;
 
@@ -772,13 +1260,34 @@ static int llog_osd_open(const struct lu_env *env, struct llog_handle *handle,
        LASSERT(ctxt->loc_exp->exp_obd);
        dt = ctxt->loc_exp->exp_obd->obd_lvfs_ctxt.dt;
        LASSERT(dt);
+       if (ctxt->loc_flags & LLOG_CTXT_FLAG_NORMAL_FID) {
+               struct lu_object_conf conf = { 0 };
+               if (logid != NULL) {
+                       logid_to_fid(logid, &lgi->lgi_fid);
+               } else {
+                       /* If logid == NULL, then it means the caller needs
+                        * to allocate new FID (llog_cat_declare_add_rec()). */
+                       rc = dt_fid_alloc(env, dt, &lgi->lgi_fid, NULL, NULL);
+                       if (rc < 0)
+                               RETURN(rc);
+                       rc = 0;
+                       conf.loc_flags = LOC_F_NEW;
+               }
+
+               o = dt_locate_at(env, dt, &lgi->lgi_fid,
+                                dt->dd_lu_dev.ld_site->ls_top_dev, &conf);
+               if (IS_ERR(o))
+                       RETURN(PTR_ERR(o));
+
+               goto after_open;
+       }
 
        ls = ls_device_get(dt);
        if (IS_ERR(ls))
                RETURN(PTR_ERR(ls));
 
        mutex_lock(&ls->ls_los_mutex);
-       los = dt_los_find(ls, FID_SEQ_LLOG);
+       los = dt_los_find(ls, name != NULL ? FID_SEQ_LLOG_NAME : FID_SEQ_LLOG);
        mutex_unlock(&ls->ls_los_mutex);
        LASSERT(los);
        ls_device_put(env, ls);
@@ -796,11 +1305,12 @@ static int llog_osd_open(const struct lu_env *env, struct llog_handle *handle,
                dt_read_lock(env, llog_dir, 0);
                rc = dt_lookup_dir(env, llog_dir, name, &lgi->lgi_fid);
                dt_read_unlock(env, llog_dir);
-               lu_object_put(env, &llog_dir->do_lu);
+               dt_object_put(env, llog_dir);
                if (rc == -ENOENT && open_param == LLOG_OPEN_NEW) {
                        /* generate fid for new llog */
                        rc = local_object_fid_generate(env, los,
                                                       &lgi->lgi_fid);
+                       new_id = true;
                }
                if (rc < 0)
                        GOTO(out, rc);
@@ -812,19 +1322,43 @@ static int llog_osd_open(const struct lu_env *env, struct llog_handle *handle,
        } else {
                LASSERTF(open_param & LLOG_OPEN_NEW, "%#x\n", open_param);
                /* generate fid for new llog */
+generate:
                rc = local_object_fid_generate(env, los, &lgi->lgi_fid);
                if (rc < 0)
                        GOTO(out, rc);
+               new_id = true;
        }
-
-       o = ls_locate(env, ls, &lgi->lgi_fid);
+       if (OBD_FAIL_PRECHECK(OBD_FAIL_MDS_LLOG_UMOUNT_RACE) &&
+           cfs_fail_val == 1) {
+               cfs_fail_val = 2;
+               OBD_RACE(OBD_FAIL_MDS_LLOG_UMOUNT_RACE);
+               msleep(MSEC_PER_SEC);
+       }
+       o = ls_locate(env, ls, &lgi->lgi_fid, NULL);
        if (IS_ERR(o))
                GOTO(out_name, rc = PTR_ERR(o));
 
+       if (dt_object_exists(o) && new_id) {
+               /* llog exists with just generated ID, e.g. some old llog file
+                * still is in use or is orphan, drop a warn and skip it. */
+               CDEBUG(D_INFO, "%s: llog exists with the same FID: "DFID
+                      ", skipping\n",
+                      o->do_lu.lo_dev->ld_obd->obd_name,
+                      PFID(lu_object_fid(&o->do_lu)));
+               dt_object_put(env, o);
+               /* just skip this llog ID, we shouldn't delete it because we
+                * don't know exactly what is its purpose and state. */
+               goto generate;
+       }
+
+after_open:
        /* No new llog is expected but doesn't exist */
-       if (open_param != LLOG_OPEN_NEW && !dt_object_exists(o))
+       if (open_param != LLOG_OPEN_NEW && !dt_object_exists(o)) {
+               CDEBUG(D_INFO, "%s: llog FID: "DFID" obj %p doesn`t exist\n",
+                      o->do_lu.lo_dev->ld_obd->obd_name,
+                      PFID(lu_object_fid(&o->do_lu)), o);
                GOTO(out_put, rc = -ENOENT);
-
+       }
        fid_to_logid(&lgi->lgi_fid, &handle->lgh_id);
        handle->lgh_obj = o;
        handle->private_data = los;
@@ -833,26 +1367,131 @@ static int llog_osd_open(const struct lu_env *env, struct llog_handle *handle,
        RETURN(rc);
 
 out_put:
-       lu_object_put(env, &o->do_lu);
+       dt_object_put(env, o);
 out_name:
        if (handle->lgh_name != NULL)
                OBD_FREE(handle->lgh_name, strlen(name) + 1);
 out:
-       dt_los_put(los);
+       if (los != NULL)
+               dt_los_put(los);
        RETURN(rc);
 }
 
-static int llog_osd_exist(struct llog_handle *handle)
+/**
+ * Get dir for regular fid log object
+ *
+ * Get directory for regular fid log object, and these regular fid log
+ * object will be inserted under this directory, to satisfy the FS
+ * consistency check, e2fsck etc.
+ *
+ * \param [in] env     execution environment
+ * \param [in] dto     llog object
+ *
+ * \retval             pointer to the directory if it is found.
+ * \retval             ERR_PTR(negative errno) if it fails.
+ */
+struct dt_object *llog_osd_get_regular_fid_dir(const struct lu_env *env,
+                                              struct dt_object *dto)
 {
-       LASSERT(handle->lgh_obj);
-       return (dt_object_exists(handle->lgh_obj) &&
-               !lu_object_is_dying(handle->lgh_obj->do_lu.lo_header));
+       struct llog_thread_info *lgi = llog_info(env);
+       struct seq_server_site *ss = dto->do_lu.lo_dev->ld_site->ld_seq_site;
+       struct lu_seq_range     *range = &lgi->lgi_range;
+       struct lu_fid           *dir_fid = &lgi->lgi_fid;
+       struct dt_object        *dir;
+       int                     rc;
+       ENTRY;
+
+       fld_range_set_any(range);
+       LASSERT(ss != NULL);
+       rc = ss->ss_server_fld->lsf_seq_lookup(env, ss->ss_server_fld,
+                                  fid_seq(lu_object_fid(&dto->do_lu)), range);
+       if (rc < 0)
+               RETURN(ERR_PTR(rc));
+
+       lu_update_log_dir_fid(dir_fid, range->lsr_index);
+       dir = dt_locate(env, lu2dt_dev(dto->do_lu.lo_dev), dir_fid);
+       if (IS_ERR(dir))
+               RETURN(dir);
+
+       if (!dt_try_as_dir(env, dir)) {
+               dt_object_put(env, dir);
+               RETURN(ERR_PTR(-ENOTDIR));
+       }
+
+       RETURN(dir);
 }
 
+/**
+ * Add llog object with regular FID to name entry
+ *
+ * Add llog object with regular FID to name space, and each llog
+ * object on each MDT will be /update_log_dir/[seq:oid:ver],
+ * so to satisfy the namespace consistency check, e2fsck etc.
+ *
+ * \param [in] env     execution environment
+ * \param [in] dto     llog object
+ * \param [in] th      thandle
+ * \param [in] declare if it is declare or execution
+ *
+ * \retval             0 if insertion succeeds.
+ * \retval             negative errno if insertion fails.
+ */
+static int
+llog_osd_regular_fid_add_name_entry(const struct lu_env *env,
+                                   struct dt_object *dto,
+                                   struct thandle *th, bool declare)
+{
+       struct llog_thread_info *lgi = llog_info(env);
+       const struct lu_fid     *fid = lu_object_fid(&dto->do_lu);
+       struct dt_insert_rec    *rec = &lgi->lgi_dt_rec;
+       struct dt_object        *dir;
+       char                    *name = lgi->lgi_name;
+       int                     rc;
+       ENTRY;
+
+       if (!fid_is_norm(fid))
+               RETURN(0);
+
+       dir = llog_osd_get_regular_fid_dir(env, dto);
+       if (IS_ERR(dir))
+               RETURN(PTR_ERR(dir));
+
+       rec->rec_fid = fid;
+       rec->rec_type = S_IFREG;
+       snprintf(name, sizeof(lgi->lgi_name), DFID, PFID(fid));
+       dt_write_lock(env, dir, 0);
+       if (declare) {
+               rc = dt_declare_insert(env, dir, (struct dt_rec *)rec,
+                              (struct dt_key *)name, th);
+       } else {
+               rc = dt_insert(env, dir, (struct dt_rec *)rec,
+                              (struct dt_key *)name, th);
+       }
+       dt_write_unlock(env, dir);
+
+       dt_object_put(env, dir);
+       RETURN(rc);
+}
+
+
+/**
+ * Implementation of the llog_operations::lop_declare_create
+ *
+ * This function declares the llog create. It declares also name insert
+ * into llog directory in case of named llog.
+ *
+ * \param[in] env      execution environment
+ * \param[in] res      llog handle of the current llog
+ * \param[in] th       current transaction handle
+ *
+ * \retval             0 on successful create declaration
+ * \retval             negative value on error
+ */
 static int llog_osd_declare_create(const struct lu_env *env,
                                   struct llog_handle *res, struct thandle *th)
 {
        struct llog_thread_info         *lgi = llog_info(env);
+       struct dt_insert_rec            *rec = &lgi->lgi_dt_rec;
        struct local_oid_storage        *los;
        struct dt_object                *o;
        int                              rc;
@@ -867,6 +1506,24 @@ static int llog_osd_declare_create(const struct lu_env *env,
        if (dt_object_exists(o))
                RETURN(0);
 
+       if (res->lgh_ctxt->loc_flags & LLOG_CTXT_FLAG_NORMAL_FID) {
+               struct llog_thread_info *lgi = llog_info(env);
+
+               lgi->lgi_attr.la_valid = LA_MODE | LA_SIZE;
+               lgi->lgi_attr.la_size = 0;
+               lgi->lgi_attr.la_mode = S_IFREG | S_IRUGO | S_IWUSR;
+               lgi->lgi_dof.dof_type = dt_mode_to_dft(S_IFREG);
+
+               rc = dt_declare_create(env, o, &lgi->lgi_attr, NULL,
+                                      &lgi->lgi_dof, th);
+               if (rc < 0)
+                       RETURN(rc);
+
+
+               rc = llog_osd_regular_fid_add_name_entry(env, o, th, true);
+
+               RETURN(rc);
+       }
        los = res->private_data;
        LASSERT(los);
 
@@ -874,9 +1531,8 @@ static int llog_osd_declare_create(const struct lu_env *env,
        if (rc)
                RETURN(rc);
 
-       rc = dt_declare_record_write(env, o, LLOG_CHUNK_SIZE, 0, th);
-       if (rc)
-               RETURN(rc);
+       /* do not declare header initialization here as it's declared
+        * in llog_osd_declare_write_rec() which is always called */
 
        if (res->lgh_name) {
                struct dt_object *llog_dir;
@@ -884,12 +1540,13 @@ static int llog_osd_declare_create(const struct lu_env *env,
                llog_dir = llog_osd_dir_get(env, res->lgh_ctxt);
                if (IS_ERR(llog_dir))
                        RETURN(PTR_ERR(llog_dir));
-               dt_declare_ref_add(env, o, th);
                logid_to_fid(&res->lgh_id, &lgi->lgi_fid);
+               rec->rec_fid = &lgi->lgi_fid;
+               rec->rec_type = S_IFREG;
                rc = dt_declare_insert(env, llog_dir,
-                                      (struct dt_rec *)&lgi->lgi_fid,
+                                      (struct dt_rec *)rec,
                                       (struct dt_key *)res->lgh_name, th);
-               lu_object_put(env, &llog_dir->do_lu);
+               dt_object_put(env, llog_dir);
                if (rc)
                        CERROR("%s: can't declare named llog %s: rc = %d\n",
                               o->do_lu.lo_dev->ld_obd->obd_name,
@@ -898,12 +1555,24 @@ static int llog_osd_declare_create(const struct lu_env *env,
        RETURN(rc);
 }
 
-/* This is a callback from the llog_* functions.
- * Assumes caller has already pushed us into the kernel context. */
+/**
+ * Implementation of the llog_operations::lop_create
+ *
+ * This function creates the llog according with llog_handle::lgh_obj
+ * and llog_handle::lgh_name.
+ *
+ * \param[in] env      execution environment
+ * \param[in] res      llog handle of the current llog
+ * \param[in] th       current transaction handle
+ *
+ * \retval             0 on successful create
+ * \retval             negative value on error
+ */
 static int llog_osd_create(const struct lu_env *env, struct llog_handle *res,
                           struct thandle *th)
 {
        struct llog_thread_info *lgi = llog_info(env);
+       struct dt_insert_rec    *rec = &lgi->lgi_dt_rec;
        struct local_oid_storage *los;
        struct dt_object        *o;
        int                      rc = 0;
@@ -918,6 +1587,26 @@ static int llog_osd_create(const struct lu_env *env, struct llog_handle *res,
        if (dt_object_exists(o))
                RETURN(-EEXIST);
 
+       if (res->lgh_ctxt->loc_flags & LLOG_CTXT_FLAG_NORMAL_FID) {
+               struct llog_thread_info *lgi = llog_info(env);
+
+               lgi->lgi_attr.la_valid = LA_MODE | LA_SIZE | LA_TYPE;
+               lgi->lgi_attr.la_size = 0;
+               lgi->lgi_attr.la_mode = S_IFREG | S_IRUGO | S_IWUSR;
+               lgi->lgi_dof.dof_type = dt_mode_to_dft(S_IFREG);
+
+               dt_write_lock(env, o, 0);
+               rc = dt_create(env, o, &lgi->lgi_attr, NULL,
+                              &lgi->lgi_dof, th);
+               dt_write_unlock(env, o);
+               if (rc < 0)
+                       RETURN(rc);
+
+               rc = llog_osd_regular_fid_add_name_entry(env, o, th, false);
+
+               RETURN(rc);
+       }
+
        los = res->private_data;
        LASSERT(los);
 
@@ -926,8 +1615,7 @@ static int llog_osd_create(const struct lu_env *env, struct llog_handle *res,
                rc = llog_osd_create_new_object(env, los, o, th);
        else
                rc = -EEXIST;
-       if (res->lgh_name)
-               dt_ref_add(env, o, th);
+
        dt_write_unlock(env, o);
        if (rc)
                RETURN(rc);
@@ -940,13 +1628,13 @@ static int llog_osd_create(const struct lu_env *env, struct llog_handle *res,
                        RETURN(PTR_ERR(llog_dir));
 
                logid_to_fid(&res->lgh_id, &lgi->lgi_fid);
+               rec->rec_fid = &lgi->lgi_fid;
+               rec->rec_type = S_IFREG;
                dt_read_lock(env, llog_dir, 0);
-               rc = dt_insert(env, llog_dir,
-                              (struct dt_rec *)&lgi->lgi_fid,
-                              (struct dt_key *)res->lgh_name,
-                              th, BYPASS_CAPA, 1);
+               rc = dt_insert(env, llog_dir, (struct dt_rec *)rec,
+                              (struct dt_key *)res->lgh_name, th);
                dt_read_unlock(env, llog_dir);
-               lu_object_put(env, &llog_dir->do_lu);
+               dt_object_put(env, llog_dir);
                if (rc)
                        CERROR("%s: can't create named llog %s: rc = %d\n",
                               o->do_lu.lo_dev->ld_obd->obd_name,
@@ -955,6 +1643,18 @@ static int llog_osd_create(const struct lu_env *env, struct llog_handle *res,
        RETURN(rc);
 }
 
+/**
+ * Implementation of the llog_operations::lop_close
+ *
+ * This function closes the llog. It just put llog object and referenced
+ * local storage.
+ *
+ * \param[in] env      execution environment
+ * \param[in] handle   llog handle of the current llog
+ *
+ * \retval             0 on successful llog close
+ * \retval             negative value on error
+ */
 static int llog_osd_close(const struct lu_env *env, struct llog_handle *handle)
 {
        struct local_oid_storage        *los;
@@ -964,8 +1664,15 @@ static int llog_osd_close(const struct lu_env *env, struct llog_handle *handle)
 
        LASSERT(handle->lgh_obj);
 
-       lu_object_put(env, &handle->lgh_obj->do_lu);
-
+       if (handle->lgh_ctxt->loc_flags & LLOG_CTXT_FLAG_NORMAL_FID) {
+               /* Remove the object from the cache, otherwise it may
+                * hold LOD being released during cleanup process */
+               dt_object_put_nocache(env, handle->lgh_obj);
+               LASSERT(handle->private_data == NULL);
+               RETURN(rc);
+       } else {
+               dt_object_put(env, handle->lgh_obj);
+       }
        los = handle->private_data;
        LASSERT(los);
        dt_los_put(los);
@@ -976,15 +1683,71 @@ static int llog_osd_close(const struct lu_env *env, struct llog_handle *handle)
        RETURN(rc);
 }
 
-static int llog_osd_destroy(const struct lu_env *env,
-                           struct llog_handle *loghandle)
+/**
+ * delete llog object name entry
+ *
+ * Delete llog object (with regular FID) from name space (under
+ * update_log_dir).
+ *
+ * \param [in] env     execution environment
+ * \param [in] dto     llog object
+ * \param [in] th      thandle
+ * \param [in] declare if it is declare or execution
+ *
+ * \retval             0 if deletion succeeds.
+ * \retval             negative errno if deletion fails.
+ */
+static int
+llog_osd_regular_fid_del_name_entry(const struct lu_env *env,
+                                   struct dt_object *dto,
+                                   struct thandle *th, bool declare)
 {
        struct llog_thread_info *lgi = llog_info(env);
+       const struct lu_fid     *fid = lu_object_fid(&dto->do_lu);
+       struct dt_object        *dir;
+       char                    *name = lgi->lgi_name;
+       int                     rc;
+       ENTRY;
+
+       if (!fid_is_norm(fid))
+               RETURN(0);
+
+       dir = llog_osd_get_regular_fid_dir(env, dto);
+       if (IS_ERR(dir))
+               RETURN(PTR_ERR(dir));
+
+       snprintf(name, sizeof(lgi->lgi_name), DFID, PFID(fid));
+       dt_write_lock(env, dir, 0);
+       if (declare) {
+               rc = dt_declare_delete(env, dir, (struct dt_key *)name,
+                                      th);
+       } else {
+               rc = dt_delete(env, dir, (struct dt_key *)name, th);
+       }
+       dt_write_unlock(env, dir);
+
+       dt_object_put(env, dir);
+       RETURN(rc);
+}
+
+/**
+ * Implementation of the llog_operations::lop_declare_destroy
+ *
+ * This function declare destroys the llog and deletes also entry in the
+ * llog directory in case of named llog. Llog should be opened prior that.
+ *
+ * \param[in] env              execution environment
+ * \param[in] loghandle        llog handle of the current llog
+ *
+ * \retval             0 on successful destroy
+ * \retval             negative value on error
+ */
+static int llog_osd_declare_destroy(const struct lu_env *env,
+                                   struct llog_handle *loghandle,
+                                   struct thandle *th)
+{
        struct llog_ctxt        *ctxt;
        struct dt_object        *o, *llog_dir = NULL;
-       struct dt_device        *d;
-       struct thandle          *th;
-       char                    *name = NULL;
        int                      rc;
 
        ENTRY;
@@ -995,87 +1758,132 @@ static int llog_osd_destroy(const struct lu_env *env,
        o = loghandle->lgh_obj;
        LASSERT(o);
 
-       d = lu2dt_dev(o->do_lu.lo_dev);
-       LASSERT(d);
-       LASSERT(d == ctxt->loc_exp->exp_obd->obd_lvfs_ctxt.dt);
-
-       th = dt_trans_create(env, d);
-       if (IS_ERR(th))
-               RETURN(PTR_ERR(th));
-
        if (loghandle->lgh_name) {
                llog_dir = llog_osd_dir_get(env, ctxt);
                if (IS_ERR(llog_dir))
-                       GOTO(out_trans, rc = PTR_ERR(llog_dir));
+                       RETURN(PTR_ERR(llog_dir));
 
-               dt_declare_ref_del(env, o, th);
-               name = loghandle->lgh_name;
                rc = dt_declare_delete(env, llog_dir,
-                                      (struct dt_key *)name, th);
-               if (rc)
-                       GOTO(out_trans, rc);
+                                      (struct dt_key *)loghandle->lgh_name,
+                                      th);
+               if (rc < 0)
+                       GOTO(out_put, rc);
        }
 
-       dt_declare_ref_del(env, o, th);
+       rc = dt_declare_ref_del(env, o, th);
+       if (rc < 0)
+               GOTO(out_put, rc);
 
        rc = dt_declare_destroy(env, o, th);
-       if (rc)
-               GOTO(out_trans, rc);
+       if (rc < 0)
+               GOTO(out_put, rc);
 
-       rc = dt_trans_start_local(env, d, th);
-       if (rc)
-               GOTO(out_trans, rc);
+       if (loghandle->lgh_ctxt->loc_flags & LLOG_CTXT_FLAG_NORMAL_FID) {
+               rc = llog_osd_regular_fid_del_name_entry(env, o, th, true);
+               if (rc < 0)
+                       GOTO(out_put, rc);
+       }
+
+out_put:
+       if (!(IS_ERR_OR_NULL(llog_dir)))
+               dt_object_put(env, llog_dir);
+
+       RETURN(rc);
+}
+
+
+/**
+ * Implementation of the llog_operations::lop_destroy
+ *
+ * This function destroys the llog and deletes also entry in the
+ * llog directory in case of named llog. Llog should be opened prior that.
+ * Destroy method is not part of external transaction and does everything
+ * inside.
+ *
+ * \param[in] env              execution environment
+ * \param[in] loghandle        llog handle of the current llog
+ *
+ * \retval             0 on successful destroy
+ * \retval             negative value on error
+ */
+static int llog_osd_destroy(const struct lu_env *env,
+                           struct llog_handle *loghandle, struct thandle *th)
+{
+       struct llog_ctxt        *ctxt;
+       struct dt_object        *o, *llog_dir = NULL;
+       int                      rc;
+
+       ENTRY;
+
+       ctxt = loghandle->lgh_ctxt;
+       LASSERT(ctxt != NULL);
+
+       o = loghandle->lgh_obj;
+       LASSERT(o != NULL);
 
        dt_write_lock(env, o, 0);
-       if (dt_object_exists(o)) {
-               if (name) {
-                       dt_ref_del(env, o, th);
-                       dt_read_lock(env, llog_dir, 0);
-                       rc = dt_delete(env, llog_dir,
-                                      (struct dt_key *) name,
-                                      th, BYPASS_CAPA);
-                       dt_read_unlock(env, llog_dir);
-                       if (rc) {
-                               CERROR("%s: can't remove llog %s: rc = %d\n",
-                                      o->do_lu.lo_dev->ld_obd->obd_name,
-                                      name, rc);
-                               GOTO(out_unlock, rc);
-                       }
-               }
-               /*
-                * XXX: compatibility bits
-                *      on old filesystems llogs are referenced by the name
-                *      on the new ones they are referenced by OI and by
-                *      the name
-                */
-               rc = dt_attr_get(env, o, &lgi->lgi_attr, NULL);
-               if (rc)
+       if (!llog_osd_exist(loghandle))
+               GOTO(out_unlock, rc = 0);
+
+       if (loghandle->lgh_name) {
+               llog_dir = llog_osd_dir_get(env, ctxt);
+               if (IS_ERR(llog_dir))
+                       GOTO(out_unlock, rc = PTR_ERR(llog_dir));
+
+               dt_read_lock(env, llog_dir, 0);
+               rc = dt_delete(env, llog_dir,
+                              (struct dt_key *)loghandle->lgh_name,
+                              th);
+               dt_read_unlock(env, llog_dir);
+               if (rc) {
+                       CERROR("%s: can't remove llog %s: rc = %d\n",
+                              o->do_lu.lo_dev->ld_obd->obd_name,
+                              loghandle->lgh_name, rc);
                        GOTO(out_unlock, rc);
-               LASSERT(lgi->lgi_attr.la_nlink < 2);
-               if (lgi->lgi_attr.la_nlink == 1)
-                       dt_ref_del(env, o, th);
-               rc = dt_destroy(env, o, th);
-               if (rc)
+               }
+       }
+
+       dt_ref_del(env, o, th);
+       rc = dt_destroy(env, o, th);
+       if (rc < 0)
+               GOTO(out_unlock, rc);
+
+       loghandle->lgh_destroyed = true;
+       if (loghandle->lgh_ctxt->loc_flags & LLOG_CTXT_FLAG_NORMAL_FID) {
+               rc = llog_osd_regular_fid_del_name_entry(env, o, th, false);
+               if (rc < 0)
                        GOTO(out_unlock, rc);
        }
+
 out_unlock:
        dt_write_unlock(env, o);
-out_trans:
-       dt_trans_stop(env, d, th);
-       if (llog_dir != NULL)
-               lu_object_put(env, &llog_dir->do_lu);
+       if (!(IS_ERR_OR_NULL(llog_dir)))
+               dt_object_put(env, llog_dir);
        RETURN(rc);
 }
 
+/**
+ * Implementation of the llog_operations::lop_setup
+ *
+ * This function setup the llog on local storage.
+ *
+ * \param[in] env      execution environment
+ * \param[in] obd      obd device the llog belongs to
+ * \param[in] olg      the llog group, it is always zero group now.
+ * \param[in] ctxt_idx the llog index, it defines the purpose of this llog.
+ *                     Every new llog type have to use own index.
+ * \param[in] disk_obd the storage obd, where llog is stored.
+ *
+ * \retval             0 on successful llog setup
+ * \retval             negative value on error
+ */
 static int llog_osd_setup(const struct lu_env *env, struct obd_device *obd,
                          struct obd_llog_group *olg, int ctxt_idx,
                          struct obd_device *disk_obd)
 {
-       struct local_oid_storage        *los;
        struct llog_thread_info         *lgi = llog_info(env);
        struct llog_ctxt                *ctxt;
        int                              rc = 0;
-
        ENTRY;
 
        LASSERT(obd);
@@ -1084,37 +1892,60 @@ static int llog_osd_setup(const struct lu_env *env, struct obd_device *obd,
        ctxt = llog_ctxt_get(olg->olg_ctxts[ctxt_idx]);
        LASSERT(ctxt);
 
+       if (disk_obd == NULL)
+               GOTO(out, rc = 0);
+
        /* initialize data allowing to generate new fids,
         * literally we need a sequece */
        lgi->lgi_fid.f_seq = FID_SEQ_LLOG;
        lgi->lgi_fid.f_oid = 1;
        lgi->lgi_fid.f_ver = 0;
        rc = local_oid_storage_init(env, disk_obd->obd_lvfs_ctxt.dt,
-                                   &lgi->lgi_fid, &los);
+                                   &lgi->lgi_fid,
+                                   &ctxt->loc_los_nameless);
+       if (rc != 0)
+               GOTO(out, rc);
+
+       lgi->lgi_fid.f_seq = FID_SEQ_LLOG_NAME;
+       lgi->lgi_fid.f_oid = 1;
+       lgi->lgi_fid.f_ver = 0;
+       rc = local_oid_storage_init(env, disk_obd->obd_lvfs_ctxt.dt,
+                                   &lgi->lgi_fid,
+                                   &ctxt->loc_los_named);
+       if (rc != 0) {
+               local_oid_storage_fini(env, ctxt->loc_los_nameless);
+               ctxt->loc_los_nameless = NULL;
+       }
+
+       GOTO(out, rc);
+
+out:
        llog_ctxt_put(ctxt);
        return rc;
 }
 
+/**
+ * Implementation of the llog_operations::lop_cleanup
+ *
+ * This function cleanups the llog on local storage.
+ *
+ * \param[in] env      execution environment
+ * \param[in] ctxt     the llog context
+ *
+ * \retval             0
+ */
 static int llog_osd_cleanup(const struct lu_env *env, struct llog_ctxt *ctxt)
 {
-       struct dt_device                *dt;
-       struct ls_device                *ls;
-       struct local_oid_storage        *los;
-
-       LASSERT(ctxt->loc_exp->exp_obd);
-       dt = ctxt->loc_exp->exp_obd->obd_lvfs_ctxt.dt;
-       ls = ls_device_get(dt);
-       if (IS_ERR(ls))
-               RETURN(PTR_ERR(ls));
+       if (ctxt->loc_los_nameless != NULL) {
+               local_oid_storage_fini(env, ctxt->loc_los_nameless);
+               ctxt->loc_los_nameless = NULL;
+       }
 
-       mutex_lock(&ls->ls_los_mutex);
-       los = dt_los_find(ls, FID_SEQ_LLOG);
-       mutex_unlock(&ls->ls_los_mutex);
-       if (los != NULL) {
-               dt_los_put(los);
-               local_oid_storage_fini(env, los);
+       if (ctxt->loc_los_named != NULL) {
+               local_oid_storage_fini(env, ctxt->loc_los_named);
+               ctxt->loc_los_named = NULL;
        }
-       ls_device_put(env, ls);
+
        return 0;
 }
 
@@ -1122,6 +1953,7 @@ struct llog_operations llog_osd_ops = {
        .lop_next_block         = llog_osd_next_block,
        .lop_prev_block         = llog_osd_prev_block,
        .lop_read_header        = llog_osd_read_header,
+       .lop_declare_destroy    = llog_osd_declare_destroy,
        .lop_destroy            = llog_osd_destroy,
        .lop_setup              = llog_osd_setup,
        .lop_cleanup            = llog_osd_cleanup,
@@ -1135,9 +1967,50 @@ struct llog_operations llog_osd_ops = {
 };
 EXPORT_SYMBOL(llog_osd_ops);
 
-/* reads the catalog list */
+struct llog_operations llog_common_cat_ops = {
+       .lop_next_block         = llog_osd_next_block,
+       .lop_prev_block         = llog_osd_prev_block,
+       .lop_read_header        = llog_osd_read_header,
+       .lop_declare_destroy    = llog_osd_declare_destroy,
+       .lop_destroy            = llog_osd_destroy,
+       .lop_setup              = llog_osd_setup,
+       .lop_cleanup            = llog_osd_cleanup,
+       .lop_open               = llog_osd_open,
+       .lop_exist              = llog_osd_exist,
+       .lop_declare_create     = llog_osd_declare_create,
+       .lop_create             = llog_osd_create,
+       .lop_declare_write_rec  = llog_osd_declare_write_rec,
+       .lop_write_rec          = llog_osd_write_rec,
+       .lop_close              = llog_osd_close,
+       .lop_add                = llog_cat_add_rec,
+       .lop_declare_add        = llog_cat_declare_add_rec,
+};
+EXPORT_SYMBOL(llog_common_cat_ops);
+
+/**
+ * Read the special file which contains the list of llog catalogs IDs
+ *
+ * This function reads the CATALOGS file which contains the array of llog
+ * catalogs IDs. The main purpose of this file is to store OSP llogs indexed
+ * by OST/MDT number.
+ *
+ * \param[in]  env             execution environment
+ * \param[in]  d               corresponding storage device
+ * \param[in]  idx             position to start from, usually OST/MDT index
+ * \param[in]  count           how many catalog IDs to read
+ * \param[out] idarray         the buffer for the data. If it is NULL then
+ *                             function returns just number of catalog IDs
+ *                             in the file.
+ * \param[in]  fid             LLOG_CATALOGS_OID for CATALOG object
+ *
+ * \retval                     0 on successful read of catalog IDs
+ * \retval                     negative value on error
+ * \retval                     positive value which is number of records in
+ *                             the file if \a idarray is NULL
+ */
 int llog_osd_get_cat_list(const struct lu_env *env, struct dt_device *d,
-                         int idx, int count, struct llog_catid *idarray)
+                         int idx, int count, struct llog_catid *idarray,
+                         const struct lu_fid *fid)
 {
        struct llog_thread_info *lgi = llog_info(env);
        struct dt_object        *o = NULL;
@@ -1151,7 +2024,7 @@ int llog_osd_get_cat_list(const struct lu_env *env, struct dt_device *d,
        size = sizeof(*idarray) * count;
        lgi->lgi_off = idx *  sizeof(*idarray);
 
-       lu_local_obj_fid(&lgi->lgi_fid, LLOG_CATALOGS_OID);
+       lgi->lgi_fid = *fid;
        o = dt_locate(env, d, &lgi->lgi_fid);
        if (IS_ERR(o))
                RETURN(PTR_ERR(o));
@@ -1161,10 +2034,16 @@ int llog_osd_get_cat_list(const struct lu_env *env, struct dt_device *d,
                if (IS_ERR(th))
                        GOTO(out, rc = PTR_ERR(th));
 
-               lgi->lgi_attr.la_valid = LA_MODE;
+               lgi->lgi_attr.la_valid = LA_MODE | LA_TYPE;
                lgi->lgi_attr.la_mode = S_IFREG | S_IRUGO | S_IWUSR;
                lgi->lgi_dof.dof_type = dt_mode_to_dft(S_IFREG);
 
+               th->th_wait_submit = 1;
+               /* Make the llog object creation synchronization, so
+                * it will be reliable to the reference, especially
+                * for remote reference */
+               th->th_sync = 1;
+
                rc = dt_declare_create(env, o, &lgi->lgi_attr, NULL,
                                       &lgi->lgi_dof, th);
                if (rc)
@@ -1185,7 +2064,7 @@ out_trans:
                        GOTO(out, rc);
        }
 
-       rc = dt_attr_get(env, o, &lgi->lgi_attr, BYPASS_CAPA);
+       rc = dt_attr_get(env, o, &lgi->lgi_attr);
        if (rc)
                GOTO(out, rc);
 
@@ -1207,7 +2086,7 @@ out_trans:
 
        /* read for new ost index or for empty file */
        memset(idarray, 0, size);
-       if (lgi->lgi_attr.la_size < lgi->lgi_off + size)
+       if (lgi->lgi_attr.la_size <= lgi->lgi_off)
                GOTO(out, rc = 0);
        if (lgi->lgi_attr.la_size < lgi->lgi_off + size)
                size = lgi->lgi_attr.la_size - lgi->lgi_off;
@@ -1215,7 +2094,9 @@ out_trans:
        lgi->lgi_buf.lb_buf = idarray;
        lgi->lgi_buf.lb_len = size;
        rc = dt_record_read(env, o, &lgi->lgi_buf, &lgi->lgi_off);
-       if (rc) {
+       /* -EFAULT means the llog is a sparse file. This is not an error
+        * after arbitrary OST index is supported. */
+       if (rc < 0 && rc != -EFAULT) {
                CERROR("%s: error reading CATALOGS: rc = %d\n",
                       o->do_lu.lo_dev->ld_obd->obd_name,  rc);
                GOTO(out, rc);
@@ -1223,29 +2104,46 @@ out_trans:
 
        EXIT;
 out:
-       lu_object_put(env, &o->do_lu);
+       dt_object_put(env, o);
        RETURN(rc);
 }
 EXPORT_SYMBOL(llog_osd_get_cat_list);
 
-/* writes the cat list */
+/**
+ * Write the special file which contains the list of llog catalogs IDs
+ *
+ * This function writes the CATALOG file which contains the array of llog
+ * catalogs IDs. It is used mostly to store OSP llogs indexed by OST/MDT
+ * number.
+ *
+ * \param[in]  env     execution environment
+ * \param[in]  d       corresponding storage device
+ * \param[in]  idx     position to start from, usually OST/MDT index
+ * \param[in]  count   how many catalog IDs to write
+ * \param[out] idarray the buffer with the data to write.
+ * \param[in]  fid     LLOG_CATALOGS_OID for CATALOG object
+ *
+ * \retval             0 on successful write of catalog IDs
+ * \retval             negative value on error
+ */
 int llog_osd_put_cat_list(const struct lu_env *env, struct dt_device *d,
-                         int idx, int count, struct llog_catid *idarray)
+                         int idx, int count, struct llog_catid *idarray,
+                         const struct lu_fid *fid)
 {
        struct llog_thread_info *lgi = llog_info(env);
        struct dt_object        *o = NULL;
        struct thandle          *th;
        int                      rc, size;
 
-       if (!count)
+       if (count == 0)
                RETURN(0);
 
        LASSERT(d);
 
        size = sizeof(*idarray) * count;
        lgi->lgi_off = idx * sizeof(*idarray);
+       lgi->lgi_fid = *fid;
 
-       lu_local_obj_fid(&lgi->lgi_fid, LLOG_CATALOGS_OID);
        o = dt_locate(env, d, &lgi->lgi_fid);
        if (IS_ERR(o))
                RETURN(PTR_ERR(o));
@@ -1253,7 +2151,7 @@ int llog_osd_put_cat_list(const struct lu_env *env, struct dt_device *d,
        if (!dt_object_exists(o))
                GOTO(out, rc = -ENOENT);
 
-       rc = dt_attr_get(env, o, &lgi->lgi_attr, BYPASS_CAPA);
+       rc = dt_attr_get(env, o, &lgi->lgi_attr);
        if (rc)
                GOTO(out, rc);
 
@@ -1268,23 +2166,35 @@ int llog_osd_put_cat_list(const struct lu_env *env, struct dt_device *d,
        if (IS_ERR(th))
                GOTO(out, rc = PTR_ERR(th));
 
-       rc = dt_declare_record_write(env, o, size, lgi->lgi_off, th);
+       lgi->lgi_buf.lb_len = size;
+       lgi->lgi_buf.lb_buf = idarray;
+       rc = dt_declare_record_write(env, o, &lgi->lgi_buf, lgi->lgi_off, th);
        if (rc)
-               GOTO(out, rc);
+               GOTO(out_trans, rc);
+
+       /* For update log, this happens during initialization,
+        * see lod_sub_prep_llog(), and we need make sure catlog
+        * file ID is written to catlist file(committed) before
+        * cross-MDT operation write update records to catlog FILE,
+        * otherwise, during failover these update records might
+        * missing */
+       if (fid_is_update_log(fid))
+               th->th_sync = 1;
 
        rc = dt_trans_start_local(env, d, th);
        if (rc)
                GOTO(out_trans, rc);
 
-       lgi->lgi_buf.lb_buf = idarray;
-       lgi->lgi_buf.lb_len = size;
+       th->th_wait_submit = 1;
+
        rc = dt_record_write(env, o, &lgi->lgi_buf, &lgi->lgi_off, th);
        if (rc)
-               CDEBUG(D_INODE, "error writeing CATALOGS: rc = %d\n", rc);
+               CDEBUG(D_INODE, "can't write CATALOGS at index %d: rc = %d\n",
+                      idx, rc);
 out_trans:
        dt_trans_stop(env, d, th);
 out:
-       lu_object_put(env, &o->do_lu);
+       dt_object_put(env, o);
        RETURN(rc);
 }
 EXPORT_SYMBOL(llog_osd_put_cat_list);