From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeremy Fitzhardinge Subject: Re: pv_ops dom0 kernel failure with ata_piix / irq problems Date: Fri, 30 Jan 2009 10:50:51 -0800 Message-ID: <49834C0B.1030309@goop.org> References: <495BE5E1.40602@goop.org> <20090102064444.GN15052@edu.joroinen.fi> <20090109174006.GR15052@edu.joroinen.fi> <20090122194645.GJ15052@edu.joroinen.fi> <4978D5BE.8090601@goop.org> <20090124140930.GP15052@edu.joroinen.fi> <20090124145844.GQ15052@edu.joroinen.fi> <497B64B8.5040709@goop.org> <20090124201702.GR15052@edu.joroinen.fi> <20090124234920.GS15052@edu.joroinen.fi> <497BB984.9080803@goop.org> <1233339176.9970.44.camel@zakaz.uk.xensource.com> <1233339380.9970.45.camel@zakaz.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1233339380.9970.45.camel@zakaz.uk.xensource.com> 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: Todd Deshane , Andrew Lyon , xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org Ian Campbell wrote: > On Fri, 2009-01-30 at 18:12 +0000, Ian Campbell wrote: > >> Possibly the correct fix might be to use xen_register_gsi() here >> instead of xen_allocate_pirq() and get rid of the special case for IRQ >> 14 and 15 in xen_pci_pirq_enable(). Maybe only 14 and 15 need this >> special treatment. >> > > FWIW this also Works For Me. > OK. I'd noticed that the native code just sets up all the legacy interrupts at once. I guess we should follow the lead to avoid these kinds of init order problems. J