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] [RESEND] remove redundent call to hvm_do_resume

To: "Li, Xin B" <xin.b.li@xxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] [RESEND] remove redundent call to hvm_do_resume
From: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Date: Mon, 23 Oct 2006 11:51:08 +0100
Delivery-date: Mon, 23 Oct 2006 03:51:38 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <B30DA1341B0CFA4893EF8A36B40B5C5D4BE013@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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acb0CVpf06Hxya4XT6uW1ULXYRyKAQCGO58QABu3EN4=
Thread-topic: [Xen-devel] [PATCH] [RESEND] remove redundent call to hvm_do_resume
User-agent: Microsoft-Entourage/11.2.5.060620
On 22/10/06 22:38, "Li, Xin B" <xin.b.li@xxxxxxxxx> wrote:

> Remove redundant call to hvm_do_resume.
> In this patch IO event channels in qemu are used for IO done
> notification only, so we can simplify the event checking handling in
> hvm_do_resume.
> Besides current intr notification from qemu is complete meanless,
> because eithor it's done with an IO event, or acturally do thing since
> the new xen consumed event channel, we will find a better way to
> implement that functionality.

I like avoiding hvm_do_resume() on every vm entry, so I took that part of
the patch. The change to ioemu was odd -- there are other places that set
send_event to 1 (the code that adds IRQs to the PIC IRR). We know that code
is somewhat broken for SMP guests, but you *do* need to notify Xen when
interrupts happen, right?

 -- Keir




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

<Prev in Thread] Current Thread [Next in Thread>