Bug 79242

Summary: [SNB/BDW bisected]igt/gem_userptr_blits costs long time to execute sporadically
Product: DRI Reporter: Guo Jinxian <jinxianx.guo>
Component: DRM/IntelAssignee: Chris Wilson <chris>
Status: CLOSED FIXED QA Contact: Intel GFX Bugs mailing list <intel-gfx-bugs>
Severity: normal    
Priority: highest CC: intel-gfx-bugs, yi.sun
Version: unspecified   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
dmesg none

Description Guo Jinxian 2014-05-26 06:25:41 UTC 
Created attachment 99828 [details]
dmesg

==System Environment==
--------------------------
Regression: Yes. 
It's a igt regression issue, the result before was skipped

Non-working platforms: SNB BDW

==kernel==
--------------------------
origin/drm-intel-nightly: c37f30ddc2921eccbf0be87c7742b56fbde53a81(fails)
    drm-intel-nightly: 2014y-05m-23d-13h-57m-55s integration manifest
origin/drm-intel-next-queued: f83d6518a13020e3cf7abbcc6b4e6d34459d9a9f(fails)
    ddrm/i915: Kill private_default_ctx off    
origin/drm-intel-fixes: c1240bd2842f6ec26d7dc1d9c845a197a41d33fe(skips)
    drm/i915: Prevent negative relocation deltas from wrapping

==Bug detailed description==
-----------------------------
]igt/gem_userptr_blits costs long time to execute sporadically

Output:
time ./gem_userptr_blits
IGT-Version: 1.6-gff3c122 (x86_64) (Linux: 3.15.0-rc3_drm-intel-next-queued_f83d65_20140525+ x86_64)
Aperture size is 2048 MiB
Total RAM is 7901 MiB
Subtest input-checking: SUCCESS
Subtest usage-restrictions: SUCCESS
Subtest invalid-mapping: SUCCESS
Subtest forbidden-operations: SUCCESS
Testing unsynchronized mappings...
Subtest create-destroy-unsync: SUCCESS
Subtest unsync-overlap: SUCCESS
Subtest unsync-unmap: SUCCESS
Subtest unsync-unmap-cycles: SUCCESS
Subtest unsync-unmap-after-close: SUCCESS
Using 2x1365 1MiB buffers
Verifying initialisation...
Cyclic blits cpu->gpu, forward...
Cyclic blits gpu->cpu, backward...
Random blits...
Subtest coherency-unsync: SUCCESS
Subtest dmabuf-unsync: SUCCESS
Subtest forked-unsync-normal: SUCCESS
Subtest forked-unsync-interruptible: SUCCESS
Subtest forked-unsync-swapping-normal: SUCCESS
Subtest forked-unsync-swapping-interruptible: SUCCESS
Subtest forked-unsync-multifd-normal: SUCCESS
Subtest forked-unsync-multifd-interruptible: SUCCESS
Subtest forked-unsync-swapping-multifd-normal: SUCCESS
Subtest forked-unsync-swapping-multifd-interruptible: SUCCESS
child 0 died with signal 9, Killed
Subtest forked-unsync-mempressure-normal: FAIL
Test assertion failure function __igt_fork, file igt_core.c:889:
Last errno: 12, Cannot allocate memory
Failed assertion: 0
Subtest forked-unsync-mempressure-interruptible: FAIL
Test requirement not met in function forking_evictions, file eviction_common.c:181:
Last errno: 12, Cannot allocate memory
Test requirement: (!(intel_get_total_ram_mb() / 4 < intel_get_total_swap_mb()))
Subtest forked-unsync-swapping-mempressure-normal: SKIP
Test requirement not met in function forking_evictions, file eviction_common.c:181:
Last errno: 12, Cannot allocate memory
Test requirement: (!(intel_get_total_ram_mb() / 4 < intel_get_total_swap_mb()))
Subtest forked-unsync-swapping-mempressure-interruptible: SKIP
Test assertion failure function __igt_fork, file igt_core.c:889:
Last errno: 12, Cannot allocate memory
Failed assertion: 0
Subtest forked-unsync-multifd-mempressure-normal: FAIL
Test assertion failure function __igt_fork, file igt_core.c:889:
Last errno: 12, Cannot allocate memory
Failed assertion: 0
Subtest forked-unsync-multifd-mempressure-interruptible: FAIL
Test requirement not met in function forking_evictions, file eviction_common.c:181:
Last errno: 12, Cannot allocate memory
Test requirement: (!(intel_get_total_ram_mb() / 4 < intel_get_total_swap_mb()))
Subtest forked-unsync-swapping-multifd-mempressure-normal: SKIP
Test requirement not met in function forking_evictions, file eviction_common.c:181:
Last errno: 12, Cannot allocate memory
Test requirement: (!(intel_get_total_ram_mb() / 4 < intel_get_total_swap_mb()))
Subtest forked-unsync-swapping-multifd-mempressure-interruptible: SKIP

Subtest swapping-unsync-normal: SUCCESS
Subtest minor-unsync-normal: SUCCESS
Subtest major-unsync-normal: SUCCESS

Subtest swapping-unsync-interruptible: SUCCESS
Subtest minor-unsync-interruptible: SUCCESS
Subtest major-unsync-interruptible: SUCCESS
Testing synchronized mappings...
Subtest create-destroy-sync: SUCCESS
Subtest sync-overlap: SUCCESS
Subtest sync-unmap: SUCCESS
Subtest sync-unmap-cycles: SUCCESS
Subtest sync-unmap-after-close: SUCCESS
Using 2x1365 1MiB buffers
Verifying initialisation...
Cyclic blits cpu->gpu, forward...
Cyclic blits gpu->cpu, backward...
Random blits...
Subtest coherency-sync: SUCCESS
Test assertion failure function check_bo, file gem_userptr_blits.c:599:
Last errno: 0, Success
Failed assertion: ptr1[i] == counter
Subtest dmabuf-sync: FAIL
Unhandled failure in child 0
gem_userptr_blits: igt_core.c:859: children_exit_handler: Assertion `ret == 0' failed.
Aborted (core dumped)

real    41m45.523s
user    0m6.311s
sys     29m14.948s

==Reproduce steps==
---------------------------- 
1.  ./gem_userptr_blits


==Bisect results==
----------------------------
commit 9911f3f0cf202444f1ef2399f5961605880b7360
Author:     Chris Wilson <chris@chris-wilson.co.uk>
AuthorDate: Thu May 22 10:20:33 2014 +0100
Commit:     Chris Wilson <chris@chris-wilson.co.uk>
CommitDate: Thu May 22 10:22:04 2014 +0100

    igt/gem_userptr_blits: Fix up last minute API changes

    When the patch was merged, the ioctl numbers had to be adjusted to leave
    no holes. Also there was a final piece of munging of the API to
    downgrade unsynced userptr for export over dma-buf.

    Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Comment 1 Guo Jinxian 2014-05-26 06:27:16 UTC 
The reproducible rate is about 1 out of 5.
Comment 2 Chris Wilson 2014-07-11 10:00:45 UTC 

*** This bug has been marked as a duplicate of bug 80208 ***
Comment 3 Guo Jinxian 2014-07-23 06:53:16 UTC 
Reopen it because bug 80208 had fixed, this bug still able to reproduce.

[root@x-hsw24 tests]# time ./gem_userptr_blits --run-subtest swapping-normal-sync
IGT-Version: 1.7-g70e6ed9 (x86_64) (Linux: 3.16.0-rc6_drm-intel-nightly_d62845_20140723+ x86_64)
Aperture size is 2048 MiB
Total RAM is 3861 MiB
Testing unsynchronized mappings...
Testing synchronized mappings...


^Z
[1]+  Stopped                 ./gem_userptr_blits --run-subtest swapping-normal-sync

real    21m30.163s
user    0m0.001s
sys     0m0.001s
Comment 4 Chris Wilson 2014-07-23 08:01:18 UTC 
(In reply to comment #3)
> Reopen it because bug 80208 had fixed, this bug still able to reproduce.
> 
> [root@x-hsw24 tests]# time ./gem_userptr_blits --run-subtest
> swapping-normal-sync
> IGT-Version: 1.7-g70e6ed9 (x86_64) (Linux:
> 3.16.0-rc6_drm-intel-nightly_d62845_20140723+ x86_64)
> Aperture size is 2048 MiB
> Total RAM is 3861 MiB
> Testing unsynchronized mappings...
> Testing synchronized mappings...
> 
> 
> ^Z
> [1]+  Stopped                 ./gem_userptr_blits --run-subtest
> swapping-normal-sync
> 
> real    21m30.163s
> user    0m0.001s
> sys     0m0.001s

That's not the same bug...
Comment 5 Guo Jinxian 2014-08-01 06:14:03 UTC 
(In reply to comment #4)
> (In reply to comment #3)
> > Reopen it because bug 80208 had fixed, this bug still able to reproduce.
> > 
> > [root@x-hsw24 tests]# time ./gem_userptr_blits --run-subtest
> > swapping-normal-sync
> > IGT-Version: 1.7-g70e6ed9 (x86_64) (Linux:
> > 3.16.0-rc6_drm-intel-nightly_d62845_20140723+ x86_64)
> > Aperture size is 2048 MiB
> > Total RAM is 3861 MiB
> > Testing unsynchronized mappings...
> > Testing synchronized mappings...
> > 
> > 
> > ^Z
> > [1]+  Stopped                 ./gem_userptr_blits --run-subtest
> > swapping-normal-sync
> > 
> > real    21m30.163s
> > user    0m0.001s
> > sys     0m0.001s
> 
> That's not the same bug...

New Bug 81832 tracked this issue, close this one.
Comment 6 Jari Tahvanainen 2016-10-19 09:26:23 UTC 
Closing verified+fixed.

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.