From 9719e2d984de2f2921e65566765e24a98ea22930 Mon Sep 17 00:00:00 2001 From: Diego Wilson Date: Mon, 21 Sep 2020 19:36:49 +0000 Subject: [PATCH] Revert "Change HIDL fmq tests to use int32_t for payload" Revert "Change payload type to int32_t for fmq device tests" Revert submission 1431616-thirtytwo Reason for revert: This topic broke the aosp_x86-userdebug target. Details at http://b/169077648 Reverted Changes: If736389eb:Add template parameters to MQDescriptor Ice9c28c3d:Change HIDL fmq tests to use int32_t for payload Ia18d8459b:Add template arguments to MQDescriptor for AIDL I95586cc83:Change payload type to int32_t for fmq device test... Change-Id: Iaf37a5dca7a915f098e49ecb076c0932eb60e0c3 --- tests/msgq/1.0/ITestMsgQ.hal | 4 ++-- tests/msgq/1.0/default/TestMsgQ.cpp | 16 ++++++++-------- tests/msgq/1.0/default/TestMsgQ.h | 8 ++++---- 3 files changed, 14 insertions(+), 14 deletions(-) diff --git a/tests/msgq/1.0/ITestMsgQ.hal b/tests/msgq/1.0/ITestMsgQ.hal index bd10237c90..112270a9e8 100644 --- a/tests/msgq/1.0/ITestMsgQ.hal +++ b/tests/msgq/1.0/ITestMsgQ.hal @@ -31,7 +31,7 @@ interface ITestMsgQ { * * @return ret True if the setup is successful. */ - configureFmqSyncReadWrite(fmq_sync mqDesc) generates(bool ret); + configureFmqSyncReadWrite(fmq_sync mqDesc) generates(bool ret); /** * This method requests the service to return an MQDescriptor to @@ -46,7 +46,7 @@ interface ITestMsgQ { * @return mqDesc This structure describes the unsynchronized FMQ that was * set up by the service. Client can use it to set up the FMQ at its end. */ - getFmqUnsyncWrite(bool configureFmq) generates(bool ret, fmq_unsync mqDesc); + getFmqUnsyncWrite(bool configureFmq) generates(bool ret, fmq_unsync mqDesc); /** * This method request the service to write into the synchronized read/write diff --git a/tests/msgq/1.0/default/TestMsgQ.cpp b/tests/msgq/1.0/default/TestMsgQ.cpp index 44737378b3..ba665c9204 100644 --- a/tests/msgq/1.0/default/TestMsgQ.cpp +++ b/tests/msgq/1.0/default/TestMsgQ.cpp @@ -25,7 +25,7 @@ namespace implementation { // Methods from ::android::hardware::tests::msgq::V1_0::ITestMsgQ follow. Return TestMsgQ::configureFmqSyncReadWrite( - const android::hardware::MQDescriptorSync& mqDesc) { + const android::hardware::MQDescriptorSync& mqDesc) { mFmqSynchronized.reset(new (std::nothrow) MessageQueueSync(mqDesc)); if ((mFmqSynchronized == nullptr) || (mFmqSynchronized->isValid() == false)) { return false; @@ -56,7 +56,7 @@ Return TestMsgQ::getFmqUnsyncWrite(bool configureFmq, getFmqUnsyncWrite_cb } Return TestMsgQ::requestWriteFmqSync(int32_t count) { - std::vector data(count); + std::vector data(count); for (int i = 0; i < count; i++) { data[i] = i; } @@ -65,14 +65,14 @@ Return TestMsgQ::requestWriteFmqSync(int32_t count) { } Return TestMsgQ::requestReadFmqSync(int32_t count) { - std::vector data(count); + std::vector data(count); bool result = mFmqSynchronized->read(&data[0], count) && verifyData(&data[0], count); return result; } Return TestMsgQ::requestWriteFmqUnsync(int32_t count) { - std::vector data(count); + std::vector data(count); for (int i = 0; i < count; i++) { data[i] = i; } @@ -81,14 +81,14 @@ Return TestMsgQ::requestWriteFmqUnsync(int32_t count) { } Return TestMsgQ::requestReadFmqUnsync(int32_t count) { - std::vector data(count); + std::vector data(count); bool result = mFmqUnsynchronized->read(&data[0], count) && verifyData(&data[0], count); return result; } Return TestMsgQ::requestBlockingRead(int32_t count) { - std::vector data(count); + std::vector data(count); bool result = mFmqSynchronized->readBlocking( &data[0], count, @@ -103,7 +103,7 @@ Return TestMsgQ::requestBlockingRead(int32_t count) { } Return TestMsgQ::requestBlockingReadDefaultEventFlagBits(int32_t count) { - std::vector data(count); + std::vector data(count); bool result = mFmqSynchronized->readBlocking( &data[0], count); @@ -116,7 +116,7 @@ Return TestMsgQ::requestBlockingReadDefaultEventFlagBits(int32_t count) { } Return TestMsgQ::requestBlockingReadRepeat(int32_t count, int32_t numIter) { - std::vector data(count); + std::vector data(count); for (int i = 0; i < numIter; i++) { bool result = mFmqSynchronized->readBlocking( &data[0], diff --git a/tests/msgq/1.0/default/TestMsgQ.h b/tests/msgq/1.0/default/TestMsgQ.h index 8a204b725d..f9fcddd4bc 100644 --- a/tests/msgq/1.0/default/TestMsgQ.h +++ b/tests/msgq/1.0/default/TestMsgQ.h @@ -49,13 +49,13 @@ using android::hardware::MQDescriptorUnsync; using android::hardware::MessageQueue; struct TestMsgQ : public ITestMsgQ { - typedef MessageQueue MessageQueueSync; - typedef MessageQueue MessageQueueUnsync; + typedef MessageQueue MessageQueueSync; + typedef MessageQueue MessageQueueUnsync; TestMsgQ() : mFmqSynchronized(nullptr), mFmqUnsynchronized(nullptr) {} // Methods from ::android::hardware::tests::msgq::V1_0::ITestMsgQ follow. - Return configureFmqSyncReadWrite(const MQDescriptorSync& mqDesc) override; + Return configureFmqSyncReadWrite(const MQDescriptorSync& mqDesc) override; Return getFmqUnsyncWrite(bool configureFmq, getFmqUnsyncWrite_cb _hidl_cb) override; Return requestWriteFmqSync(int32_t count) override; Return requestReadFmqSync(int32_t count) override; @@ -73,7 +73,7 @@ private: /* * Utility function to verify data read from the fast message queue. */ - bool verifyData(int32_t* data, int count) { + bool verifyData(uint16_t* data, int count) { for (int i = 0; i < count; i++) { if (data[i] != i) return false; }