New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 780734 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Feature



Sign in to add a comment

Add jpeg decoder performance test.

Project Member Reported by mojahsu@chromium.org, Nov 2 2017

Issue description

Add a test to measure cpu/power of HW/SW jpeg decoder.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 7 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/cb2e65e087e44e4c3fae11dd2020d9d5f4e9c64a

commit cb2e65e087e44e4c3fae11dd2020d9d5f4e9c64a
Author: Moja Hsu <mojahsu@chromium.org>
Date: Tue Nov 07 05:45:50 2017

JDA unittest: Add performance unit test cases

Add unit test cases for performance measurement by decoding many jpeg
images by JDA or by SW.

BUG= chromium:780734 
TEST=Run the 2 tests on samus and elm.

Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I61628d7f0ee52624b8db6e939556dd34216e8071
Reviewed-on: https://chromium-review.googlesource.com/750744
Reviewed-by: Kuang-che Wu <kcwu@chromium.org>
Commit-Queue: Hsu Wei-Cheng <mojahsu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#514405}
[modify] https://crrev.com/cb2e65e087e44e4c3fae11dd2020d9d5f4e9c64a/media/gpu/jpeg_decode_accelerator_unittest.cc

Comment 2 by ovanieva@google.com, Jan 19 2018

Labels: Build-Toolchain
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 29 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/364fed407966647ef13d00cb2e86f57547649804

commit 364fed407966647ef13d00cb2e86f57547649804
Author: Moja Hsu <mojahsu@chromium.org>
Date: Mon Jan 29 13:24:44 2018

autotest-chrome: add JpegDecodeAcceleratorPerf to ebuild

BUG= chromium:780734 
TEST=Run a build
CQ-DEPEND=CL:753169

Change-Id: I62a0867b556b1e639cb722e499ce35876f0350e9
Reviewed-on: https://chromium-review.googlesource.com/754203
Commit-Ready: Hsu Wei-Cheng <mojahsu@chromium.org>
Tested-by: Hsu Wei-Cheng <mojahsu@chromium.org>
Reviewed-by: Heng-ruey Hsu <henryhsu@chromium.org>

[modify] https://crrev.com/364fed407966647ef13d00cb2e86f57547649804/chromeos-base/autotest-chrome/autotest-chrome-9999.ebuild

Project Member

Comment 4 by bugdroid1@chromium.org, Jan 29 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/2434bb27c1b6460124bf7db782292f2912d1ceb3

commit 2434bb27c1b6460124bf7db782292f2912d1ceb3
Author: Moja Hsu <mojahsu@chromium.org>
Date: Mon Jan 29 13:24:44 2018

Add video_JpegDecodeAcceleratorPerf test.

The video_JpegDecodeAcceleratorPerf test measures the cpu usage and
power consumption of jpeg decoding.

BUG= chromium:780734 
TEST=Run the test on samus and elm.
CQ-DEPEND=CL:750744
CQ-DEPEND=CL:754203

Change-Id: I56f609b44b4e9568492d2c7039032beeb9313dd3
Reviewed-on: https://chromium-review.googlesource.com/753169
Commit-Ready: Hsu Wei-Cheng <mojahsu@chromium.org>
Tested-by: Hsu Wei-Cheng <mojahsu@chromium.org>
Reviewed-by: Ricky Liang <jcliang@chromium.org>

[add] https://crrev.com/2434bb27c1b6460124bf7db782292f2912d1ceb3/client/site_tests/video_JpegDecodeAcceleratorPerf/control.cpu
[modify] https://crrev.com/2434bb27c1b6460124bf7db782292f2912d1ceb3/client/cros/power/power_utils.py
[add] https://crrev.com/2434bb27c1b6460124bf7db782292f2912d1ceb3/client/site_tests/video_JpegDecodeAcceleratorPerf/video_JpegDecodeAcceleratorPerf.py
[add] https://crrev.com/2434bb27c1b6460124bf7db782292f2912d1ceb3/client/site_tests/video_JpegDecodeAcceleratorPerf/control.power

Does this measure both software and hardware decoder?
Yes, it outputs the results of software and hardware decoder.
Components: Tools>ChromeOS-Toolchain
Can you rename the test to video_JDAPerf to be consistent with the other tests?
video_VDAPerf
video_VEAPerf
video_JEAPerf
Labels: -Pri-3 Pri-2
Have you filed a monitor request? You also need to file a monitor request after rename.

Like this. https://bugs.chromium.org/p/chromium/issues/detail?id=807865
Project Member

Comment 10 by bugdroid1@chromium.org, Feb 28 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/0e0c8a96e0468901724bdf2add5eb709a42c7cf8

commit 0e0c8a96e0468901724bdf2add5eb709a42c7cf8
Author: Moja Hsu <mojahsu@chromium.org>
Date: Wed Feb 28 04:17:14 2018

autotest-chrome: Rename JpegDecodeAcceleratorPerf to JDAPerf

BUG= chromium:780734 
TEST=Run a build
CQ-DEPEND=CL:934005

Change-Id: Ie8f8501231784fb6902488bafceba1005562dcce
Reviewed-on: https://chromium-review.googlesource.com/934007
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Hsu Wei-Cheng <mojahsu@chromium.org>
Reviewed-by: Ricky Liang <jcliang@chromium.org>

[modify] https://crrev.com/0e0c8a96e0468901724bdf2add5eb709a42c7cf8/chromeos-base/autotest-chrome/autotest-chrome-9999.ebuild

Project Member

Comment 11 by bugdroid1@chromium.org, Feb 28 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/4e1e4ef7376bba3408d7d0f613fef91edcdcadd0

commit 4e1e4ef7376bba3408d7d0f613fef91edcdcadd0
Author: Moja Hsu <mojahsu@chromium.org>
Date: Wed Feb 28 04:17:13 2018

video_JDAPerf: Rename test name to video_JDAPerf

Rename video_JpegDecodeAcceleratorPerf to video_JDAPerf.

BUG= chromium:780734 
TEST=Run the test on samus.
CQ-DEPEND=CL:934007

Change-Id: I585704b35616da897d8a491e4e84fa2d605645cb
Reviewed-on: https://chromium-review.googlesource.com/934005
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Hsu Wei-Cheng <mojahsu@chromium.org>
Reviewed-by: Ricky Liang <jcliang@chromium.org>

[rename] https://crrev.com/4e1e4ef7376bba3408d7d0f613fef91edcdcadd0/client/site_tests/video_JDAPerf/video_JDAPerf.py
[rename] https://crrev.com/4e1e4ef7376bba3408d7d0f613fef91edcdcadd0/client/site_tests/video_JDAPerf/control.cpu
[rename] https://crrev.com/4e1e4ef7376bba3408d7d0f613fef91edcdcadd0/client/site_tests/video_JDAPerf/control.power

Project Member

Comment 12 by bugdroid1@chromium.org, Mar 20 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/f685d346d6a09aa507f81a759acc26deb01832c5

commit f685d346d6a09aa507f81a759acc26deb01832c5
Author: Moja Hsu <mojahsu@chromium.org>
Date: Tue Mar 20 09:22:04 2018

video_JDAPerf: upload test results

BUG= chromium:780734 
TEST=None

Change-Id: I616db260fdece3e91dc0b013b97d155505cad6da
Reviewed-on: https://chromium-review.googlesource.com/948424
Commit-Ready: Hsu Wei-Cheng <mojahsu@chromium.org>
Tested-by: Hsu Wei-Cheng <mojahsu@chromium.org>
Reviewed-by: Heng-ruey Hsu <henryhsu@chromium.org>

[modify] https://crrev.com/f685d346d6a09aa507f81a759acc26deb01832c5/tko/perf_upload/perf_dashboard_config.json

Status: Verified (was: Assigned)

Sign in to add a comment