Bug 19317 - [G45 GEM] readpix fails with tiling on when using two different sizes memory banks(1G,512MB)
[G45 GEM] readpix fails with tiling on when using two different sizes memory ...
Product: Mesa
Classification: Unclassified
Component: Drivers/DRI/i965
x86-64 (AMD64) Linux (All)
: medium normal
Assigned To: Eric Anholt
Depends on:
Blocks: 18841 intel-3d-release
  Show dependency treegraph
Reported: 2008-12-28 21:24 UTC by lin, jiewen
Modified: 2009-11-05 06:02 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:

log (53.29 KB, text/plain)
2008-12-28 21:24 UTC, lin, jiewen
conf (3.81 KB, text/plain)
2008-12-28 21:25 UTC, lin, jiewen

Note You need to log in before you can comment on or make changes to this bug.
Description lin, jiewen 2008-12-28 21:24:45 UTC
Created attachment 21536 [details]

System Environment:

--Platform: G45
--Architecture(32-bit,64-bit,compatiblity): 64-bit
--2D driver:
--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]
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