Re: [Xen-ia64-devel] [RFC] The "vcpu->arch.privregs" is doubly used

[Masaki Kanno <kanno.masaki@xxxxxxxxxxxxxx>]

Hi,

Sorry. Because size is not same as the mapped_regs_t structure as for 
the vpd structure, my patch cannot solve a problem. Maybe the "destroy" 
processing will become wrong.

Best regards,
 Kan

Masaki Kanno wrote:
>Hi,
>
>I found the "vcpu->arch.privregs" being doubly used.
>Please refer to the following functions.
>
> - alloc_vcpu_struct()         : xen/arch/ia64/xen/domain.c
> - vmx_final_setup_guest()     : xen/arch/ia64/vmx/vmx_init.c
>
>I suggest the following patch. Please comment.
>
>Best regards,
> Kan
>
>diff -r 25003dd43a92 xen/arch/ia64/vmx/vmx_init.c
>--- a/xen/arch/ia64/vmx/vmx_init.c      Fri Mar 17 15:37:28 2006 -0700
>+++ b/xen/arch/ia64/vmx/vmx_init.c      Mon Mar 20 17:35:23 2006 +0900
>@@ -261,6 +261,8 @@ vmx_final_setup_guest(struct vcpu *v)
> {
>        vpd_t *vpd;
> 
>+       free_xenheap_pages(v->arch.privregs, get_order(sizeof(mapped_regs_t)));
>+
>        vpd = alloc_vpd();
>        ASSERT(vpd);
>
>
>
>_______________________________________________
>Xen-ia64-devel mailing list
>Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
>http://lists.xensource.com/xen-ia64-devel
>


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