Whamcloud - gitweb
LU-4612 lvfs: correct call to pop_ctxt 24/9224/4
authorBob Glossman <bob.glossman@intel.com>
Tue, 11 Feb 2014 16:51:13 +0000 (08:51 -0800)
committerOleg Drokin <oleg.drokin@intel.com>
Wed, 12 Feb 2014 22:44:24 +0000 (22:44 +0000)
Earlier commit 3e7573cc14a331f01150814495e2345793e22f06 that converted
a call of osd_pop_ctxt() to pop_ctxt() ignored the fact the argument
order of these routines were different. This led to panics.

This patch fixes the pop_ctxt() call by putting the arguments
in the correct order.

Signed-off-by: Bob Glossman <bob.glossman@intel.com>
Change-Id: I352250fe3ed91cb8a23a5b8e88b944dc8309b481
Reviewed-on: http://review.whamcloud.com/9224
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Minh Diep <minh.diep@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/osd-ldiskfs/osd_compat.c

index b4a018c..6ec1129 100644 (file)
@@ -411,7 +411,7 @@ static int osd_ost_init(const struct lu_env *env, struct osd_device *dev)
        GOTO(cleanup, rc);
 
 cleanup:
        GOTO(cleanup, rc);
 
 cleanup:
-       pop_ctxt(&new, &save);
+       pop_ctxt(&save, &new);
         if (IS_ERR(d)) {
                 OBD_FREE_PTR(dev->od_ost_map);
                 RETURN(PTR_ERR(d));
         if (IS_ERR(d)) {
                 OBD_FREE_PTR(dev->od_ost_map);
                 RETURN(PTR_ERR(d));