Re: [Xen-devel] Regression in OSSTest Windows install test case

[Wei Liu <wei.liu2@xxxxxxxxxx>]

On Wed, Jul 15, 2015 at 03:49:44PM +0100, Paul Durrant wrote:
> > -----Original Message-----
> > From: Wei Liu [mailto:wei.liu2@xxxxxxxxxx]
> > Sent: 15 July 2015 15:46
> > To: Andrew Cooper
> > Cc: Wei Liu; xen-devel@xxxxxxxxxxxxxxxxxxxx; Jan Beulich; Paul Durrant
> > Subject: Re: Regression in OSSTest Windows install test case
> > 
> > On Wed, Jul 15, 2015 at 03:43:09PM +0100, Andrew Cooper wrote:
> > > On 15/07/15 15:40, Wei Liu wrote:
> > > > Hi all
> > > >
> > > > The Windows install test case has been failing reliably in OSSTest for
> > > > the last 6 runs.
> > > >
> > > > The issue manifest as Windows hit blue screen with garbled text that
> > > > cannot be recognised when booting. Xen doesn't complain about any
> > > > failures.
> > > >
> > > > I bisected that and found this commit is to be blamed.
> > > >
> > > > commit 3bbaaec09b1b942f5624dee176da6e416d31f982
> > > > Author: Paul Durrant <paul.durrant@xxxxxxxxxx>
> > > > Date:   Thu Jul 9 19:04:00 2015 +0200
> > > >
> > > >     x86/hvm: unify stdvga mmio intercept with standard mmio intercept
> > > >
> > > >     It's clear from the following check in hvmemul_rep_movs:
> > > >
> > > >         if ( sp2mt == p2m_mmio_direct || dp2mt == p2m_mmio_direct ||
> > > >              (sp2mt == p2m_mmio_dm && dp2mt == p2m_mmio_dm) )
> > > >             return X86EMUL_UNHANDLEABLE;
> > > >
> > > >     that mmio <-> mmio copy is not handled. This means the code in the
> > > >     stdvga mmio intercept that explicitly handles mmio <-> mmio copy
> > when
> > > >     hvm_copy_to/from_guest_phys() fails is never going to be executed.
> > > >
> > > >     This patch therefore adds a check in hvmemul_do_io_addr() to make
> > sure
> > > >     mmio <-> mmio is disallowed and then registers standard mmio
> > intercept ops
> > > >     in stdvga_init().
> > > >
> > > >     With this patch all mmio and portio handled within Xen now goes
> > through
> > > >     process_io_intercept().
> > > >
> > > >     Signed-off-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
> > > >     Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> > > >
> > > > Tell me if you need more information or want me to test a patch. I do
> > > > have test environment at hand.
> > >
> > > Given the nature of the patch, I would not be unduly surprised that the
> > > text is garbled.
> > >
> > > Which windows and qemu are you using?  We have not encountered a
> > single
> > > failure like this in XenServer testing.
> > >
> > 
> > win7-x64.iso from OSSTest.
> > 
> > QEMU emulator version 2.2.1, Copyright (c) 2003-2008 Fabrice Bellard
> > 
> 
> BTW, does the xl cfg specify cirrus or stdvga?
> 
>   Paul

Here is the config I use (slightly modified from the one in OSSTest). No
setting with regard to video card, so it should be the cirrus.

---
name        = 'win'
memory = 512
vif         = [ 'type=ioemu,mac=5a:36:0e:98:50:33' ]
#
on_poweroff = 'destroy'
on_reboot   = 'preserve'
on_crash    = 'preserve'
#
vcpus = 2
#
kernel      = 'hvmloader'
builder     = 'hvm'
#
disk        = [
                'phy:/dev/DATA/win,hda,w',              
'file:/data/win7-x64.iso,hdc:cdrom,r'
              ]
#
usb=1
usbdevice='tablet'
#
#stdvga=1
keymap='en-gb';
#
sdl=0
opengl=0
vnc=1
vncunused=1
vncdisplay=0
vnclisten='0.0.0.0'

#
#boot = 'dc'
boot = 'c'
device_model_version='qemu-xen'
serial='file:/dev/stderr'

#
viridian=1


> 
> > 
> > > ~Andrew

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel