summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRobert Foss <robert.foss@collabora.com>2016-10-20 17:49:43 -0400
committerRobert Foss <robert.foss@collabora.com>2017-03-22 14:38:43 -0400
commit12eca20cc77db2819e2ca53e97c82023a03ab6b3 (patch)
tree9be4defeda20760324d4fb529af3011bfed3cee7
parent397e501dfbddbeb87b8278fbb501e9af501cdcc4 (diff)
downloaddrm_hwcomposer-12eca20cc77db2819e2ca53e97c82023a03ab6b3.tar.gz
drm_hwcomposer-12eca20cc77db2819e2ca53e97c82023a03ab6b3.tar.xz
drm_hwcomposer: Add out-fence support
Change-Id: I63f878efe2ac07f3a3d637cf08f1c220308182e8 Signed-off-by: Robert Foss <robert.foss@collabora.com>
-rw-r--r--drmdisplaycomposition.h9
-rw-r--r--drmdisplaycompositor.cpp16
-rw-r--r--drmhwctwo.cpp9
3 files changed, 27 insertions, 7 deletions
diff --git a/drmdisplaycomposition.h b/drmdisplaycomposition.h
index b165adc..0586d58 100644
--- a/drmdisplaycomposition.h
+++ b/drmdisplaycomposition.h
@@ -189,6 +189,14 @@ class DrmDisplayComposition {
return planner_;
}
+ int take_out_fence() {
+ return out_fence_.Release();
+ }
+
+ void set_out_fence(int out_fence) {
+ out_fence_.Set(dup(out_fence));
+ }
+
void Dump(std::ostringstream *out) const;
private:
@@ -215,6 +223,7 @@ class DrmDisplayComposition {
int timeline_current_ = 0;
int timeline_squash_done_ = 0;
int timeline_pre_comp_done_ = 0;
+ UniqueFd out_fence_ = -1;
bool geometry_changed_;
std::vector<DrmHwcLayer> layers_;
diff --git a/drmdisplaycompositor.cpp b/drmdisplaycompositor.cpp
index 71c0451..251cdb5 100644
--- a/drmdisplaycompositor.cpp
+++ b/drmdisplaycompositor.cpp
@@ -492,6 +492,7 @@ int DrmDisplayCompositor::CommitFrame(DrmDisplayComposition *display_comp,
display_comp->composition_planes();
std::vector<DrmCompositionRegion> &pre_comp_regions =
display_comp->pre_comp_regions();
+ uint64_t out_fences[drm_->GetCrtcCount()];
DrmConnector *connector = drm_->GetConnectorForDisplay(display_);
if (!connector) {
@@ -510,6 +511,16 @@ int DrmDisplayCompositor::CommitFrame(DrmDisplayComposition *display_comp,
return -ENOMEM;
}
+ if (crtc->out_fence_ptr_property().id() != 0) {
+ ret = drmModeAtomicAddProperty(pset, crtc->id(), crtc->out_fence_ptr_property().id(),
+ (uint64_t) &out_fences[0]);
+ if (ret < 0) {
+ ALOGE("Failed to add OUT_FENCE_PTR property to pset: %d", ret);
+ drmModeAtomicFree(pset);
+ return ret;
+ }
+ }
+
if (mode_.needs_modeset) {
ret = drmModeAtomicAddProperty(pset, crtc->id(), crtc->mode_property().id(),
mode_.blob_id) < 0 ||
@@ -708,6 +719,11 @@ out:
mode_.needs_modeset = false;
}
+ if (crtc->out_fence_ptr_property().id()) {
+ display_comp->set_out_fence((int) out_fences[crtc->pipe()]);
+ close((int) out_fences[crtc->pipe()]);
+ }
+
return ret;
}
diff --git a/drmhwctwo.cpp b/drmhwctwo.cpp
index a00762e..c836930 100644
--- a/drmhwctwo.cpp
+++ b/drmhwctwo.cpp
@@ -561,19 +561,14 @@ HWC2::Error DrmHwcTwo::HwcDisplay::PresentDisplay(int32_t *retire_fence) {
i = overlay_planes.erase(i);
}
+ AddFenceToRetireFence(composition->take_out_fence());
+
ret = compositor_.ApplyComposition(std::move(composition));
if (ret) {
ALOGE("Failed to apply the frame composition ret=%d", ret);
return HWC2::Error::BadParameter;
}
- // Now that the release fences have been generated by the compositor, make
- // sure they're managed properly
- for (std::pair<const uint32_t, DrmHwcTwo::HwcLayer *> &l : z_map) {
- l.second->manage_release_fence();
- AddFenceToRetireFence(l.second->release_fence());
- }
-
// The retire fence returned here is for the last frame, so return it and
// promote the next retire fence
*retire_fence = retire_fence_.Release();