]> git.sesse.net Git - casparcg/blobdiff - core/producer/frame_producer.cpp
[frame_producer] Created workaround for bug in FFmpeg where every new thread used...
[casparcg] / core / producer / frame_producer.cpp
index 2b78c61778998ba10c57bf77552893eb2a830224..de25a488e792be9a12a95192375557188a4634e4 100644 (file)
@@ -22,6 +22,7 @@
 #include "../StdAfx.h"
 
 #include "frame_producer.h"
+#include "cg_proxy.h"
 
 #include "../frame/draw_frame.h"
 #include "../frame/frame_transform.h"
@@ -39,7 +40,6 @@
 #include <boost/thread.hpp>
 
 namespace caspar { namespace core {
-
 struct frame_producer_registry::impl
 {
        std::vector<producer_factory_t>         producer_factories;
@@ -72,11 +72,13 @@ frame_producer_dependencies::frame_producer_dependencies(
                const spl::shared_ptr<core::frame_factory>& frame_factory,
                const std::vector<spl::shared_ptr<video_channel>>& channels,
                const video_format_desc& format_desc,
-               const spl::shared_ptr<const frame_producer_registry> producer_registry)
+               const spl::shared_ptr<const frame_producer_registry> producer_registry,
+               const spl::shared_ptr<const cg_producer_registry> cg_registry)
        : frame_factory(frame_factory)
        , channels(channels)
        , format_desc(format_desc)
        , producer_registry(producer_registry)
+       , cg_registry(cg_registry)
 {
 }
 
@@ -103,7 +105,7 @@ struct frame_producer_base::impl
                frame_number_ = 0;
                paused_ = false;
        }
-       
+
        draw_frame receive()
        {
                if(paused_)
@@ -148,7 +150,7 @@ draw_frame frame_producer_base::last_frame()
        return impl_->last_frame();
 }
 
-std::future<std::wstring> frame_producer_base::call(const std::vector<std::wstring>&) 
+std::future<std::wstring> frame_producer_base::call(const std::vector<std::wstring>&)
 {
        CASPAR_THROW_EXCEPTION(not_supported());
 }
@@ -176,7 +178,7 @@ const std::vector<std::wstring>& frame_producer_base::get_variables() const
        return empty;
 }
 
-const spl::shared_ptr<frame_producer>& frame_producer::empty() 
+const spl::shared_ptr<frame_producer>& frame_producer::empty()
 {
        class empty_frame_producer : public frame_producer
        {
@@ -186,7 +188,7 @@ const spl::shared_ptr<frame_producer>& frame_producer::empty()
                void paused(bool value) override{}
                uint32_t nb_frames() const override {return 0;}
                std::wstring print() const override { return L"empty";}
-               monitor::subject& monitor_output() override {static monitor::subject monitor_subject(""); return monitor_subject;}                                                                              
+               monitor::subject& monitor_output() override {static monitor::subject monitor_subject(""); return monitor_subject;}
                std::wstring name() const override {return L"empty";}
                uint32_t frame_number() const override {return 0;}
                std::future<std::wstring> call(const std::vector<std::wstring>& params) override{CASPAR_THROW_EXCEPTION(not_implemented());}
@@ -194,7 +196,7 @@ const spl::shared_ptr<frame_producer>& frame_producer::empty()
                const std::vector<std::wstring>& get_variables() const override { static std::vector<std::wstring> empty; return empty; }
                draw_frame last_frame() {return draw_frame::empty();}
                constraints& pixel_constraints() override { static constraints c; return c; }
-       
+
                boost::property_tree::wptree info() const override
                {
                        boost::property_tree::wptree info;
@@ -207,6 +209,18 @@ const spl::shared_ptr<frame_producer>& frame_producer::empty()
        return producer;
 }
 
+std::shared_ptr<executor>& producer_destroyer()
+{
+       static auto destroyer = []
+       {
+               auto result = std::make_shared<executor>(L"Producer destroyer");
+               result->set_capacity(std::numeric_limits<unsigned int>::max());
+               return result;
+       }();;
+
+       return destroyer;
+}
+
 tbb::atomic<bool>& destroy_producers_in_separate_thread()
 {
        static tbb::atomic<bool> state;
@@ -217,13 +231,15 @@ tbb::atomic<bool>& destroy_producers_in_separate_thread()
 void destroy_producers_synchronously()
 {
        destroy_producers_in_separate_thread() = false;
+       // Join destroyer, executing rest of producers in queue synchronously.
+       producer_destroyer().reset();
 }
 
 class destroy_producer_proxy : public frame_producer
-{      
+{
        std::shared_ptr<frame_producer> producer_;
 public:
-       destroy_producer_proxy(spl::shared_ptr<frame_producer>&& producer) 
+       destroy_producer_proxy(spl::shared_ptr<frame_producer>&& producer)
                : producer_(std::move(producer))
        {
                destroy_producers_in_separate_thread() = true;
@@ -231,32 +247,31 @@ public:
 
        virtual ~destroy_producer_proxy()
        {
-               static tbb::atomic<int> counter;
-               static std::once_flag counter_init_once;
-               std::call_once(counter_init_once, []{ counter = 0; });
-               
                if(producer_ == core::frame_producer::empty() || !destroy_producers_in_separate_thread())
                        return;
 
-               ++counter;
-               CASPAR_VERIFY(counter < 8);
-               
+               auto destroyer = producer_destroyer();
+
+               if (!destroyer)
+                       return;
+
+               CASPAR_VERIFY(destroyer->size() < 8);
+
                auto producer = new spl::shared_ptr<frame_producer>(std::move(producer_));
-               boost::thread([=]
+
+               destroyer->begin_invoke([=]
                {
                        std::unique_ptr<spl::shared_ptr<frame_producer>> pointer_guard(producer);
                        auto str = (*producer)->print();
                        try
                        {
-                               ensure_gpf_handler_installed_for_thread(u8(L"Destroyer: " + str).c_str());
-
                                if (!producer->unique())
                                        CASPAR_LOG(debug) << str << L" Not destroyed on asynchronous destruction thread: " << producer->use_count();
                                else
                                        CASPAR_LOG(debug) << str << L" Destroying on asynchronous destruction thread.";
                        }
                        catch(...){}
-                       
+
                        try
                        {
                                pointer_guard.reset();
@@ -266,11 +281,9 @@ public:
                        {
                                CASPAR_LOG_CURRENT_EXCEPTION();
                        }
-
-                       --counter;
-               }).detach(); 
+               });
        }
-       
+
        draw_frame                                                                                      receive() override                                                                                                                                                                                                              {return producer_->receive();}
        std::wstring                                                                            print() const override                                                                                                                  {return producer_->print();}
        void                                                                                            paused(bool value) override                                                                                                             {producer_->paused(value);}
@@ -283,7 +296,7 @@ public:
        void                                                                                            leading_producer(const spl::shared_ptr<frame_producer>& producer) override              {return producer_->leading_producer(producer);}
        uint32_t                                                                                        nb_frames() const override                                                                                                              {return producer_->nb_frames();}
        draw_frame                                                                                      last_frame()                                                                                                                                    {return producer_->last_frame();}
-       monitor::subject&                                                                       monitor_output() override                                                                                                               {return producer_->monitor_output();}                                                                           
+       monitor::subject&                                                                       monitor_output() override                                                                                                               {return producer_->monitor_output();}
        bool                                                                                            collides(double x, double y) const override                                                                             {return producer_->collides(x, y);}
        void                                                                                            on_interaction(const interaction_event::ptr& event)     override                                        {return producer_->on_interaction(event);}
        constraints&                                                                            pixel_constraints() override                                                                                                    {return producer_->pixel_constraints();}
@@ -298,7 +311,7 @@ spl::shared_ptr<core::frame_producer> do_create_producer(const frame_producer_de
 {
        if(params.empty())
                CASPAR_THROW_EXCEPTION(invalid_argument() << msg_info("params cannot be empty"));
-       
+
        auto producer = frame_producer::empty();
        std::any_of(factories.begin(), factories.end(), [&](const producer_factory_t& factory) -> bool
                {
@@ -325,7 +338,7 @@ spl::shared_ptr<core::frame_producer> do_create_producer(const frame_producer_de
 
        if(producer == frame_producer::empty())
                return producer;
-               
+
        return producer;
 }
 
@@ -356,38 +369,41 @@ draw_frame frame_producer_registry::create_thumbnail(const frame_producer_depend
 
        if (key_frame == draw_frame::empty())
                key_frame = do_create_thumbnail_frame(dependencies, media_file + L"_ALPHA", thumbnail_producers);
-  
+
        if (fill_frame != draw_frame::empty() && key_frame != draw_frame::empty())
                return draw_frame::mask(fill_frame, key_frame);
-  
+
        return fill_frame;
 }
 
 spl::shared_ptr<core::frame_producer> frame_producer_registry::create_producer(const frame_producer_dependencies& dependencies, const std::vector<std::wstring>& params) const
-{      
+{
        auto& producer_factories = impl_->producer_factories;
        auto producer = do_create_producer(dependencies, params, producer_factories);
        auto key_producer = frame_producer::empty();
-       
-       try // to find a key file.
+
+       if (!params.empty() && !boost::contains(params.at(0), L"://"))
        {
-               auto params_copy = params;
-               if(params_copy.size() > 0)
+               try // to find a key file.
                {
-                       params_copy[0] += L"_A";
-                       key_producer = do_create_producer(dependencies, params_copy, producer_factories);
-                       if(key_producer == frame_producer::empty())
+                       auto params_copy = params;
+                       if (params_copy.size() > 0)
                        {
-                               params_copy[0] += L"LPHA";
+                               params_copy[0] += L"_A";
                                key_producer = do_create_producer(dependencies, params_copy, producer_factories);
+                               if (key_producer == frame_producer::empty())
+                               {
+                                       params_copy[0] += L"LPHA";
+                                       key_producer = do_create_producer(dependencies, params_copy, producer_factories);
+                               }
                        }
                }
+               catch (...) {}
        }
-       catch(...){}
 
        if(producer != frame_producer::empty() && key_producer != frame_producer::empty())
                return create_separated_producer(producer, key_producer);
-       
+
        if(producer == frame_producer::empty())
        {
                std::wstring str;
@@ -408,5 +424,4 @@ spl::shared_ptr<core::frame_producer> frame_producer_registry::create_producer(c
        std::copy(iterator(iss),  iterator(), std::back_inserter(tokens));
        return create_producer(dependencies, tokens);
 }
-
 }}