Whamcloud - gitweb
LU-6017 obd: remove destroy cookie handling
[fs/lustre-release.git] / lustre / ofd / ofd_io.c
1 /*
2  * GPL HEADER START
3  *
4  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
5  *
6  * This program is free software; you can redistribute it and/or modify
7  * it under the terms of the GNU General Public License version 2 only,
8  * as published by the Free Software Foundation.
9  *
10  * This program is distributed in the hope that it will be useful, but
11  * WITHOUT ANY WARRANTY; without even the implied warranty of
12  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
13  * General Public License version 2 for more details (a copy is included
14  * in the LICENSE file that accompanied this code).
15  *
16  * You should have received a copy of the GNU General Public License
17  * version 2 along with this program; If not, see
18  * http://www.gnu.org/licenses/gpl-2.0.html
19  *
20  * GPL HEADER END
21  */
22 /*
23  * Copyright (c) 2009, 2010, Oracle and/or its affiliates. All rights reserved.
24  * Use is subject to license terms.
25  *
26  * Copyright (c) 2012, 2014 Intel Corporation.
27  */
28 /*
29  * This file is part of Lustre, http://www.lustre.org/
30  * Lustre is a trademark of Sun Microsystems, Inc.
31  *
32  * lustre/ofd/ofd_io.c
33  *
34  * This file provides functions to handle IO requests from clients and
35  * also LFSCK routines to check parent file identifier (PFID) consistency.
36  *
37  * Author: Alexey Zhuravlev <alexey.zhuravlev@intel.com>
38  * Author: Fan Yong <fan.yong@intel.com>
39  */
40
41 #define DEBUG_SUBSYSTEM S_FILTER
42
43 #include <linux/kthread.h>
44 #include "ofd_internal.h"
45
46 struct ofd_inconsistency_item {
47         struct list_head         oii_list;
48         struct ofd_object       *oii_obj;
49         struct lu_fid            oii_pfid;
50 };
51
52 /**
53  * Verify single object for parent FID consistency.
54  *
55  * Part of LFSCK processing which checks single object PFID stored in extended
56  * attribute (XATTR) against real FID of MDT parent object received by LFSCK.
57  * This verifies that the OST object is being referenced by only a single MDT
58  * object.
59  *
60  * \param[in] env       execution environment
61  * \param[in] ofd       OFD device
62  * \param[in] oii       object-related local data
63  * \param[in] lr        LFSCK request data
64  */
65 static void ofd_inconsistency_verify_one(const struct lu_env *env,
66                                          struct ofd_device *ofd,
67                                          struct ofd_inconsistency_item *oii,
68                                          struct lfsck_request *lr)
69 {
70         struct ofd_object       *fo     = oii->oii_obj;
71         struct lu_fid           *pfid   = &fo->ofo_pfid;
72         int                      rc;
73
74         LASSERT(fo->ofo_pfid_checking);
75         LASSERT(!fo->ofo_pfid_verified);
76
77         lr->lr_fid = fo->ofo_header.loh_fid; /* OST-object itself FID. */
78         lr->lr_fid2 = oii->oii_pfid; /* client given PFID. */
79         lr->lr_fid3 = *pfid; /* OST local stored PFID. */
80
81         rc = lfsck_in_notify(env, ofd->ofd_osd, lr, NULL);
82         ofd_write_lock(env, fo);
83         switch (lr->lr_status) {
84         case LPVS_INIT:
85                 LASSERT(rc <= 0);
86
87                 if (rc < 0)
88                         CDEBUG(D_LFSCK, "%s: fail to verify OST local stored "
89                                "PFID xattr for "DFID", the client given PFID "
90                                DFID", OST local stored PFID "DFID": rc = %d\n",
91                                ofd_name(ofd), PFID(&fo->ofo_header.loh_fid),
92                                PFID(&oii->oii_pfid), PFID(pfid), rc);
93                 else
94                         fo->ofo_pfid_verified = 1;
95                 break;
96         case LPVS_INCONSISTENT:
97                 LASSERT(rc != 0);
98
99                 ofd->ofd_inconsistency_self_detected++;
100                 if (rc < 0)
101                         CDEBUG(D_LFSCK, "%s: fail to verify the client given "
102                                "PFID for "DFID", the client given PFID "DFID
103                                ", local stored PFID "DFID": rc = %d\n",
104                                ofd_name(ofd), PFID(&fo->ofo_header.loh_fid),
105                                PFID(&oii->oii_pfid), PFID(pfid), rc);
106                 else
107                         CDEBUG(D_LFSCK, "%s: both the client given PFID and "
108                                "the OST local stored PFID are stale for the "
109                                "OST-object "DFID", client given PFID is "DFID
110                                ", local stored PFID is "DFID"\n",
111                                ofd_name(ofd), PFID(&fo->ofo_header.loh_fid),
112                                PFID(&oii->oii_pfid), PFID(pfid));
113                 break;
114         case LPVS_INCONSISTENT_TOFIX:
115                 ofd->ofd_inconsistency_self_detected++;
116                 if (rc == 0) {
117                         ofd->ofd_inconsistency_self_repaired++;
118                         CDEBUG(D_LFSCK, "%s: fixed the staled OST PFID xattr "
119                                "for "DFID", with the client given PFID "DFID
120                                ", the old stored PFID "DFID"\n",
121                                ofd_name(ofd), PFID(&fo->ofo_header.loh_fid),
122                                PFID(&oii->oii_pfid), PFID(pfid));
123                 } else if (rc < 0) {
124                         CDEBUG(D_LFSCK, "%s: fail to fix the OST PFID xattr "
125                                "for "DFID", client given PFID "DFID", local "
126                                "stored PFID "DFID": rc = %d\n",
127                                ofd_name(ofd), PFID(&fo->ofo_header.loh_fid),
128                                PFID(&oii->oii_pfid), PFID(pfid), rc);
129                 }
130                 *pfid = oii->oii_pfid;
131                 fo->ofo_pfid_verified = 1;
132                 break;
133         default:
134                 break;
135         }
136         fo->ofo_pfid_checking = 0;
137         ofd_write_unlock(env, fo);
138
139         lu_object_put(env, &fo->ofo_obj.do_lu);
140         OBD_FREE_PTR(oii);
141 }
142
143 /**
144  * Verification thread to check parent FID consistency.
145  *
146  * Kernel thread to check consistency of parent FID for any
147  * new item added for checking by ofd_add_inconsistency_item().
148  *
149  * \param[in] args      OFD device
150  *
151  * \retval              0 on successful thread termination
152  * \retval              negative value if thread can't start
153  */
154 static int ofd_inconsistency_verification_main(void *args)
155 {
156         struct lu_env                  env;
157         struct ofd_device             *ofd    = args;
158         struct ptlrpc_thread          *thread = &ofd->ofd_inconsistency_thread;
159         struct ofd_inconsistency_item *oii;
160         struct lfsck_request          *lr     = NULL;
161         struct l_wait_info             lwi    = { 0 };
162         int                            rc;
163         ENTRY;
164
165         rc = lu_env_init(&env, LCT_DT_THREAD);
166         spin_lock(&ofd->ofd_inconsistency_lock);
167         thread_set_flags(thread, rc != 0 ? SVC_STOPPED : SVC_RUNNING);
168         wake_up_all(&thread->t_ctl_waitq);
169         spin_unlock(&ofd->ofd_inconsistency_lock);
170         if (rc != 0)
171                 RETURN(rc);
172
173         OBD_ALLOC_PTR(lr);
174         if (unlikely(lr == NULL))
175                 GOTO(out, rc = -ENOMEM);
176
177         lr->lr_event = LE_PAIRS_VERIFY;
178         lr->lr_active = LFSCK_TYPE_LAYOUT;
179
180         spin_lock(&ofd->ofd_inconsistency_lock);
181         while (1) {
182                 if (unlikely(!thread_is_running(thread)))
183                         break;
184
185                 while (!list_empty(&ofd->ofd_inconsistency_list)) {
186                         oii = list_entry(ofd->ofd_inconsistency_list.next,
187                                          struct ofd_inconsistency_item,
188                                          oii_list);
189                         list_del_init(&oii->oii_list);
190                         spin_unlock(&ofd->ofd_inconsistency_lock);
191                         ofd_inconsistency_verify_one(&env, ofd, oii, lr);
192                         spin_lock(&ofd->ofd_inconsistency_lock);
193                 }
194
195                 spin_unlock(&ofd->ofd_inconsistency_lock);
196                 l_wait_event(thread->t_ctl_waitq,
197                              !list_empty(&ofd->ofd_inconsistency_list) ||
198                              !thread_is_running(thread),
199                              &lwi);
200                 spin_lock(&ofd->ofd_inconsistency_lock);
201         }
202
203         while (!list_empty(&ofd->ofd_inconsistency_list)) {
204                 struct ofd_object *fo;
205
206                 oii = list_entry(ofd->ofd_inconsistency_list.next,
207                                  struct ofd_inconsistency_item,
208                                  oii_list);
209                 list_del_init(&oii->oii_list);
210                 fo = oii->oii_obj;
211                 spin_unlock(&ofd->ofd_inconsistency_lock);
212
213                 ofd_write_lock(&env, fo);
214                 fo->ofo_pfid_checking = 0;
215                 ofd_write_unlock(&env, fo);
216
217                 lu_object_put(&env, &fo->ofo_obj.do_lu);
218                 OBD_FREE_PTR(oii);
219                 spin_lock(&ofd->ofd_inconsistency_lock);
220         }
221
222         OBD_FREE_PTR(lr);
223
224         GOTO(out, rc = 0);
225
226 out:
227         thread_set_flags(thread, SVC_STOPPED);
228         wake_up_all(&thread->t_ctl_waitq);
229         spin_unlock(&ofd->ofd_inconsistency_lock);
230         lu_env_fini(&env);
231
232         return rc;
233 }
234
235 /**
236  * Start parent FID verification thread.
237  *
238  * See ofd_inconsistency_verification_main().
239  *
240  * \param[in] ofd       OFD device
241  *
242  * \retval              0 on successful start of thread
243  * \retval              negative value on error
244  */
245 int ofd_start_inconsistency_verification_thread(struct ofd_device *ofd)
246 {
247         struct ptlrpc_thread    *thread = &ofd->ofd_inconsistency_thread;
248         struct l_wait_info       lwi    = { 0 };
249         struct task_struct      *task;
250         int                      rc;
251
252         spin_lock(&ofd->ofd_inconsistency_lock);
253         if (unlikely(thread_is_running(thread))) {
254                 spin_unlock(&ofd->ofd_inconsistency_lock);
255
256                 return -EALREADY;
257         }
258
259         thread_set_flags(thread, 0);
260         spin_unlock(&ofd->ofd_inconsistency_lock);
261         task = kthread_run(ofd_inconsistency_verification_main, ofd,
262                            "inconsistency_verification");
263         if (IS_ERR(task)) {
264                 rc = PTR_ERR(task);
265                 CERROR("%s: cannot start self_repair thread: rc = %d\n",
266                        ofd_name(ofd), rc);
267         } else {
268                 rc = 0;
269                 l_wait_event(thread->t_ctl_waitq,
270                              thread_is_running(thread) ||
271                              thread_is_stopped(thread),
272                              &lwi);
273         }
274
275         return rc;
276 }
277
278 /**
279  * Stop parent FID verification thread.
280  *
281  * \param[in] ofd       OFD device
282  *
283  * \retval              0 on successful start of thread
284  * \retval              -EALREADY if thread is already stopped
285  */
286 int ofd_stop_inconsistency_verification_thread(struct ofd_device *ofd)
287 {
288         struct ptlrpc_thread    *thread = &ofd->ofd_inconsistency_thread;
289         struct l_wait_info       lwi    = { 0 };
290
291         spin_lock(&ofd->ofd_inconsistency_lock);
292         if (thread_is_init(thread) || thread_is_stopped(thread)) {
293                 spin_unlock(&ofd->ofd_inconsistency_lock);
294
295                 return -EALREADY;
296         }
297
298         thread_set_flags(thread, SVC_STOPPING);
299         spin_unlock(&ofd->ofd_inconsistency_lock);
300         wake_up_all(&thread->t_ctl_waitq);
301         l_wait_event(thread->t_ctl_waitq,
302                      thread_is_stopped(thread),
303                      &lwi);
304
305         return 0;
306 }
307
308 /**
309  * Add new item for parent FID verification.
310  *
311  * Prepare new verification item and pass it to the dedicated
312  * verification thread for further processing.
313  *
314  * \param[in] env       execution environment
315  * \param[in] fo        OFD object
316  * \param[in] oa        OBDO structure with PFID
317  */
318 static void ofd_add_inconsistency_item(const struct lu_env *env,
319                                        struct ofd_object *fo, struct obdo *oa)
320 {
321         struct ofd_device               *ofd    = ofd_obj2dev(fo);
322         struct ofd_inconsistency_item   *oii;
323         bool                             wakeup = false;
324
325         OBD_ALLOC_PTR(oii);
326         if (oii == NULL)
327                 return;
328
329         INIT_LIST_HEAD(&oii->oii_list);
330         lu_object_get(&fo->ofo_obj.do_lu);
331         oii->oii_obj = fo;
332         oii->oii_pfid.f_seq = oa->o_parent_seq;
333         oii->oii_pfid.f_oid = oa->o_parent_oid;
334         oii->oii_pfid.f_stripe_idx = oa->o_stripe_idx;
335
336         spin_lock(&ofd->ofd_inconsistency_lock);
337         if (fo->ofo_pfid_checking || fo->ofo_pfid_verified) {
338                 spin_unlock(&ofd->ofd_inconsistency_lock);
339                 OBD_FREE_PTR(oii);
340
341                 return;
342         }
343
344         fo->ofo_pfid_checking = 1;
345         if (list_empty(&ofd->ofd_inconsistency_list))
346                 wakeup = true;
347         list_add_tail(&oii->oii_list, &ofd->ofd_inconsistency_list);
348         spin_unlock(&ofd->ofd_inconsistency_lock);
349         if (wakeup)
350                 wake_up_all(&ofd->ofd_inconsistency_thread.t_ctl_waitq);
351
352         /* XXX: When the found inconsistency exceeds some threshold,
353          *      we can trigger the LFSCK to scan part of the system
354          *      or the whole system, which depends on how to define
355          *      the threshold, a simple way maybe like that: define
356          *      the absolute value of how many inconsisteny allowed
357          *      to be repaired via self detect/repair mechanism, if
358          *      exceeded, then trigger the LFSCK to scan the layout
359          *      inconsistency within the whole system. */
360 }
361
362 /**
363  * Verify parent FID of an object.
364  *
365  * Check the parent FID is sane and start extended
366  * verification procedure otherwise.
367  *
368  * \param[in] env       execution environment
369  * \param[in] fo        OFD object
370  * \param[in] oa        OBDO structure with PFID
371  *
372  * \retval              0 on successful verification
373  * \retval              -EINPROGRESS if PFID is being repaired
374  * \retval              -EPERM if PFID was verified but still insane
375  */
376 int ofd_verify_ff(const struct lu_env *env, struct ofd_object *fo,
377                   struct obdo *oa)
378 {
379         struct lu_fid   *pfid   = &fo->ofo_pfid;
380         int              rc     = 0;
381         ENTRY;
382
383         if (fid_is_sane(pfid)) {
384                 if (likely(oa->o_parent_seq == pfid->f_seq &&
385                            oa->o_parent_oid == pfid->f_oid &&
386                            oa->o_stripe_idx == pfid->f_stripe_idx))
387                         RETURN(0);
388
389                 if (fo->ofo_pfid_verified)
390                         RETURN(-EPERM);
391         }
392
393         /* The OST-object may be inconsistent, and we need further verification.
394          * To avoid block the RPC service thread, return -EINPROGRESS to client
395          * and make it retry later. */
396         if (fo->ofo_pfid_checking)
397                 RETURN(-EINPROGRESS);
398
399         rc = ofd_object_ff_load(env, fo);
400         if (rc == -ENODATA)
401                 RETURN(0);
402
403         if (rc < 0)
404                 RETURN(rc);
405
406         if (likely(oa->o_parent_seq == pfid->f_seq &&
407                    oa->o_parent_oid == pfid->f_oid &&
408                    oa->o_stripe_idx == pfid->f_stripe_idx))
409                 RETURN(0);
410
411         /* Push it to the dedicated thread for further verification. */
412         ofd_add_inconsistency_item(env, fo, oa);
413
414         RETURN(-EINPROGRESS);
415 }
416
417 /**
418  * Prepare buffers for read request processing.
419  *
420  * This function converts remote buffers from client to local buffers
421  * and prepares the latter.
422  *
423  * \param[in] env       execution environment
424  * \param[in] exp       OBD export of client
425  * \param[in] ofd       OFD device
426  * \param[in] fid       FID of object
427  * \param[in] la        object attributes
428  * \param[in] oa        OBDO structure from client
429  * \param[in] niocount  number of remote buffers
430  * \param[in] rnb       remote buffers
431  * \param[in] nr_local  number of local buffers
432  * \param[in] lnb       local buffers
433  * \param[in] jobid     job ID name
434  *
435  * \retval              0 on successful prepare
436  * \retval              negative value on error
437  */
438 static int ofd_preprw_read(const struct lu_env *env, struct obd_export *exp,
439                            struct ofd_device *ofd, const struct lu_fid *fid,
440                            struct lu_attr *la, struct obdo *oa, int niocount,
441                            struct niobuf_remote *rnb, int *nr_local,
442                            struct niobuf_local *lnb, char *jobid)
443 {
444         struct ofd_object       *fo;
445         int                      i, j, rc, tot_bytes = 0;
446
447         ENTRY;
448         LASSERT(env != NULL);
449
450         fo = ofd_object_find(env, ofd, fid);
451         if (IS_ERR(fo))
452                 RETURN(PTR_ERR(fo));
453         LASSERT(fo != NULL);
454
455         ofd_read_lock(env, fo);
456         if (!ofd_object_exists(fo))
457                 GOTO(unlock, rc = -ENOENT);
458
459         if (ofd->ofd_lfsck_verify_pfid && oa->o_valid & OBD_MD_FLFID) {
460                 rc = ofd_verify_ff(env, fo, oa);
461                 if (rc != 0)
462                         GOTO(unlock, rc);
463         }
464
465         *nr_local = 0;
466         for (i = 0, j = 0; i < niocount; i++) {
467                 rc = dt_bufs_get(env, ofd_object_child(fo), rnb + i,
468                                  lnb + j, 0);
469                 if (unlikely(rc < 0))
470                         GOTO(buf_put, rc);
471                 LASSERT(rc <= PTLRPC_MAX_BRW_PAGES);
472                 /* correct index for local buffers to continue with */
473                 j += rc;
474                 *nr_local += rc;
475                 LASSERT(j <= PTLRPC_MAX_BRW_PAGES);
476                 tot_bytes += rnb[i].rnb_len;
477         }
478
479         LASSERT(*nr_local > 0 && *nr_local <= PTLRPC_MAX_BRW_PAGES);
480         rc = dt_attr_get(env, ofd_object_child(fo), la);
481         if (unlikely(rc))
482                 GOTO(buf_put, rc);
483
484         rc = dt_read_prep(env, ofd_object_child(fo), lnb, *nr_local);
485         if (unlikely(rc))
486                 GOTO(buf_put, rc);
487
488         ofd_counter_incr(exp, LPROC_OFD_STATS_READ, jobid, tot_bytes);
489         RETURN(0);
490
491 buf_put:
492         dt_bufs_put(env, ofd_object_child(fo), lnb, *nr_local);
493 unlock:
494         ofd_read_unlock(env, fo);
495         ofd_object_put(env, fo);
496         return rc;
497 }
498
499 /**
500  * Prepare buffers for write request processing.
501  *
502  * This function converts remote buffers from client to local buffers
503  * and prepares the latter. If there is recovery in progress and required
504  * object is missing then it can be re-created before write.
505  *
506  * \param[in] env       execution environment
507  * \param[in] exp       OBD export of client
508  * \param[in] ofd       OFD device
509  * \param[in] fid       FID of object
510  * \param[in] la        object attributes
511  * \param[in] oa        OBDO structure from client
512  * \param[in] objcount  always 1
513  * \param[in] obj       object data
514  * \param[in] rnb       remote buffers
515  * \param[in] nr_local  number of local buffers
516  * \param[in] lnb       local buffers
517  * \param[in] jobid     job ID name
518  *
519  * \retval              0 on successful prepare
520  * \retval              negative value on error
521  */
522 static int ofd_preprw_write(const struct lu_env *env, struct obd_export *exp,
523                             struct ofd_device *ofd, const struct lu_fid *fid,
524                             struct lu_attr *la, struct obdo *oa,
525                             int objcount, struct obd_ioobj *obj,
526                             struct niobuf_remote *rnb, int *nr_local,
527                             struct niobuf_local *lnb, char *jobid)
528 {
529         struct ofd_object       *fo;
530         int                      i, j, k, rc = 0, tot_bytes = 0;
531
532         ENTRY;
533         LASSERT(env != NULL);
534         LASSERT(objcount == 1);
535
536         if (unlikely(exp->exp_obd->obd_recovering)) {
537                 u64 seq = fid_seq(fid);
538                 u64 oid = fid_oid(fid);
539                 struct ofd_seq *oseq;
540
541                 oseq = ofd_seq_load(env, ofd, seq);
542                 if (IS_ERR(oseq)) {
543                         CERROR("%s: Can't find FID Sequence "LPX64": rc = %d\n",
544                                ofd_name(ofd), seq, (int)PTR_ERR(oseq));
545                         GOTO(out, rc = -EINVAL);
546                 }
547
548                 if (oid > ofd_seq_last_oid(oseq)) {
549                         int sync = 0;
550                         int diff;
551
552                         mutex_lock(&oseq->os_create_lock);
553                         diff = oid - ofd_seq_last_oid(oseq);
554
555                         /* Do sync create if the seq is about to used up */
556                         if (fid_seq_is_idif(seq) || fid_seq_is_mdt0(seq)) {
557                                 if (unlikely(oid >= IDIF_MAX_OID - 1))
558                                         sync = 1;
559                         } else if (fid_seq_is_norm(seq)) {
560                                 if (unlikely(oid >=
561                                              LUSTRE_DATA_SEQ_MAX_WIDTH - 1))
562                                         sync = 1;
563                         } else {
564                                 CERROR("%s : invalid o_seq "DOSTID"\n",
565                                        ofd_name(ofd), POSTID(&oa->o_oi));
566                                 mutex_unlock(&oseq->os_create_lock);
567                                 ofd_seq_put(env, oseq);
568                                 GOTO(out, rc = -EINVAL);
569                         }
570
571                         while (diff > 0) {
572                                 u64 next_id = ofd_seq_last_oid(oseq) + 1;
573                                 int count = ofd_precreate_batch(ofd, diff);
574
575                                 rc = ofd_precreate_objects(env, ofd, next_id,
576                                                            oseq, count, sync);
577                                 if (rc < 0) {
578                                         mutex_unlock(&oseq->os_create_lock);
579                                         ofd_seq_put(env, oseq);
580                                         GOTO(out, rc);
581                                 }
582
583                                 diff -= rc;
584                         }
585
586                         mutex_unlock(&oseq->os_create_lock);
587                 }
588
589                 ofd_seq_put(env, oseq);
590         }
591
592         fo = ofd_object_find(env, ofd, fid);
593         if (IS_ERR(fo))
594                 GOTO(out, rc = PTR_ERR(fo));
595         LASSERT(fo != NULL);
596
597         ofd_read_lock(env, fo);
598         if (!ofd_object_exists(fo)) {
599                 CERROR("%s: BRW to missing obj "DOSTID"\n",
600                        exp->exp_obd->obd_name, POSTID(&obj->ioo_oid));
601                 ofd_read_unlock(env, fo);
602                 ofd_object_put(env, fo);
603                 GOTO(out, rc = -ENOENT);
604         }
605
606         if (ofd->ofd_lfsck_verify_pfid && oa->o_valid & OBD_MD_FLFID) {
607                 rc = ofd_verify_ff(env, fo, oa);
608                 if (rc != 0) {
609                         ofd_read_unlock(env, fo);
610                         ofd_object_put(env, fo);
611                         GOTO(out, rc);
612                 }
613         }
614
615         /* Process incoming grant info, set OBD_BRW_GRANTED flag and grant some
616          * space back if possible */
617         ofd_grant_prepare_write(env, exp, oa, rnb, obj->ioo_bufcnt);
618
619         /* parse remote buffers to local buffers and prepare the latter */
620         *nr_local = 0;
621         for (i = 0, j = 0; i < obj->ioo_bufcnt; i++) {
622                 rc = dt_bufs_get(env, ofd_object_child(fo),
623                                  rnb + i, lnb + j, 1);
624                 if (unlikely(rc < 0))
625                         GOTO(err, rc);
626                 LASSERT(rc <= PTLRPC_MAX_BRW_PAGES);
627                 /* correct index for local buffers to continue with */
628                 for (k = 0; k < rc; k++) {
629                         lnb[j+k].lnb_flags = rnb[i].rnb_flags;
630                         if (!(rnb[i].rnb_flags & OBD_BRW_GRANTED))
631                                 lnb[j+k].lnb_rc = -ENOSPC;
632
633                         /* remote client can't break through quota */
634                         if (exp_connect_rmtclient(exp))
635                                 lnb[j+k].lnb_flags &= ~OBD_BRW_NOQUOTA;
636                 }
637                 j += rc;
638                 *nr_local += rc;
639                 LASSERT(j <= PTLRPC_MAX_BRW_PAGES);
640                 tot_bytes += rnb[i].rnb_len;
641         }
642         LASSERT(*nr_local > 0 && *nr_local <= PTLRPC_MAX_BRW_PAGES);
643
644         rc = dt_write_prep(env, ofd_object_child(fo), lnb, *nr_local);
645         if (unlikely(rc != 0))
646                 GOTO(err, rc);
647
648         ofd_counter_incr(exp, LPROC_OFD_STATS_WRITE, jobid, tot_bytes);
649         RETURN(0);
650 err:
651         dt_bufs_put(env, ofd_object_child(fo), lnb, *nr_local);
652         ofd_read_unlock(env, fo);
653         ofd_object_put(env, fo);
654         /* ofd_grant_prepare_write() was called, so we must commit */
655         ofd_grant_commit(env, exp, rc);
656 out:
657         /* let's still process incoming grant information packed in the oa,
658          * but without enforcing grant since we won't proceed with the write.
659          * Just like a read request actually. */
660         ofd_grant_prepare_read(env, exp, oa);
661         return rc;
662 }
663
664 /**
665  * Prepare bulk IO requests for processing.
666  *
667  * This function does initial checks of IO and calls corresponding
668  * functions for read/write processing.
669  *
670  * \param[in] env       execution environment
671  * \param[in] cmd       IO type (read/write)
672  * \param[in] exp       OBD export of client
673  * \param[in] oa        OBDO structure from request
674  * \param[in] objcount  always 1
675  * \param[in] obj       object data
676  * \param[in] rnb       remote buffers
677  * \param[in] nr_local  number of local buffers
678  * \param[in] lnb       local buffers
679  *
680  * \retval              0 on successful prepare
681  * \retval              negative value on error
682  */
683 int ofd_preprw(const struct lu_env *env, int cmd, struct obd_export *exp,
684                struct obdo *oa, int objcount, struct obd_ioobj *obj,
685                struct niobuf_remote *rnb, int *nr_local,
686                struct niobuf_local *lnb)
687 {
688         struct tgt_session_info *tsi = tgt_ses_info(env);
689         struct ofd_device       *ofd = ofd_exp(exp);
690         struct ofd_thread_info  *info;
691         char                    *jobid;
692         const struct lu_fid     *fid = &oa->o_oi.oi_fid;
693         int                      rc = 0;
694
695         if (*nr_local > PTLRPC_MAX_BRW_PAGES) {
696                 CERROR("%s: bulk has too many pages %d, which exceeds the"
697                        "maximum pages per RPC of %d\n",
698                        exp->exp_obd->obd_name, *nr_local, PTLRPC_MAX_BRW_PAGES);
699                 RETURN(-EPROTO);
700         }
701
702         if (tgt_ses_req(tsi) == NULL) { /* echo client case */
703                 info = ofd_info_init(env, exp);
704                 jobid = NULL;
705         } else {
706                 info = tsi2ofd_info(tsi);
707                 jobid = tsi->tsi_jobid;
708         }
709
710         LASSERT(oa != NULL);
711
712         if (OBD_FAIL_CHECK(OBD_FAIL_OST_ENOENT)) {
713                 struct ofd_seq          *oseq;
714
715                 oseq = ofd_seq_load(env, ofd, ostid_seq(&oa->o_oi));
716                 if (IS_ERR(oseq)) {
717                         CERROR("%s: Can not find seq for "DOSTID
718                                ": rc = %ld\n", ofd_name(ofd), POSTID(&oa->o_oi),
719                                PTR_ERR(oseq));
720                         RETURN(-EINVAL);
721                 }
722
723                 if (oseq->os_destroys_in_progress == 0) {
724                         /* don't fail lookups for orphan recovery, it causes
725                          * later LBUGs when objects still exist during
726                          * precreate */
727                         ofd_seq_put(env, oseq);
728                         RETURN(-ENOENT);
729                 }
730                 ofd_seq_put(env, oseq);
731         }
732
733         LASSERT(objcount == 1);
734         LASSERT(obj->ioo_bufcnt > 0);
735
736         if (cmd == OBD_BRW_WRITE) {
737                 la_from_obdo(&info->fti_attr, oa, OBD_MD_FLGETATTR);
738                 rc = ofd_preprw_write(env, exp, ofd, fid, &info->fti_attr, oa,
739                                       objcount, obj, rnb, nr_local, lnb, jobid);
740         } else if (cmd == OBD_BRW_READ) {
741                 ofd_grant_prepare_read(env, exp, oa);
742                 rc = ofd_preprw_read(env, exp, ofd, fid, &info->fti_attr, oa,
743                                      obj->ioo_bufcnt, rnb, nr_local, lnb,
744                                      jobid);
745                 obdo_from_la(oa, &info->fti_attr, LA_ATIME);
746         } else {
747                 CERROR("%s: wrong cmd %d received!\n",
748                        exp->exp_obd->obd_name, cmd);
749                 rc = -EPROTO;
750         }
751         RETURN(rc);
752 }
753
754 /**
755  * Drop reference on local buffers for read bulk IO.
756  *
757  * This will free all local buffers use by this read request.
758  *
759  * \param[in] env       execution environment
760  * \param[in] ofd       OFD device
761  * \param[in] fid       object FID
762  * \param[in] objcount  always 1
763  * \param[in] niocount  number of local buffers
764  * \param[in] lnb       local buffers
765  *
766  * \retval              0 on successful execution
767  * \retval              negative value on error
768  */
769 static int
770 ofd_commitrw_read(const struct lu_env *env, struct ofd_device *ofd,
771                   const struct lu_fid *fid, int objcount, int niocount,
772                   struct niobuf_local *lnb)
773 {
774         struct ofd_object *fo;
775
776         ENTRY;
777
778         LASSERT(niocount > 0);
779
780         fo = ofd_object_find(env, ofd, fid);
781         if (IS_ERR(fo))
782                 RETURN(PTR_ERR(fo));
783         LASSERT(fo != NULL);
784         LASSERT(ofd_object_exists(fo));
785         dt_bufs_put(env, ofd_object_child(fo), lnb, niocount);
786
787         ofd_read_unlock(env, fo);
788         ofd_object_put(env, fo);
789         /* second put is pair to object_get in ofd_preprw_read */
790         ofd_object_put(env, fo);
791
792         RETURN(0);
793 }
794
795 /**
796  * Set attributes of object during write bulk IO processing.
797  *
798  * Change object attributes and write parent FID into extended
799  * attributes when needed.
800  *
801  * \param[in] env       execution environment
802  * \param[in] ofd       OFD device
803  * \param[in] ofd_obj   OFD object
804  * \param[in] la        object attributes
805  * \param[in] ff        parent FID
806  *
807  * \retval              0 on successful attributes update
808  * \retval              negative value on error
809  */
810 static int
811 ofd_write_attr_set(const struct lu_env *env, struct ofd_device *ofd,
812                    struct ofd_object *ofd_obj, struct lu_attr *la,
813                    struct filter_fid *ff)
814 {
815         struct ofd_thread_info  *info = ofd_info(env);
816         __u64                    valid = la->la_valid;
817         int                      rc;
818         struct thandle          *th;
819         struct dt_object        *dt_obj;
820         int                      ff_needed = 0;
821
822         ENTRY;
823
824         LASSERT(la);
825
826         dt_obj = ofd_object_child(ofd_obj);
827         LASSERT(dt_obj != NULL);
828
829         la->la_valid &= LA_UID | LA_GID;
830
831         rc = ofd_attr_handle_ugid(env, ofd_obj, la, 0 /* !is_setattr */);
832         if (rc != 0)
833                 GOTO(out, rc);
834
835         if (ff != NULL) {
836                 rc = ofd_object_ff_load(env, ofd_obj);
837                 if (rc == -ENODATA)
838                         ff_needed = 1;
839                 else if (rc < 0)
840                         GOTO(out, rc);
841         }
842
843         if (!la->la_valid && !ff_needed)
844                 /* no attributes to set */
845                 GOTO(out, rc = 0);
846
847         th = ofd_trans_create(env, ofd);
848         if (IS_ERR(th))
849                 GOTO(out, rc = PTR_ERR(th));
850
851         if (la->la_valid) {
852                 rc = dt_declare_attr_set(env, dt_obj, la, th);
853                 if (rc)
854                         GOTO(out_tx, rc);
855         }
856
857         if (ff_needed) {
858                 if (OBD_FAIL_CHECK(OBD_FAIL_LFSCK_UNMATCHED_PAIR1))
859                         ff->ff_parent.f_oid = cpu_to_le32(1UL << 31);
860                 if (OBD_FAIL_CHECK(OBD_FAIL_LFSCK_UNMATCHED_PAIR2))
861                         ff->ff_parent.f_oid =
862                         cpu_to_le32(le32_to_cpu(ff->ff_parent.f_oid) - 1);
863
864                 info->fti_buf.lb_buf = ff;
865                 info->fti_buf.lb_len = sizeof(*ff);
866                 rc = dt_declare_xattr_set(env, dt_obj, &info->fti_buf,
867                                           XATTR_NAME_FID, 0, th);
868                 if (rc)
869                         GOTO(out_tx, rc);
870         }
871
872         /* We don't need a transno for this operation which will be re-executed
873          * anyway when the OST_WRITE (with a transno assigned) is replayed */
874         rc = dt_trans_start_local(env, ofd->ofd_osd , th);
875         if (rc)
876                 GOTO(out_tx, rc);
877
878         /* set uid/gid */
879         if (la->la_valid) {
880                 rc = dt_attr_set(env, dt_obj, la, th);
881                 if (rc)
882                         GOTO(out_tx, rc);
883         }
884
885         /* set filter fid EA */
886         if (ff_needed) {
887                 if (OBD_FAIL_CHECK(OBD_FAIL_LFSCK_NOPFID))
888                         GOTO(out_tx, rc);
889
890                 rc = dt_xattr_set(env, dt_obj, &info->fti_buf, XATTR_NAME_FID,
891                                   0, th);
892                 if (rc == 0) {
893                         ofd_obj->ofo_pfid.f_seq = le64_to_cpu(ff->ff_parent.f_seq);
894                         ofd_obj->ofo_pfid.f_oid = le32_to_cpu(ff->ff_parent.f_oid);
895                         /* Currently, the filter_fid::ff_parent::f_ver is not
896                          * the real parent MDT-object's FID::f_ver, instead it
897                          * is the OST-object index in its parent MDT-object's
898                          * layout EA. */
899                         ofd_obj->ofo_pfid.f_stripe_idx =
900                                         le32_to_cpu(ff->ff_parent.f_stripe_idx);
901                 }
902         }
903
904         GOTO(out_tx, rc);
905
906 out_tx:
907         dt_trans_stop(env, ofd->ofd_osd, th);
908 out:
909         la->la_valid = valid;
910         return rc;
911 }
912
913 struct ofd_soft_sync_callback {
914         struct dt_txn_commit_cb  ossc_cb;
915         struct obd_export       *ossc_exp;
916 };
917
918 /**
919  * Callback function for "soft sync" update.
920  *
921  * Reset fed_soft_sync_count upon committing the "soft_sync" update.
922  * See ofd_soft_sync_cb_add() below for more details on soft sync.
923  *
924  * \param[in] env       execution environment
925  * \param[in] th        transaction handle
926  * \param[in] cb        callback data
927  * \param[in] err       error code
928  */
929 static void ofd_cb_soft_sync(struct lu_env *env, struct thandle *th,
930                              struct dt_txn_commit_cb *cb, int err)
931 {
932         struct ofd_soft_sync_callback   *ossc;
933
934         ossc = container_of(cb, struct ofd_soft_sync_callback, ossc_cb);
935
936         CDEBUG(D_INODE, "export %p soft sync count is reset\n", ossc->ossc_exp);
937         atomic_set(&ossc->ossc_exp->exp_filter_data.fed_soft_sync_count, 0);
938
939         class_export_cb_put(ossc->ossc_exp);
940         OBD_FREE_PTR(ossc);
941 }
942
943 /**
944  * Add callback for "soft sync" processing.
945  *
946  * The "soft sync" mechanism does asynchronous commit when OBD_BRW_SOFT_SYNC
947  * flag is set in client buffers. The intention is for this operation to
948  * commit pages belonging to a client which has "too many" outstanding
949  * unstable pages in its cache. See LU-2139 for details.
950  *
951  * This function adds callback to be called when commit is done.
952  *
953  * \param[in] th        transaction handle
954  * \param[in] exp       OBD export of client
955  *
956  * \retval              0 on successful callback adding
957  * \retval              negative value on error
958  */
959 static int ofd_soft_sync_cb_add(struct thandle *th, struct obd_export *exp)
960 {
961         struct ofd_soft_sync_callback           *ossc;
962         struct dt_txn_commit_cb                 *dcb;
963         int                                      rc;
964
965         OBD_ALLOC_PTR(ossc);
966         if (ossc == NULL)
967                 return -ENOMEM;
968
969         ossc->ossc_exp = class_export_cb_get(exp);
970
971         dcb = &ossc->ossc_cb;
972         dcb->dcb_func = ofd_cb_soft_sync;
973         INIT_LIST_HEAD(&dcb->dcb_linkage);
974         strlcpy(dcb->dcb_name, "ofd_cb_soft_sync", sizeof(dcb->dcb_name));
975
976         rc = dt_trans_cb_add(th, dcb);
977         if (rc) {
978                 class_export_cb_put(exp);
979                 OBD_FREE_PTR(ossc);
980         }
981
982         return rc;
983 }
984
985 /**
986  * Commit bulk IO buffers to the storage.
987  *
988  * This function finalizes write IO processing by writing data to the disk.
989  * That write can be synchronous or asynchronous depending on buffers flags.
990  *
991  * \param[in] env       execution environment
992  * \param[in] exp       OBD export of client
993  * \param[in] ofd       OFD device
994  * \param[in] fid       FID of object
995  * \param[in] la        object attributes
996  * \param[in] ff        parent FID of object
997  * \param[in] objcount  always 1
998  * \param[in] niocount  number of local buffers
999  * \param[in] lnb       local buffers
1000  * \param[in] old_rc    result of processing at this point
1001  *
1002  * \retval              0 on successful commit
1003  * \retval              negative value on error
1004  */
1005 static int
1006 ofd_commitrw_write(const struct lu_env *env, struct obd_export *exp,
1007                    struct ofd_device *ofd, const struct lu_fid *fid,
1008                    struct lu_attr *la, struct filter_fid *ff, int objcount,
1009                    int niocount, struct niobuf_local *lnb, int old_rc)
1010 {
1011         struct ofd_thread_info  *info = ofd_info(env);
1012         struct ofd_object       *fo;
1013         struct dt_object        *o;
1014         struct thandle          *th;
1015         int                      rc = 0;
1016         int                      retries = 0;
1017         int                      i;
1018         struct filter_export_data *fed = &exp->exp_filter_data;
1019         bool                     soft_sync = false;
1020         bool                     cb_registered = false;
1021
1022         ENTRY;
1023
1024         LASSERT(objcount == 1);
1025
1026         fo = ofd_object_find(env, ofd, fid);
1027         LASSERT(fo != NULL);
1028         LASSERT(ofd_object_exists(fo));
1029
1030         o = ofd_object_child(fo);
1031         LASSERT(o != NULL);
1032
1033         if (old_rc)
1034                 GOTO(out, rc = old_rc);
1035
1036         /*
1037          * The first write to each object must set some attributes.  It is
1038          * important to set the uid/gid before calling
1039          * dt_declare_write_commit() since quota enforcement is now handled in
1040          * declare phases.
1041          */
1042         rc = ofd_write_attr_set(env, ofd, fo, la, ff);
1043         if (rc)
1044                 GOTO(out, rc);
1045
1046         la->la_valid &= LA_ATIME | LA_MTIME | LA_CTIME;
1047
1048 retry:
1049         th = ofd_trans_create(env, ofd);
1050         if (IS_ERR(th))
1051                 GOTO(out, rc = PTR_ERR(th));
1052
1053         th->th_sync |= ofd->ofd_syncjournal;
1054         if (th->th_sync == 0) {
1055                 for (i = 0; i < niocount; i++) {
1056                         if (!(lnb[i].lnb_flags & OBD_BRW_ASYNC)) {
1057                                 th->th_sync = 1;
1058                                 break;
1059                         }
1060                         if (lnb[i].lnb_flags & OBD_BRW_SOFT_SYNC)
1061                                 soft_sync = true;
1062                 }
1063         }
1064
1065         if (OBD_FAIL_CHECK(OBD_FAIL_OST_DQACQ_NET))
1066                 GOTO(out_stop, rc = -EINPROGRESS);
1067
1068         rc = dt_declare_write_commit(env, o, lnb, niocount, th);
1069         if (rc)
1070                 GOTO(out_stop, rc);
1071
1072         if (la->la_valid) {
1073                 /* update [mac]time if needed */
1074                 rc = dt_declare_attr_set(env, o, la, th);
1075                 if (rc)
1076                         GOTO(out_stop, rc);
1077         }
1078
1079         rc = ofd_trans_start(env, ofd, fo, th);
1080         if (rc)
1081                 GOTO(out_stop, rc);
1082
1083         rc = dt_write_commit(env, o, lnb, niocount, th);
1084         if (rc)
1085                 GOTO(out_stop, rc);
1086
1087         if (la->la_valid) {
1088                 rc = dt_attr_set(env, o, la, th);
1089                 if (rc)
1090                         GOTO(out_stop, rc);
1091         }
1092
1093         /* get attr to return */
1094         rc = dt_attr_get(env, o, la);
1095
1096 out_stop:
1097         /* Force commit to make the just-deleted blocks
1098          * reusable. LU-456 */
1099         if (rc == -ENOSPC)
1100                 th->th_sync = 1;
1101
1102         /* do this before trans stop in case commit has finished */
1103         if (!th->th_sync && soft_sync && !cb_registered) {
1104                 ofd_soft_sync_cb_add(th, exp);
1105                 cb_registered = true;
1106         }
1107
1108         ofd_trans_stop(env, ofd, th, rc);
1109         if (rc == -ENOSPC && retries++ < 3) {
1110                 CDEBUG(D_INODE, "retry after force commit, retries:%d\n",
1111                        retries);
1112                 goto retry;
1113         }
1114
1115         if (!soft_sync)
1116                 /* reset fed_soft_sync_count upon non-SOFT_SYNC RPC */
1117                 atomic_set(&fed->fed_soft_sync_count, 0);
1118         else if (atomic_inc_return(&fed->fed_soft_sync_count) ==
1119                  ofd->ofd_soft_sync_limit)
1120                 dt_commit_async(env, ofd->ofd_osd);
1121
1122 out:
1123         dt_bufs_put(env, o, lnb, niocount);
1124         ofd_read_unlock(env, fo);
1125         ofd_object_put(env, fo);
1126         /* second put is pair to object_get in ofd_preprw_write */
1127         ofd_object_put(env, fo);
1128         ofd_grant_commit(env, info->fti_exp, old_rc);
1129         RETURN(rc);
1130 }
1131
1132 /**
1133  * Commit bulk IO to the storage.
1134  *
1135  * This is companion function to the ofd_preprw(). It finishes bulk IO
1136  * request processing by committing buffers to the storage (WRITE) and/or
1137  * freeing those buffers (read/write). See ofd_commitrw_read() and
1138  * ofd_commitrw_write() for details about each type of IO.
1139  *
1140  * \param[in] env       execution environment
1141  * \param[in] cmd       IO type (READ/WRITE)
1142  * \param[in] exp       OBD export of client
1143  * \param[in] oa        OBDO structure from client
1144  * \param[in] objcount  always 1
1145  * \param[in] obj       object data
1146  * \param[in] rnb       remote buffers
1147  * \param[in] npages    number of local buffers
1148  * \param[in] lnb       local buffers
1149  * \param[in] old_rc    result of processing at this point
1150  *
1151  * \retval              0 on successful commit
1152  * \retval              negative value on error
1153  */
1154 int ofd_commitrw(const struct lu_env *env, int cmd, struct obd_export *exp,
1155                  struct obdo *oa, int objcount, struct obd_ioobj *obj,
1156                  struct niobuf_remote *rnb, int npages,
1157                  struct niobuf_local *lnb, int old_rc)
1158 {
1159         struct ofd_thread_info  *info = ofd_info(env);
1160         struct ofd_mod_data     *fmd;
1161         __u64                    valid;
1162         struct ofd_device       *ofd = ofd_exp(exp);
1163         struct filter_fid       *ff = NULL;
1164         const struct lu_fid     *fid = &oa->o_oi.oi_fid;
1165         int                      rc = 0;
1166
1167         LASSERT(npages > 0);
1168
1169         if (cmd == OBD_BRW_WRITE) {
1170                 /* Don't update timestamps if this write is older than a
1171                  * setattr which modifies the timestamps. b=10150 */
1172
1173                 /* XXX when we start having persistent reservations this needs
1174                  * to be changed to ofd_fmd_get() to create the fmd if it
1175                  * doesn't already exist so we can store the reservation handle
1176                  * there. */
1177                 valid = OBD_MD_FLUID | OBD_MD_FLGID;
1178                 fmd = ofd_fmd_find(exp, fid);
1179                 if (!fmd || fmd->fmd_mactime_xid < info->fti_xid)
1180                         valid |= OBD_MD_FLATIME | OBD_MD_FLMTIME |
1181                                  OBD_MD_FLCTIME;
1182                 ofd_fmd_put(exp, fmd);
1183                 la_from_obdo(&info->fti_attr, oa, valid);
1184
1185                 if (oa->o_valid & OBD_MD_FLFID) {
1186                         ff = &info->fti_mds_fid;
1187                         ofd_prepare_fidea(ff, oa);
1188                 }
1189
1190                 rc = ofd_commitrw_write(env, exp, ofd, fid, &info->fti_attr,
1191                                         ff, objcount, npages, lnb, old_rc);
1192                 if (rc == 0)
1193                         obdo_from_la(oa, &info->fti_attr,
1194                                      OFD_VALID_FLAGS | LA_GID | LA_UID);
1195                 else
1196                         obdo_from_la(oa, &info->fti_attr, LA_GID | LA_UID);
1197
1198                 /* don't report overquota flag if we failed before reaching
1199                  * commit */
1200                 if (old_rc == 0 && (rc == 0 || rc == -EDQUOT)) {
1201                         /* return the overquota flags to client */
1202                         if (lnb[0].lnb_flags & OBD_BRW_OVER_USRQUOTA) {
1203                                 if (oa->o_valid & OBD_MD_FLFLAGS)
1204                                         oa->o_flags |= OBD_FL_NO_USRQUOTA;
1205                                 else
1206                                         oa->o_flags = OBD_FL_NO_USRQUOTA;
1207                         }
1208
1209                         if (lnb[0].lnb_flags & OBD_BRW_OVER_GRPQUOTA) {
1210                                 if (oa->o_valid & OBD_MD_FLFLAGS)
1211                                         oa->o_flags |= OBD_FL_NO_GRPQUOTA;
1212                                 else
1213                                         oa->o_flags = OBD_FL_NO_GRPQUOTA;
1214                         }
1215
1216                         oa->o_valid |= OBD_MD_FLFLAGS;
1217                         oa->o_valid |= OBD_MD_FLUSRQUOTA | OBD_MD_FLGRPQUOTA;
1218                 }
1219         } else if (cmd == OBD_BRW_READ) {
1220                 struct ldlm_namespace *ns = ofd->ofd_namespace;
1221
1222                 /* If oa != NULL then ofd_preprw_read updated the inode
1223                  * atime and we should update the lvb so that other glimpses
1224                  * will also get the updated value. bug 5972 */
1225                 if (oa && ns && ns->ns_lvbo && ns->ns_lvbo->lvbo_update) {
1226                          struct ldlm_resource *rs = NULL;
1227
1228                         ost_fid_build_resid(fid, &info->fti_resid);
1229                         rs = ldlm_resource_get(ns, NULL, &info->fti_resid,
1230                                                LDLM_EXTENT, 0);
1231                         if (!IS_ERR(rs)) {
1232                                 ldlm_res_lvbo_update(rs, NULL, 1);
1233                                 ldlm_resource_putref(rs);
1234                         }
1235                 }
1236                 rc = ofd_commitrw_read(env, ofd, fid, objcount,
1237                                        npages, lnb);
1238                 if (old_rc)
1239                         rc = old_rc;
1240         } else {
1241                 LBUG();
1242                 rc = -EPROTO;
1243         }
1244
1245         RETURN(rc);
1246 }