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