New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 656786 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

drm-tests: get null_platform_test working on tegra

Project Member Reported by h...@chromium.org, Oct 17 2016

Issue description

There's still a few things needed to get null_platform_test working on tegra. See also my recent patches for waffle.

1. Get rid of the gbm_bo_get_stride_or_tiling hack and use the format modifiers (I believe hoegsberg@ has an outstanding CL)

2. Add the external flush (see also https://chromium-review.googlesource.com/#/c/391246/)
 
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 20 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/28e0d42a080069c1c1bfd13163b8f33d6cf905ba

commit 28e0d42a080069c1c1bfd13163b8f33d6cf905ba
Author: Haixia Shi <hshi@chromium.org>
Date: Wed Oct 19 21:50:26 2016

mesa[-img]: do not install opengles headers from mesa[-img]

We should always use x11-drivers/opengles-headers. The headers from
mesa[-img] are older and cause file collissions.

Explicitly add DEPEND on opengles-headers in the following packages that
need the GLES headers at compile time: deqp, drm-tests, eglinfo, waffle.

BUG= chromium:656786 
TEST=confirm no file collission when emerging opengles-headers

Change-Id: I5a7d3d6e1365e93540a1d3332929f0df41d50ae6
Reviewed-on: https://chromium-review.googlesource.com/400846
Commit-Ready: Haixia Shi <hshi@chromium.org>
Tested-by: Haixia Shi <hshi@chromium.org>
Reviewed-by: Haixia Shi <hshi@chromium.org>

[rename] https://crrev.com/28e0d42a080069c1c1bfd13163b8f33d6cf905ba/media-libs/waffle/waffle-1.6.0-r10.ebuild
[rename] https://crrev.com/28e0d42a080069c1c1bfd13163b8f33d6cf905ba/media-libs/mesa-img/mesa-img-12.1.0-r5.ebuild
[modify] https://crrev.com/28e0d42a080069c1c1bfd13163b8f33d6cf905ba/media-libs/mesa-img/mesa-img-9999.ebuild
[modify] https://crrev.com/28e0d42a080069c1c1bfd13163b8f33d6cf905ba/chromeos-base/drm-tests/drm-tests-9999.ebuild
[rename] https://crrev.com/28e0d42a080069c1c1bfd13163b8f33d6cf905ba/media-libs/mesa/mesa-12.1.0-r8.ebuild
[rename] https://crrev.com/28e0d42a080069c1c1bfd13163b8f33d6cf905ba/chromeos-base/drm-tests/drm-tests-0.0.1-r62.ebuild
[rename] https://crrev.com/28e0d42a080069c1c1bfd13163b8f33d6cf905ba/media-gfx/deqp/deqp-2016.06.20-r3.ebuild
[modify] https://crrev.com/28e0d42a080069c1c1bfd13163b8f33d6cf905ba/media-libs/mesa/mesa-9999.ebuild
[modify] https://crrev.com/28e0d42a080069c1c1bfd13163b8f33d6cf905ba/media-libs/mesa/mesa-12.1.0.ebuild
[rename] https://crrev.com/28e0d42a080069c1c1bfd13163b8f33d6cf905ba/app-misc/eglinfo/eglinfo-8.2.0-r2.ebuild
[modify] https://crrev.com/28e0d42a080069c1c1bfd13163b8f33d6cf905ba/media-libs/mesa-img/mesa-img-12.1.0.ebuild

Project Member

Comment 3 by bugdroid1@chromium.org, Oct 20 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/drm-tests/+/fe4e2ff46371b278f32681be2ceb69759bf37a2c

commit fe4e2ff46371b278f32681be2ceb69759bf37a2c
Author: Haixia Shi <hshi@chromium.org>
Date: Mon Oct 17 21:28:55 2016

drm-tests: Add support for EXT_image_flush_external_extension

The null_platform_test needs to call EGLImageFlushExternalEXT before every
page flip if this extension is supported by the driver.

BUG= chromium:656786 
CQ-DEPEND=CL:400846
TEST=run test on nyan_big

Change-Id: I133e175fec2193fcc0d85861204c8e64f52521b8
Reviewed-on: https://chromium-review.googlesource.com/399958
Commit-Ready: Haixia Shi <hshi@chromium.org>
Tested-by: Haixia Shi <hshi@chromium.org>
Reviewed-by: Haixia Shi <hshi@chromium.org>

[modify] https://crrev.com/fe4e2ff46371b278f32681be2ceb69759bf37a2c/bsdrm/src/egl.c
[modify] https://crrev.com/fe4e2ff46371b278f32681be2ceb69759bf37a2c/bsdrm/include/bs_drm.h
[modify] https://crrev.com/fe4e2ff46371b278f32681be2ceb69759bf37a2c/null_platform_test.c

Comment 4 by h...@chromium.org, Oct 25 2016

Status: Fixed (was: Available)
With the latest driver and drm-test changes this is all working now.
Project Member

Comment 5 by bugdroid1@chromium.org, Dec 8 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/725c34dd8ee28b64fd83d90e460d6ec780d7345e

commit 725c34dd8ee28b64fd83d90e460d6ec780d7345e
Author: Haixia Shi <hshi@chromium.org>
Date: Thu Dec 08 00:12:40 2016

autotest-deps-glbench: add DEPEND on opengles-headers

Forgot this package (see CL:400846) which sometimes causes the package
to fail to build due to missing GL headers.

BUG= chromium:656786 
TEST=unmerge opengles-headers, then merge autotest-deps-glbench

Change-Id: I61f86338b221db0d874f5bbce516e14eb7287ea5
Reviewed-on: https://chromium-review.googlesource.com/417197
Reviewed-by: Haixia Shi <hshi@chromium.org>
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>
Commit-Queue: Haixia Shi <hshi@chromium.org>
Tested-by: Haixia Shi <hshi@chromium.org>
Trybot-Ready: Haixia Shi <hshi@chromium.org>

[modify] https://crrev.com/725c34dd8ee28b64fd83d90e460d6ec780d7345e/chromeos-base/autotest-deps-glbench/autotest-deps-glbench-9999.ebuild

Comment 6 by ka...@chromium.org, Jan 13 2017

Status: Verified (was: Fixed)

Sign in to add a comment