]> xenbits.xensource.com Git - xen.git/commitdiff
tmem: fixup 2010 cleanup patch that breaks tmem save/restore
authorDan Magenheimer <dan.magenheimer@oracle.com>
Tue, 11 Sep 2012 12:19:03 +0000 (14:19 +0200)
committerDan Magenheimer <dan.magenheimer@oracle.com>
Tue, 11 Sep 2012 12:19:03 +0000 (14:19 +0200)
20918:a3fa6d444b25 "Fix domain reference leaks" (in Feb 2010, by Jan)
does some cleanup in addition to the leak fixes.  Unfortunately, that
cleanup inadvertently resulted in an incorrect fallthrough in a switch
statement which breaks tmem save/restore.

That broken patch was apparently applied to 4.0-testing and 4.1-testing
so those are broken as well.

What is the process now for requesting back-patches to 4.0 and 4.1?

(Side note: This does not by itself entirely fix save/restore in 4.2.)

Signed-off-by: Dan Magenheimer <dan.magenheimer@oracle.com>
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Committed-by: Jan Beulich <jbeulich@suse.com>
xen/common/tmem.c

index 091080e3f1d77d9500572ba754b18559bd53690a..6af8faafec9d2b462f824dc5b6e7594fcb487b54 100644 (file)
@@ -2419,6 +2419,7 @@ static NOINLINE int tmemc_save_subop(int cli_id, uint32_t pool_id,
              break;
         tmh_copy_to_client_buf(buf, pool->uuid, 2);
         rc = 0;
+        break;
     case TMEMC_SAVE_END:
         if ( client == NULL )
             break;
@@ -2429,6 +2430,7 @@ static NOINLINE int tmemc_save_subop(int cli_id, uint32_t pool_id,
                 pgp_free_from_inv_list(client,pgp);
         client->frozen = client->was_frozen;
         rc = 0;
+        break;
     }
     return rc;
 }