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

Unified Diff: content/common/gpu/media/vaapi_drm_picture.cc

Issue 490233002: VaapiVideoAccelerator: make Vaapi accelerator work with ozone (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update after Pawel's review Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/common/gpu/media/vaapi_drm_picture.cc
diff --git a/content/common/gpu/media/vaapi_drm_picture.cc b/content/common/gpu/media/vaapi_drm_picture.cc
new file mode 100644
index 0000000000000000000000000000000000000000..1cb60f8646fe31daea91fcb51aedb0127d83c070
--- /dev/null
+++ b/content/common/gpu/media/vaapi_drm_picture.cc
@@ -0,0 +1,144 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "base/file_descriptor_posix.h"
+#include "content/common/gpu/media/va_surface.h"
+#include "content/common/gpu/media/vaapi_drm_picture.h"
+#include "content/common/gpu/media/vaapi_wrapper.h"
+#include "third_party/libva/va/drm/va_drm.h"
+#include "third_party/libva/va/va.h"
+#include "third_party/libva/va/va_drmcommon.h"
+#include "ui/gfx/gpu_memory_buffer.h"
+#include "ui/gl/gl_bindings.h"
+#include "ui/gl/gl_image_linux_dma_buffer.h"
+#include "ui/gl/scoped_binders.h"
+#include "ui/ozone/public/native_pixmap.h"
+#include "ui/ozone/public/ozone_platform.h"
+#include "ui/ozone/public/surface_factory_ozone.h"
+
+namespace content {
+
+VaapiDrmPicture::VaapiDrmPicture(
+ base::WeakPtr<VaapiWrapper> vaapi_wrapper,
+ const base::Callback<bool(void)> make_context_current,
+ int32 picture_buffer_id,
+ uint32 texture_id,
+ const gfx::Size& size)
+ : VaapiPicture(picture_buffer_id, texture_id, size),
+ vaapi_wrapper_(vaapi_wrapper),
+ make_context_current_(make_context_current) {
+}
+
+VaapiDrmPicture::~VaapiDrmPicture() {
+ // At this point |vaapi_wrapper_| shouldn't be available anymore.
+ DCHECK(!vaapi_wrapper_.get());
+
+ if (egl_image_.get() && make_context_current_.Run()) {
+ egl_image_->ReleaseTexImage(GL_TEXTURE_EXTERNAL_OES);
+ egl_image_->Destroy(true);
+
+ DCHECK_EQ(glGetError(), GL_NO_ERROR);
hshi1 2014/11/22 00:10:26 nit: warning: comparison between signed and unsign
+ }
+}
+
+bool VaapiDrmPicture::Initialize() {
+ VASurfaceAttrib va_attribs[2];
+ VASurfaceAttribExternalBuffers va_attrib_extbuf;
+
+ ui::OzonePlatform* platform = ui::OzonePlatform::GetInstance();
+ ui::SurfaceFactoryOzone* factory = platform->GetSurfaceFactoryOzone();
+
+ pixmap_ =
+ factory->CreateNativePixmap(size(), ui::SurfaceFactoryOzone::RGBA_8888);
hshi1 2014/11/22 00:19:30 need rebase - this function now takes 4 arguments
+ if (!pixmap_.get()) {
+ LOG(ERROR) << "Failed creating an Ozone NativePixmap";
+ return false;
+ }
+
+ int dmabuf_fd = pixmap_->GetDmaBufFd();
+ if (dmabuf_fd < 0) {
+ LOG(ERROR) << "Failed get dmabuf from an Ozone NativePixmap";
+ return false;
+ }
+ int dmabuf_pitch = pixmap_->GetDmaBufPitch();
+
+ memset(&va_attrib_extbuf, 0, sizeof(va_attrib_extbuf));
+ va_attrib_extbuf.pixel_format = VA_FOURCC_BGRX;
+ va_attrib_extbuf.width = size().width();
+ va_attrib_extbuf.height = size().height();
+ va_attrib_extbuf.data_size = size().height() * dmabuf_pitch;
+ va_attrib_extbuf.num_planes = 1;
+ va_attrib_extbuf.pitches[0] = dmabuf_pitch;
+ va_attrib_extbuf.offsets[0] = 0;
+ va_attrib_extbuf.buffers = reinterpret_cast<unsigned long*>(&dmabuf_fd);
+ va_attrib_extbuf.num_buffers = 1;
+ va_attrib_extbuf.flags = 0;
+ va_attrib_extbuf.private_data = NULL;
+
+ va_attribs[0].type = VASurfaceAttribMemoryType;
+ va_attribs[0].flags = VA_SURFACE_ATTRIB_SETTABLE;
+ va_attribs[0].value.type = VAGenericValueTypeInteger;
+ va_attribs[0].value.value.i = VA_SURFACE_ATTRIB_MEM_TYPE_DRM_PRIME;
+
+ va_attribs[1].type = VASurfaceAttribExternalBufferDescriptor;
+ va_attribs[1].flags = VA_SURFACE_ATTRIB_SETTABLE;
+ va_attribs[1].value.type = VAGenericValueTypePointer;
+ va_attribs[1].value.value.p = &va_attrib_extbuf;
+
+ if (!vaapi_wrapper_.get()) {
+ LOG(ERROR) << "VaapiWrapper not available";
+ return false;
+ }
+
+ va_surface_ = vaapi_wrapper_->CreateUnownedSurface(
+ VA_RT_FORMAT_RGB32, size(), va_attribs, arraysize(va_attribs));
+ if (!va_surface_.get()) {
+ LOG(ERROR) << "Failed to create VASurface for an Ozone NativePixmap";
+ return false;
+ }
+
+ if (!make_context_current_.Run())
+ return false;
+
+ egl_image_ = new gfx::GLImageLinuxDMABuffer(size(), GL_RGBA);
+ if (!egl_image_->Initialize(base::FileDescriptor(dmabuf_fd, false),
+ gfx::GpuMemoryBuffer::BGRA_8888,
+ dmabuf_pitch)) {
+ LOG(ERROR) << "Failed to create an EGLImage for an Ozone NativePixmap";
+ return false;
+ }
+
+ gfx::ScopedTextureBinder texture_binder(GL_TEXTURE_EXTERNAL_OES,
+ texture_id());
+ if (!egl_image_->BindTexImage(GL_TEXTURE_EXTERNAL_OES)) {
+ LOG(ERROR) << "Failed to bind texture to EGLImage";
+ return false;
+ }
+
+ return true;
+}
+
+bool VaapiDrmPicture::DownloadFromSurface(
+ const scoped_refptr<VASurface>& va_surface) {
+ if (!vaapi_wrapper_.get()) {
+ LOG(ERROR) << "VaapiWrapper not available";
+ return false;
+ }
+
+ return vaapi_wrapper_->BlitSurface(va_surface->id(),
+ va_surface->size(),
+ va_surface_->id(),
+ va_surface_->size());
+}
+
+void VaapiDrmPicture::DisposeVaapi() {
+ va_surface_ = NULL;
+}
+
+// static
+uint32 VaapiPicture::GetGLTextureTarget() {
+ return GL_TEXTURE_EXTERNAL_OES;
+}
+
+} // namespace

Powered by Google App Engine
This is Rietveld 408576698