[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] Ping: [PATCH] x86: compat_show_guest_stack() should not truncate MFN


  • To: "xen-devel" <xen-devel@xxxxxxxxxxxxx>
  • From: "Jan Beulich" <JBeulich@xxxxxxxx>
  • Date: Mon, 07 Jan 2013 12:00:49 +0000
  • Delivery-date: Mon, 07 Jan 2013 12:01:22 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xen.org>

>>> On 20.12.12 at 17:54, Jan Beulich wrote:
> Re-using "addr" here was a mistake, as it is a 32-bit quantity.
> 
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> 
> --- a/xen/arch/x86/x86_64/compat/traps.c
> +++ b/xen/arch/x86/x86_64/compat/traps.c
> @@ -20,11 +20,12 @@ void compat_show_guest_stack(struct vcpu
>      if ( v != current )
>      {
>          struct vcpu *vcpu;
> +        unsigned long mfn;
>  
>          ASSERT(guest_kernel_mode(v, regs));
> -        addr = read_cr3() >> PAGE_SHIFT;
> +        mfn = read_cr3() >> PAGE_SHIFT;
>          for_each_vcpu( v->domain, vcpu )
> -            if ( pagetable_get_pfn(vcpu->arch.guest_table) == addr )
> +            if ( pagetable_get_pfn(vcpu->arch.guest_table) == mfn )
>                  break;
>          if ( !vcpu )
>          {
> 
> 
> 




_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.