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][VTD][PATCH][RESEND]add a timer for the sharedinterrupt i

To: "Keir Fraser" <Keir.Fraser@xxxxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-devel][VTD][PATCH][RESEND]add a timer for the sharedinterrupt issue for vt-d
From: "Xin, Xiaohui" <xiaohui.xin@xxxxxxxxx>
Date: Sat, 20 Oct 2007 19:32:46 +0800
Delivery-date: Sat, 20 Oct 2007 04:33:35 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <C33FA3E3.F324%Keir.Fraser@xxxxxxxxxxxx>
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: <9A1462408D6D394C8A7A812E98F00A4D0226ACE5@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <C33FA3E3.F324%Keir.Fraser@xxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcgQw+ProgCV0z3VTpKlJoAxcNdgrQApmSLpACXmX1AADhg+kgAzHlZQAAE+5vIAAA37gA==
Thread-topic: [Xen-devel][VTD][PATCH][RESEND]add a timer for the sharedinterrupt issue for vt-d

The timer value is an experimental value to tolerate. My thought is that is there some possibility that assert-to-inject is too large and then the inject-to-eoi is too small that real interrupt for the HVM cannot be finished normally?

 

Thanks

Xiaohui  

 


From: Keir Fraser [mailto:Keir.Fraser@xxxxxxxxxxxx]
Sent: 2007
1020 19:24
To: Xin, Xiaohui; xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: Re: [Xen-devel][VTD][PATCH][RESEND]add a timer for the sharedinterrupt issue for vt-d

 

On 20/10/07 12:04, "Xin, Xiaohui" <xiaohui.xin@xxxxxxxxx> wrote:

For 2) Yes, firstly I think to use the flag of the domain as “is_pasued_by_controller”, but from the experience and from the log file, at the situation which IDE shares interrupt with NIC card, and when the NIC is assigned to HVM guest, “is pasued_by_controller” can deal with most cases, but after unpaused, there are still one or more interrupts can be scheduled. The “is_paused_by_controller” can not deal with all the cases.  So I still use timer to deal with that because timer is simple.


How about setting the timer unconditionally in hvm_do_IRQ_dpci() (i.e., as you do now), but then not touching it at all in vmx_dirq_assist()? Is there much point in resetting the timer in vmx_dirq_assist()? It seems to me you should have an assert-to-eoi latency that you are prepared to tolerate. Does separating out assert-to-inject and inject-to-eoi latencies actually make sense?

 -- Keir

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