Jan Beulich
2010-Jan-12 11:09 UTC
[Xen-devel] [PATCH] x86: minor cleanup to arch_memory_op()
There''s a function-wide variable rc, so no need to re-declare it in
individual case handling blocks.
Signed-off-by: Jan Beulich <jbeulich@novell.com>
--- 2010-01-06.orig/xen/arch/x86/mm.c 2010-01-06 11:23:46.000000000 +0100
+++ 2010-01-06/xen/arch/x86/mm.c 2010-01-12 12:03:44.000000000 +0100
@@ -4237,7 +4237,6 @@ long arch_memory_op(int op, XEN_GUEST_HA
struct xen_add_to_physmap xatp;
unsigned long prev_mfn, mfn = 0, gpfn;
struct domain *d;
- int rc;
if ( copy_from_guest(&xatp, arg, 1) )
return -EFAULT;
@@ -4349,7 +4348,6 @@ long arch_memory_op(int op, XEN_GUEST_HA
{
struct xen_foreign_memory_map fmap;
struct domain *d;
- int rc;
if ( copy_from_guest(&fmap, arg, 1) )
return -EFAULT;
@@ -4401,7 +4399,6 @@ long arch_memory_op(int op, XEN_GUEST_HA
struct memory_map_context ctxt;
XEN_GUEST_HANDLE(e820entry_t) buffer;
unsigned int i;
- int rc;
if ( !IS_PRIV(current->domain) )
return -EINVAL;
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel