Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(203)

Issue 137023008: Add support for Tegra V4L2 VDA (Closed)

Created:
6 years, 10 months ago by shivdasp
Modified:
6 years, 8 months ago
CC:
chromium-reviews, fischman+watch_chromium.org, jam, mcasas+watch_chromium.org, joi+watch-content_chromium.org, feature-media-reviews_chromium.org, darin-cc_chromium.org, piman+watch_chromium.org, wjia+watch_chromium.org, jln+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

Add support for Tegra V4L2 VDA This change add a TegraV4L2Device and extends the support of V4L2 VDA on Tegra platform. BUG=chromium-os-partner:23082 TEST=Run video playback Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=260661

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

Patch Set 4 : #

Patch Set 5 : #

Total comments: 90

Patch Set 6 : #

Total comments: 28

Patch Set 7 : #

Total comments: 7

Patch Set 8 : #

Total comments: 7

Patch Set 9 : #

Patch Set 10 : Fixed minor nit #

Patch Set 11 : fixed a small issue #

Total comments: 31

Patch Set 12 : #

Total comments: 26

Patch Set 13 : #

Patch Set 14 : #

Total comments: 16

Patch Set 15 : use scopedFD for dmabuf fds. #

Total comments: 16

Patch Set 16 : Addressed a few more comments #

Total comments: 2

Patch Set 17 : #

Total comments: 37

Patch Set 18 : #

Total comments: 9

Patch Set 19 : LazyInstance related changes #

Patch Set 20 : rebased #

Total comments: 6

Patch Set 21 : addressed nits #

Total comments: 2

Patch Set 22 : minor nit #

Unified diffs Side-by-side diffs Delta from patch set Stats (+460 lines, -91 lines) Patch
M content/common/gpu/media/exynos_v4l2_video_device.h View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 1 chunk +20 lines, -13 lines 0 comments Download
M content/common/gpu/media/exynos_v4l2_video_device.cc View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 3 chunks +63 lines, -0 lines 0 comments Download
M content/common/gpu/media/gpu_video_decode_accelerator.cc View 1 2 3 4 5 6 7 1 chunk +2 lines, -1 line 0 comments Download
M content/common/gpu/media/rendering_helper.h View 1 2 3 4 5 1 chunk +3 lines, -0 lines 0 comments Download
M content/common/gpu/media/rendering_helper.cc View 1 2 3 4 5 6 1 chunk +2 lines, -0 lines 0 comments Download
A content/common/gpu/media/tegra_v4l2_video_device.h View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 1 chunk +57 lines, -0 lines 0 comments Download
A content/common/gpu/media/tegra_v4l2_video_device.cc View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 1 chunk +189 lines, -0 lines 0 comments Download
M content/common/gpu/media/v4l2_video_decode_accelerator.h View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 4 chunks +12 lines, -4 lines 0 comments Download
M content/common/gpu/media/v4l2_video_decode_accelerator.cc View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 17 chunks +57 lines, -63 lines 0 comments Download
M content/common/gpu/media/v4l2_video_device.h View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 3 chunks +30 lines, -2 lines 0 comments Download
M content/common/gpu/media/v4l2_video_device.cc View 1 2 3 4 5 2 chunks +12 lines, -7 lines 0 comments Download
M content/common/gpu/media/video_decode_accelerator_unittest.cc View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 1 chunk +2 lines, -1 line 0 comments Download
M content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 1 chunk +5 lines, -0 lines 0 comments Download
M content/content_common.gypi View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 1 chunk +2 lines, -0 lines 0 comments Download
M media/test/data/test-25fps.h264.md5 View 1 2 3 4 5 6 7 1 chunk +2 lines, -0 lines 0 comments Download
M media/test/data/test-25fps.vp8.md5 View 1 2 3 4 5 6 7 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 155 (0 generated)
shivdasp
Do not review yet. This is first draft version with some more changes needed.
6 years, 10 months ago (2014-01-28 13:49:36 UTC) #1
shivdasp
This change adds support for Tegra V4L2Device into the V4L2VDA. Please have a look.
6 years, 10 months ago (2014-02-06 09:48:46 UTC) #2
shivdasp
Is there a change in gles2_cmd_decoder.cc recently ? Because this CL used to work on ...
6 years, 10 months ago (2014-02-06 17:37:40 UTC) #3
Ami GONE FROM CHROMIUM
On Thu, Feb 6, 2014 at 9:37 AM, <shivdasp@nvidia.com> wrote: > Is there a change ...
6 years, 10 months ago (2014-02-06 17:46:18 UTC) #4
shivdasp
That was the first place I checked but there are 4-5 changes recently in that ...
6 years, 10 months ago (2014-02-07 05:33:52 UTC) #5
Ami GONE FROM CHROMIUM
@posciak: as usual I rely on you to review the called platform code for calling ...
6 years, 10 months ago (2014-02-07 09:09:30 UTC) #6
Pawel Osciak
https://chromiumcodereview.appspot.com/137023008/diff/80001/content/common/gpu/media/exynos_v4l2_video_device.cc File content/common/gpu/media/exynos_v4l2_video_device.cc (right): https://chromiumcodereview.appspot.com/137023008/diff/80001/content/common/gpu/media/exynos_v4l2_video_device.cc#newcode123 content/common/gpu/media/exynos_v4l2_video_device.cc:123: EGLImageKHR egl_image = eglCreateImageKHR( We need to make GL ...
6 years, 10 months ago (2014-02-10 06:36:17 UTC) #7
shivdasp
https://codereview.chromium.org/137023008/diff/80001/content/common/gpu/media/tegra_v4l2_video_device.cc File content/common/gpu/media/tegra_v4l2_video_device.cc (right): https://codereview.chromium.org/137023008/diff/80001/content/common/gpu/media/tegra_v4l2_video_device.cc#newcode23 content/common/gpu/media/tegra_v4l2_video_device.cc:23: const char kDevice[] = "/dev/video0"; This is a v4l2 ...
6 years, 10 months ago (2014-02-10 13:31:16 UTC) #8
shivdasp
Hi Pawel, Could you check my comments especially regarding the EglImage section ? Your views ...
6 years, 10 months ago (2014-02-11 14:41:14 UTC) #9
Pawel Osciak
https://chromiumcodereview.appspot.com/137023008/diff/80001/content/common/gpu/media/tegra_v4l2_video_device.cc File content/common/gpu/media/tegra_v4l2_video_device.cc (right): https://chromiumcodereview.appspot.com/137023008/diff/80001/content/common/gpu/media/tegra_v4l2_video_device.cc#newcode23 content/common/gpu/media/tegra_v4l2_video_device.cc:23: const char kDevice[] = "/dev/video0"; On 2014/02/10 13:31:17, shivdasp ...
6 years, 10 months ago (2014-02-12 09:15:12 UTC) #10
shivdasp
https://codereview.chromium.org/137023008/diff/80001/content/common/gpu/media/tegra_v4l2_video_device.cc File content/common/gpu/media/tegra_v4l2_video_device.cc (right): https://codereview.chromium.org/137023008/diff/80001/content/common/gpu/media/tegra_v4l2_video_device.cc#newcode23 content/common/gpu/media/tegra_v4l2_video_device.cc:23: const char kDevice[] = "/dev/video0"; This library internally talks ...
6 years, 10 months ago (2014-02-12 10:11:55 UTC) #11
Pawel Osciak
https://chromiumcodereview.appspot.com/137023008/diff/80001/content/common/gpu/media/tegra_v4l2_video_device.cc File content/common/gpu/media/tegra_v4l2_video_device.cc (right): https://chromiumcodereview.appspot.com/137023008/diff/80001/content/common/gpu/media/tegra_v4l2_video_device.cc#newcode23 content/common/gpu/media/tegra_v4l2_video_device.cc:23: const char kDevice[] = "/dev/video0"; On 2014/02/12 10:11:55, shivdasp ...
6 years, 10 months ago (2014-02-13 10:42:54 UTC) #12
shivdasp
https://codereview.chromium.org/137023008/diff/80001/content/common/gpu/media/exynos_v4l2_video_device.cc File content/common/gpu/media/exynos_v4l2_video_device.cc (right): https://codereview.chromium.org/137023008/diff/80001/content/common/gpu/media/exynos_v4l2_video_device.cc#newcode123 content/common/gpu/media/exynos_v4l2_video_device.cc:123: EGLImageKHR egl_image = eglCreateImageKHR( make_context_current_ is already done before ...
6 years, 10 months ago (2014-02-14 03:06:44 UTC) #13
Pawel Osciak
https://chromiumcodereview.appspot.com/137023008/diff/80001/content/common/gpu/media/exynos_v4l2_video_device.cc File content/common/gpu/media/exynos_v4l2_video_device.cc (right): https://chromiumcodereview.appspot.com/137023008/diff/80001/content/common/gpu/media/exynos_v4l2_video_device.cc#newcode123 content/common/gpu/media/exynos_v4l2_video_device.cc:123: EGLImageKHR egl_image = eglCreateImageKHR( On 2014/02/14 03:06:45, shivdasp wrote: ...
6 years, 10 months ago (2014-02-14 07:36:10 UTC) #14
shivdasp
https://codereview.chromium.org/137023008/diff/80001/content/common/gpu/media/tegra_v4l2_video_device.cc File content/common/gpu/media/tegra_v4l2_video_device.cc (right): https://codereview.chromium.org/137023008/diff/80001/content/common/gpu/media/tegra_v4l2_video_device.cc#newcode23 content/common/gpu/media/tegra_v4l2_video_device.cc:23: const char kDevice[] = "/dev/video0"; Whitelisted will not be ...
6 years, 10 months ago (2014-02-14 09:18:58 UTC) #15
shivdasp
Incorporated review comments, please take a look.
6 years, 10 months ago (2014-02-19 21:23:58 UTC) #16
shivdasp
https://codereview.chromium.org/137023008/diff/510001/content/common/gpu/media/v4l2_video_decode_accelerator.cc File content/common/gpu/media/v4l2_video_decode_accelerator.cc (right): https://codereview.chromium.org/137023008/diff/510001/content/common/gpu/media/v4l2_video_decode_accelerator.cc#newcode713 content/common/gpu/media/v4l2_video_decode_accelerator.cc:713: if (!AppendToInputFrame(data, size)) I think there is a bug ...
6 years, 10 months ago (2014-02-20 08:51:01 UTC) #17
Pawel Osciak
On 2014/02/19 21:23:58, shivdasp wrote: > Incorporated review comments, please take a look. Hi Shivdas, ...
6 years, 10 months ago (2014-02-21 05:32:43 UTC) #18
Pawel Osciak
https://codereview.chromium.org/137023008/diff/510001/content/common/gpu/media/v4l2_video_decode_accelerator.cc File content/common/gpu/media/v4l2_video_decode_accelerator.cc (right): https://codereview.chromium.org/137023008/diff/510001/content/common/gpu/media/v4l2_video_decode_accelerator.cc#newcode713 content/common/gpu/media/v4l2_video_decode_accelerator.cc:713: if (!AppendToInputFrame(data, size)) On 2014/02/20 08:51:02, shivdasp wrote: > ...
6 years, 10 months ago (2014-02-21 05:36:42 UTC) #19
shivdasp
On 2014/02/21 05:32:43, Pawel Osciak wrote: > On 2014/02/19 21:23:58, shivdasp wrote: > > Incorporated ...
6 years, 10 months ago (2014-02-21 07:06:53 UTC) #20
shivdasp
Ping on whether there is a need to change EglImage creation part. On 2014/02/21 07:06:53, ...
6 years, 10 months ago (2014-02-25 09:37:55 UTC) #21
shivdasp
https://codereview.chromium.org/137023008/diff/510001/content/common/gpu/media/v4l2_video_decode_accelerator.cc File content/common/gpu/media/v4l2_video_decode_accelerator.cc (right): https://codereview.chromium.org/137023008/diff/510001/content/common/gpu/media/v4l2_video_decode_accelerator.cc#newcode713 content/common/gpu/media/v4l2_video_decode_accelerator.cc:713: if (!AppendToInputFrame(data, size)) The issue is that we try ...
6 years, 10 months ago (2014-02-25 09:38:14 UTC) #22
Pawel Osciak
On 2014/02/14 09:18:58, shivdasp wrote: > > Yes, although it's not "synchronized", but the ownership ...
6 years, 10 months ago (2014-02-26 10:57:59 UTC) #23
Pawel Osciak
On 2014/02/21 07:06:53, shivdasp wrote: > On 2014/02/21 05:32:43, Pawel Osciak wrote: > > On ...
6 years, 10 months ago (2014-02-26 11:02:28 UTC) #24
Pawel Osciak
On 2014/02/25 09:38:14, shivdasp wrote: > https://codereview.chromium.org/137023008/diff/510001/content/common/gpu/media/v4l2_video_decode_accelerator.cc > File content/common/gpu/media/v4l2_video_decode_accelerator.cc (right): > > https://codereview.chromium.org/137023008/diff/510001/content/common/gpu/media/v4l2_video_decode_accelerator.cc#newcode713 > ...
6 years, 10 months ago (2014-02-26 11:06:14 UTC) #25
shivdasp
On 2014/02/26 10:57:59, Pawel Osciak wrote: > On 2014/02/14 09:18:58, shivdasp wrote: > > > ...
6 years, 10 months ago (2014-02-26 14:39:52 UTC) #26
shivdasp
On 2014/02/26 11:06:14, Pawel Osciak wrote: > On 2014/02/25 09:38:14, shivdasp wrote: > > > ...
6 years, 10 months ago (2014-02-26 14:45:26 UTC) #27
Pawel Osciak
On 2014/02/26 14:39:52, shivdasp wrote: > On 2014/02/26 10:57:59, Pawel Osciak wrote: > > On ...
6 years, 9 months ago (2014-03-03 05:10:48 UTC) #28
Pawel Osciak
On 2014/02/26 14:45:26, shivdasp wrote: > On 2014/02/26 11:06:14, Pawel Osciak wrote: > > On ...
6 years, 9 months ago (2014-03-03 05:16:34 UTC) #29
shivdasp
On 2014/03/03 05:10:48, Pawel Osciak wrote: > On 2014/02/26 14:39:52, shivdasp wrote: > > On ...
6 years, 9 months ago (2014-03-03 16:37:45 UTC) #30
shivdasp
On 2014/03/03 05:16:34, Pawel Osciak wrote: > On 2014/02/26 14:45:26, shivdasp wrote: > > On ...
6 years, 9 months ago (2014-03-03 16:42:27 UTC) #31
Pawel Osciak
On 2014/03/03 16:42:27, shivdasp wrote: > On 2014/03/03 05:16:34, Pawel Osciak wrote: > > On ...
6 years, 9 months ago (2014-03-04 08:37:04 UTC) #32
Pawel Osciak
On 2014/03/03 16:42:27, shivdasp wrote: > On 2014/03/03 05:16:34, Pawel Osciak wrote: > > On ...
6 years, 9 months ago (2014-03-04 08:46:42 UTC) #33
sheu
https://chromiumcodereview.appspot.com/137023008/diff/510001/content/common/gpu/media/exynos_v4l2_video_device.h File content/common/gpu/media/exynos_v4l2_video_device.h (right): https://chromiumcodereview.appspot.com/137023008/diff/510001/content/common/gpu/media/exynos_v4l2_video_device.h#newcode36 content/common/gpu/media/exynos_v4l2_video_device.h:36: unsigned int GetTextureTarget() OVERRIDE; While we're at this -- ...
6 years, 9 months ago (2014-03-06 08:51:34 UTC) #34
shivdasp
I did not notice sheu's comments while making patchset#7. Please have a look regarding the ...
6 years, 9 months ago (2014-03-06 11:10:08 UTC) #35
sheu
https://chromiumcodereview.appspot.com/137023008/diff/510001/content/common/gpu/media/tegra_v4l2_video_device.cc File content/common/gpu/media/tegra_v4l2_video_device.cc (right): https://chromiumcodereview.appspot.com/137023008/diff/510001/content/common/gpu/media/tegra_v4l2_video_device.cc#newcode120 content/common/gpu/media/tegra_v4l2_video_device.cc:120: TEGRAV4L2_DLSYM_OR_RETURN_ON_ERROR(UseEglImage); On 2014/03/06 11:10:09, shivdasp wrote: > Initialize() is ...
6 years, 9 months ago (2014-03-07 00:18:07 UTC) #36
sheu
I'm considering a fix for the stuck DecodeBufferInitial() in: http://chromiumcodereview.appspot.com/189993002/
6 years, 9 months ago (2014-03-07 04:06:22 UTC) #37
shivdasp
https://codereview.chromium.org/137023008/diff/660001/content/common/gpu/media/v4l2_video_decode_accelerator.cc File content/common/gpu/media/v4l2_video_decode_accelerator.cc (right): https://codereview.chromium.org/137023008/diff/660001/content/common/gpu/media/v4l2_video_decode_accelerator.cc#newcode719 content/common/gpu/media/v4l2_video_decode_accelerator.cc:719: goto chk_format_info; Jumping to GetFormatInfo() will move us out ...
6 years, 9 months ago (2014-03-07 17:31:40 UTC) #38
sheu
https://chromiumcodereview.appspot.com/137023008/diff/660001/content/common/gpu/media/v4l2_video_decode_accelerator.cc File content/common/gpu/media/v4l2_video_decode_accelerator.cc (right): https://chromiumcodereview.appspot.com/137023008/diff/660001/content/common/gpu/media/v4l2_video_decode_accelerator.cc#newcode719 content/common/gpu/media/v4l2_video_decode_accelerator.cc:719: goto chk_format_info; On 2014/03/07 17:31:40, shivdasp wrote: > Jumping ...
6 years, 9 months ago (2014-03-07 20:31:49 UTC) #39
sheu
https://chromiumcodereview.appspot.com/137023008/diff/660001/content/common/gpu/media/v4l2_video_decode_accelerator.cc File content/common/gpu/media/v4l2_video_decode_accelerator.cc (right): https://chromiumcodereview.appspot.com/137023008/diff/660001/content/common/gpu/media/v4l2_video_decode_accelerator.cc#newcode719 content/common/gpu/media/v4l2_video_decode_accelerator.cc:719: goto chk_format_info; On 2014/03/07 20:31:50, sheu wrote: > On ...
6 years, 9 months ago (2014-03-07 21:36:29 UTC) #40
shivdasp
https://codereview.chromium.org/137023008/diff/660001/content/common/gpu/media/v4l2_video_decode_accelerator.cc File content/common/gpu/media/v4l2_video_decode_accelerator.cc (right): https://codereview.chromium.org/137023008/diff/660001/content/common/gpu/media/v4l2_video_decode_accelerator.cc#newcode719 content/common/gpu/media/v4l2_video_decode_accelerator.cc:719: goto chk_format_info; Ohh that's why this never happens on ...
6 years, 9 months ago (2014-03-10 05:58:12 UTC) #41
Pawel Osciak
On 2014/03/07 21:36:29, sheu wrote: > https://chromiumcodereview.appspot.com/137023008/diff/660001/content/common/gpu/media/v4l2_video_decode_accelerator.cc > File content/common/gpu/media/v4l2_video_decode_accelerator.cc (right): > > https://chromiumcodereview.appspot.com/137023008/diff/660001/content/common/gpu/media/v4l2_video_decode_accelerator.cc#newcode719 > ...
6 years, 9 months ago (2014-03-10 11:31:48 UTC) #42
Pawel Osciak
On 2014/03/10 05:58:12, shivdasp wrote: > https://codereview.chromium.org/137023008/diff/660001/content/common/gpu/media/v4l2_video_decode_accelerator.cc > File content/common/gpu/media/v4l2_video_decode_accelerator.cc (right): > > https://codereview.chromium.org/137023008/diff/660001/content/common/gpu/media/v4l2_video_decode_accelerator.cc#newcode719 > ...
6 years, 9 months ago (2014-03-10 11:32:54 UTC) #43
sheu
On 2014/03/10 11:31:48, Pawel Osciak wrote: > Hm, this is actually quite a neat idea... ...
6 years, 9 months ago (2014-03-10 19:42:18 UTC) #44
Pawel Osciak
On 2014/03/10 19:42:18, sheu wrote: > On 2014/03/10 11:31:48, Pawel Osciak wrote: > > Hm, ...
6 years, 9 months ago (2014-03-11 00:03:14 UTC) #45
shivdasp
On 2014/03/10 11:32:54, Pawel Osciak wrote: > On 2014/03/10 05:58:12, shivdasp wrote: > > > ...
6 years, 9 months ago (2014-03-11 06:25:19 UTC) #46
Pawel Osciak
On 2014/03/11 06:25:19, shivdasp wrote: > On 2014/03/10 11:32:54, Pawel Osciak wrote: > > On ...
6 years, 9 months ago (2014-03-12 06:30:33 UTC) #47
shivdasp
On 2014/03/12 06:30:33, Pawel Osciak wrote: > On 2014/03/11 06:25:19, shivdasp wrote: > > On ...
6 years, 9 months ago (2014-03-12 06:46:15 UTC) #48
Pawel Osciak
On 2014/03/12 06:46:15, shivdasp wrote: > On 2014/03/12 06:30:33, Pawel Osciak wrote: > > On ...
6 years, 9 months ago (2014-03-12 06:49:57 UTC) #49
shivdasp
On 2014/03/12 06:49:57, Pawel Osciak wrote: > On 2014/03/12 06:46:15, shivdasp wrote: > > On ...
6 years, 9 months ago (2014-03-12 09:35:08 UTC) #50
Pawel Osciak
On 2014/03/12 09:35:08, shivdasp wrote: > On 2014/03/12 06:49:57, Pawel Osciak wrote: > > On ...
6 years, 9 months ago (2014-03-12 09:54:09 UTC) #51
shivdasp
On 2014/03/12 09:54:09, Pawel Osciak wrote: > On 2014/03/12 09:35:08, shivdasp wrote: > > On ...
6 years, 9 months ago (2014-03-12 12:16:14 UTC) #52
shivdasp
On 2014/03/12 12:16:14, shivdasp wrote: > On 2014/03/12 09:54:09, Pawel Osciak wrote: > > On ...
6 years, 9 months ago (2014-03-13 07:14:56 UTC) #53
shivdasp
On 2014/03/13 07:14:56, shivdasp wrote: > On 2014/03/12 12:16:14, shivdasp wrote: > > On 2014/03/12 ...
6 years, 9 months ago (2014-03-13 10:48:08 UTC) #54
shivdasp
On 2014/03/13 10:48:08, shivdasp wrote: > On 2014/03/13 07:14:56, shivdasp wrote: > > On 2014/03/12 ...
6 years, 9 months ago (2014-03-14 04:48:28 UTC) #55
Pawel Osciak
On 2014/03/14 04:48:28, shivdasp wrote: > Pawel, > How do you suggest we go ahead ...
6 years, 9 months ago (2014-03-18 05:49:59 UTC) #56
shivdasp
On 2014/03/18 05:49:59, Pawel Osciak wrote: > On 2014/03/14 04:48:28, shivdasp wrote: > > Pawel, ...
6 years, 9 months ago (2014-03-18 06:08:49 UTC) #57
Pawel Osciak
On 2014/03/18 06:08:49, shivdasp wrote: > On 2014/03/18 05:49:59, Pawel Osciak wrote: > > On ...
6 years, 9 months ago (2014-03-18 06:15:28 UTC) #58
shivdasp
On 2014/03/18 06:15:28, Pawel Osciak wrote: > On 2014/03/18 06:08:49, shivdasp wrote: > > On ...
6 years, 9 months ago (2014-03-18 12:22:17 UTC) #59
sheu
On 2014/03/18 12:22:17, shivdasp wrote: > I fixed the rendering helper to reallocate the textures ...
6 years, 9 months ago (2014-03-18 20:15:22 UTC) #60
sheu
Can we split this CL into two? I think the changes to the V4L2VideoDevice interface ...
6 years, 9 months ago (2014-03-18 22:45:42 UTC) #61
shivdasp
On 2014/03/18 20:15:22, sheu wrote: > On 2014/03/18 12:22:17, shivdasp wrote: > > I fixed ...
6 years, 9 months ago (2014-03-19 04:23:20 UTC) #62
Pawel Osciak
On 2014/03/18 12:22:17, shivdasp wrote: > On 2014/03/18 06:15:28, Pawel Osciak wrote: > > On ...
6 years, 9 months ago (2014-03-19 05:35:15 UTC) #63
Pawel Osciak
On 2014/03/19 04:23:20, shivdasp wrote: > On 2014/03/18 20:15:22, sheu wrote: > > On 2014/03/18 ...
6 years, 9 months ago (2014-03-19 05:44:09 UTC) #64
shivdasp
On 2014/03/19 05:44:09, Pawel Osciak wrote: > On 2014/03/19 04:23:20, shivdasp wrote: > > On ...
6 years, 9 months ago (2014-03-19 06:17:55 UTC) #65
Pawel Osciak
On 2014/03/19 06:17:55, shivdasp wrote: > On 2014/03/19 05:44:09, Pawel Osciak wrote: > > On ...
6 years, 9 months ago (2014-03-19 06:21:45 UTC) #66
shivdasp
On 2014/03/19 06:21:45, Pawel Osciak wrote: > On 2014/03/19 06:17:55, shivdasp wrote: > > On ...
6 years, 9 months ago (2014-03-20 04:44:16 UTC) #67
Pawel Osciak
On 2014/03/20 04:44:16, shivdasp wrote: > On 2014/03/19 06:21:45, Pawel Osciak wrote: > > On ...
6 years, 9 months ago (2014-03-20 05:29:41 UTC) #68
sheu
On 2014/03/20 05:29:41, Pawel Osciak wrote: > Great, thank you. Please let me know if ...
6 years, 9 months ago (2014-03-20 21:30:00 UTC) #69
shivdasp
On 2014/03/20 21:30:00, sheu wrote: > On 2014/03/20 05:29:41, Pawel Osciak wrote: > > Great, ...
6 years, 9 months ago (2014-03-21 01:16:19 UTC) #70
shivdasp
Addressed all the previous comments too, PTAL. Thanks https://codereview.chromium.org/137023008/diff/680001/content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc File content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc (right): https://codereview.chromium.org/137023008/diff/680001/content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc#newcode112 content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc:112: write_whitelist->push_back(kDevVicPath); ...
6 years, 9 months ago (2014-03-21 10:53:44 UTC) #71
sheu
https://chromiumcodereview.appspot.com/137023008/diff/680001/content/common/gpu/media/v4l2_video_decode_accelerator.cc File content/common/gpu/media/v4l2_video_decode_accelerator.cc (right): https://chromiumcodereview.appspot.com/137023008/diff/680001/content/common/gpu/media/v4l2_video_decode_accelerator.cc#newcode1031 content/common/gpu/media/v4l2_video_decode_accelerator.cc:1031: } Pawel brought up the point that it might ...
6 years, 9 months ago (2014-03-21 23:19:53 UTC) #72
shivdasp
https://codereview.chromium.org/137023008/diff/680001/content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc File content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc (right): https://codereview.chromium.org/137023008/diff/680001/content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc#newcode112 content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc:112: write_whitelist->push_back(kDevVicPath); On 2014/03/21 23:19:54, sheu wrote: > On 2014/03/21 ...
6 years, 9 months ago (2014-03-24 03:48:59 UTC) #73
shivdasp
Addressed John's comments, PTAL. https://codereview.chromium.org/137023008/diff/680001/content/common/gpu/media/v4l2_video_decode_accelerator.cc File content/common/gpu/media/v4l2_video_decode_accelerator.cc (right): https://codereview.chromium.org/137023008/diff/680001/content/common/gpu/media/v4l2_video_decode_accelerator.cc#newcode1031 content/common/gpu/media/v4l2_video_decode_accelerator.cc:1031: } Done, please refer patchset#9. ...
6 years, 9 months ago (2014-03-24 06:01:26 UTC) #74
shivdasp
> > Great, thank you. Please let me know if the EOS event also solves ...
6 years, 9 months ago (2014-03-24 06:30:34 UTC) #75
shivdasp
This is tested with all the cases for resolution change events. PTAL
6 years, 9 months ago (2014-03-24 12:14:23 UTC) #76
Pawel Osciak
I'll defer reviewing the sandboxing/library loading stuff to Jorge. https://chromiumcodereview.appspot.com/137023008/diff/740001/content/common/gpu/media/tegra_v4l2_video_device.cc File content/common/gpu/media/tegra_v4l2_video_device.cc (right): https://chromiumcodereview.appspot.com/137023008/diff/740001/content/common/gpu/media/tegra_v4l2_video_device.cc#newcode119 content/common/gpu/media/tegra_v4l2_video_device.cc:119: ...
6 years, 9 months ago (2014-03-25 08:21:07 UTC) #77
shivdasp
https://codereview.chromium.org/137023008/diff/740001/content/common/gpu/media/tegra_v4l2_video_device.cc File content/common/gpu/media/tegra_v4l2_video_device.cc (right): https://codereview.chromium.org/137023008/diff/740001/content/common/gpu/media/tegra_v4l2_video_device.cc#newcode119 content/common/gpu/media/tegra_v4l2_video_device.cc:119: const EGLint* /*attrib*/, On 2014/03/25 08:21:08, Pawel Osciak wrote: ...
6 years, 9 months ago (2014-03-25 10:36:39 UTC) #78
Pawel Osciak
https://codereview.chromium.org/137023008/diff/740001/content/common/gpu/media/tegra_v4l2_video_device.cc File content/common/gpu/media/tegra_v4l2_video_device.cc (right): https://codereview.chromium.org/137023008/diff/740001/content/common/gpu/media/tegra_v4l2_video_device.cc#newcode119 content/common/gpu/media/tegra_v4l2_video_device.cc:119: const EGLint* /*attrib*/, On 2014/03/25 10:36:40, shivdasp wrote: > ...
6 years, 9 months ago (2014-03-25 11:20:55 UTC) #79
Jorge Lucangeli Obes
shivdasp@: please coordinate with davidung@ who's working on https://chromiumcodereview.appspot.com/179983006/ for the sandbox changes. Thanks! https://chromiumcodereview.appspot.com/137023008/diff/740001/content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc ...
6 years, 9 months ago (2014-03-25 21:15:13 UTC) #80
shivdasp
Jorge, Yes either of us have to rebase if the other CL lands before. Shivdas ...
6 years, 9 months ago (2014-03-26 02:54:24 UTC) #81
Jorge Lucangeli Obes
https://codereview.chromium.org/137023008/diff/740001/content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc File content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc (right): https://codereview.chromium.org/137023008/diff/740001/content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc#newcode225 content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc:225: dlopen("/usr/lib/libtegrav4l2.so", dlopen_flag); On 2014/03/26 02:54:25, shivdasp wrote: > Yes ...
6 years, 9 months ago (2014-03-26 03:15:47 UTC) #82
shivdasp
https://codereview.chromium.org/137023008/diff/740001/content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc File content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc (right): https://codereview.chromium.org/137023008/diff/740001/content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc#newcode225 content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc:225: dlopen("/usr/lib/libtegrav4l2.so", dlopen_flag); I get your point. I think all ...
6 years, 9 months ago (2014-03-26 03:34:28 UTC) #83
Jorge Lucangeli Obes
https://codereview.chromium.org/137023008/diff/740001/content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc File content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc (right): https://codereview.chromium.org/137023008/diff/740001/content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc#newcode225 content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc:225: dlopen("/usr/lib/libtegrav4l2.so", dlopen_flag); On 2014/03/26 03:34:29, shivdasp wrote: > I ...
6 years, 9 months ago (2014-03-26 16:47:23 UTC) #84
shivdasp
https://codereview.chromium.org/137023008/diff/740001/content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc File content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc (right): https://codereview.chromium.org/137023008/diff/740001/content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc#newcode225 content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc:225: dlopen("/usr/lib/libtegrav4l2.so", dlopen_flag); On 2014/03/26 16:47:24, Jorge Lucangeli Obes wrote: ...
6 years, 9 months ago (2014-03-26 21:30:50 UTC) #85
shivdasp
Hi Pawel, Tried to address your proposal of moving the device specific egl allocation in ...
6 years, 9 months ago (2014-03-26 21:35:56 UTC) #86
sheu
https://chromiumcodereview.appspot.com/137023008/diff/810001/content/common/gpu/media/exynos_v4l2_video_device.cc File content/common/gpu/media/exynos_v4l2_video_device.cc (right): https://chromiumcodereview.appspot.com/137023008/diff/810001/content/common/gpu/media/exynos_v4l2_video_device.cc#newcode8 content/common/gpu/media/exynos_v4l2_video_device.cc:8: #include <libdrm/drm_fourcc.h> Header ordering -- this one above. https://chromiumcodereview.appspot.com/137023008/diff/810001/content/common/gpu/media/exynos_v4l2_video_device.cc#newcode194 ...
6 years, 9 months ago (2014-03-27 02:00:22 UTC) #87
shivdasp
https://codereview.chromium.org/137023008/diff/810001/content/common/gpu/media/exynos_v4l2_video_device.cc File content/common/gpu/media/exynos_v4l2_video_device.cc (right): https://codereview.chromium.org/137023008/diff/810001/content/common/gpu/media/exynos_v4l2_video_device.cc#newcode194 content/common/gpu/media/exynos_v4l2_video_device.cc:194: device_output_buffer_map_.clear(); Seems this is not needed now after reading ...
6 years, 9 months ago (2014-03-27 02:41:03 UTC) #88
Pawel Osciak
https://chromiumcodereview.appspot.com/137023008/diff/740001/content/common/gpu/media/v4l2_video_device.h File content/common/gpu/media/v4l2_video_device.h (right): https://chromiumcodereview.appspot.com/137023008/diff/740001/content/common/gpu/media/v4l2_video_device.h#newcode60 content/common/gpu/media/v4l2_video_device.h:60: // may have its own format. The texture_id is ...
6 years, 9 months ago (2014-03-27 05:18:05 UTC) #89
shivdasp
https://chromiumcodereview.appspot.com/137023008/diff/740001/content/common/gpu/media/v4l2_video_device.h File content/common/gpu/media/v4l2_video_device.h (right): https://chromiumcodereview.appspot.com/137023008/diff/740001/content/common/gpu/media/v4l2_video_device.h#newcode60 content/common/gpu/media/v4l2_video_device.h:60: // may have its own format. The texture_id is ...
6 years, 9 months ago (2014-03-27 05:40:36 UTC) #90
Pawel Osciak
https://chromiumcodereview.appspot.com/137023008/diff/810001/content/common/gpu/media/v4l2_video_device.h File content/common/gpu/media/v4l2_video_device.h (right): https://chromiumcodereview.appspot.com/137023008/diff/810001/content/common/gpu/media/v4l2_video_device.h#newcode70 content/common/gpu/media/v4l2_video_device.h:70: virtual void DestroyEGLImage(unsigned int buffer_index) = 0; On 2014/03/27 ...
6 years, 9 months ago (2014-03-27 05:58:34 UTC) #91
shivdasp
https://codereview.chromium.org/137023008/diff/810001/content/common/gpu/media/v4l2_video_device.h File content/common/gpu/media/v4l2_video_device.h (right): https://codereview.chromium.org/137023008/diff/810001/content/common/gpu/media/v4l2_video_device.h#newcode76 content/common/gpu/media/v4l2_video_device.h:76: virtual uint32 GetCapturePixelFormat() = 0; On 2014/03/27 05:58:35, Pawel ...
6 years, 9 months ago (2014-03-27 06:51:45 UTC) #92
Pawel Osciak
https://codereview.chromium.org/137023008/diff/810001/content/common/gpu/media/v4l2_video_device.h File content/common/gpu/media/v4l2_video_device.h (right): https://codereview.chromium.org/137023008/diff/810001/content/common/gpu/media/v4l2_video_device.h#newcode76 content/common/gpu/media/v4l2_video_device.h:76: virtual uint32 GetCapturePixelFormat() = 0; On 2014/03/27 06:51:46, shivdasp ...
6 years, 9 months ago (2014-03-27 06:59:55 UTC) #93
sheu
https://codereview.chromium.org/137023008/diff/810001/content/common/gpu/media/v4l2_video_device.h File content/common/gpu/media/v4l2_video_device.h (right): https://codereview.chromium.org/137023008/diff/810001/content/common/gpu/media/v4l2_video_device.h#newcode70 content/common/gpu/media/v4l2_video_device.h:70: virtual void DestroyEGLImage(unsigned int buffer_index) = 0; On 2014/03/27 ...
6 years, 9 months ago (2014-03-27 07:08:48 UTC) #94
shivdasp
https://codereview.chromium.org/137023008/diff/810001/content/common/gpu/media/v4l2_video_device.h File content/common/gpu/media/v4l2_video_device.h (right): https://codereview.chromium.org/137023008/diff/810001/content/common/gpu/media/v4l2_video_device.h#newcode76 content/common/gpu/media/v4l2_video_device.h:76: virtual uint32 GetCapturePixelFormat() = 0; On 2014/03/27 07:08:49, sheu ...
6 years, 9 months ago (2014-03-27 07:40:06 UTC) #95
Pawel Osciak
https://codereview.chromium.org/137023008/diff/810001/content/common/gpu/media/v4l2_video_device.h File content/common/gpu/media/v4l2_video_device.h (right): https://codereview.chromium.org/137023008/diff/810001/content/common/gpu/media/v4l2_video_device.h#newcode76 content/common/gpu/media/v4l2_video_device.h:76: virtual uint32 GetCapturePixelFormat() = 0; > Apart from DCHECK ...
6 years, 9 months ago (2014-03-27 07:45:05 UTC) #96
shivdasp
https://codereview.chromium.org/137023008/diff/810001/content/common/gpu/media/v4l2_video_device.h File content/common/gpu/media/v4l2_video_device.h (right): https://codereview.chromium.org/137023008/diff/810001/content/common/gpu/media/v4l2_video_device.h#newcode76 content/common/gpu/media/v4l2_video_device.h:76: virtual uint32 GetCapturePixelFormat() = 0; On 2014/03/27 07:45:06, Pawel ...
6 years, 9 months ago (2014-03-27 07:54:54 UTC) #97
shivdasp
Addressed review comments, PTAL.
6 years, 9 months ago (2014-03-27 08:00:15 UTC) #98
Pawel Osciak
On 2014/03/27 07:54:54, shivdasp wrote: > https://codereview.chromium.org/137023008/diff/810001/content/common/gpu/media/v4l2_video_device.h > File content/common/gpu/media/v4l2_video_device.h (right): > > https://codereview.chromium.org/137023008/diff/810001/content/common/gpu/media/v4l2_video_device.h#newcode76 > ...
6 years, 9 months ago (2014-03-27 08:01:16 UTC) #99
shivdasp
On 2014/03/27 08:01:16, Pawel Osciak wrote: > On 2014/03/27 07:54:54, shivdasp wrote: > > > ...
6 years, 9 months ago (2014-03-27 08:13:27 UTC) #100
shivdasp
Addressed review comments regarding num_planes.
6 years, 9 months ago (2014-03-27 08:52:49 UTC) #101
Pawel Osciak
https://codereview.chromium.org/137023008/diff/850001/content/common/gpu/media/exynos_v4l2_video_device.cc File content/common/gpu/media/exynos_v4l2_video_device.cc (right): https://codereview.chromium.org/137023008/diff/850001/content/common/gpu/media/exynos_v4l2_video_device.cc#newcode126 content/common/gpu/media/exynos_v4l2_video_device.cc:126: int dmabuf_fds[2] = {-1, -1}; As I mentioned before, ...
6 years, 9 months ago (2014-03-27 09:09:49 UTC) #102
shivdasp
https://codereview.chromium.org/137023008/diff/850001/content/common/gpu/media/exynos_v4l2_video_device.cc File content/common/gpu/media/exynos_v4l2_video_device.cc (right): https://codereview.chromium.org/137023008/diff/850001/content/common/gpu/media/exynos_v4l2_video_device.cc#newcode126 content/common/gpu/media/exynos_v4l2_video_device.cc:126: int dmabuf_fds[2] = {-1, -1}; I think this is ...
6 years, 9 months ago (2014-03-27 09:33:21 UTC) #103
sheu
For what it's worth -- I think we're really close. Just bits now. https://codereview.chromium.org/137023008/diff/850001/content/common/gpu/media/exynos_v4l2_video_device.cc File ...
6 years, 9 months ago (2014-03-27 09:40:40 UTC) #104
shivdasp
https://codereview.chromium.org/137023008/diff/850001/content/common/gpu/media/exynos_v4l2_video_device.cc File content/common/gpu/media/exynos_v4l2_video_device.cc (right): https://codereview.chromium.org/137023008/diff/850001/content/common/gpu/media/exynos_v4l2_video_device.cc#newcode126 content/common/gpu/media/exynos_v4l2_video_device.cc:126: int dmabuf_fds[2] = {-1, -1}; On 2014/03/27 09:40:41, sheu ...
6 years, 9 months ago (2014-03-27 10:06:44 UTC) #105
shivdasp
Used the scopedFD for dmabuf fds. Waiting for consensus on other comments. PTAL
6 years, 9 months ago (2014-03-27 10:51:41 UTC) #106
kpurandare
On 2014/03/27 10:51:41, shivdasp wrote: Hi Pawel, Just wanted to confirm, if we are now ...
6 years, 9 months ago (2014-03-28 05:12:20 UTC) #107
Pawel Osciak
https://chromiumcodereview.appspot.com/137023008/diff/850001/content/common/gpu/media/exynos_v4l2_video_device.cc File content/common/gpu/media/exynos_v4l2_video_device.cc (right): https://chromiumcodereview.appspot.com/137023008/diff/850001/content/common/gpu/media/exynos_v4l2_video_device.cc#newcode126 content/common/gpu/media/exynos_v4l2_video_device.cc:126: int dmabuf_fds[2] = {-1, -1}; On 2014/03/27 10:06:46, shivdasp ...
6 years, 9 months ago (2014-03-28 05:33:51 UTC) #108
Pawel Osciak
By the way, for the sake of not holding this CL up too much longer ...
6 years, 9 months ago (2014-03-28 05:39:19 UTC) #109
shivdasp
https://codereview.chromium.org/137023008/diff/850001/content/common/gpu/media/exynos_v4l2_video_device.cc File content/common/gpu/media/exynos_v4l2_video_device.cc (right): https://codereview.chromium.org/137023008/diff/850001/content/common/gpu/media/exynos_v4l2_video_device.cc#newcode126 content/common/gpu/media/exynos_v4l2_video_device.cc:126: int dmabuf_fds[2] = {-1, -1}; On 2014/03/28 05:33:53, Pawel ...
6 years, 9 months ago (2014-03-28 05:42:37 UTC) #110
Pawel Osciak
On 2014/03/28 05:42:37, shivdasp wrote: > We made change in the buffer allocation to have ...
6 years, 9 months ago (2014-03-28 05:59:55 UTC) #111
Pawel Osciak
https://chromiumcodereview.appspot.com/137023008/diff/860017/content/common/gpu/media/v4l2_video_decode_accelerator.cc File content/common/gpu/media/v4l2_video_decode_accelerator.cc (left): https://chromiumcodereview.appspot.com/137023008/diff/860017/content/common/gpu/media/v4l2_video_decode_accelerator.cc#oldcode1539 content/common/gpu/media/v4l2_video_decode_accelerator.cc:1539: Let's not remove this please. https://chromiumcodereview.appspot.com/137023008/diff/860017/content/common/gpu/media/v4l2_video_decode_accelerator.cc File content/common/gpu/media/v4l2_video_decode_accelerator.cc (right): ...
6 years, 9 months ago (2014-03-28 06:21:44 UTC) #112
shivdasp
https://codereview.chromium.org/137023008/diff/860017/content/common/gpu/media/v4l2_video_decode_accelerator.cc File content/common/gpu/media/v4l2_video_decode_accelerator.cc (left): https://codereview.chromium.org/137023008/diff/860017/content/common/gpu/media/v4l2_video_decode_accelerator.cc#oldcode1539 content/common/gpu/media/v4l2_video_decode_accelerator.cc:1539: On 2014/03/28 06:21:46, Pawel Osciak wrote: > Let's not ...
6 years, 9 months ago (2014-03-28 06:40:42 UTC) #113
Pawel Osciak
https://chromiumcodereview.appspot.com/137023008/diff/860017/content/common/gpu/media/v4l2_video_decode_accelerator.cc File content/common/gpu/media/v4l2_video_decode_accelerator.cc (right): https://chromiumcodereview.appspot.com/137023008/diff/860017/content/common/gpu/media/v4l2_video_decode_accelerator.cc#newcode1887 content/common/gpu/media/v4l2_video_decode_accelerator.cc:1887: if (frame_buffer_size_.IsEmpty()) On 2014/03/28 06:40:43, shivdasp wrote: > This ...
6 years, 9 months ago (2014-03-28 06:52:32 UTC) #114
shivdasp
https://codereview.chromium.org/137023008/diff/860017/content/common/gpu/media/v4l2_video_decode_accelerator.cc File content/common/gpu/media/v4l2_video_decode_accelerator.cc (right): https://codereview.chromium.org/137023008/diff/860017/content/common/gpu/media/v4l2_video_decode_accelerator.cc#newcode1887 content/common/gpu/media/v4l2_video_decode_accelerator.cc:1887: if (frame_buffer_size_.IsEmpty()) On 2014/03/28 06:52:34, Pawel Osciak wrote: > ...
6 years, 9 months ago (2014-03-28 07:00:18 UTC) #115
Pawel Osciak
https://codereview.chromium.org/137023008/diff/860017/content/common/gpu/media/v4l2_video_decode_accelerator.cc File content/common/gpu/media/v4l2_video_decode_accelerator.cc (right): https://codereview.chromium.org/137023008/diff/860017/content/common/gpu/media/v4l2_video_decode_accelerator.cc#newcode1887 content/common/gpu/media/v4l2_video_decode_accelerator.cc:1887: if (frame_buffer_size_.IsEmpty()) On 2014/03/28 07:00:19, shivdasp wrote: > On ...
6 years, 9 months ago (2014-03-28 07:25:52 UTC) #116
shivdasp
https://codereview.chromium.org/137023008/diff/860017/content/common/gpu/media/v4l2_video_decode_accelerator.cc File content/common/gpu/media/v4l2_video_decode_accelerator.cc (right): https://codereview.chromium.org/137023008/diff/860017/content/common/gpu/media/v4l2_video_decode_accelerator.cc#newcode1887 content/common/gpu/media/v4l2_video_decode_accelerator.cc:1887: if (frame_buffer_size_.IsEmpty()) > But then the driver should not ...
6 years, 9 months ago (2014-03-28 08:00:33 UTC) #117
shivdasp
Made the number of planes parameterized and addressed other comments. PTAL
6 years, 9 months ago (2014-03-28 08:52:48 UTC) #118
shivdasp
Do let me know if this looks alright.
6 years, 9 months ago (2014-03-28 10:06:57 UTC) #119
Pawel Osciak
https://codereview.chromium.org/137023008/diff/880001/content/common/gpu/media/v4l2_video_decode_accelerator.h File content/common/gpu/media/v4l2_video_decode_accelerator.h (right): https://codereview.chromium.org/137023008/diff/880001/content/common/gpu/media/v4l2_video_decode_accelerator.h#newcode436 content/common/gpu/media/v4l2_video_decode_accelerator.h:436: uint8 output_planes_count_; s/uint8/size_t/
6 years, 9 months ago (2014-03-28 10:18:41 UTC) #120
Pawel Osciak
What's the plan for sandbox changes? In any case, I think it's time to ask ...
6 years, 9 months ago (2014-03-28 10:19:12 UTC) #121
shivdasp
On 2014/03/28 10:19:12, Pawel Osciak wrote: > What's the plan for sandbox changes? I explained ...
6 years, 9 months ago (2014-03-28 10:24:48 UTC) #122
shivdasp
https://codereview.chromium.org/137023008/diff/880001/content/common/gpu/media/v4l2_video_decode_accelerator.h File content/common/gpu/media/v4l2_video_decode_accelerator.h (right): https://codereview.chromium.org/137023008/diff/880001/content/common/gpu/media/v4l2_video_decode_accelerator.h#newcode436 content/common/gpu/media/v4l2_video_decode_accelerator.h:436: uint8 output_planes_count_; On 2014/03/28 10:18:42, Pawel Osciak wrote: > ...
6 years, 9 months ago (2014-03-28 10:53:35 UTC) #123
Ami GONE FROM CHROMIUM
Mostly nits; I think this is really close to being landable! https://codereview.chromium.org/137023008/diff/900001/content/common/gpu/media/tegra_v4l2_video_device.cc File content/common/gpu/media/tegra_v4l2_video_device.cc (right): ...
6 years, 9 months ago (2014-03-28 17:10:00 UTC) #124
shivdasp
Ami and John could you take a look at this please. Thanks
6 years, 9 months ago (2014-03-28 17:34:33 UTC) #125
Ami GONE FROM CHROMIUM
On 2014/03/28 17:34:33, shivdasp wrote: > Ami and John could you take a look at ...
6 years, 9 months ago (2014-03-28 17:44:34 UTC) #126
shivdasp
Patch incoming to address these comments. https://codereview.chromium.org/137023008/diff/900001/content/common/gpu/media/tegra_v4l2_video_device.cc File content/common/gpu/media/tegra_v4l2_video_device.cc (right): https://codereview.chromium.org/137023008/diff/900001/content/common/gpu/media/tegra_v4l2_video_device.cc#newcode4 content/common/gpu/media/tegra_v4l2_video_device.cc:4: // On 2014/03/28 ...
6 years, 9 months ago (2014-03-28 18:54:10 UTC) #127
shivdasp
Syncing my code (taking more time than usual) to rebase because of a change in ...
6 years, 9 months ago (2014-03-28 19:06:34 UTC) #128
Ami GONE FROM CHROMIUM
https://codereview.chromium.org/137023008/diff/900001/content/common/gpu/media/video_decode_accelerator_unittest.cc File content/common/gpu/media/video_decode_accelerator_unittest.cc (right): https://codereview.chromium.org/137023008/diff/900001/content/common/gpu/media/video_decode_accelerator_unittest.cc#newcode1548 content/common/gpu/media/video_decode_accelerator_unittest.cc:1548: dlopen("/usr/lib/libtegrav4l2.so", RTLD_NOW | RTLD_GLOBAL | RTLD_NODELETE); On 2014/03/28 18:54:11, ...
6 years, 9 months ago (2014-03-28 19:33:38 UTC) #129
shivdasp
https://codereview.chromium.org/137023008/diff/900001/content/common/gpu/media/video_decode_accelerator_unittest.cc File content/common/gpu/media/video_decode_accelerator_unittest.cc (right): https://codereview.chromium.org/137023008/diff/900001/content/common/gpu/media/video_decode_accelerator_unittest.cc#newcode1548 content/common/gpu/media/video_decode_accelerator_unittest.cc:1548: dlopen("/usr/lib/libtegrav4l2.so", RTLD_NOW | RTLD_GLOBAL | RTLD_NODELETE); On 2014/03/28 19:33:40, ...
6 years, 9 months ago (2014-03-28 20:39:24 UTC) #130
Ami GONE FROM CHROMIUM
On Fri, Mar 28, 2014 at 1:39 PM, <shivdasp@nvidia.com> wrote: > > https://codereview.chromium.org/137023008/diff/900001/ > content/common/gpu/media/video_decode_accelerator_unittest.cc ...
6 years, 9 months ago (2014-03-28 20:51:08 UTC) #131
shivdasp
Can't reply inline for some reason. Regarding GetFormatInfo(), it must succeed since we have received ...
6 years, 9 months ago (2014-03-28 22:03:08 UTC) #132
shivdasp
git cl format does not like "spaces around ##" and that's what happened when I ...
6 years, 9 months ago (2014-03-28 22:15:47 UTC) #133
Ami GONE FROM CHROMIUM
Yes, obey git cl format. Yes, put TegraSymbolFinder can be in tegra_v4l2_video_device.cc Let me know ...
6 years, 9 months ago (2014-03-28 23:25:24 UTC) #134
shivdasp
Updated with rebased patchset, PTAL.
6 years, 9 months ago (2014-03-29 00:17:01 UTC) #135
shivdasp
Hi Ami, Pawel, Hope there are no more comments here to be addressed. Thanks for ...
6 years, 9 months ago (2014-03-29 01:10:21 UTC) #136
kpurandare
Hi Ami, Pawel, I hope the final patchset which is rebased is good to go ...
6 years, 9 months ago (2014-03-29 02:47:43 UTC) #137
Pawel Osciak
Hi Shivdas, Could you please address my yesterday's question about the capture format (V4L2_PIX_FMT_NV12M)? Are ...
6 years, 9 months ago (2014-03-29 04:01:30 UTC) #138
kpurandare
On 2014/03/29 04:01:30, Pawel Osciak wrote: Hi Pawel, As per my understanding the capture format ...
6 years, 9 months ago (2014-03-29 06:51:04 UTC) #139
Ami GONE FROM CHROMIUM
LGTM % nits & posciak's say-so. https://codereview.chromium.org/137023008/diff/960001/content/common/gpu/media/tegra_v4l2_video_device.cc File content/common/gpu/media/tegra_v4l2_video_device.cc (right): https://codereview.chromium.org/137023008/diff/960001/content/common/gpu/media/tegra_v4l2_video_device.cc#newcode81 content/common/gpu/media/tegra_v4l2_video_device.cc:81: } nit: newline ...
6 years, 9 months ago (2014-03-29 08:23:50 UTC) #140
Pawel Osciak
On 2014/03/29 06:51:04, kpurandare wrote: > On 2014/03/29 04:01:30, Pawel Osciak wrote: > > Hi ...
6 years, 8 months ago (2014-03-29 11:14:20 UTC) #141
Pawel Osciak
LGTM % one nit, and assuming sandboxing owners approval and that the format issue is ...
6 years, 8 months ago (2014-03-29 11:16:11 UTC) #142
shivdasp
Patchset incoming for addressing nits. https://codereview.chromium.org/137023008/diff/960001/content/common/gpu/media/exynos_v4l2_video_device.cc File content/common/gpu/media/exynos_v4l2_video_device.cc (right): https://codereview.chromium.org/137023008/diff/960001/content/common/gpu/media/exynos_v4l2_video_device.cc#newcode142 content/common/gpu/media/exynos_v4l2_video_device.cc:142: } On 2014/03/29 11:16:12, ...
6 years, 8 months ago (2014-03-29 17:47:36 UTC) #143
shivdasp
On 2014/03/29 11:14:20, Pawel Osciak wrote: > On 2014/03/29 06:51:04, kpurandare wrote: > > On ...
6 years, 8 months ago (2014-03-29 17:58:16 UTC) #144
shivdasp
Addressed minor nits. PTAL
6 years, 8 months ago (2014-03-29 18:05:17 UTC) #145
shivdasp
Jorge, May I request you to take a look at the sandboxing related changes in ...
6 years, 8 months ago (2014-03-29 18:10:17 UTC) #146
shivdasp
On 2014/03/29 18:10:17, shivdasp wrote: > Jorge, > May I request you to take a ...
6 years, 8 months ago (2014-03-31 13:06:21 UTC) #147
shivdasp
Jorgelo@ and piman@ OWNERS please.
6 years, 8 months ago (2014-03-31 17:52:14 UTC) #148
Jorge Lucangeli Obes
https://codereview.chromium.org/137023008/diff/980001/content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc File content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc (right): https://codereview.chromium.org/137023008/diff/980001/content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc#newcode183 content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc:183: dlopen("/usr/lib/libtegrav4l2.so", dlopen_flag); Please add a comment above this line ...
6 years, 8 months ago (2014-03-31 18:07:23 UTC) #149
shivdasp
Addressed comment from Jorgelo@. PTAL https://codereview.chromium.org/137023008/diff/980001/content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc File content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc (right): https://codereview.chromium.org/137023008/diff/980001/content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc#newcode183 content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc:183: dlopen("/usr/lib/libtegrav4l2.so", dlopen_flag); On 2014/03/31 ...
6 years, 8 months ago (2014-03-31 18:17:04 UTC) #150
Jorge Lucangeli Obes
Sandbox lgtm.
6 years, 8 months ago (2014-03-31 18:21:20 UTC) #151
shivdasp
The CQ bit was checked by shivdasp@nvidia.com
6 years, 8 months ago (2014-03-31 18:37:21 UTC) #152
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/shivdasp@nvidia.com/137023008/1000001
6 years, 8 months ago (2014-03-31 18:40:33 UTC) #153
piman
lgtm
6 years, 8 months ago (2014-03-31 21:39:03 UTC) #154
commit-bot: I haz the power
6 years, 8 months ago (2014-03-31 21:42:17 UTC) #155
Message was sent while issue was closed.
Change committed as 260661

Powered by Google App Engine
This is Rietveld 408576698