Temporarily disable detach and promote
After disccusion, we decided that our migrationg will start with
detached buffer. Therefore, until we refractor both producer buffer and
detached buffer to use binder, it makes no sense to write a converter
between pdx and binder backend.
As detached buffer is neither used outside nor exposed via API, this
change should not have any negative effect.
Some tests are also disabled since after refractoring the behavior might
change, which means we have to rewrite them after refracting. Put TODOs
above them.
Change-Id: I937403a4a8ae9f3ea94718211e12326ace9060bf
Test: 'atest buffer_hub-test': passed
diff --git a/libs/vr/libbufferhub/producer_buffer.cpp b/libs/vr/libbufferhub/producer_buffer.cpp
index c4f1a3b..7b6f77a 100644
--- a/libs/vr/libbufferhub/producer_buffer.cpp
+++ b/libs/vr/libbufferhub/producer_buffer.cpp
@@ -222,6 +222,10 @@
}
Status<LocalChannelHandle> ProducerBuffer::Detach() {
+ // TODO(b/112338294) remove after migrate producer buffer to binder
+ ALOGW("ProducerBuffer::Detach: not supported operation during migration");
+ return {};
+
uint64_t buffer_state = buffer_state_->load();
if (!BufferHubDefs::IsBufferGained(buffer_state)) {
// Can only detach a ProducerBuffer when it's in gained state.