• Hirokazu Honda's avatar
    Reland "media/gpu/test/VFValidator: Workaround the buffer synchronization problem on intel 3.18" · a5cc8aa3
    Hirokazu Honda authored
    This reverts commit 1980f0cb.
    
    Reason for revert: Kernel uprev for SKL is not complete in M89.
    
    Original change's description:
    > Revert "media/gpu/test/VFValidator: Workaround the buffer synchronization problem on intel 3.18"
    >
    > This reverts commit 276dd2ba.
    >
    > Reason for revert: The root cause of the synchronization is fixed.
    > Removes this workaround.
    >
    > Original change's description:
    > > media/gpu/test/VFValidator: Workaround the buffer synchronization problem on intel 3.18
    > >
    > > There is a synchronization problem on the mapped buffer on intel
    > > skylake devices. This is due to some missing 9165 patches in kernel
    > > 3.18, and thus will be fixed if the kernel is upreved to 4.4 or
    > > newer. This bug causes a decoded content validation failure in
    > > video_decode_accelerator_tests.
    > > This is a workaround to this bug; usleep() is added after the
    > > buffer is mapped and before the frame validation if the kernel
    > > version is 3.18 and the processor is Skylake.
    > >
    > > Bug: b:149808895
    > > Test: video_decode_accelerator_tests on sentry
    > > Change-Id: Ib77ebe0b83834e4d91bc994cbe6912348862e021
    > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2371724
    > > Commit-Queue: Hirokazu Honda <hiroh@chromium.org>
    > > Reviewed-by: Miguel Casas <mcasas@chromium.org>
    > > Cr-Commit-Position: refs/heads/master@{#801614}
    >
    > TBR=mcasas@chromium.org,hiroh@chromium.org,andrescj@chromium.org
    >
    > # Not skipping CQ checks because original CL landed > 1 day ago.
    >
    > Bug: b:149808895
    > Change-Id: Id0f1bdcc486effef2c6d3fd64275d71051020028
    > Test: None
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2631645
    > Reviewed-by: Hirokazu Honda <hiroh@chromium.org>
    > Commit-Queue: Hirokazu Honda <hiroh@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#843919}
    
    TBR=mcasas@chromium.org,hiroh@chromium.org,andrescj@chromium.org,chromium-scoped@luci-project-accounts.iam.gserviceaccount.com
    
    # Not skipping CQ checks because this is a reland.
    
    Bug: b:149808895
    Change-Id: I1f592a49c0276154101c59044c63c652c173e6af
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2631808Reviewed-by: default avatarHirokazu Honda <hiroh@chromium.org>
    Commit-Queue: Hirokazu Honda <hiroh@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#843956}
    a5cc8aa3
video_frame_validator.cc 17.7 KB