[Xen-devel] [PATCH] xen, s3: no need to check idle vcpu

To:	"xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject:	[Xen-devel] [PATCH] xen, s3: no need to check idle vcpu
From:	"Tian, Kevin" <kevin.tian@xxxxxxxxx>
Date:	Wed, 19 Nov 2008 10:35:50 +0800
Accept-language:	en-US
Acceptlanguage:	en-US
Delivery-date:	Tue, 18 Nov 2008 18:36:31 -0800
Envelope-to:	www-data@xxxxxxxxxxxxxxxxxxx
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/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe:	<http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender:	xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index:	AclJ74kp4/JQIxF0TDSUrp9xBqaYdw==
Thread-topic:	[PATCH] xen, s3: no need to check idle vcpu

No need to check idle vcpu, as update_vcpu_system_time
already checks v->vcpu_info which is null for idle one.

Signed-off-by Kevin Tian <kevin.tian@xxxxxxxxx>
diff -r df226bb1e95d xen/arch/x86/time.c
--- a/xen/arch/x86/time.c       Wed Nov 19 02:03:00 2008 +0800
+++ b/xen/arch/x86/time.c       Wed Nov 19 02:04:10 2008 +0800
@@ -1239,8 +1239,7 @@ int time_resume(void)
 
     do_settime(get_cmos_time() + cmos_utc_offset, 0, NOW());
 
-    if ( !is_idle_vcpu(current) )
-        update_vcpu_system_time(current);
+    update_vcpu_system_time(current);
 
     return 0;
 }

Thanks,
Kevin

idle_check.patch
Description: idle_check.patch

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

WARNING - OLD ARCHIVES

xen-devel

[Xen-devel] [PATCH] xen, s3: no need to check idle vcpu