WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

RE: [Xen-devel] [PATCH 0/4] HVM Virtual S3 --- Revised and resent

To: "Ke, Liping" <liping.ke@xxxxxxxxx>, "Keir Fraser" <keir.fraser@xxxxxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH 0/4] HVM Virtual S3 --- Revised and resent
From: "Ke, Liping" <liping.ke@xxxxxxxxx>
Date: Sat, 24 May 2008 08:02:00 +0800
Delivery-date: Fri, 23 May 2008 17:02:47 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <391BF3CDD2DC0848B40ACB72FA97AD59036A7B7C@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <391BF3CDD2DC0848B40ACB72FA97AD5903667B4B@xxxxxxxxxxxxxxxxxxxxxxxxxxxx><C45C5F82.2123E%keir.fraser@xxxxxxxxxxxxx> <391BF3CDD2DC0848B40ACB72FA97AD59036A7B7C@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Aci6Rwan30LCwMVNRjCgsXSrb2rzYwAOb9TQAAFWQHAAAIxGwAAAHx6dAACz6YAAAGA27AAfNwegAG4iWxMAGvX4wAAAeUDA
Thread-topic: [Xen-devel] [PATCH 0/4] HVM Virtual S3 --- Revised and resent
Hi, Keir
I have a rough look and just one thing, I noticed that in arch_vcpu_reset, if 
it is hvm, we don't do destroy_pagetables.
Maybe it will have some problems. Since now s3 sleep down in protected mode
yet wake up in real mode, so cr3 used in protected mode when sleeping down is 
not freed. It will cause
Domain_heap use count != 0, domain_destroy could not be completed totally, some 
resource is not freed.

We found the problem when trying to create a vtd device assigned hvm guest, 
then destroy it and redo create, yet
It failed saying "vtd device is assigned aready".
We tried to log this cr3, put_page of this cr3, it works, so we need 
vcpu_destroy_pagetables.

Yet I am not sure whether the problem still exist after your restructure. I 
will do further try next Monday:)

Thanks& Regards,
Criping



 void arch_vcpu_reset(struct vcpu *v)
 {
-    destroy_gdt(v);
-    vcpu_destroy_pagetables(v);
+    if ( !is_hvm_vcpu(v) )
+    {
+        destroy_gdt(v);
+        vcpu_destroy_pagetables(v);
+    }
+    else
+    {
+        vcpu_end_shutdown_deferral(v);
+    }
 }

Ke, Liping wrote:
> Sure, I will try it on Monday.
> Thanks a lot!
> Criping
> Keir Fraser wrote:
>> I think all these issues are fixed as of c/s 17713. However, when I
>> s3resume a Linux guest I find it is unresponsive and the VGA display
>> is corrupted by re-printing of BIOS start-of-day messages. Perhaps
>> the BIOS is taking an incorrect path on S3 resume? It would be good
>> if you can look into this now -- I think the hypervisor issues at
>> least are now resolved and this is probably something in the
>> higher-level rombios or ioemu logic.
>> 
>>  -- Keir
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel


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