[Xen-changelog] [xen-3.4-testing] Revert 20954:b4041e7bbe1b "paging_domctl: Add missing breaks in switch stmt"

["Xen patchbot-3.4-testing" <patchbot-3.4-testing@xxxxxxxxxxxxxxxxxxx>]

# HG changeset patch
# User Keir Fraser <keir.fraser@xxxxxxxxxx>
# Date 1267204248 0
# Node ID fa6ee088d915e53c075fb5c6e225f5200e1a0a99
# Parent  75bb7e5fc20725cf1a15d6408e4ac2b4cd0f848b
Revert 20954:b4041e7bbe1b "paging_domctl: Add missing breaks in switch stmt"

This fixed a fairly innocuous bug (OP_ENABLE/OP_OFF both don't work
properly) but unmasked a much nastier one (turning off shadow mode on
a PV guest crashes the hypervisor).

So, for now, we pick the less of two evils. We don't really much rely
on OP_ENABLE/OP_OFF anyway, as it happens.

Signed-off-by: Keir Fraser <keir.fraser@xxxxxxxxxx>
xen-unstable changeset:   20987:c4301c2c727d
xen-unstable date:        Fri Feb 26 17:09:50 2010 +0000
---
 xen/arch/x86/mm/paging.c |    2 --
 1 files changed, 2 deletions(-)

diff -r 75bb7e5fc207 -r fa6ee088d915 xen/arch/x86/mm/paging.c
--- a/xen/arch/x86/mm/paging.c  Fri Feb 26 08:14:22 2010 +0000
+++ b/xen/arch/x86/mm/paging.c  Fri Feb 26 17:10:48 2010 +0000
@@ -713,13 +713,11 @@ int paging_domctl(struct domain *d, xen_
                 hap_logdirty_init(d);
             return paging_log_dirty_enable(d);
         }
-        break;
 
     case XEN_DOMCTL_SHADOW_OP_OFF:
         if ( paging_mode_log_dirty(d) )
             if ( (rc = paging_log_dirty_disable(d)) != 0 )
                 return rc;
-        break;
 
     case XEN_DOMCTL_SHADOW_OP_CLEAN:
     case XEN_DOMCTL_SHADOW_OP_PEEK:

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog