From patchwork Tue Oct 24 23:40:03 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Rob Herring \(Arm\)" X-Patchwork-Id: 116997 Delivered-To: patch@linaro.org Received: by 10.140.22.164 with SMTP id 33csp231376qgn; Tue, 24 Oct 2017 16:40:38 -0700 (PDT) X-Google-Smtp-Source: ABhQp+TtMWIcBO+H8bqQ/QR3Osg8g+Nzafa+qFp9qTzz827vdswCz3WjxsB1N6BYrPV3bfoEAQ3c X-Received: by 10.98.37.68 with SMTP id l65mr286603pfl.253.1508888438588; Tue, 24 Oct 2017 16:40:38 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1508888438; cv=none; d=google.com; s=arc-20160816; b=V7q6z8Ufk5XbhWoLeiqP42+YnhO+Nb9uU+1Ivq9ycmzezWXoPwg6YN1Y3IE+BVnSnz vLJxD0kcRIR1LF8/Tmk6VFZPQdPyobski6NkRA5zNPu3GcDEFLykeLquv5n3LoI/XhV8 D1ewnPDDS/gdpX2SS4paWWOCCmFDWwtVXh3/Ps8DrUGo1iSVp/B2j/edUnug25ivN6/n gNQo6X2kw/Ne3RfkeTMed+vKJghVxcGGTT9PB5wzGvxwGLY/BwefPq3dDjzRevLaLN9p enDcW0l3I9TGx2bkKy4hH8Xe2KYtHQjMCC5j93imW+EsrP3HPA6xKOX1FWl/Lvm79iSv c18Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:mime-version :list-subscribe:list-help:list-post:list-archive:list-unsubscribe :list-id:precedence:message-id:date:subject:to:from:delivered-to :arc-authentication-results; bh=5GIpVBa4fwn6rDOp75gV7grxnFP7MZ8lI87BIkfdoNg=; b=Xm/5EXh34nkS3Cc01e+Y6hthvaSvaLINg84EnT9Nv31fPZLalJz87kAQHhz2mWefnS AJ8UHzuQy2WmJlmaEBz+BSjbaEiHY5YOHd5l07iaTlEIMTojyyxZNzoWQvzXkYeq13D8 qu9Du0WtpJB3Q8WK9++xrKBc5hK1S5Wtb9hbLzGA6DgpK9cloyDK8GPJcncdcB2yocrY FClUUje9pxheaTO9kxv4yNzcJrNAnKfGZPm3f232SZ3oIm2DQQcs149f4v/VA3gbKFiI NXwm7+PoOckyzGNFbJiXowZmKBpIOtI1O+sakDJSric1kikun7OPdqLgKmI7AdlKdvmU Iqyw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of dri-devel-bounces@lists.freedesktop.org designates 131.252.210.177 as permitted sender) smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Return-Path: Received: from gabe.freedesktop.org (gabe.freedesktop.org. [131.252.210.177]) by mx.google.com with ESMTPS id q8si935204pfa.310.2017.10.24.16.40.37 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 24 Oct 2017 16:40:38 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of dri-devel-bounces@lists.freedesktop.org designates 131.252.210.177 as permitted sender) client-ip=131.252.210.177; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of dri-devel-bounces@lists.freedesktop.org designates 131.252.210.177 as permitted sender) smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 36DA46E5F8; Tue, 24 Oct 2017 23:40:07 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mail-oi0-f68.google.com (mail-oi0-f68.google.com [209.85.218.68]) by gabe.freedesktop.org (Postfix) with ESMTPS id D31C46E5F9 for ; Tue, 24 Oct 2017 23:40:05 +0000 (UTC) Received: by mail-oi0-f68.google.com with SMTP id w197so40188434oif.6 for ; Tue, 24 Oct 2017 16:40:05 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id; bh=EN8L1nX6N908awn8PMV9zYl7xyoZx4VhTa4wgscQJeo=; b=DWtIokkAam04MlNLYpyg29BkPoDbKsMoq7fPpvqoEUCS0rrdLVUN7866iVGiV+0QG+ PzHv8xksHFgLnlDEwMdiS2SEbAuFxzPGfH83uYseUFOu74gElgWwDGwP4v07v/cnS1OX qec84in4Tg1FtwRGRABZDbXzGQykGUvOinKTbCEpeOX7MJAodWE82/9m2cAD1ZG0n0Km +wHdDPIaDE61bhiiXwWTZw38VNWOvmStvFV8MCBD/nRRsyD43oc/JnL8DYdcUf2hf5jv GbD6QnHiLjy0Ced0D28azPjh4TxR4GMNmVLQEUIL3Ax84Teu0uOkkvXvXRwN2FW5BXDG XFow== X-Gm-Message-State: AMCzsaVsMWV25G+EXpo4WpemwTuR9Qj1gIunMNajWPkcl4E0dRMmAmPE Aq2rNLCVDeSZ2nsuVflnqTy9+YE= X-Received: by 10.157.51.174 with SMTP id u46mr167468otc.428.1508888404699; Tue, 24 Oct 2017 16:40:04 -0700 (PDT) Received: from xps15.herring.priv (216-188-254-6.dyn.grandenetworks.net. [216.188.254.6]) by smtp.googlemail.com with ESMTPSA id v3sm814231otd.15.2017.10.24.16.40.03 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 24 Oct 2017 16:40:04 -0700 (PDT) From: Rob Herring To: dri-devel@lists.freedesktop.org Subject: [RFC PATCH hwc] drm_hwcomposer: provide a common gralloc handle definition Date: Tue, 24 Oct 2017 18:40:03 -0500 Message-Id: <20171024234003.1204-1-robh@kernel.org> X-Mailer: git-send-email 2.14.1 X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" EGL, gralloc, and HWC must all have a common definition of fd's and int's in native_handle_t to share the fd and width, height, format, etc. of a dmabuf. Move the definition into HWC so we aren't dependent on a specific gralloc implementation and so we don't have to create an importer just for different native_handle_t layouts. This will allow supporting multiple gralloc implementations that conform to this layout. Perhaps we should allow for multiple levels of subclassing by only defining the fields we care about here and not validating magic and the size. Signed-off-by: Rob Herring --- I'm not sure if there's a better way, but I can't find a way to remove the gralloc dependency. At least for EGL, only the fd is needed as it has the ANativeWindow and can get all the info from that. Android.mk | 1 - gralloc_drm_handle.h | 85 ++++++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 85 insertions(+), 1 deletion(-) create mode 100644 gralloc_drm_handle.h diff --git a/Android.mk b/Android.mk index 611fcb75190b..ee5b8bfb44d0 100644 --- a/Android.mk +++ b/Android.mk @@ -47,7 +47,6 @@ LOCAL_SHARED_LIBRARIES := \ LOCAL_STATIC_LIBRARIES := libdrmhwc_utils LOCAL_C_INCLUDES := \ - external/gbm_gralloc \ system/core/libsync LOCAL_SRC_FILES := \ diff --git a/gralloc_drm_handle.h b/gralloc_drm_handle.h new file mode 100644 index 000000000000..aa89e46db619 --- /dev/null +++ b/gralloc_drm_handle.h @@ -0,0 +1,85 @@ +/* + * Copyright (C) 2010-2011 Chia-I Wu + * Copyright (C) 2010-2011 LunarG Inc. + * Copyright (C) 2016 Linaro, Ltd., Rob Herring + * + * Permission is hereby granted, free of charge, to any person obtaining a + * copy of this software and associated documentation files (the "Software"), + * to deal in the Software without restriction, including without limitation + * the rights to use, copy, modify, merge, publish, distribute, sublicense, + * and/or sell copies of the Software, and to permit persons to whom the + * Software is furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included + * in all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER + * DEALINGS IN THE SOFTWARE. + */ + +#ifndef _GRALLOC_DRM_HANDLE_H_ +#define _GRALLOC_DRM_HANDLE_H_ + +#include + +#ifdef __cplusplus +extern "C" { +#endif + +struct gralloc_drm_handle_t { + native_handle_t base; + + /* file descriptors */ + int prime_fd; + + /* integers */ + int magic; + + int width; + int height; + int format; + int usage; + int stride; /* the stride in bytes */ + uint64_t modifier; /* buffer modifiers */ + + int data_owner; /* owner of data (for validation) */ + union { + void *data; /* private pointer for gralloc */ + uint64_t reserved; + } __attribute__((aligned(8))); +}; +#define GRALLOC_HANDLE_MAGIC 0x5f47424d +#define GRALLOC_HANDLE_NUM_FDS 1 +#define GRALLOC_HANDLE_NUM_INTS ( \ + ((sizeof(struct gralloc_drm_handle_t) - sizeof(native_handle_t))/sizeof(int)) \ + - GRALLOC_HANDLE_NUM_FDS) + +static inline struct gralloc_drm_handle_t *gralloc_drm_handle(buffer_handle_t _handle) +{ + struct gralloc_drm_handle_t *handle = + (struct gralloc_drm_handle_t *) _handle; + + if (handle && (handle->base.version != sizeof(handle->base) || + handle->base.numInts != GRALLOC_HANDLE_NUM_INTS || + handle->base.numFds != GRALLOC_HANDLE_NUM_FDS || + handle->magic != GRALLOC_HANDLE_MAGIC)) + return NULL; + + return handle; +} + +static inline int gralloc_drm_get_prime_fd(buffer_handle_t _handle) +{ + struct gralloc_drm_handle_t *handle = gralloc_drm_handle(_handle); + return (handle) ? handle->prime_fd : -1; +} + +#ifdef __cplusplus +} +#endif +#endif /* _GRALLOC_DRM_HANDLE_H_ */