| commit | 26fe6823066a0cf50b396d24b234756398be6f55 | [log] [tgz] |
|---|---|---|
| author | David Stevens <[email protected]> | Mon Mar 09 12:23:42 2020 +0000 |
| committer | Commit Bot <[email protected]> | Tue Mar 10 04:17:30 2020 +0000 |
| tree | 43af6a7537f6872e69eddb8da3df92f14e065cf7 | |
| parent | 08d8dbf094fdc5aa9b92c7257e292edcf28602b2 [diff] |
Reland "minigbm: introduce test allocation" This reverts commit 08d8dbf094fdc5aa9b92c7257e292edcf28602b2. The original change returned early from drv_bo_create_with_modifiers, which broke reference counting. This must have hit some race condition in the tests, as they no longer flake after fixing reference counting. Original change's description: > Revert "minigbm: introduce test allocation" > > This reverts commit f0e607c7d436134b53fd45a4ead36d714451be8a. > > Reason for revert: caused flaky regressions across the board. > > BUG=b:150997559 > Exempt-From-Owner-Approval: revert. > > Original change's description: > > minigbm: introduce test allocation > > > > This change introduces a GBM_TEST_ALLOC flag to minigbm, which allows > > for the creation of fake buffers that can be used to determine buffer > > metadata without actually allocating a full buffer. The new flag is > > supported by the i915 backends. This flag also alleviates the need to > > cache buffers when virtio_gpu queries metadata properties. > > > > BUG=b:145994510 > > TEST=play youtube with arcvm demo image plus this and virgl change > > > > Change-Id: I9c6819aa3b5b674e4bb33b0656f2a9f155b0884e > > Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/1980688 > > Tested-by: David Stevens <[email protected]> > > Reviewed-by: Gurchetan Singh <[email protected]> > > Commit-Queue: David Stevens <[email protected]> > > Bug: b:145994510 > Change-Id: I50079b7f0aabf38e1f373cac0f28c0e057eed760 > Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/2093923 > Commit-Queue: Ilja H. Friedel <[email protected]> > Tested-by: Ilja H. Friedel <[email protected]> > Reviewed-by: Ilja H. Friedel <[email protected]> Bug: b:150997559, b:145994510 Change-Id: If0f02a4701bb6960b6413d6b0c00b481146914d9 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/2094068 Reviewed-by: Gurchetan Singh <[email protected]> Commit-Queue: David Stevens <[email protected]> Tested-by: David Stevens <[email protected]>