Re: [Xen-devel] pv_ops dom0 kernel failure with ata_piix / irq problems

[Pasi Kärkkäinen <pasik@xxxxxx>]

On Mon, Feb 02, 2009 at 09:22:17PM +0200, Pasi Kärkkäinen wrote:
> On Mon, Feb 02, 2009 at 10:42:30AM +0200, Pasi Kärkkäinen wrote:
> > On Fri, Jan 30, 2009 at 10:50:51AM -0800, Jeremy Fitzhardinge wrote:
> > > 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.
> > > 
> > 
> > OK.
> > 
> > I've been busy, and haven't yet had time to try the patch.
> > 
> > Hopefully I can try it later today!
> > 
> 
> And now I tried the latest bits. 
> 
> I also applied Ian's legacy irq fix/patch. 
> 
> bootlog:
> http://pasik.reaktio.net/xen/pv_ops-dom0-debug/pv_ops-dom0-bootlog-8-xen331-linux-2.6.29-rc3.txt
> 
> Now my (IDE) disk is detected, but it ata_piix still seems to fail..
> 
> I guess I'll compile new kernel again with libata debug enabled.. 
> 

I enabled the following in include/linux/libata.h:

#define ATA_DEBUG
#define ATA_VERBOSE_DEBUG
#define ATA_IRQ_TRAP

And here's dom0 bootlog with those debugging options enabled:
http://pasik.reaktio.net/xen/pv_ops-dom0-debug/pv_ops-dom0-bootlog-9-xen331-linux-2.6.29-rc3-atadebug.txt

Is there something you'd like me to try/test? 

-- Pasi

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