Bug 75237 - [BDW]igt/gem_pwrite_pread/*-correctness cases fail
Summary: [BDW]igt/gem_pwrite_pread/*-correctness cases fail
Status: CLOSED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Intel (show other bugs)
Version: unspecified
Hardware: All Linux (All)
: high critical
Assignee: Chris Wilson
QA Contact: Intel GFX Bugs mailing list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-02-20 01:58 UTC by lu hua
Modified: 2017-10-06 14:39 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
dmesg (96.28 KB, text/plain)
2014-02-20 01:58 UTC, lu hua
no flags Details
dmesg(072d358) (92.86 KB, text/plain)
2014-03-03 05:05 UTC, lu hua
no flags Details

Description lu hua 2014-02-20 01:58:11 UTC
Created attachment 94400 [details]
dmesg

System Environment:
--------------------------
Platform: Broadwell
Kernel(drm-intel-nightly)1be8f2b4dd6d3db00af24d4891c82d2650bd282d

Bug detailed description:
------------------------- 
New case gem_pwrite_pread fails on Broadwell with -nightly kernel.
Following cases fail:
igt/gem_pwrite_pread/display-copy-correctness
igt/gem_pwrite_pread/display-pwrite-blt-gtt_mmap-correctness
igt/gem_pwrite_pread/snooped-copy-correctness
igt/gem_pwrite_pread/snooped-pwrite-blt-cpu_mmap-correctness
igt/gem_pwrite_pread/uncached-copy-correctness
igt/gem_pwrite_pread/uncached-pwrite-blt-gtt_mmap-correctness

run ./gem_pwrite_pread --run-subtest display-copy-correctness
output:
IGT-Version: 1.5-g9597836 (x86_64) (Linux: 3.13.0_drm-intel-nightly_1be8f2_20140220+ x86_64)
Test assertion failure function test_copy, file gem_pwrite_pread.c:239:
Last errno: 2, No such file or directory
Failed assertion: buf[i] == i
Subtest display-copy-correctness: FAIL

Reproduce steps:
-------------------------
1.  ./gem_pwrite_pread --run-subtest display-copy-correctness
Comment 1 Yang Kun (YK) 2014-02-27 02:51:32 UTC
We encounter this issue as well. bumping the importance to high. Please advice if this is not appropriate.

thanks
-YK
Comment 2 Gordon Jin 2014-02-28 02:59:54 UTC
I'm not sure if the severity should be critical. any comments?
Comment 3 Chris Wilson 2014-02-28 08:02:53 UTC
This is one of the fundamental tests as to whether anything we render will be displayed correctly. Timo Aaltonen (Canonical) reported seeing display corruption in the form of dirty cache lines that may have been this bug which went away with a BIOS update.

Basically if this test fails, I would not expect any render correctness tests to pass either.
Comment 4 Chris Wilson 2014-02-28 21:49:06 UTC
Just a broken test.

commit 072d358bf03d0c39dc83bce0b93801d030d2324f
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Fri Feb 28 21:44:22 2014 +0000

    igt/gem_pwrite_pread: Fix XY_SRC_COPY command length for gen8+
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=75237
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Comment 5 lu hua 2014-03-03 05:05:28 UTC
It still fails.
output:
IGT-Version: 1.5-g072d358 (x86_64) (Linux: 3.14.0-rc4_drm-intel-nightly_0dfe3c_20140303+ x86_64)
Test assertion failure function test_copy, file gem_pwrite_pread.c:240:
Last errno: 2, No such file or directory
Failed assertion: buf[i] == i
Subtest display-copy-correctness: FAIL
Comment 6 lu hua 2014-03-03 05:05:58 UTC
Created attachment 94999 [details]
dmesg(072d358)
Comment 7 Chris Wilson 2014-03-05 11:11:08 UTC
Ah, the relocation entries were also wrong.

Try:

commit a3b80912a993b4b6952917afea41906f88dda01e
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Wed Mar 5 11:09:46 2014 +0000

    igt/gem_pwrite_pread: Fix relocation offsets for gen8+
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=75237
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Comment 8 lu hua 2014-03-07 05:52:30 UTC
Test on latest -nightly kernel and latest igt, this issue still occurs.
IGT-Version: 1.5-gcdf74b6 (x86_64) (Linux: 3.14.0-rc5_drm-intel-nightly_2e7638_20140307+ x86_64)
Test assertion failure function test_copy, file gem_pwrite_pread.c:240:
Last errno: 2, No such file or directory
Failed assertion: buf[i] == i
Subtest display-copy-correctness: FAIL
Comment 9 Chris Wilson 2014-03-07 07:53:14 UTC
Third time lucky.

commit 856afd588ef31d02c0dd8fd827a0af2725edc464
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Wed Mar 5 11:09:46 2014 +0000

    igt/gem_pwrite_pread: Fix relocation offsets for gen8+
    
    Oops, I didn't check to see if there was more than one call site where
    we built and execbuffer.
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=75237
    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Comment 10 lu hua 2014-03-10 06:44:56 UTC
Verified.Fixed.
Comment 11 Elizabeth 2017-10-06 14:39:46 UTC
Closing old verified.


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.