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

[Wei Liu <wei.liu2@xxxxxxxxxx>]

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


> ~Andrew

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