Update VtsHalEvsV1_*TargetTest

- Removes a requirement of the landscape aspect ratio of the captured
  frames because the Extended View System now allows the video streams
  in any aspect ratio
- Increases the maximum latency of the video stream start request from
  500 ms to 1 second to accommodate the camera system design with the
  additional processing units outside of the head unit
- Reduces a number of additional buffers to allocate from 6 to 2 in
  CameraStreamBuffering test, to be consistent with comments

Bug: 204835095
Bug: 305642713
Test: atest VtsHalEvsV1_0TargetTest and
      atest VtsHalEvsV1_1TargetTest
Change-Id: Id41bde10964b650f78e76d4f97a4588367f7c7fc
Merged-In: Id41bde10964b650f78e76d4f97a4588367f7c7fc
(cherry picked from commit 7bc98ce6ed)
This commit is contained in:
Changyeon Jo 2021-11-08 08:12:33 -08:00
parent 775b6eca77
commit ab78601274
2 changed files with 3 additions and 14 deletions

View file

@ -17,8 +17,7 @@
#define LOG_TAG "VtsHalEvsTest"
// These values are called out in the EVS design doc (as of Mar 8, 2017)
static const int kMaxStreamStartMilliseconds = 500;
static const int kMaxStreamStartMilliseconds = 1000;
static const int kMinimumFramesPerSecond = 10;
static const int kSecondsToMilliseconds = 1000;
@ -332,11 +331,6 @@ TEST_P(EvsHidlTest, CameraStreamPerformance) {
printf("Measured time to first frame %0.2f ms\n", timeToFirstFrame * kNanoToMilliseconds);
ALOGI("Measured time to first frame %0.2f ms", timeToFirstFrame * kNanoToMilliseconds);
// Check aspect ratio
unsigned width = 0, height = 0;
frameHandler->getFrameDimension(&width, &height);
EXPECT_GE(width, height);
// Wait a bit, then ensure we get at least the required minimum number of frames
sleep(5);
nsecs_t end = systemTime(SYSTEM_TIME_MONOTONIC);
@ -370,7 +364,7 @@ TEST_P(EvsHidlTest, CameraStreamBuffering) {
ALOGI("Starting CameraStreamBuffering test");
// Arbitrary constant (should be > 1 and not too big)
static const unsigned int kBuffersToHold = 6;
static const unsigned int kBuffersToHold = 2;
// Get the camera list
loadCameraList();

View file

@ -498,11 +498,6 @@ TEST_P(EvsHidlTest, CameraStreamPerformance) {
<< std::scientific << timeToFirstFrame * kNanoToMilliseconds
<< " ms.";
// Check aspect ratio
unsigned width = 0, height = 0;
frameHandler->getFrameDimension(&width, &height);
EXPECT_GE(width, height);
// Wait a bit, then ensure we get at least the required minimum number of frames
sleep(5);
nsecs_t end = systemTime(SYSTEM_TIME_MONOTONIC);
@ -540,7 +535,7 @@ TEST_P(EvsHidlTest, CameraStreamBuffering) {
LOG(INFO) << "Starting CameraStreamBuffering test";
// Arbitrary constant (should be > 1 and not too big)
static const unsigned int kBuffersToHold = 6;
static const unsigned int kBuffersToHold = 2;
// Get the camera list
loadCameraList();