{"id":2231,"url":"https://patchwork.libcamera.org/api/1.1/covers/2231/?format=json","web_url":"https://patchwork.libcamera.org/cover/2231/","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":"<20191027203335.26888-1-jacopo@jmondi.org>","date":"2019-10-27T20:33:29","name":"[libcamera-devel,0/6] Add support for blocking method invocation","submitter":{"id":3,"url":"https://patchwork.libcamera.org/api/1.1/people/3/?format=json","name":"Jacopo Mondi","email":"jacopo@jmondi.org"},"mbox":"https://patchwork.libcamera.org/cover/2231/mbox/","series":[{"id":558,"url":"https://patchwork.libcamera.org/api/1.1/series/558/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=558","date":"2019-10-27T20:33:29","name":"Add support for blocking method invocation","version":1,"mbox":"https://patchwork.libcamera.org/series/558/mbox/"}],"comments":"https://patchwork.libcamera.org/api/covers/2231/comments/","headers":{"Return-Path":"","Received":["from relay6-d.mail.gandi.net (relay6-d.mail.gandi.net\n\t[217.70.183.198])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 4715B6017A\n\tfor ;\n\tSun, 27 Oct 2019 21:31:48 +0100 (CET)","from uno.localdomain (143.121.2.93.rev.sfr.net [93.2.121.143])\n\t(Authenticated sender: jacopo@jmondi.org)\n\tby relay6-d.mail.gandi.net (Postfix) with ESMTPSA id C3E62C0002;\n\tSun, 27 Oct 2019 20:31:47 +0000 (UTC)"],"X-Originating-IP":"93.2.121.143","From":"Jacopo Mondi ","To":"libcamera-devel@lists.libcamera.org","Date":"Sun, 27 Oct 2019 21:33:29 +0100","Message-Id":"<20191027203335.26888-1-jacopo@jmondi.org>","X-Mailer":"git-send-email 2.23.0","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","Subject":"[libcamera-devel] [PATCH 0/6] Add support for blocking method\n\tinvocation","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","X-List-Received-Date":"Sun, 27 Oct 2019 20:31:48 -0000"},"content":"Hello,\n this small series generalise the blocking method invocation implemented in\nthe Android Camera HAL to be a generic libcamera core concept.\n\nPatch 1/6 contains the definition of message invocation types, which I've been\nsuggested to 'be insipired' from the QT defined ones. unfortunately I'm not sure\nhow to handle 3 on 5 of them, so I made it clear in the documentation which\nshall be changed in the v2 of this series.\n\n2/6 passes the newly defined message invocation type to the\nObject::invokeMethod() operation, 3/6 creates a synchronization class for\nObjects that wraps an std::conditional_variable and 4/6 uses them in the method\ninvocation path of the Object and Message classes to block until a method has\nnot been invoked on the receiver Object.\n\nPatch 5/6 makes the objec-invoke test use the newly introduced blocking method\ninvocation mode, and patch 6/6 removes the ThreadRPC class from the Android\ncamera HAL implementation.\n\nThanks\n j\n\nJacopo Mondi (6):\n libcamera: object: Define message invocation type\n libcamera: object: Invoke method with invokeType\n libcamera: object: Add ObjectConditionVariable class\n libcamera: Support blocking method invocation\n test: object-invoke: invoke method in blocking mode\n android: Replace ThreadRPC with blocking method call\n\n include/libcamera/object.h | 32 +++++++++--\n src/android/camera_device.cpp | 34 +++---------\n src/android/camera_device.h | 5 +-\n src/android/camera_proxy.cpp | 20 ++-----\n src/android/camera_proxy.h | 3 --\n src/android/meson.build | 1 -\n src/android/thread_rpc.cpp | 26 ---------\n src/android/thread_rpc.h | 39 --------------\n src/libcamera/event_notifier.cpp | 3 +-\n src/libcamera/include/message.h | 5 ++\n src/libcamera/message.cpp | 11 +++-\n src/libcamera/object.cpp | 92 ++++++++++++++++++++++++++++++--\n src/libcamera/timer.cpp | 2 +-\n test/object-invoke.cpp | 6 +--\n 14 files changed, 149 insertions(+), 130 deletions(-)\n delete mode 100644 src/android/thread_rpc.cpp\n delete mode 100644 src/android/thread_rpc.h\n\n--\n2.23.0"}