Whamcloud - gitweb
LU-2139 ofd: Do async commit if SOFT_SYNC is seen
[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.sun.com/software/products/lustre/docs/GPLv2.pdf
19  *
20  * Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara,
21  * CA 95054 USA or visit www.sun.com if you need additional information or
22  * have any questions.
23  *
24  * GPL HEADER END
25  */
26 /*
27  * Copyright (c) 2009, 2010, Oracle and/or its affiliates. All rights reserved.
28  * Use is subject to license terms.
29  *
30  * Copyright (c) 2012, 2013, Intel Corporation.
31  */
32 /*
33  * This file is part of Lustre, http://www.lustre.org/
34  * Lustre is a trademark of Sun Microsystems, Inc.
35  *
36  * lustre/ofd/ofd_io.c
37  *
38  * Author: Alex Tomas <bzzz@whamcloud.com>
39  */
40
41 #define DEBUG_SUBSYSTEM S_FILTER
42
43 #include "ofd_internal.h"
44
45 static int ofd_preprw_read(const struct lu_env *env, struct obd_export *exp,
46                            struct ofd_device *ofd, struct lu_fid *fid,
47                            struct lu_attr *la, int niocount,
48                            struct niobuf_remote *rnb, int *nr_local,
49                            struct niobuf_local *lnb, char *jobid)
50 {
51         struct ofd_object       *fo;
52         int                      i, j, rc, tot_bytes = 0;
53
54         ENTRY;
55         LASSERT(env != NULL);
56
57         fo = ofd_object_find(env, ofd, fid);
58         if (IS_ERR(fo))
59                 RETURN(PTR_ERR(fo));
60         LASSERT(fo != NULL);
61
62         ofd_read_lock(env, fo);
63         if (!ofd_object_exists(fo))
64                 GOTO(unlock, rc = -ENOENT);
65
66         /* parse remote buffers to local buffers and prepare the latter */
67         *nr_local = 0;
68         for (i = 0, j = 0; i < niocount; i++) {
69                 rc = dt_bufs_get(env, ofd_object_child(fo), rnb + i,
70                                  lnb + j, 0, ofd_object_capa(env, fo));
71                 if (unlikely(rc < 0))
72                         GOTO(buf_put, rc);
73                 LASSERT(rc <= PTLRPC_MAX_BRW_PAGES);
74                 /* correct index for local buffers to continue with */
75                 j += rc;
76                 *nr_local += rc;
77                 LASSERT(j <= PTLRPC_MAX_BRW_PAGES);
78                 tot_bytes += rnb[i].rnb_len;
79         }
80
81         LASSERT(*nr_local > 0 && *nr_local <= PTLRPC_MAX_BRW_PAGES);
82         rc = dt_attr_get(env, ofd_object_child(fo), la,
83                          ofd_object_capa(env, fo));
84         if (unlikely(rc))
85                 GOTO(buf_put, rc);
86
87         rc = dt_read_prep(env, ofd_object_child(fo), lnb, *nr_local);
88         if (unlikely(rc))
89                 GOTO(buf_put, rc);
90
91         ofd_counter_incr(exp, LPROC_OFD_STATS_READ, jobid, tot_bytes);
92         RETURN(0);
93
94 buf_put:
95         dt_bufs_put(env, ofd_object_child(fo), lnb, *nr_local);
96 unlock:
97         ofd_read_unlock(env, fo);
98         ofd_object_put(env, fo);
99         return rc;
100 }
101
102 static int ofd_preprw_write(const struct lu_env *env, struct obd_export *exp,
103                             struct ofd_device *ofd, struct lu_fid *fid,
104                             struct lu_attr *la, struct obdo *oa,
105                             int objcount, struct obd_ioobj *obj,
106                             struct niobuf_remote *rnb, int *nr_local,
107                             struct niobuf_local *lnb, char *jobid)
108 {
109         struct ofd_object       *fo;
110         int                      i, j, k, rc = 0, tot_bytes = 0;
111
112         ENTRY;
113         LASSERT(env != NULL);
114         LASSERT(objcount == 1);
115
116         if (unlikely(exp->exp_obd->obd_recovering)) {
117                 struct ofd_thread_info *info = ofd_info(env);
118
119                 /* copied from ofd_precreate_object */
120                 /* XXX this should be consolidated to use the same code
121                  *     instead of a copy, due to the ongoing risk of bugs. */
122                 memset(&info->fti_attr, 0, sizeof(info->fti_attr));
123                 info->fti_attr.la_valid = LA_TYPE | LA_MODE;
124                 info->fti_attr.la_mode = S_IFREG | S_ISUID | S_ISGID | 0666;
125                 info->fti_attr.la_valid |= LA_ATIME | LA_MTIME | LA_CTIME;
126                 /* Initialize a/c/m time so any client timestamp will always
127                  * be newer and update the inode. ctime = 0 is also handled
128                  * specially in osd_inode_setattr().  See LU-221, LU-1042 */
129                 info->fti_attr.la_atime = 0;
130                 info->fti_attr.la_mtime = 0;
131                 info->fti_attr.la_ctime = 0;
132
133                 fo = ofd_object_find_or_create(env, ofd, fid, &info->fti_attr);
134         } else {
135                 fo = ofd_object_find(env, ofd, fid);
136         }
137
138         if (IS_ERR(fo))
139                 GOTO(out, rc = PTR_ERR(fo));
140         LASSERT(fo != NULL);
141
142         ofd_read_lock(env, fo);
143         if (!ofd_object_exists(fo)) {
144                 CERROR("%s: BRW to missing obj "DOSTID"\n",
145                        exp->exp_obd->obd_name, POSTID(&obj->ioo_oid));
146                 ofd_read_unlock(env, fo);
147                 ofd_object_put(env, fo);
148                 GOTO(out, rc = -ENOENT);
149         }
150
151         /* Process incoming grant info, set OBD_BRW_GRANTED flag and grant some
152          * space back if possible */
153         ofd_grant_prepare_write(env, exp, oa, rnb, obj->ioo_bufcnt);
154
155         /* parse remote buffers to local buffers and prepare the latter */
156         *nr_local = 0;
157         for (i = 0, j = 0; i < obj->ioo_bufcnt; i++) {
158                 rc = dt_bufs_get(env, ofd_object_child(fo),
159                                  rnb + i, lnb + j, 1,
160                                  ofd_object_capa(env, fo));
161                 if (unlikely(rc < 0))
162                         GOTO(err, rc);
163                 LASSERT(rc <= PTLRPC_MAX_BRW_PAGES);
164                 /* correct index for local buffers to continue with */
165                 for (k = 0; k < rc; k++) {
166                         lnb[j+k].lnb_flags = rnb[i].rnb_flags;
167                         if (!(rnb[i].rnb_flags & OBD_BRW_GRANTED))
168                                 lnb[j+k].lnb_rc = -ENOSPC;
169
170                         /* remote client can't break through quota */
171                         if (exp_connect_rmtclient(exp))
172                                 lnb[j+k].lnb_flags &= ~OBD_BRW_NOQUOTA;
173                 }
174                 j += rc;
175                 *nr_local += rc;
176                 LASSERT(j <= PTLRPC_MAX_BRW_PAGES);
177                 tot_bytes += rnb[i].rnb_len;
178         }
179         LASSERT(*nr_local > 0 && *nr_local <= PTLRPC_MAX_BRW_PAGES);
180
181         rc = dt_write_prep(env, ofd_object_child(fo), lnb, *nr_local);
182         if (unlikely(rc != 0))
183                 GOTO(err, rc);
184
185         ofd_counter_incr(exp, LPROC_OFD_STATS_WRITE, jobid, tot_bytes);
186         RETURN(0);
187 err:
188         dt_bufs_put(env, ofd_object_child(fo), lnb, *nr_local);
189         ofd_read_unlock(env, fo);
190         /* ofd_grant_prepare_write() was called, so we must commit */
191         ofd_grant_commit(env, exp, rc);
192 out:
193         /* let's still process incoming grant information packed in the oa,
194          * but without enforcing grant since we won't proceed with the write.
195          * Just like a read request actually. */
196         ofd_grant_prepare_read(env, exp, oa);
197         return rc;
198 }
199
200 int ofd_preprw(const struct lu_env *env, int cmd, struct obd_export *exp,
201                struct obdo *oa, int objcount, struct obd_ioobj *obj,
202                struct niobuf_remote *rnb, int *nr_local,
203                struct niobuf_local *lnb, struct obd_trans_info *oti,
204                struct lustre_capa *capa)
205 {
206         struct tgt_session_info *tsi = tgt_ses_info(env);
207         struct ofd_device       *ofd = ofd_exp(exp);
208         struct ofd_thread_info  *info;
209         char                    *jobid;
210         int                      rc = 0;
211
212         if (*nr_local > PTLRPC_MAX_BRW_PAGES) {
213                 CERROR("%s: bulk has too many pages %d, which exceeds the"
214                        "maximum pages per RPC of %d\n",
215                        exp->exp_obd->obd_name, *nr_local, PTLRPC_MAX_BRW_PAGES);
216                 RETURN(-EPROTO);
217         }
218
219         if (tgt_ses_req(tsi) == NULL) { /* echo client case */
220                 LASSERT(oti != NULL);
221                 lu_env_refill((struct lu_env *)env);
222                 info = ofd_info_init(env, exp);
223                 ofd_oti2info(info, oti);
224                 jobid = oti->oti_jobid;
225         } else {
226                 info = tsi2ofd_info(tsi);
227                 jobid = tsi->tsi_jobid;
228         }
229
230         LASSERT(oa != NULL);
231
232         if (OBD_FAIL_CHECK(OBD_FAIL_OST_ENOENT)) {
233                 struct ofd_seq          *oseq;
234
235                 oseq = ofd_seq_load(env, ofd, ostid_seq(&oa->o_oi));
236                 if (IS_ERR(oseq)) {
237                         CERROR("%s: Can not find seq for "DOSTID
238                                ": rc = %ld\n", ofd_name(ofd), POSTID(&oa->o_oi),
239                                PTR_ERR(oseq));
240                         RETURN(-EINVAL);
241                 }
242
243                 if (oseq->os_destroys_in_progress == 0) {
244                         /* don't fail lookups for orphan recovery, it causes
245                          * later LBUGs when objects still exist during
246                          * precreate */
247                         ofd_seq_put(env, oseq);
248                         RETURN(-ENOENT);
249                 }
250                 ofd_seq_put(env, oseq);
251         }
252
253         LASSERT(objcount == 1);
254         LASSERT(obj->ioo_bufcnt > 0);
255
256         rc = ostid_to_fid(&info->fti_fid, &oa->o_oi, 0);
257         if (unlikely(rc != 0))
258                 RETURN(rc);
259
260         if (cmd == OBD_BRW_WRITE) {
261                 rc = ofd_auth_capa(exp, &info->fti_fid, ostid_seq(&oa->o_oi),
262                                    capa, CAPA_OPC_OSS_WRITE);
263                 if (rc == 0) {
264                         la_from_obdo(&info->fti_attr, oa, OBD_MD_FLGETATTR);
265                         rc = ofd_preprw_write(env, exp, ofd, &info->fti_fid,
266                                               &info->fti_attr, oa, objcount,
267                                               obj, rnb, nr_local, lnb, jobid);
268                 }
269         } else if (cmd == OBD_BRW_READ) {
270                 rc = ofd_auth_capa(exp, &info->fti_fid, ostid_seq(&oa->o_oi),
271                                    capa, CAPA_OPC_OSS_READ);
272                 if (rc == 0) {
273                         ofd_grant_prepare_read(env, exp, oa);
274                         rc = ofd_preprw_read(env, exp, ofd, &info->fti_fid,
275                                              &info->fti_attr, obj->ioo_bufcnt,
276                                              rnb, nr_local, lnb, jobid);
277                         obdo_from_la(oa, &info->fti_attr, LA_ATIME);
278                 }
279         } else {
280                 CERROR("%s: wrong cmd %d received!\n",
281                        exp->exp_obd->obd_name, cmd);
282                 rc = -EPROTO;
283         }
284         RETURN(rc);
285 }
286
287 static int
288 ofd_commitrw_read(const struct lu_env *env, struct ofd_device *ofd,
289                   struct lu_fid *fid, int objcount, int niocount,
290                   struct niobuf_local *lnb)
291 {
292         struct ofd_object *fo;
293
294         ENTRY;
295
296         LASSERT(niocount > 0);
297
298         fo = ofd_object_find(env, ofd, fid);
299         if (IS_ERR(fo))
300                 RETURN(PTR_ERR(fo));
301         LASSERT(fo != NULL);
302         LASSERT(ofd_object_exists(fo));
303         dt_bufs_put(env, ofd_object_child(fo), lnb, niocount);
304
305         ofd_read_unlock(env, fo);
306         ofd_object_put(env, fo);
307         /* second put is pair to object_get in ofd_preprw_read */
308         ofd_object_put(env, fo);
309
310         RETURN(0);
311 }
312
313 static int
314 ofd_write_attr_set(const struct lu_env *env, struct ofd_device *ofd,
315                    struct ofd_object *ofd_obj, struct lu_attr *la,
316                    struct filter_fid *ff)
317 {
318         struct ofd_thread_info  *info = ofd_info(env);
319         __u64                    valid = la->la_valid;
320         int                      rc;
321         struct thandle          *th;
322         struct dt_object        *dt_obj;
323         int                      ff_needed = 0;
324
325         ENTRY;
326
327         LASSERT(la);
328
329         dt_obj = ofd_object_child(ofd_obj);
330         LASSERT(dt_obj != NULL);
331
332         la->la_valid &= LA_UID | LA_GID;
333
334         rc = ofd_attr_handle_ugid(env, ofd_obj, la, 0 /* !is_setattr */);
335         if (rc != 0)
336                 GOTO(out, rc);
337
338         if (ff != NULL) {
339                 rc = ofd_object_ff_check(env, ofd_obj);
340                 if (rc == -ENODATA)
341                         ff_needed = 1;
342                 else if (rc < 0)
343                         GOTO(out, rc);
344         }
345
346         if (!la->la_valid && !ff_needed)
347                 /* no attributes to set */
348                 GOTO(out, rc = 0);
349
350         th = ofd_trans_create(env, ofd);
351         if (IS_ERR(th))
352                 GOTO(out, rc = PTR_ERR(th));
353
354         if (la->la_valid) {
355                 rc = dt_declare_attr_set(env, dt_obj, la, th);
356                 if (rc)
357                         GOTO(out_tx, rc);
358         }
359
360         if (ff_needed) {
361                 info->fti_buf.lb_buf = ff;
362                 info->fti_buf.lb_len = sizeof(*ff);
363                 rc = dt_declare_xattr_set(env, dt_obj, &info->fti_buf,
364                                           XATTR_NAME_FID, 0, th);
365                 if (rc)
366                         GOTO(out_tx, rc);
367         }
368
369         /* We don't need a transno for this operation which will be re-executed
370          * anyway when the OST_WRITE (with a transno assigned) is replayed */
371         rc = dt_trans_start_local(env, ofd->ofd_osd , th);
372         if (rc)
373                 GOTO(out_tx, rc);
374
375         /* set uid/gid */
376         if (la->la_valid) {
377                 rc = dt_attr_set(env, dt_obj, la, th,
378                                  ofd_object_capa(env, ofd_obj));
379                 if (rc)
380                         GOTO(out_tx, rc);
381         }
382
383         /* set filter fid EA */
384         if (ff_needed) {
385                 rc = dt_xattr_set(env, dt_obj, &info->fti_buf, XATTR_NAME_FID,
386                                   0, th, BYPASS_CAPA);
387                 if (rc)
388                         GOTO(out_tx, rc);
389         }
390
391         EXIT;
392 out_tx:
393         dt_trans_stop(env, ofd->ofd_osd, th);
394 out:
395         la->la_valid = valid;
396         return rc;
397 }
398
399 struct ofd_soft_sync_callback {
400         struct dt_txn_commit_cb  ossc_cb;
401         struct obd_export       *ossc_exp;
402 };
403
404 static void ofd_cb_soft_sync(struct lu_env *env, struct thandle *th,
405                              struct dt_txn_commit_cb *cb, int err)
406 {
407         struct ofd_soft_sync_callback   *ossc;
408
409         ossc = container_of(cb, struct ofd_soft_sync_callback, ossc_cb);
410
411         CDEBUG(D_INODE, "export %p soft sync count is reset\n", ossc->ossc_exp);
412         atomic_set(&ossc->ossc_exp->exp_filter_data.fed_soft_sync_count, 0);
413
414         class_export_cb_put(ossc->ossc_exp);
415         OBD_FREE_PTR(ossc);
416 }
417
418 static int ofd_soft_sync_cb_add(struct thandle *th, struct obd_export *exp)
419 {
420         struct ofd_soft_sync_callback           *ossc;
421         struct dt_txn_commit_cb                 *dcb;
422         int                                      rc;
423
424         OBD_ALLOC_PTR(ossc);
425         if (ossc == NULL)
426                 return -ENOMEM;
427
428         ossc->ossc_exp = class_export_cb_get(exp);
429
430         dcb = &ossc->ossc_cb;
431         dcb->dcb_func = ofd_cb_soft_sync;
432         CFS_INIT_LIST_HEAD(&dcb->dcb_linkage);
433         strncpy(dcb->dcb_name, "ofd_cb_soft_sync", MAX_COMMIT_CB_STR_LEN);
434         dcb->dcb_name[MAX_COMMIT_CB_STR_LEN - 1] = '\0';
435
436         rc = dt_trans_cb_add(th, dcb);
437         if (rc) {
438                 class_export_cb_put(exp);
439                 OBD_FREE_PTR(ossc);
440         }
441
442         return rc;
443 }
444
445 static int
446 ofd_commitrw_write(const struct lu_env *env, struct obd_export *exp,
447                    struct ofd_device *ofd, struct lu_fid *fid,
448                    struct lu_attr *la, struct filter_fid *ff, int objcount,
449                    int niocount, struct niobuf_local *lnb, int old_rc)
450 {
451         struct ofd_thread_info  *info = ofd_info(env);
452         struct ofd_object       *fo;
453         struct dt_object        *o;
454         struct thandle          *th;
455         int                      rc = 0;
456         int                      retries = 0;
457         int                      i;
458         struct filter_export_data *fed = &exp->exp_filter_data;
459         bool                     soft_sync = false;
460         bool                     cb_registered = false;
461
462         ENTRY;
463
464         LASSERT(objcount == 1);
465
466         fo = ofd_object_find(env, ofd, fid);
467         LASSERT(fo != NULL);
468         LASSERT(ofd_object_exists(fo));
469
470         o = ofd_object_child(fo);
471         LASSERT(o != NULL);
472
473         if (old_rc)
474                 GOTO(out, rc = old_rc);
475
476         /*
477          * The first write to each object must set some attributes.  It is
478          * important to set the uid/gid before calling
479          * dt_declare_write_commit() since quota enforcement is now handled in
480          * declare phases.
481          */
482         rc = ofd_write_attr_set(env, ofd, fo, la, ff);
483         if (rc)
484                 GOTO(out, rc);
485
486         la->la_valid &= LA_ATIME | LA_MTIME | LA_CTIME;
487
488 retry:
489         th = ofd_trans_create(env, ofd);
490         if (IS_ERR(th))
491                 GOTO(out, rc = PTR_ERR(th));
492
493         th->th_sync |= ofd->ofd_syncjournal;
494         if (th->th_sync == 0) {
495                 for (i = 0; i < niocount; i++) {
496                         if (!(lnb[i].lnb_flags & OBD_BRW_ASYNC)) {
497                                 th->th_sync = 1;
498                                 break;
499                         }
500                         if (lnb[i].lnb_flags & OBD_BRW_SOFT_SYNC)
501                                 soft_sync = true;
502                 }
503         }
504
505         if (OBD_FAIL_CHECK(OBD_FAIL_OST_DQACQ_NET))
506                 GOTO(out_stop, rc = -EINPROGRESS);
507
508         rc = dt_declare_write_commit(env, o, lnb, niocount, th);
509         if (rc)
510                 GOTO(out_stop, rc);
511
512         if (la->la_valid) {
513                 /* update [mac]time if needed */
514                 rc = dt_declare_attr_set(env, o, la, th);
515                 if (rc)
516                         GOTO(out_stop, rc);
517         }
518
519         rc = ofd_trans_start(env, ofd, fo, th);
520         if (rc)
521                 GOTO(out_stop, rc);
522
523         rc = dt_write_commit(env, o, lnb, niocount, th);
524         if (rc)
525                 GOTO(out_stop, rc);
526
527         if (la->la_valid) {
528                 rc = dt_attr_set(env, o, la, th, ofd_object_capa(env, fo));
529                 if (rc)
530                         GOTO(out_stop, rc);
531         }
532
533         /* get attr to return */
534         rc = dt_attr_get(env, o, la, ofd_object_capa(env, fo));
535
536 out_stop:
537         /* Force commit to make the just-deleted blocks
538          * reusable. LU-456 */
539         if (rc == -ENOSPC)
540                 th->th_sync = 1;
541
542         /* do this before trans stop in case commit has finished */
543         if (!th->th_sync && soft_sync && !cb_registered) {
544                 ofd_soft_sync_cb_add(th, exp);
545                 cb_registered = true;
546         }
547
548         ofd_trans_stop(env, ofd, th, rc);
549         if (rc == -ENOSPC && retries++ < 3) {
550                 CDEBUG(D_INODE, "retry after force commit, retries:%d\n",
551                        retries);
552                 goto retry;
553         }
554
555         if (!soft_sync)
556                 /* reset fed_soft_sync_count upon non-SOFT_SYNC RPC */
557                 atomic_set(&fed->fed_soft_sync_count, 0);
558         else if (atomic_inc_return(&fed->fed_soft_sync_count) ==
559                  ofd->ofd_soft_sync_limit)
560                 dt_commit_async(env, ofd->ofd_osd);
561
562 out:
563         dt_bufs_put(env, o, lnb, niocount);
564         ofd_read_unlock(env, fo);
565         ofd_object_put(env, fo);
566         /* second put is pair to object_get in ofd_preprw_write */
567         ofd_object_put(env, fo);
568         ofd_grant_commit(env, info->fti_exp, old_rc);
569         RETURN(rc);
570 }
571
572 int ofd_commitrw(const struct lu_env *env, int cmd, struct obd_export *exp,
573                  struct obdo *oa, int objcount, struct obd_ioobj *obj,
574                  struct niobuf_remote *rnb, int npages,
575                  struct niobuf_local *lnb, struct obd_trans_info *oti,
576                  int old_rc)
577 {
578         struct ofd_thread_info  *info = ofd_info(env);
579         struct ofd_mod_data     *fmd;
580         __u64                    valid;
581         struct ofd_device       *ofd = ofd_exp(exp);
582         struct filter_fid       *ff = NULL;
583         int                      rc = 0;
584
585         LASSERT(npages > 0);
586
587         rc = ostid_to_fid(&info->fti_fid, &oa->o_oi, 0);
588         if (unlikely(rc != 0))
589                 RETURN(rc);
590         if (cmd == OBD_BRW_WRITE) {
591                 /* Don't update timestamps if this write is older than a
592                  * setattr which modifies the timestamps. b=10150 */
593
594                 /* XXX when we start having persistent reservations this needs
595                  * to be changed to ofd_fmd_get() to create the fmd if it
596                  * doesn't already exist so we can store the reservation handle
597                  * there. */
598                 valid = OBD_MD_FLUID | OBD_MD_FLGID;
599                 fmd = ofd_fmd_find(exp, &info->fti_fid);
600                 if (!fmd || fmd->fmd_mactime_xid < info->fti_xid)
601                         valid |= OBD_MD_FLATIME | OBD_MD_FLMTIME |
602                                  OBD_MD_FLCTIME;
603                 ofd_fmd_put(exp, fmd);
604                 la_from_obdo(&info->fti_attr, oa, valid);
605
606                 if (oa->o_valid & OBD_MD_FLFID) {
607                         ff = &info->fti_mds_fid;
608                         ofd_prepare_fidea(ff, oa);
609                 }
610
611                 rc = ofd_commitrw_write(env, exp, ofd, &info->fti_fid,
612                                         &info->fti_attr, ff, objcount, npages,
613                                         lnb, old_rc);
614                 if (rc == 0)
615                         obdo_from_la(oa, &info->fti_attr,
616                                      OFD_VALID_FLAGS | LA_GID | LA_UID);
617                 else
618                         obdo_from_la(oa, &info->fti_attr, LA_GID | LA_UID);
619
620                 /* don't report overquota flag if we failed before reaching
621                  * commit */
622                 if (old_rc == 0 && (rc == 0 || rc == -EDQUOT)) {
623                         /* return the overquota flags to client */
624                         if (lnb[0].lnb_flags & OBD_BRW_OVER_USRQUOTA) {
625                                 if (oa->o_valid & OBD_MD_FLFLAGS)
626                                         oa->o_flags |= OBD_FL_NO_USRQUOTA;
627                                 else
628                                         oa->o_flags = OBD_FL_NO_USRQUOTA;
629                         }
630
631                         if (lnb[0].lnb_flags & OBD_BRW_OVER_GRPQUOTA) {
632                                 if (oa->o_valid & OBD_MD_FLFLAGS)
633                                         oa->o_flags |= OBD_FL_NO_GRPQUOTA;
634                                 else
635                                         oa->o_flags = OBD_FL_NO_GRPQUOTA;
636                         }
637
638                         oa->o_valid |= OBD_MD_FLFLAGS;
639                         oa->o_valid |= OBD_MD_FLUSRQUOTA | OBD_MD_FLGRPQUOTA;
640                 }
641         } else if (cmd == OBD_BRW_READ) {
642                 struct ldlm_namespace *ns = ofd->ofd_namespace;
643
644                 /* If oa != NULL then ofd_preprw_read updated the inode
645                  * atime and we should update the lvb so that other glimpses
646                  * will also get the updated value. bug 5972 */
647                 if (oa && ns && ns->ns_lvbo && ns->ns_lvbo->lvbo_update) {
648                          struct ldlm_resource *rs = NULL;
649
650                         ost_fid_build_resid(&info->fti_fid, &info->fti_resid);
651                         rs = ldlm_resource_get(ns, NULL, &info->fti_resid,
652                                                LDLM_EXTENT, 0);
653                         if (rs != NULL) {
654                                 ns->ns_lvbo->lvbo_update(rs, NULL, 1);
655                                 ldlm_resource_putref(rs);
656                         }
657                 }
658                 rc = ofd_commitrw_read(env, ofd, &info->fti_fid, objcount,
659                                        npages, lnb);
660                 if (old_rc)
661                         rc = old_rc;
662         } else {
663                 LBUG();
664                 rc = -EPROTO;
665         }
666
667         if (oti != NULL)
668                 ofd_info2oti(info, oti);
669         RETURN(rc);
670 }