]> xenbits.xensource.com Git - legacy/linux-2.6.18-xen.git/commitdiff
ia64, xencomm: fix 1018:b7eb9756e522
authorKeir Fraser <keir.fraser@citrix.com>
Fri, 11 Jun 2010 08:37:25 +0000 (09:37 +0100)
committerKeir Fraser <keir.fraser@citrix.com>
Fri, 11 Jun 2010 08:37:25 +0000 (09:37 +0100)
This fixes the following errors:
  /arch/ia64/xen/xcom_privcmd.c: In function `xencomm_privcmd_sysctl':
  /arch/ia64/xen/xcom_privcmd.c:295: error: case label not within a
  switch statement
  /arch/ia64/xen/xcom_privcmd.c:305: error: break statement not within
  loop or switch

Since 1018:b7eb9756e522 inserted lines in outside of a switch
statement. This patch corrects it.

Signed-off-by: KUWAMURA Shin'ya <kuwa@jp.fujitsu.com>
arch/ia64/xen/xcom_privcmd.c

index 37742117a897cc8bd43ce748d90b5edce6afd9d6..03f6630481344b4cc399c06ecb03b7fb46c53bd3 100644 (file)
@@ -282,16 +282,6 @@ xencomm_privcmd_sysctl(privcmd_hypercall_t *hypercall)
                break;
        }
 
-       default:
-               printk("%s: unknown sysctl cmd %d\n", __func__, kern_op.cmd);
-               return -ENOSYS;
-       }
-
-       if (ret) {
-               /* error mapping the nested pointer */
-               return ret;
-       }
-
        case XEN_SYSCTL_cpupool_op:
                desc = xencomm_map(
                        xen_guest_handle(kern_op.u.cpupool_op.cpumap.bitmap),
@@ -304,6 +294,16 @@ xencomm_privcmd_sysctl(privcmd_hypercall_t *hypercall)
                                     (void *)desc);
                break;
 
+       default:
+               printk("%s: unknown sysctl cmd %d\n", __func__, kern_op.cmd);
+               return -ENOSYS;
+       }
+
+       if (ret) {
+               /* error mapping the nested pointer */
+               return ret;
+       }
+
        ret = xencomm_arch_hypercall_sysctl(op_desc);
 
        /* FIXME: should we restore the handles?  */