Bug 727886

Summary: qxl-win: regression: revision 2 not supported because of too strict io range check
Product: Red Hat Enterprise Linux 8 Reporter: Alon Levy <alevy>
Component: spice-qxl-xddmAssignee: Alon Levy <alevy>
Status: CLOSED CURRENTRELEASE QA Contact: Desktop QE <desktop-qa-list>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: ---CC: acathrow, dblechte, pvine
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-07-15 12:20:49 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
fix none

Description Alon Levy 2011-08-03 14:21:17 UTC 
Created attachment 516526 [details]
fix

Description of problem:
driver qxl-win-0.1-8 fails to load on a vm running qxl revision 2 with an io bar of revision 2, i.e. [0..QXL_IO_DESTROY_ALL_SURFACES]

Version-Release number of selected component (if applicable):
qemu-kvm-0.12.1.2-2.160.el6.x86_64

How reproducible:
100%

Steps to Reproduce:
1. start qemu-kvm of the above version with a qxl device (-vga qxl) with a windows vm (xp 32, win7 32 tested by me, others reported to fail the same way: win7 64)
2. try installing the brew built qxl-win-0.1-8 driver (https://brewweb.devel.redhat.com/buildinfo?buildID=174558)
  
Actual results:
Installation fails with Code 10.
Looking in c:\windows\setupapi.log you will see:
#I163 Device not started: Device has problemn: 0x0a: CM_PROB_FAILED_START.
Device manager will show a yellow exclamation mark next to the device name "Red Hat QXL GPU" under "Display Adapter"

Expected results:
Installation successful.

Additional info:
Comment 1 Alon Levy 2011-08-03 14:22:36 UTC 
Note: for beta 1 this is irrelevant, since we are using 2.2 driver. For beta 2, assuming we get the driver to be acceptable otherwise, this is a must.