From patchwork Sun Aug 29 21:06:32 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vedant Paranjape X-Patchwork-Id: 13565 Return-Path: X-Original-To: parsemail@patchwork.libcamera.org Delivered-To: parsemail@patchwork.libcamera.org Received: from lancelot.ideasonboard.com (lancelot.ideasonboard.com [92.243.16.209]) by patchwork.libcamera.org (Postfix) with ESMTPS id BCA47BDC71 for ; Sun, 29 Aug 2021 21:06:42 +0000 (UTC) Received: from lancelot.ideasonboard.com (localhost [IPv6:::1]) by lancelot.ideasonboard.com (Postfix) with ESMTP id 0A17D6916C; Sun, 29 Aug 2021 23:06:42 +0200 (CEST) Authentication-Results: lancelot.ideasonboard.com; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b="i/dTZyEM"; dkim-atps=neutral Received: from mail-pf1-x42b.google.com (mail-pf1-x42b.google.com [IPv6:2607:f8b0:4864:20::42b]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id EF50C68891 for ; Sun, 29 Aug 2021 23:06:40 +0200 (CEST) Received: by mail-pf1-x42b.google.com with SMTP id 2so10568290pfo.8 for ; Sun, 29 Aug 2021 14:06:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=Tmf6X8jcvYT13ISt60CwTn13OxJxoNO4cLbySD807mM=; b=i/dTZyEMUl0LTrW3V7sS40cMzgW8kXsaN2NAhReygL5UqDy+onClLoKkoTOvaq3Pc2 tvgv8ms9blxDMWsriA6BOJMDhK7gef8E2oAq1opNJZaNOm/2scfYpLe2pYerm5VpB1M7 H1RIG3okOZdwx+dtIofLmT4dBirvcr15zLmtw+XZPxEsCYSnLXqerEtHPc69sJspMZjO +DU5h7CXsqadHjHssW22LMprPn8cyYoJSPkncgxeF/jE67iQH0K0ynavJYea8hVfrJ/I sAiIBx0Lc8n8shV3Xylv09sZXcV8SujzjT8CKiTGlvtJzIGXiFhmqTHaeGnYcH0ifhrt ZrYQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=Tmf6X8jcvYT13ISt60CwTn13OxJxoNO4cLbySD807mM=; b=pqUfrxSGk+LTEaEkKM9V1w4HCLhwTinLeza5XcEqBGehdZnbX7ErBMPfNoDqA/+BXV 6DARExlvB20CqUuYXJwS1tGB+lx7F4VYreVepHAaq71DI0Y98z9DF5evmbOziTKi/iF5 LcIbBGvp4mzChTzaukRomxf8WxlHUkUorpsNWq2p2FbdghKZSvTuxxOl/aWF9QLsWrPj E+SBP34JZcCasF/P5txQEjwQupCnEKjbob8KRqNb0/cT+YHgZskeuNHgUixWw9io7N0W tQf+CttXSwYts1gRbulliViJ6FFZA2s8689NgypjDKlcmDhCt7MqMoRLXkD5+vdjw0iE RLrQ== X-Gm-Message-State: AOAM531Fn4bLjioOkcJxapJHACOkHz0eXPq7pSgkhdq30acagcX0jdB5 +LQx8oIGxYwVtAe0cGQfUvECjjGbq4wyfw== X-Google-Smtp-Source: ABdhPJxvJGBuY+nGSStHjVlSZOweCt4LXUUPYwE4Dlk6qfFaCro/cjUGRN4TTH9jwWHf0j+4DoZDxA== X-Received: by 2002:aa7:9f9c:0:b0:3fe:ba:96d1 with SMTP id z28-20020aa79f9c000000b003fe00ba96d1mr4047619pfr.72.1630271198919; Sun, 29 Aug 2021 14:06:38 -0700 (PDT) Received: from localhost.localdomain ([1.186.166.201]) by smtp.googlemail.com with ESMTPSA id n13sm12106271pff.164.2021.08.29.14.06.37 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 29 Aug 2021 14:06:38 -0700 (PDT) From: Vedant Paranjape To: libcamera-devel@lists.libcamera.org Date: Mon, 30 Aug 2021 02:36:32 +0530 Message-Id: <20210829210632.432207-1-vedantparanjape160201@gmail.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Subject: [libcamera-devel] [PATCH v1] test: gstreamer: Factor out code into a base class X-BeenThere: libcamera-devel@lists.libcamera.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Vedant Paranjape Errors-To: libcamera-devel-bounces@lists.libcamera.org Sender: "libcamera-devel" Lot of code used in single stream test is boiler plate and common across every gstreamer test. Factored out this code into a base class called GstreamerTest. Also updated the gstreamer_single_stream_test to use the GstreamerTest base class Signed-off-by: Vedant Paranjape --- .../gstreamer_single_stream_test.cpp | 138 +++------------ test/gstreamer/gstreamer_test.cpp | 157 ++++++++++++++++++ test/gstreamer/gstreamer_test.h | 38 +++++ test/gstreamer/meson.build | 2 +- 4 files changed, 216 insertions(+), 119 deletions(-) create mode 100644 test/gstreamer/gstreamer_test.cpp create mode 100644 test/gstreamer/gstreamer_test.h diff --git a/test/gstreamer/gstreamer_single_stream_test.cpp b/test/gstreamer/gstreamer_single_stream_test.cpp index 4c8d4804..021ac269 100644 --- a/test/gstreamer/gstreamer_single_stream_test.cpp +++ b/test/gstreamer/gstreamer_single_stream_test.cpp @@ -14,86 +14,35 @@ #include +#include "gstreamer_test.h" #include "test.h" using namespace std; -extern "C" { -const char *__asan_default_options() -{ - /* - * Disable leak detection due to a known global variable initialization - * leak in glib's g_quark_init(). This should ideally be handled by - * using a suppression file instead of disabling leak detection. - */ - return "detect_leaks=false"; -} -} - -class GstreamerSingleStreamTest : public Test +class GstreamerSingleStreamTest : public GstreamerTest, public Test { +public: + GstreamerSingleStreamTest() + : GstreamerTest() + { + } + protected: int init() override { - /* - * GStreamer by default spawns a process to run the - * gst-plugin-scanner helper. If libcamera is compiled with ASan - * enabled, and as GStreamer is most likely not, this causes the - * ASan link order check to fail when gst-plugin-scanner - * dlopen()s the plugin as many libraries will have already been - * loaded by then. Fix this issue by disabling spawning of a - * child helper process when scanning the build directory for - * plugins. - */ - gst_registry_fork_set_enabled(false); - - /* Initialize GStreamer */ - g_autoptr(GError) errInit = NULL; - if (!gst_init_check(nullptr, nullptr, &errInit)) { - g_printerr("Could not initialize GStreamer: %s\n", - errInit ? errInit->message : "unknown error"); - - return TestFail; - } - - /* - * Remove the system libcamera plugin, if any, and add the - * plugin from the build directory. - */ - GstRegistry *registry = gst_registry_get(); - GstPlugin *plugin = gst_registry_lookup(registry, "libgstlibcamera.so"); - if (plugin) { - gst_registry_remove_plugin(registry, plugin); - gst_object_unref(plugin); - } - - std::string path = libcamera::utils::libcameraBuildPath() - + "src/gstreamer"; - if (!gst_registry_scan_path(registry, path.c_str())) { - g_printerr("Failed to add plugin to registry\n"); - gst_deinit(); - return TestFail; - } + if (status_ != TestPass) + return status_; - /* Create the elements */ - libcameraSrc_ = gst_element_factory_make("libcamerasrc", "libcamera"); convert0_ = gst_element_factory_make("videoconvert", "convert0"); sink0_ = gst_element_factory_make("fakesink", "sink0"); - /* Create the empty pipeline_ */ - pipeline_ = gst_pipeline_new("test-pipeline"); - - if (!pipeline_ || !convert0_ || !sink0_ || !libcameraSrc_) { - g_printerr("Not all elements could be created. %p.%p.%p.%p\n", - pipeline_, convert0_, sink0_, libcameraSrc_); - if (pipeline_) - gst_object_unref(pipeline_); + if (!convert0_ || !sink0_) { + g_printerr("Not all elements could be created. %p.%p\n", + convert0_, sink0_); if (convert0_) gst_object_unref(convert0_); if (sink0_) gst_object_unref(sink0_); - if (libcameraSrc_) - gst_object_unref(libcameraSrc_); gst_deinit(); return TestFail; @@ -102,16 +51,8 @@ protected: return TestPass; } - void cleanup() override - { - gst_object_unref(pipeline_); - gst_deinit(); - } - int run() override { - GstStateChangeReturn ret; - /* Build the pipeline */ gst_bin_add_many(GST_BIN(pipeline_), libcameraSrc_, convert0_, sink0_, NULL); if (gst_element_link_many(libcameraSrc_, convert0_, sink0_, NULL) != TRUE) { @@ -119,57 +60,18 @@ protected: return TestFail; } - /* Start playing */ - ret = gst_element_set_state(pipeline_, GST_STATE_PLAYING); - if (ret == GST_STATE_CHANGE_FAILURE) { - g_printerr("Unable to set the pipeline to the playing state.\n"); - return TestFail; - } + gstreamer_start_pipeline(); + if (status_ != TestPass) + return status_; - /* Wait until error or EOS or timeout after 2 seconds */ - constexpr GstMessageType msgType = - static_cast(GST_MESSAGE_ERROR | GST_MESSAGE_EOS); - constexpr GstClockTime timeout = 2 * GST_SECOND; - - g_autoptr(GstBus) bus = gst_element_get_bus(pipeline_); - g_autoptr(GstMessage) msg = gst_bus_timed_pop_filtered(bus, timeout, msgType); - - gst_element_set_state(pipeline_, GST_STATE_NULL); - - /* Parse error message */ - if (msg == NULL) - return TestPass; - - switch (GST_MESSAGE_TYPE(msg)) { - case GST_MESSAGE_ERROR: - gstreamer_print_error(msg); - break; - case GST_MESSAGE_EOS: - g_print("End-Of-Stream reached.\n"); - break; - default: - g_printerr("Unexpected message received.\n"); - break; - } + gstreamer_wait_for_event(); + if (status_ != TestPass) + return status_; - return TestFail; + return TestPass; } private: - void gstreamer_print_error(GstMessage *msg) - { - g_autoptr(GError) err = NULL; - g_autofree gchar *debug_info = NULL; - - gst_message_parse_error(msg, &err, &debug_info); - g_printerr("Error received from element %s: %s\n", - GST_OBJECT_NAME(msg->src), err->message); - g_printerr("Debugging information: %s\n", - debug_info ? debug_info : "none"); - } - - GstElement *pipeline_; - GstElement *libcameraSrc_; GstElement *convert0_; GstElement *sink0_; }; diff --git a/test/gstreamer/gstreamer_test.cpp b/test/gstreamer/gstreamer_test.cpp new file mode 100644 index 00000000..34608a02 --- /dev/null +++ b/test/gstreamer/gstreamer_test.cpp @@ -0,0 +1,157 @@ +/* SPDX-License-Identifier: GPL-2.0-or-later */ +/* + * Copyright (C) 2021, Vedant Paranjape + * + * libcamera Gstreamer element API tests + */ + +#include "gstreamer_test.h" + +#include "test.h" + +using namespace std; + +extern "C" { +const char *__asan_default_options() +{ + /* + * Disable leak detection due to a known global variable initialization + * leak in glib's g_quark_init(). This should ideally be handled by + * using a suppression file instead of disabling leak detection. + */ + return "detect_leaks=false"; +} +} + +GstreamerTest::GstreamerTest() +{ + /* + * GStreamer by default spawns a process to run the + * gst-plugin-scanner helper. If libcamera is compiled with ASan + * enabled, and as GStreamer is most likely not, this causes the + * ASan link order check to fail when gst-plugin-scanner + * dlopen()s the plugin as many libraries will have already been + * loaded by then. Fix this issue by disabling spawning of a + * child helper process when scanning the build directory for + * plugins. + */ + gst_registry_fork_set_enabled(false); + + /* Initialize GStreamer */ + g_autoptr(GError) errInit = NULL; + if (!gst_init_check(nullptr, nullptr, &errInit)) { + g_printerr("Could not initialize GStreamer: %s\n", + errInit ? errInit->message : "unknown error"); + + status_ = TestFail; + return; + } + + /* + * Remove the system libcamera plugin, if any, and add the + * plugin from the build directory. + */ + GstRegistry *registry = gst_registry_get(); + GstPlugin *plugin = gst_registry_lookup(registry, "libgstlibcamera.so"); + if (plugin) { + gst_registry_remove_plugin(registry, plugin); + gst_object_unref(plugin); + } + + std::string path = libcamera::utils::libcameraBuildPath() + "src/gstreamer"; + if (!gst_registry_scan_path(registry, path.c_str())) { + g_printerr("Failed to add plugin to registry\n"); + gst_deinit(); + status_ = TestFail; + return; + } + + /* Create the elements */ + libcameraSrc_ = gst_element_factory_make("libcamerasrc", "libcamera"); + pipeline_ = gst_pipeline_new("test-pipeline"); + + if (!pipeline_ || !libcameraSrc_) { + g_printerr("Not all elements could be created. %p.%p\n", + pipeline_, libcameraSrc_); + if (pipeline_) + gst_object_unref(pipeline_); + if (libcameraSrc_) + gst_object_unref(libcameraSrc_); + gst_deinit(); + status_ = TestFail; + return; + } + + status_ = TestPass; +} + +GstreamerTest::~GstreamerTest() +{ + gst_object_unref(pipeline_); + if (status_ == TestFail) { + gst_object_unref(libcameraSrc_); + } + + gst_deinit(); +} + +void GstreamerTest::gstreamer_start_pipeline() +{ + GstStateChangeReturn ret; + + /* Start playing */ + ret = gst_element_set_state(pipeline_, GST_STATE_PLAYING); + if (ret == GST_STATE_CHANGE_FAILURE) { + g_printerr("Unable to set the pipeline to the playing state.\n"); + status_ = TestFail; + return; + } + + status_ = TestPass; +} + +void GstreamerTest::gstreamer_wait_for_event() +{ + /* Wait until error or EOS or timeout after 2 seconds */ + constexpr GstMessageType msgType = + static_cast(GST_MESSAGE_ERROR | GST_MESSAGE_EOS); + constexpr GstClockTime timeout = 2 * GST_SECOND; + + g_autoptr(GstBus) bus = gst_element_get_bus(pipeline_); + g_autoptr(GstMessage) msg = gst_bus_timed_pop_filtered(bus, timeout, msgType); + + gst_element_set_state(pipeline_, GST_STATE_NULL); + + /* Parse error message */ + if (msg == NULL) { + status_ = TestPass; + return; + } + + switch (GST_MESSAGE_TYPE(msg)) { + case GST_MESSAGE_ERROR: + gstreamer_print_error(msg); + break; + case GST_MESSAGE_EOS: + g_print("End-Of-Stream reached.\n"); + break; + default: + g_printerr("Unexpected message received.\n"); + break; + } + + status_ = TestPass; +} + +void GstreamerTest::gstreamer_print_error(GstMessage *msg) +{ + g_autoptr(GError) err = NULL; + g_autofree gchar *debug_info = NULL; + + gst_message_parse_error(msg, &err, &debug_info); + g_printerr("Error received from element %s: %s\n", + GST_OBJECT_NAME(msg->src), err->message); + g_printerr("Debugging information: %s\n", + debug_info ? debug_info : "none"); +} + diff --git a/test/gstreamer/gstreamer_test.h b/test/gstreamer/gstreamer_test.h new file mode 100644 index 00000000..8e117166 --- /dev/null +++ b/test/gstreamer/gstreamer_test.h @@ -0,0 +1,38 @@ +/* SPDX-License-Identifier: GPL-2.0-or-later */ +/* + * Copyright (C) 2021, Vedant Paranjape + * + * gstreamer_test.cpp - GStreamer test base class + */ + +#ifndef __LIBCAMERA_GSTREAMER_TEST_H__ +#define __LIBCAMERA_GSTREAMER_TEST_H__ + +#include +#include + +#include + +#include "libcamera/internal/source_paths.h" + +#include + +using namespace std; + +class GstreamerTest +{ +public: + GstreamerTest(); + ~GstreamerTest(); + +protected: + void gstreamer_start_pipeline(); + void gstreamer_wait_for_event(); + void gstreamer_print_error(GstMessage *msg); + + GstElement *pipeline_; + GstElement *libcameraSrc_; + int status_; +}; + +#endif /* __LIBCAMERA_GSTREAMER_TEST_H__ */ \ No newline at end of file diff --git a/test/gstreamer/meson.build b/test/gstreamer/meson.build index b99aa0da..aca53b92 100644 --- a/test/gstreamer/meson.build +++ b/test/gstreamer/meson.build @@ -10,7 +10,7 @@ gstreamer_tests = [ gstreamer_dep = dependency('gstreamer-1.0', required: true) foreach t : gstreamer_tests - exe = executable(t[0], t[1], + exe = executable(t[0], t[1], 'gstreamer_test.cpp', dependencies : [libcamera_private, gstreamer_dep], link_with : test_libraries, include_directories : test_includes_internal)