debuggers.hg

changeset 20840:3f8fd65732cc

find changesets by author, revision, files, or words in the commit message
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>
author Keir Fraser <keir.fraser@citrix.com>
date Wed Jan 13 08:17:00 2010 +0000 (2010-01-13)
parents 4573383ec3dd
children 6512068aa0db
files xen/arch/x86/mm.c
line diff
     1.1 --- a/xen/arch/x86/mm.c	Wed Jan 13 08:16:37 2010 +0000
     1.2 +++ b/xen/arch/x86/mm.c	Wed Jan 13 08:17:00 2010 +0000
     1.3 @@ -4235,7 +4235,6 @@ long arch_memory_op(int op, XEN_GUEST_HA
     1.4          struct xen_add_to_physmap xatp;
     1.5          unsigned long prev_mfn, mfn = 0, gpfn;
     1.6          struct domain *d;
     1.7 -        int rc;
     1.8  
     1.9          if ( copy_from_guest(&xatp, arg, 1) )
    1.10              return -EFAULT;
    1.11 @@ -4347,7 +4346,6 @@ long arch_memory_op(int op, XEN_GUEST_HA
    1.12      {
    1.13          struct xen_foreign_memory_map fmap;
    1.14          struct domain *d;
    1.15 -        int rc;
    1.16  
    1.17          if ( copy_from_guest(&fmap, arg, 1) )
    1.18              return -EFAULT;
    1.19 @@ -4399,7 +4397,6 @@ long arch_memory_op(int op, XEN_GUEST_HA
    1.20          struct memory_map_context ctxt;
    1.21          XEN_GUEST_HANDLE(e820entry_t) buffer;
    1.22          unsigned int i;
    1.23 -        int rc;
    1.24  
    1.25          if ( !IS_PRIV(current->domain) )
    1.26              return -EINVAL;