Whamcloud - gitweb
LU-3749 recovery: save versions from reply into lastrcvd 86/7786/4
authorMikhail Pershin <mike.pershin@intel.com>
Fri, 27 Sep 2013 15:08:30 +0000 (19:08 +0400)
committerOleg Drokin <oleg.drokin@intel.com>
Wed, 2 Oct 2013 16:50:51 +0000 (16:50 +0000)
Fix bug with using the wrong buffer for reply versions.
Versions should be saved from rq_repmsg instead of rq_reqmsg.
This bug was introduced in commit
4f5830f769f9cc9bacfcdfee9de5962324fb5c81.

Test-Parameters: testgroup=review,failover
Signed-off-by: Mikhail Pershin <mike.pershin@intel.com>
Change-Id: I6881b5d23ec9349b8fe4e9a21e32a1c8be338182
Reviewed-on: http://review.whamcloud.com/7786
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/target/tgt_lastrcvd.c

index 88dd030..2694aaa 100644 (file)
@@ -788,7 +788,7 @@ int tgt_last_rcvd_update(const struct lu_env *env, struct lu_target *tgt,
                ted->ted_lcd->lcd_last_close_result = th->th_result;
        } else {
                /* VBR: save versions in last_rcvd for reconstruct. */
-               __u64 *pre_versions = lustre_msg_get_versions(req->rq_reqmsg);
+               __u64 *pre_versions = lustre_msg_get_versions(req->rq_repmsg);
 
                if (pre_versions) {
                        ted->ted_lcd->lcd_pre_versions[0] = pre_versions[0];