Whamcloud - gitweb
- Fixed serious LDLM bugs, including mistaking the lock _type_ for the lock
[fs/lustre-release.git] / lustre / mds / mds_reint.c
1 /* -*- mode: c; c-basic-offset: 8; indent-tabs-mode: nil; -*-
2  * vim:expandtab:shiftwidth=8:tabstop=8:
3  *
4  *  linux/mds/mds_reint.c
5  *
6  *  Lustre Metadata Server (mds) reintegration routines
7  *
8  *  Copyright (C) 2002  Cluster File Systems, Inc.
9  *  author: Peter Braam <braam@clusterfs.com>
10  *
11  *  This code is issued under the GNU General Public License.
12  *  See the file COPYING in this distribution
13  *
14  */
15
16 // XXX - add transaction sequence numbers
17
18 #define EXPORT_SYMTAB
19 #define DEBUG_SUBSYSTEM S_MDS
20
21 #include <linux/obd_support.h>
22 #include <linux/obd_class.h>
23 #include <linux/obd.h>
24 #include <linux/lustre_lib.h>
25 #include <linux/lustre_idl.h>
26 #include <linux/lustre_mds.h>
27 #include <linux/lustre_dlm.h>
28 #include <linux/obd_class.h>
29
30 struct mds_client_info *mds_uuid_to_mci(struct mds_obd *mds, __u8 *uuid)
31 {
32         struct list_head *p;
33
34         if (!uuid)
35                 return NULL;
36
37         list_for_each(p, &mds->mds_client_info) {
38                 struct mds_client_info *mci;
39
40                 mci = list_entry(p, struct mds_client_info, mci_list);
41                 CDEBUG(D_INFO, "checking client UUID '%s'\n",
42                        mci->mci_mcd->mcd_uuid);
43                 if (!strncmp(mci->mci_mcd->mcd_uuid, uuid,
44                              sizeof(mci->mci_mcd->mcd_uuid)))
45                         return mci;
46         }
47         CDEBUG(D_INFO, "no mds client info found for  UUID '%s'\n", uuid);
48         return NULL;
49 }
50
51 int mds_update_last_rcvd(struct mds_obd *mds, void *handle,
52                          struct ptlrpc_request *req)
53 {
54         /* get from req->rq_connection-> or req->rq_client */
55         struct obd_run_ctxt saved;
56         struct mds_client_info *mci;
57         loff_t off;
58         int rc;
59
60         mci = mds_uuid_to_mci(mds, req->rq_connection->c_remote_uuid);
61         if (!mci) {
62                 CERROR("unable to locate MDS client data for UUID '%s'\n",
63                        ptlrpc_req_to_uuid(req));
64                 /* This will be a real error once everything is working */
65                 //LBUG();
66                 RETURN(0);
67         }
68
69         off = MDS_LR_CLIENT + mci->mci_off * MDS_LR_SIZE;
70
71         ++mds->mds_last_rcvd;   /* lock this, or make it an LDLM function? */
72         req->rq_repmsg->transno = HTON__u64(mds->mds_last_rcvd);
73         mci->mci_mcd->mcd_last_rcvd = cpu_to_le64(mds->mds_last_rcvd);
74         mci->mci_mcd->mcd_mount_count = cpu_to_le64(mds->mds_mount_count);
75         mci->mci_mcd->mcd_last_xid = cpu_to_le64(req->rq_xid);
76
77         mds_fs_set_last_rcvd(mds, handle);
78         push_ctxt(&saved, &mds->mds_ctxt);
79         rc = lustre_fwrite(mds->mds_rcvd_filp, (char *)mci->mci_mcd,
80                            sizeof(*mci->mci_mcd), &off);
81         pop_ctxt(&saved);
82         CDEBUG(D_INODE, "wrote trans #%Ld for client '%s' at #%d: rc = %d\n",
83                mds->mds_last_rcvd, mci->mci_mcd->mcd_uuid, mci->mci_off, rc);
84         // store new value and last committed value in req struct
85
86         if (rc == sizeof(*mci->mci_mcd))
87                 rc = 0;
88         else {
89                 CERROR("error writing to last_rcvd file: rc = %d\n", rc);
90                 if (rc >= 0)
91                         rc = -EIO;
92         }
93
94         return rc;
95 }
96
97 static int mds_reint_setattr(struct mds_update_record *rec, int offset,
98                              struct ptlrpc_request *req)
99 {
100         struct mds_obd *mds = &req->rq_obd->u.mds;
101         struct dentry *de;
102         void *handle;
103         int rc = 0;
104         int err;
105
106         de = mds_fid2dentry(mds, rec->ur_fid1, NULL);
107         if (IS_ERR(de) || OBD_FAIL_CHECK(OBD_FAIL_MDS_REINT_SETATTR)) {
108                 GOTO(out_setattr, rc = -ESTALE);
109         }
110
111         CDEBUG(D_INODE, "ino %ld\n", de->d_inode->i_ino);
112
113         OBD_FAIL_WRITE(OBD_FAIL_MDS_REINT_SETATTR_WRITE,
114                        de->d_inode->i_sb->s_dev);
115
116         handle = mds_fs_start(mds, de->d_inode, MDS_FSOP_SETATTR);
117         if (!handle)
118                 GOTO(out_setattr_de, rc = PTR_ERR(handle));
119         rc = mds_fs_setattr(mds, de, handle, &rec->ur_iattr);
120
121         if (!rc)
122                 rc = mds_update_last_rcvd(mds, handle, req);
123
124         err = mds_fs_commit(mds, de->d_inode, handle);
125         if (err) {
126                 CERROR("error on commit: err = %d\n", err);
127                 if (!rc)
128                         rc = err;
129         }
130         EXIT;
131 out_setattr_de:
132         l_dput(de);
133 out_setattr:
134         req->rq_status = rc;
135         return(0);
136 }
137
138 static int mds_reint_recreate(struct mds_update_record *rec, int offset,
139                             struct ptlrpc_request *req)
140 {
141         struct dentry *de = NULL;
142         struct mds_obd *mds = &req->rq_obd->u.mds;
143         struct dentry *dchild = NULL;
144         struct inode *dir;
145         int rc = 0;
146         ENTRY;
147
148         de = mds_fid2dentry(mds, rec->ur_fid1, NULL);
149         if (IS_ERR(de) || OBD_FAIL_CHECK(OBD_FAIL_MDS_REINT_CREATE)) {
150                 LBUG();
151                 GOTO(out_create_de, rc = -ESTALE);
152         }
153         dir = de->d_inode;
154         CDEBUG(D_INODE, "parent ino %ld\n", dir->i_ino);
155
156         down(&dir->i_sem);
157         dchild = lookup_one_len(rec->ur_name, de, rec->ur_namelen - 1);
158         if (IS_ERR(dchild)) {
159                 CERROR("child lookup error %ld\n", PTR_ERR(dchild));
160                 up(&dir->i_sem);
161                 LBUG();
162                 GOTO(out_create_dchild, rc = -ESTALE);
163         }
164
165         if (dchild->d_inode) {
166                 struct mds_body *body;
167                 rc = 0;
168                 body = lustre_msg_buf(req->rq_repmsg, 0);
169                 body->ino = dchild->d_inode->i_ino;
170                 body->generation = dchild->d_inode->i_generation;
171         } else {
172                 CERROR("child doesn't exist (dir %ld, name %s)\n",
173                        dir->i_ino, rec->ur_name);
174                 rc = -ENOENT;
175                 LBUG();
176         }
177
178 out_create_dchild:
179         l_dput(dchild);
180         up(&dir->i_sem);
181 out_create_de:
182         l_dput(de);
183         req->rq_status = rc;
184         return 0;
185 }
186
187 static int mds_reint_create(struct mds_update_record *rec, int offset,
188                             struct ptlrpc_request *req)
189 {
190         struct dentry *de = NULL;
191         struct mds_obd *mds = &req->rq_obd->u.mds;
192         struct dentry *dchild = NULL;
193         struct inode *dir;
194         void *handle;
195         struct ldlm_lock *lock;
196         struct lustre_handle lockh;
197         int rc = 0, err, flags, lock_mode, type = rec->ur_mode & S_IFMT;
198         __u64 res_id[3] = {0,0,0};
199         ENTRY;
200
201         /* requests were at offset 2, replies go back at 1 */
202         if (offset)
203                 offset = 1;
204
205         if (strcmp(req->rq_obd->obd_type->typ_name, "mds") != 0)
206                 LBUG();
207
208         de = mds_fid2dentry(mds, rec->ur_fid1, NULL);
209         if (IS_ERR(de) || OBD_FAIL_CHECK(OBD_FAIL_MDS_REINT_CREATE)) {
210                 LBUG();
211                 GOTO(out_create_de, rc = -ESTALE);
212         }
213         dir = de->d_inode;
214         CDEBUG(D_INODE, "parent ino %ld name %s mode %o\n",
215                dir->i_ino, rec->ur_name, rec->ur_mode);
216
217         lock_mode = (req->rq_reqmsg->opc == MDS_REINT) ? LCK_CW : LCK_PW;
218         res_id[0] = dir->i_ino;
219
220         rc = ldlm_local_lock_match(mds->mds_local_namespace, res_id, LDLM_PLAIN,
221                                    NULL, 0, lock_mode, &lockh);
222         if (rc == 0) {
223                 rc = ldlm_cli_enqueue(mds->mds_ldlm_client, mds->mds_ldlm_conn,
224                                       NULL, mds->mds_local_namespace, NULL,
225                                       res_id, LDLM_PLAIN, NULL, 0, lock_mode,
226                                       &flags, (void *)mds_lock_callback, NULL,
227                                       0, &lockh);
228                 if (rc != ELDLM_OK) {
229                         CERROR("lock enqueue: err: %d\n", rc);
230                         GOTO(out_create_de, rc = -EIO);
231                 }
232         }
233         ldlm_lock_dump((void *)(unsigned long)lockh.addr);
234
235         down(&dir->i_sem);
236         dchild = lookup_one_len(rec->ur_name, de, rec->ur_namelen - 1);
237         if (IS_ERR(dchild)) {
238                 CERROR("child lookup error %ld\n", PTR_ERR(dchild));
239                 up(&dir->i_sem);
240                 LBUG();
241                 GOTO(out_create_dchild, rc = -ESTALE);
242         }
243
244         if (dchild->d_inode) {
245                 struct mds_body *body;
246                 struct obdo *obdo;
247                 struct inode *inode = dchild->d_inode;
248                 CERROR("child exists (dir %ld, name %s, ino %ld)\n",
249                        dir->i_ino, rec->ur_name, dchild->d_inode->i_ino);
250
251                 body = lustre_msg_buf(req->rq_repmsg, offset);
252                 mds_pack_inode2fid(&body->fid1, inode);
253                 mds_pack_inode2body(body, inode);
254 #warning FIXME: This ext3/N-specific code does not belong here
255                 /* If i_file_acl is set, this inode has an EA */
256                 if (S_ISREG(inode->i_mode) && inode->u.ext3_i.i_file_acl) {
257                         obdo = lustre_msg_buf(req->rq_repmsg, offset + 1);
258                         mds_fs_get_obdo(mds, inode, obdo);
259                 }
260                 /* now a normal case for intent locking */
261                 GOTO(out_create_dchild, rc = -EEXIST);
262         }
263
264         OBD_FAIL_WRITE(OBD_FAIL_MDS_REINT_CREATE_WRITE, dir->i_sb->s_dev);
265
266         switch (type) {
267         case S_IFREG: {
268                 handle = mds_fs_start(mds, dir, MDS_FSOP_CREATE);
269                 if (!handle)
270                         GOTO(out_create_dchild, PTR_ERR(handle));
271                 rc = vfs_create(dir, dchild, rec->ur_mode);
272                 EXIT;
273                 break;
274         }
275         case S_IFDIR: {
276                 handle = mds_fs_start(mds, dir, MDS_FSOP_MKDIR);
277                 if (!handle)
278                         GOTO(out_create_dchild, PTR_ERR(handle));
279                 rc = vfs_mkdir(dir, dchild, rec->ur_mode);
280                 EXIT;
281                 break;
282         }
283         case S_IFLNK: {
284                 handle = mds_fs_start(mds, dir, MDS_FSOP_SYMLINK);
285                 if (!handle)
286                         GOTO(out_create_dchild, PTR_ERR(handle));
287                 rc = vfs_symlink(dir, dchild, rec->ur_tgt);
288                 EXIT;
289                 break;
290         }
291         case S_IFCHR:
292         case S_IFBLK:
293         case S_IFIFO:
294         case S_IFSOCK: {
295                 int rdev = rec->ur_rdev;
296                 handle = mds_fs_start(mds, dir, MDS_FSOP_MKNOD);
297                 if (!handle)
298                         GOTO(out_create_dchild, PTR_ERR(handle));
299                 rc = vfs_mknod(dir, dchild, rec->ur_mode, rdev);
300                 EXIT;
301                 break;
302         }
303         default:
304                 CERROR("bad file type %d for create of %s\n",type,rec->ur_name);
305                 GOTO(out_create_dchild, rc = -EINVAL);
306         }
307
308         if (rc) {
309                 CERROR("error during create: %d\n", rc);
310                 if (rc != -ENOSPC) LBUG();
311                 GOTO(out_create_commit, rc);
312         } else {
313                 struct iattr iattr;
314                 struct inode *inode = dchild->d_inode;
315                 struct mds_body *body;
316
317                 CDEBUG(D_INODE, "created ino %ld\n", dchild->d_inode->i_ino);
318                 if (!offset && type == S_IFREG) {
319                         struct obdo *obdo;
320                         obdo = lustre_msg_buf(req->rq_reqmsg, 2);
321                         rc = mds_fs_set_obdo(mds, inode, handle, obdo);
322                         if (rc)
323                                 CERROR("error %d setting objid for %ld\n",
324                                        rc, inode->i_ino);
325                 }
326
327                 iattr.ia_atime = rec->ur_time;
328                 iattr.ia_ctime = rec->ur_time;
329                 iattr.ia_mtime = rec->ur_time;
330                 iattr.ia_uid = rec->ur_uid;
331                 iattr.ia_gid = rec->ur_gid;
332                 iattr.ia_valid = ATTR_UID | ATTR_GID | ATTR_ATIME |
333                         ATTR_MTIME | ATTR_CTIME;
334
335                 rc = mds_fs_setattr(mds, dchild, handle, &iattr);
336                 if (rc) {
337                         CERROR("error on setattr: rc = %d\n", rc);
338                         /* XXX should we abort here in case of error? */
339                 }
340
341                 rc = mds_update_last_rcvd(mds, handle, req);
342                 if (rc) {
343                         CERROR("error on update_last_rcvd: rc = %d\n", rc);
344                         /* XXX should we abort here in case of error? */
345                 }
346
347                 body = lustre_msg_buf(req->rq_repmsg, offset);
348                 body->ino = inode->i_ino;
349                 body->generation = inode->i_generation;
350         }
351         EXIT;
352 out_create_commit:
353         err = mds_fs_commit(mds, dir, handle);
354         if (err) {
355                 CERROR("error on commit: err = %d\n", err);
356                 if (!rc)
357                         rc = err;
358         }
359 out_create_dchild:
360         l_dput(dchild);
361         up(&dir->i_sem);
362         lock = lustre_handle2object(&lockh);
363         ldlm_lock_decref(lock, lock_mode);
364 out_create_de:
365         l_dput(de);
366         req->rq_status = rc;
367         return 0;
368 }
369
370 static int mds_reint_unlink(struct mds_update_record *rec, int offset,
371                             struct ptlrpc_request *req)
372 {
373         struct dentry *de = NULL;
374         struct dentry *dchild = NULL;
375         struct mds_obd *mds = &req->rq_obd->u.mds;
376         struct inode *dir, *inode;
377         void *handle;
378         int rc = 0;
379         int err;
380         ENTRY;
381
382         de = mds_fid2dentry(mds, rec->ur_fid1, NULL);
383         if (IS_ERR(de) || OBD_FAIL_CHECK(OBD_FAIL_MDS_REINT_UNLINK)) {
384                 LBUG();
385                 GOTO(out_unlink, rc = -ESTALE);
386         }
387         dir = de->d_inode;
388         CDEBUG(D_INODE, "parent ino %ld\n", dir->i_ino);
389
390         down(&dir->i_sem);
391         dchild = lookup_one_len(rec->ur_name, de, rec->ur_namelen - 1);
392         if (IS_ERR(dchild)) {
393                 CERROR("child lookup error %ld\n", PTR_ERR(dchild));
394                 LBUG();
395                 GOTO(out_unlink_de, rc = -ESTALE);
396         }
397
398         inode = dchild->d_inode;
399         if (!inode) {
400                 CERROR("child doesn't exist (dir %ld, name %s\n",
401                        dir->i_ino, rec->ur_name);
402                 LBUG();
403                 GOTO(out_unlink_dchild, rc = -ESTALE);
404         }
405
406         if (inode->i_ino != rec->ur_fid2->id) {
407                 CERROR("inode and FID ID do not match (%ld != %Ld)\n",
408                        inode->i_ino, rec->ur_fid2->id);
409                 LBUG();
410                 GOTO(out_unlink_dchild, rc = -ESTALE);
411         }
412         if (inode->i_generation != rec->ur_fid2->generation) {
413                 CERROR("inode and FID GENERATION do not match (%d != %d)\n",
414                        inode->i_generation, rec->ur_fid2->generation);
415                 LBUG();
416                 GOTO(out_unlink_dchild, rc = -ESTALE);
417         }
418
419         OBD_FAIL_WRITE(OBD_FAIL_MDS_REINT_UNLINK_WRITE, dir->i_sb->s_dev);
420
421         switch (dchild->d_inode->i_mode & S_IFMT) {
422         case S_IFDIR:
423                 handle = mds_fs_start(mds, dir, MDS_FSOP_RMDIR);
424                 if (!handle)
425                         GOTO(out_unlink_dchild, rc = PTR_ERR(handle));
426                 rc = vfs_rmdir(dir, dchild);
427                 break;
428         default:
429                 handle = mds_fs_start(mds, dir, MDS_FSOP_UNLINK);
430                 if (!handle)
431                         GOTO(out_unlink_dchild, rc = PTR_ERR(handle));
432                 rc = vfs_unlink(dir, dchild);
433                 break;
434         }
435
436         if (!rc)
437                 rc = mds_update_last_rcvd(mds, handle, req);
438         err = mds_fs_commit(mds, dir, handle);
439         if (err) {
440                 CERROR("error on commit: err = %d\n", err);
441                 if (!rc)
442                         rc = err;
443         }
444
445         EXIT;
446 out_unlink_dchild:
447         l_dput(dchild);
448 out_unlink_de:
449         up(&dir->i_sem);
450         l_dput(de);
451 out_unlink:
452         req->rq_status = rc;
453         return 0;
454 }
455
456 static int mds_reint_link(struct mds_update_record *rec, int offset,
457                             struct ptlrpc_request *req)
458 {
459         struct dentry *de_src = NULL;
460         struct dentry *de_tgt_dir = NULL;
461         struct dentry *dchild = NULL;
462         struct mds_obd *mds = &req->rq_obd->u.mds;
463         void *handle;
464         int rc = 0;
465         int err;
466
467         ENTRY;
468         de_src = mds_fid2dentry(mds, rec->ur_fid1, NULL);
469         if (IS_ERR(de_src) || OBD_FAIL_CHECK(OBD_FAIL_MDS_REINT_LINK)) {
470                 GOTO(out_link, rc = -ESTALE);
471         }
472
473         de_tgt_dir = mds_fid2dentry(mds, rec->ur_fid2, NULL);
474         if (IS_ERR(de_tgt_dir)) {
475                 GOTO(out_link_de_src, rc = -ESTALE);
476         }
477
478         down(&de_tgt_dir->d_inode->i_sem);
479         dchild = lookup_one_len(rec->ur_name, de_tgt_dir, rec->ur_namelen - 1);
480         if (IS_ERR(dchild)) {
481                 CERROR("child lookup error %ld\n", PTR_ERR(dchild));
482                 GOTO(out_link_de_tgt_dir, rc = -ESTALE);
483         }
484
485         if (dchild->d_inode) {
486                 CERROR("child exists (dir %ld, name %s\n",
487                        de_tgt_dir->d_inode->i_ino, rec->ur_name);
488                 GOTO(out_link_dchild, rc = -EEXIST);
489         }
490
491         OBD_FAIL_WRITE(OBD_FAIL_MDS_REINT_LINK_WRITE,
492                        de_src->d_inode->i_sb->s_dev);
493
494         handle = mds_fs_start(mds, de_tgt_dir->d_inode, MDS_FSOP_LINK);
495         if (!handle)
496                 GOTO(out_link_dchild, rc = PTR_ERR(handle));
497
498         rc = vfs_link(de_src, de_tgt_dir->d_inode, dchild);
499
500         if (!rc)
501                 rc = mds_update_last_rcvd(mds, handle, req);
502
503         err = mds_fs_commit(mds, de_tgt_dir->d_inode, handle);
504         if (err) {
505                 CERROR("error on commit: err = %d\n", err);
506                 if (!rc)
507                         rc = err;
508         }
509         EXIT;
510
511 out_link_dchild:
512         l_dput(dchild);
513 out_link_de_tgt_dir:
514         up(&de_tgt_dir->d_inode->i_sem);
515         l_dput(de_tgt_dir);
516 out_link_de_src:
517         l_dput(de_src);
518 out_link:
519         req->rq_status = rc;
520         return 0;
521 }
522
523 static int mds_reint_rename(struct mds_update_record *rec, int offset,
524                             struct ptlrpc_request *req)
525 {
526         struct dentry *de_srcdir = NULL;
527         struct dentry *de_tgtdir = NULL;
528         struct dentry *de_old = NULL;
529         struct dentry *de_new = NULL;
530         struct mds_obd *mds = &req->rq_obd->u.mds;
531         void *handle;
532         int rc = 0;
533         int err;
534         ENTRY;
535
536         de_srcdir = mds_fid2dentry(mds, rec->ur_fid1, NULL);
537         if (IS_ERR(de_srcdir)) {
538                 GOTO(out_rename, rc = -ESTALE);
539         }
540
541         de_tgtdir = mds_fid2dentry(mds, rec->ur_fid2, NULL);
542         if (IS_ERR(de_tgtdir)) {
543                 GOTO(out_rename_srcdir, rc = -ESTALE);
544         }
545
546 #warning FIXME: This needs locking attention
547
548         de_old = lookup_one_len(rec->ur_name, de_srcdir, rec->ur_namelen - 1);
549         if (IS_ERR(de_old)) {
550                 CERROR("old child lookup error %ld\n", PTR_ERR(de_old));
551                 GOTO(out_rename_tgtdir, rc = -ESTALE);
552         }
553
554         de_new = lookup_one_len(rec->ur_tgt, de_tgtdir, rec->ur_tgtlen - 1);
555         if (IS_ERR(de_new)) {
556                 CERROR("new child lookup error %ld\n", PTR_ERR(de_new));
557                 GOTO(out_rename_deold, rc = -ESTALE);
558         }
559
560         OBD_FAIL_WRITE(OBD_FAIL_MDS_REINT_RENAME_WRITE,
561                        de_srcdir->d_inode->i_sb->s_dev);
562
563         handle = mds_fs_start(mds, de_tgtdir->d_inode, MDS_FSOP_RENAME);
564         if (!handle)
565                 GOTO(out_rename_denew, rc = PTR_ERR(handle));
566         rc = vfs_rename(de_srcdir->d_inode, de_old, de_tgtdir->d_inode, de_new,
567                         NULL);
568
569         if (!rc)
570                 rc = mds_update_last_rcvd(mds, handle, req);
571
572         err = mds_fs_commit(mds, de_tgtdir->d_inode, handle);
573         if (err) {
574                 CERROR("error on commit: err = %d\n", err);
575                 if (!rc)
576                         rc = err;
577         }
578         EXIT;
579
580 out_rename_denew:
581         l_dput(de_new);
582 out_rename_deold:
583         l_dput(de_old);
584 out_rename_tgtdir:
585         l_dput(de_tgtdir);
586 out_rename_srcdir:
587         l_dput(de_srcdir);
588 out_rename:
589         req->rq_status = rc;
590         return 0;
591 }
592
593 typedef int (*mds_reinter)(struct mds_update_record *, int offset,
594                            struct ptlrpc_request *);
595
596 static mds_reinter reinters[REINT_MAX+1] = {
597         [REINT_SETATTR]   mds_reint_setattr,
598         [REINT_CREATE]    mds_reint_create,
599         [REINT_UNLINK]    mds_reint_unlink,
600         [REINT_LINK]      mds_reint_link,
601         [REINT_RENAME]    mds_reint_rename,
602         [REINT_RECREATE]  mds_reint_recreate,
603 };
604
605 int mds_reint_rec(struct mds_update_record *rec, int offset,
606                   struct ptlrpc_request *req)
607 {
608         int rc;
609
610         if (rec->ur_opcode < 1 || rec->ur_opcode > REINT_MAX) {
611                 CERROR("opcode %d not valid\n", rec->ur_opcode);
612                 rc = req->rq_status = -EINVAL;
613                 RETURN(rc);
614         }
615
616         rc = reinters[rec->ur_opcode](rec, offset, req);
617
618         return rc;
619 }