{"id":18987,"url":"https://patchwork.libcamera.org/api/1.1/patches/18987/?format=json","web_url":"https://patchwork.libcamera.org/patch/18987/","project":{"id":1,"url":"https://patchwork.libcamera.org/api/1.1/projects/1/?format=json","name":"libcamera","link_name":"libcamera","list_id":"libcamera_core","list_email":"libcamera-devel@lists.libcamera.org","web_url":"","scm_url":"","webscm_url":""},"msgid":"<20230912-gralloc-api-v4-v1-3-0f80402d8e7a@baylibre.com>","date":"2023-09-12T09:36:12","name":"[libcamera-devel,3/4] android: Stub GraphicBufferAllocator for build tests","commit_ref":null,"pull_url":null,"state":"superseded","archived":false,"hash":"e4343dd1fe0cc66c195b81b0415ba01a9806a9d5","submitter":{"id":153,"url":"https://patchwork.libcamera.org/api/1.1/people/153/?format=json","name":"Mattijs Korpershoek","email":"mkorpershoek@baylibre.com"},"delegate":null,"mbox":"https://patchwork.libcamera.org/patch/18987/mbox/","series":[{"id":4016,"url":"https://patchwork.libcamera.org/api/1.1/series/4016/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=4016","date":"2023-09-12T09:36:09","name":"android: switch over to modern gralloc API via libui","version":1,"mbox":"https://patchwork.libcamera.org/series/4016/mbox/"}],"comments":"https://patchwork.libcamera.org/api/patches/18987/comments/","check":"pending","checks":"https://patchwork.libcamera.org/api/patches/18987/checks/","tags":{},"headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 172F3C32B1\n\tfor ;\n\tTue, 12 Sep 2023 09:36:25 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 3A722628FD;\n\tTue, 12 Sep 2023 11:36:24 +0200 (CEST)","from mail-wm1-x329.google.com (mail-wm1-x329.google.com\n\t[IPv6:2a00:1450:4864:20::329])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id CB680628ED\n\tfor ;\n\tTue, 12 Sep 2023 11:36:20 +0200 (CEST)","by mail-wm1-x329.google.com with SMTP id\n\t5b1f17b1804b1-401bdff4cb4so57772895e9.3\n\tfor ;\n\tTue, 12 Sep 2023 02:36:20 -0700 (PDT)","from [192.168.2.39] ([82.66.159.240])\n\tby smtp.gmail.com with ESMTPSA id\n\ty10-20020a05600c364a00b00401b242e2e6sm15578253wmq.47.2023.09.12.02.36.20\n\t(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n\tTue, 12 Sep 2023 02:36:20 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1694511384;\n\tbh=+3dnsNjsMSU2OFhFNq6MlkaSCcQjIWhSpmvXdSKm+fE=;\n\th=Date:References:In-Reply-To:To:Subject:List-Id:List-Unsubscribe:\n\tList-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:\n\tFrom;\n\tb=a/iDAztnGS+aCzz6DFC9k6XdrBAcc+/lMDxlXD0zIIRUcOhwQNKYwINvNqLvAq9q5\n\twpqIVncquZg9LM1YxRxyCqheLS9QWolao/DMp0wyza0cA7OyABP8rCdX+cF4gYDSHu\n\tY5vyaBKVgCCnZrYij2agva0NvcT6IE9yK6UufmZ5CU33gWWXPBDWrcT+/6A+HanJ6C\n\t4Yc7LR0L2AMNTcg3quXhY1PhuGI6fOjEdEizQtcdmfRnndZIqm//hsXp6JiZ9zuIEm\n\tH/2B0uy0zRM/Q5B+b4k6PTyqKCneej9Rxn0aW6PD7JlAaVdYpDdDLGkXDbAvOZPbcG\n\thNDYXZaALd7KQ==","v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1694511380;\n\tx=1695116180; darn=lists.libcamera.org; \n\th=cc:to:in-reply-to:references:message-id:content-transfer-encoding\n\t:mime-version:subject:date:from:from:to:cc:subject:date:message-id\n\t:reply-to; bh=NpLe2ktemPeEsIRO+V+DPbMtVuaE1wO2VAeNe9UlQtU=;\n\tb=13nRBv14ZL2kBeJGMcH/70jGwctAnHuqGzJ/x6y/Zfhh5dogNNRFZIb1HaUz6P34F9\n\tFPM0yHiTdsX1cYcmP6OP65uth/X5HEs9oAZzeVCjH05iyQcqDey7TvpPJJJnb+G39JbR\n\tryUs0Sg2VtiWKJBfZ0YhebLCcRq+Tovx0pOIyDNs47KEY6GCr8hxtTZHk8Owwcgqgptb\n\t2DKjzOuZQiwWEK23O+QD8CvjGt/cPNfqS3ShJEai+mEIv0WR0RnFC7E3tn4vLjCT8bQ4\n\tc8VjhmEBdxh87yKDP6271TWiHWzxd//BNchApMqHRgFz5+SpW8MaNAHEJ02XUVlUPcJn\n\th6bw=="],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (2048-bit key; \n\tunprotected)\n\theader.d=baylibre-com.20230601.gappssmtp.com\n\theader.i=@baylibre-com.20230601.gappssmtp.com header.b=\"13nRBv14\"; \n\tdkim-atps=neutral","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20230601; t=1694511380; x=1695116180;\n\th=cc:to:in-reply-to:references:message-id:content-transfer-encoding\n\t:mime-version:subject:date:from:x-gm-message-state:from:to:cc\n\t:subject:date:message-id:reply-to;\n\tbh=NpLe2ktemPeEsIRO+V+DPbMtVuaE1wO2VAeNe9UlQtU=;\n\tb=edMWM7jI+6o/PO/Sv2dfPhoPrJRpKDxcvjOctDKjX/A4o9bYQF9G5Np8BOgeT5pvV6\n\tkiZ8Yom3u+UsUP+95KgA+YhMrQzCT/8SiEGiA8Koe6yvhVIzlqcl9fp0HnYN1aDPU0fB\n\tiKOJCwiELfif1k1pYM/E7qiCwUvF10B0gz0UMeHLYatY4MJDd80xRw9DuW5PhKa2sRrH\n\trYc2I40V5Y6Ivl56+UMXhbpZrYHuat2BxfNUaIC7oFVIjBBCI1AZyHAQLey0Vetp2YuZ\n\t8YUgstJwZPg+YZ//LeX0SLgbYZKjdkJoo/zGlWY92jU0+BmeEdyy3Lzeo4ZPxpIq4XoU\n\tQyPg==","X-Gm-Message-State":"AOJu0YzYQNQzoIeqyMdirqYMxf83VPcH359wO3v9t7Gg5nYznRsi+aB8\n\tKjmqoPDhTO0y6HeLJEy8INDq/Q==","X-Google-Smtp-Source":"AGHT+IGLszgiohfb4x3SNnZ7+oYKv52n4jVDR5imDmZ6UzN1BMudTCuEB/lGKmEPSXQ0lzZfT/l8NQ==","X-Received":"by 2002:a7b:cd82:0:b0:401:b6f6:d8fd with SMTP id\n\ty2-20020a7bcd82000000b00401b6f6d8fdmr10834831wmj.6.1694511380535; \n\tTue, 12 Sep 2023 02:36:20 -0700 (PDT)","Date":"Tue, 12 Sep 2023 11:36:12 +0200","MIME-Version":"1.0","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"7bit","Message-Id":"<20230912-gralloc-api-v4-v1-3-0f80402d8e7a@baylibre.com>","References":"<20230912-gralloc-api-v4-v1-0-0f80402d8e7a@baylibre.com>","In-Reply-To":"<20230912-gralloc-api-v4-v1-0-0f80402d8e7a@baylibre.com>","To":"libcamera-devel@lists.libcamera.org","X-Mailer":"b4 0.12.3","Subject":"[libcamera-devel] [PATCH 3/4] android: Stub GraphicBufferAllocator\n\tfor build tests","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","From":"Mattijs Korpershoek via libcamera-devel\n\t","Reply-To":"Mattijs Korpershoek ","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "},"content":"If we want to keep building libcamera on traditional Linux systems with:\n -Dandroid=enabled -Dandroid_platform=generic\n\nWe should stub GraphicBufferAllocator, which is not available.\nIt's only available when building with the VNDK or when building within the\nAOSP tree.\n\nAlso remove some deprecated methods and inclusions which are not needed for\nthe stub class.\n\nSigned-off-by: Mattijs Korpershoek \n---\n .../libs/ui/include/ui/GraphicBufferAllocator.h | 30 -------------\n src/android/mm/graphic_buffer_allocator_stub.cpp | 50 ++++++++++++++++++++++\n src/android/mm/meson.build | 1 +\n 3 files changed, 51 insertions(+), 30 deletions(-)","diff":"diff --git a/include/android/frameworks/native/libs/ui/include/ui/GraphicBufferAllocator.h b/include/android/frameworks/native/libs/ui/include/ui/GraphicBufferAllocator.h\nindex e4674d746e37..9eac5bbe8324 100644\n--- a/include/android/frameworks/native/libs/ui/include/ui/GraphicBufferAllocator.h\n+++ b/include/android/frameworks/native/libs/ui/include/ui/GraphicBufferAllocator.h\n@@ -29,15 +29,10 @@\n #include \n \n #include \n-#include \n-#include \n #include \n \n namespace android {\n \n-class GrallocAllocator;\n-class GraphicBufferMapper;\n-\n class GraphicBufferAllocator : public Singleton\n {\n public:\n@@ -52,25 +47,6 @@ public:\n uint64_t usage, buffer_handle_t* handle, uint32_t* stride,\n std::string requestorName);\n \n- /**\n- * Allocates and does NOT import a gralloc buffer. Buffers cannot be used until they have\n- * been imported. This function is for advanced use cases only.\n- *\n- * The raw native handle must be freed by calling native_handle_close() followed by\n- * native_handle_delete().\n- */\n- status_t allocateRawHandle(uint32_t w, uint32_t h, PixelFormat format, uint32_t layerCount,\n- uint64_t usage, buffer_handle_t* handle, uint32_t* stride,\n- std::string requestorName);\n-\n- /**\n- * DEPRECATED: GraphicBufferAllocator does not use the graphicBufferId.\n- */\n- status_t allocate(uint32_t w, uint32_t h, PixelFormat format,\n- uint32_t layerCount, uint64_t usage,\n- buffer_handle_t* handle, uint32_t* stride, uint64_t graphicBufferId,\n- std::string requestorName);\n-\n status_t free(buffer_handle_t handle);\n \n uint64_t getTotalSize() const;\n@@ -94,15 +70,9 @@ protected:\n uint64_t usage, buffer_handle_t* handle, uint32_t* stride,\n std::string requestorName, bool importBuffer);\n \n- static Mutex sLock;\n- static KeyedVector sAllocList;\n-\n friend class Singleton;\n GraphicBufferAllocator();\n ~GraphicBufferAllocator();\n-\n- GraphicBufferMapper& mMapper;\n- std::unique_ptr mAllocator;\n };\n \n // ---------------------------------------------------------------------------\ndiff --git a/src/android/mm/graphic_buffer_allocator_stub.cpp b/src/android/mm/graphic_buffer_allocator_stub.cpp\nnew file mode 100644\nindex 000000000000..4e2eab5754e7\n--- /dev/null\n+++ b/src/android/mm/graphic_buffer_allocator_stub.cpp\n@@ -0,0 +1,50 @@\n+/* SPDX-License-Identifier: Apache-2.0 */\n+/*\n+ * Copyright (C) 2023, Ideas on Board\n+ * Copyright (C) 2023, BayLibre\n+ *\n+ * graphic_buffer_allocator_stub.cpp - Android GraphicBufferAllocator\n+ * stub for compile-testing\n+ */\n+\n+#include \n+\n+namespace android {\n+\n+ANDROID_SINGLETON_STATIC_INSTANCE(GraphicBufferAllocator)\n+\n+GraphicBufferAllocator::GraphicBufferAllocator()\n+{\n+}\n+GraphicBufferAllocator::~GraphicBufferAllocator()\n+{\n+}\n+\n+uint64_t GraphicBufferAllocator::getTotalSize() const\n+{\n+\treturn 0;\n+}\n+\n+#pragma GCC diagnostic push\n+#pragma GCC diagnostic ignored \"-Wunused-parameter\"\n+status_t GraphicBufferAllocator::allocate(uint32_t width,\n+\t\t\t\t\t uint32_t height,\n+\t\t\t\t\t PixelFormat format,\n+\t\t\t\t\t uint32_t layerCount,\n+\t\t\t\t\t uint64_t usage,\n+\t\t\t\t\t buffer_handle_t *handle,\n+\t\t\t\t\t uint32_t *stride,\n+\t\t\t\t\t std::string requestorName)\n+{\n+\t*handle = nullptr;\n+\t*stride = 0;\n+\treturn INVALID_OPERATION;\n+}\n+\n+status_t GraphicBufferAllocator::free(buffer_handle_t handle)\n+{\n+\treturn INVALID_OPERATION;\n+}\n+#pragma GCC diagnostic pop\n+\n+}; // namespace android\ndiff --git a/src/android/mm/meson.build b/src/android/mm/meson.build\nindex e3e0484c3720..e9ceb3afba67 100644\n--- a/src/android/mm/meson.build\n+++ b/src/android/mm/meson.build\n@@ -11,6 +11,7 @@ if platform == 'generic'\n android_deps += [libhardware]\n else\n android_hal_sources += files(['libhardware_stub.c'])\n+ android_hal_sources += files(['graphic_buffer_allocator_stub.cpp'])\n endif\n elif platform == 'cros'\n android_hal_sources += files(['cros_camera_buffer.cpp',\n","prefixes":["libcamera-devel","3/4"]}