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][VT] Clear the pending interrupt on sharedpage wh

To: "Keir Fraser" <Keir.Fraser@xxxxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH][VT] Clear the pending interrupt on sharedpage when PIC initialized
From: "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx>
Date: Wed, 14 Sep 2005 18:03:48 +0800
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Wed, 14 Sep 2005 10:01:47 +0000
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/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: AcW4P5+STvRQBQ7HQSiY5foRhvquvgA06z2w
Thread-topic: [Xen-devel] [PATCH][VT] Clear the pending interrupt on sharedpage when PIC initialized
Keir Fraser wrote:
> On 13 Sep 2005, at 03:40, Jiang, Yunhong wrote:
> 
>> 1) I think the  cpu_reset_interrupt() is a common API on qemu for
>> hardware interrupt/exception etc,while clear the shared page is just
>> for hardware interrupt. Of course, this function is currently used
>> just for hardware interrupt since qemu works as a device model :)
>> But change here may cause following code on cpu_reset_interrupt().
>>      if ( mask == CPU_INTERRUPT_HARD)
>>           ......clear the shared page...
> 
> Hmm... okay, I see your point here. This would be a bit ugly.
> 
> I think placing your  patch under pic_reset() is a better plan.
> 
>   -- Keir

Attached is the changed patch. Please have a look on it.

Thanks
Yunhong Jiang

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

Attachment: irq_base.diff
Description: irq_base.diff

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
<Prev in Thread] Current Thread [Next in Thread>
  • RE: [Xen-devel] [PATCH][VT] Clear the pending interrupt on sharedpage when PIC initialized, Jiang, Yunhong <=