From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeremy Fitzhardinge Subject: Re: xen_hvm_callback_vector Date: Sat, 28 Aug 2010 14:40:19 -0700 Message-ID: <4C798243.7000301@goop.org> References: <4C7842C3.2000300@goop.org> <1283019557.3469.47.camel@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1283019557.3469.47.camel@localhost.localdomain> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Ian Campbell Cc: "Xen-devel@lists.xensource.com" , Stefano Stabellini List-Id: xen-devel@lists.xenproject.org On 08/28/2010 11:19 AM, Ian Campbell wrote: > xen_evtchn_do_upcall is used for the callback vector interrupt injection > (and regular PV obviously). > > xen_hvm_evtchn_do_upcall is used by the old-style PCI IRQ injection > route in platform-pci.c if the callback vector is not available. > > So entry_64.S should be using xen_evtchn_do_upcall. > > Also, I think smp_xen_hvm_callback_vector is simply a redundant > duplicate of xen_evtchn_do_upcall which could be removed. smp_xen_hvm_callback_vector is just missing, so I think it should be using BUILD_INTERRUPT3 with the third arg being xen_evtchn_do_upcall. J