Re: [Xen-devel] Re: Possible regression in "passthough: add no_wb option for pci conf write"

[Qing He <qing.he@xxxxxxxxx>]

On Mon, 2009-12-07 at 13:29 +0800, Qing He wrote:
> On Mon, 2009-12-07 at 11:43 +0800, Simon Horman wrote:
> > [2009-12-07 12:32:03 3360] DEBUG (DevController:139) Waiting for devices 
> > tap2.
> > [2009-12-07 12:32:03 3360] DEBUG (DevController:139) Waiting for devices 
> > tap.
> > [2009-12-07 12:32:03 3360] DEBUG (DevController:139) Waiting for devices 
> > vtpm.
> > [2009-12-07 12:32:03 3360] INFO (XendDomain:1206) Domain hvm (8) unpaused.
> > [2009-12-07 12:32:03 3360] WARNING (image:544) domain hvm: device model 
> > failure: pid 8472: died due to signal 11; see /var/log/xen/qemu-dm-hvm.log 
> > [2009-12-07 12:32:03 3360] WARNING (XendDomainInfo:1983) Domain has 
> > crashed: name=hvm id=8.
> > [2009-12-07 12:32:03 3360] ERROR (XendDomainInfo:2117) VM hvm restarting 
> > too fast (Elapsed time: 0.161757 seconds). Refusing to restart to avoid 
> > loops.
> > [2009-12-07 12:32:03 3360] DEBUG (XendDomainInfo:2961) 
> > XendDomainInfo.destroy: domid=8
> 
> >From the `bad' log, it seems that the device model actually crashes,
> by receiving a SIGSEGV. So you are obseversing that ioemu window
> disappear at boot time?

Ok, I understand the problem, if pt_find_reg_grp() fails and
returns NULL, it will jump to out:, but at this time reg is
still NULL (pt_find_reg() is not reached) which leads to a
NULL dereference.

I'll fix it, thanks for reporting.

Thanks,
Qing

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

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