Bug 19317

Summary: [G45 GEM] readpix fails with tiling on when using two different sizes memory banks(1G,512MB)
Product: Mesa Reporter: lin, jiewen <jiewen.lin>
Component: Drivers/DRI/i965Assignee: Eric Anholt <eric>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: medium CC: haihao.xiang
Version: unspecified   
Hardware: x86-64 (AMD64)   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Bug Depends on:    
Bug Blocks: 18841, 20277    
Attachments: log
conf

Description lin, jiewen 2008-12-28 21:24:45 UTC
Created attachment 21536 [details]
log

System Environment:
--------------------------

--Platform: G45
--Architecture(32-bit,64-bit,compatiblity): 64-bit
--2D driver:
(xf86-video-intel-2.6-branch)83f3c376b5942e134047a220e6e5f2432ffc492c
--3D driver: (intel-2008-q4)f96baeaac3ef41260ac3975750627ece073fdce0
--DRM: shipped with kernel 2.6.28
--libdrm: (master)0243c9f801a35de3465a0321c02f18a4d07ce5b8
--Xserver:  (server-1.6-branch)32e81074b967716865aef08b66ec29caf0fec2c5
--Kernel: 2.6.28

Bug detailed description:
--------------------------
start X and run readpix, the picture display incorrectly with tiling on when using two memory banks. It works well with tiling enabled when using one memory bank only.
Reproduce steps:
----------------
1. xinit &
2. ./readpix
Comment 1 lin, jiewen 2008-12-28 21:25:23 UTC
Created attachment 21537 [details]
conf
Comment 2 Eric Anholt 2009-01-06 23:03:30 UTC
working fine here on dual-channel g45.
Comment 3 lin, jiewen 2009-01-07 19:22:25 UTC
(In reply to comment #2)
> working fine here on dual-channel g45.
> 

Yes, but now the issue is: if one is 1G and the other is 512MB, readpix fails with tiling on . We try this with any combination of two memory slots.
Comment 4 lin, jiewen 2009-01-07 23:39:45 UTC
Our q965 does not has this issue, it may be g45 issue only.
Comment 5 lin, jiewen 2009-01-16 00:24:14 UTC
This issue still eixsts with current build.
Comment 6 Eric Anholt 2009-10-30 17:39:12 UTC
commit 7c8bed62e0165a0be3363f7abf81bf9e30341e00
Author: Eric Anholt <eric@anholt.net>
Date:   Fri Oct 30 15:33:11 2009 -0700

    intel: Use GTT mapping when available for swrast.
    
    This improves piglit quick.tests runtime from 19:33 minutes to 6:06 on
    my GM45.  It should also hide most of the A17 swizzling issues, though
    they'll still exist when swapping occurs (which is the kernel's problem
    either way)
Comment 7 fangxun 2009-11-05 06:02:12 UTC
It still fails with commit 7c8bed62e0165a0be3363f7abf81bf9e30341e00.
But it is verified with recent driver. Following is these commits:
Libdrm:        (master)b0b96636dbf93445dd532b09b21fa4fc5ce6bdc7
Mesa:          (master)1cc16e1b831cef8e1573cc998cee3e55179bb830
Xserver:        (master)d30637339963950910e5f5fb755b8465ac7dddb4
Xf86_video_intel:   (master)10946118dd3a63f1375a1bfde0b2f0542a93c1c2

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.