LU-731 ldlm: revise and export ldlm_lock_set_data

[fs/lustre-release.git] / lustre / lov / lov_page.c

diff --git a/lustre/lov/lov_page.c b/lustre/lov/lov_page.c
index 4609de2..15b9033 100644 (file)
--- a/lustre/lov/lov_page.c
+++ b/lustre/lov/lov_page.c
@@ -26,8 +26,11 @@
  * GPL HEADER END
  */
 /*
- * Copyright 2008 Sun Microsystems, Inc.  All rights reserved.
+ * Copyright (c) 2008, 2010, Oracle and/or its affiliates. All rights reserved.
  * Use is subject to license terms.
+ *
+ * Copyright (c) 2011 Whamcloud, Inc.
+ *
  */
 /*
  * This file is part of Lustre, http://www.lustre.org/
@@ -140,61 +143,58 @@ struct cl_page *lov_page_init_raid0(const struct lu_env *env,
                                     struct cl_object *obj, struct cl_page *page,
                                     cfs_page_t *vmpage)
 {
-        struct lov_page   *lpg;
         struct lov_object *loo = cl2lov(obj);
-        int result;
-
+        struct lov_layout_raid0 *r0 = lov_r0(loo);
+        struct lov_io     *lio = lov_env_io(env);
+        struct cl_page    *subpage;
+        struct cl_object  *subobj;
+        struct lov_io_sub *sub;
+        struct lov_page   *lpg;
+        struct cl_page    *result;
+        loff_t             offset;
+        obd_off            suboff;
+        int                stripe;
+        int                rc;
         ENTRY;
+
+        offset = cl_offset(obj, page->cp_index);
+        stripe = lov_stripe_number(r0->lo_lsm, offset);
+        LASSERT(stripe < r0->lo_nr);
+        rc = lov_stripe_offset(r0->lo_lsm, offset, stripe,
+                                   &suboff);
+        LASSERT(rc == 0);
+
         OBD_SLAB_ALLOC_PTR_GFP(lpg, lov_page_kmem, CFS_ALLOC_IO);
-        if (lpg != NULL) {
-                loff_t   offset;
-                int      stripe;
-                obd_off  suboff;
-                struct cl_page          *subpage;
-                struct cl_object        *subobj;
-                struct lov_layout_raid0 *r0 = lov_r0(loo);
-
-                offset = cl_offset(obj, page->cp_index);
-                stripe = lov_stripe_number(r0->lo_lsm, offset);
-                result = lov_stripe_offset(r0->lo_lsm, offset, stripe,
-                                           &suboff);
-                LASSERT(stripe < r0->lo_nr);
-                LASSERT(result == 0);
-
-                subobj  = lovsub2cl(r0->lo_sub[stripe]);
-                subpage = cl_page_find(env, subobj,
-                                       cl_index(subobj, suboff), vmpage,
-                                       page->cp_type);
-                if (!IS_ERR(subpage)) {
-                        if (subpage->cp_parent != NULL) {
-                                /*
-                                 * This is only possible when TRANSIENT page
-                                 * is being created, and CACHEABLE sub-page
-                                 * (attached to already existing top-page) has
-                                 * been found. Tell cl_page_find() to use
-                                 * existing page.
-                                 */
-                                LASSERT(subpage->cp_type == CPT_CACHEABLE);
-                                LASSERT(page->cp_type == CPT_TRANSIENT);
-                                lpg->lps_invalid = 1;
-                                cl_page_put(env, subpage);
-                                /*
-                                 * XXX This assumes that lov is in the topmost
-                                 * cl_page.
-                                 */
-                                result = PTR_ERR(cl_page_top(subpage));
-                        } else {
-                                lu_ref_add(&subpage->cp_reference, "lov", page);
-                                subpage->cp_parent = page;
-                                page->cp_child = subpage;
-                        }
-                        cl_page_slice_add(page, &lpg->lps_cl,
-                                          obj, &lov_page_ops);
-                } else
-                        result = PTR_ERR(subpage);
-        } else
-                result = -ENOMEM;
-        RETURN(ERR_PTR(result));
+        if (lpg == NULL)
+                GOTO(out, result = ERR_PTR(-ENOMEM));
+
+        lpg->lps_invalid = 1;
+        cl_page_slice_add(page, &lpg->lps_cl, obj, &lov_page_ops);
+
+        sub = lov_sub_get(env, lio, stripe);
+        if (IS_ERR(sub))
+                GOTO(out, result = (struct cl_page *)sub);
+
+        subobj = lovsub2cl(r0->lo_sub[stripe]);
+        subpage = cl_page_find_sub(sub->sub_env, subobj,
+                                   cl_index(subobj, suboff), vmpage, page);
+        lov_sub_put(sub);
+        if (IS_ERR(subpage))
+                GOTO(out, result = subpage);
+
+        if (likely(subpage->cp_parent == page)) {
+                lu_ref_add(&subpage->cp_reference, "lov", page);
+                lpg->lps_invalid = 0;
+                result = NULL;
+        } else {
+                CL_PAGE_DEBUG(D_ERROR, env, page, "parent page\n");
+                CL_PAGE_DEBUG(D_ERROR, env, subpage, "child page\n");
+                LASSERT(0);
+        }
+
+        EXIT;
+out:
+        return(result);
 }