[{"id":2051,"web_url":"https://patchwork.libcamera.org/comment/2051/","msgid":"<20190628120326.GC4779@pendragon.ideasonboard.com>","date":"2019-06-28T12:03:26","subject":"Re: [libcamera-devel] [PATCH 1/2] libcamera: ipc: unix: Add a IPC\n\tmechanism based on Unix sockets","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"Hi Niklas,\n\nThank you for the patch.\n\nOn Thu, Jun 27, 2019 at 04:09:54AM +0200, Niklas Söderlund wrote:\n> To be able to isolate an IPA component in a separate process an IPC\n> mechanism is needed to communicate with it. Add an IPC mechanism based\n> on Unix sockets which allows users to pass both data and file descriptors\n> to and from the IPA process.\n> \n> The implementation allows users to send both data and file descriptors\n> in the same message. This allows users to more easily implement\n> serialization and deserialization of objects as all elements belonging\n> to an object can be sent in one message.\n> \n> Signed-off-by: Niklas Söderlund \n> ---\n> src/libcamera/include/ipc_unixsocket.h | 59 +++++\n> src/libcamera/ipc_unixsocket.cpp | 288 +++++++++++++++++++++++++\n> src/libcamera/meson.build | 2 +\n> 3 files changed, 349 insertions(+)\n> create mode 100644 src/libcamera/include/ipc_unixsocket.h\n> create mode 100644 src/libcamera/ipc_unixsocket.cpp\n> \n> diff --git a/src/libcamera/include/ipc_unixsocket.h b/src/libcamera/include/ipc_unixsocket.h\n> new file mode 100644\n> index 0000000000000000..68edbe72e2af1298\n> --- /dev/null\n> +++ b/src/libcamera/include/ipc_unixsocket.h\n> @@ -0,0 +1,59 @@\n> +/* SPDX-License-Identifier: LGPL-2.1-or-later */\n> +/*\n> + * Copyright (C) 2019, Google Inc.\n> + *\n> + * ipc_unixsocket.h - IPC mechanism based on Unix sockets\n> + */\n> +\n> +#ifndef __LIBCAMERA_IPC_UNIXSOCKET_H__\n> +#define __LIBCAMERA_IPC_UNIXSOCKET_H__\n> +\n> +#include \n> +#include \n> +#include \n> +\n> +#include \n> +\n> +namespace libcamera {\n> +\n> +class IPCUnixSocket\n> +{\n> +public:\n> +\tstruct Payload {\n> +\t\tstd::vector data;\n> +\t\tstd::vector fds;\n> +\t};\n> +\n> +\tIPCUnixSocket();\n\nYou also need a destructor that will call close().\n\n> +\n> +\tint create();\n> +\tint attach(int fd);\n\nTo keep the socket semantics, should we call this bind() ?\n\n> +\tvoid close();\n\nShould we have a isOpen() call too, or maybe name it isBound() ?\n\n> +\n> +\tSignal payloadReceived;\n\nTo avoid copying the payload in the slot receiving the signal, would it\nmake sense to replace this with a Signal readyRead;\nsignal, and add a Payload receive(); function ? With copy ellision the\nreceive() function will just fill the Payload object from the caller,\nwhich could come anywhere, could be dynamically allocated, and kept\naround for longer than the payloadReceived signal would allow. Otherwise\na user that wants to keep the received payload around would need to make\na copy in the payloadReceived handler. You already have the receive()\nfunction implemented as recv(), so it's a minor change.\n\n> +\n> +\tint send(Payload &payload);\n\nYou can make the payload const. It will require a const_cast<> for the\nbuffer pointer in sendData(), but I think that's fine.\n\n> +\n> +private:\n> +\tstruct Header {\n> +\t\tuint32_t data;\n> +\t\tuint8_t fds;\n> +\t};\n> +\n> +\tint configure();\n> +\n> +\tint sendData(void *buffer, size_t length, const int32_t *fds, unsigned int num);\n\nHow about passing the Payload reference to this function ? It would make\nits signature simpler, and the code more readable in my opinion.\n\n> +\n> +\tint recv();\n> +\tint recvData(void *buffer, size_t length, int32_t *fds, unsigned int num);\n\nSame comment here.\n\n> +\n> +\tvoid dataNotifier(EventNotifier *notifier);\n> +\n> +\tint fd_;\n> +\tbool master_;\n\nYou never check the master flag, so I would drop it. Once the sockets\nare created and bound there's no master or slave anymore, they're\nsymmetrical.\n\n> +\tEventNotifier *notifier_;\n> +};\n> +\n> +} /* namespace libcamera */\n> +\n> +#endif /* __LIBCAMERA_IPC_UNIXSOCKET_H__ */\n> diff --git a/src/libcamera/ipc_unixsocket.cpp b/src/libcamera/ipc_unixsocket.cpp\n> new file mode 100644\n> index 0000000000000000..7b3d8995374dac1e\n> --- /dev/null\n> +++ b/src/libcamera/ipc_unixsocket.cpp\n> @@ -0,0 +1,288 @@\n> +/* SPDX-License-Identifier: LGPL-2.1-or-later */\n> +/*\n> + * Copyright (C) 2019, Google Inc.\n> + *\n> + * ipc_unixsocket.cpp - IPC mechanism based on Unix sockets\n> + */\n> +\n> +#include \"ipc_unixsocket.h\"\n> +\n> +#include \n> +#include \n> +#include \n> +\n> +#include \"log.h\"\n> +\n> +/**\n> + * \\file ipc_unixsocket.h\n> + * \\brief IPC mechanism based on Unix sockets\n> + */\n> +\n> +namespace libcamera {\n> +\n> +LOG_DEFINE_CATEGORY(IPCUnixSocket)\n> +\n> +/**\n> + * \\struct IPCUnixSocket::Payload\n> + * \\brief Container for an IPC payload\n> + *\n> + * Holds an array of bytes and an array of file descriptors that can be\n> + * transported across a IPC boundary.\n> + */\n> +\n> +/**\n> + * \\var IPCUnixSocket::Payload::data\n> + * \\brief Array of bytes to cross IPC boundary\n> + */\n> +\n> +/**\n> + * \\var IPCUnixSocket::Payload::fds\n> + * \\brief Array of file descriptors to cross IPC boundary\n> + */\n> +\n> +/**\n> + * \\class IPCUnixSocket\n> + * \\brief IPC mechanism based on Unix sockets\n> + *\n> + * The IPC mechanism provided by libcamera are centred around passing arrays of\n> + * raw bytes and file descriptors between processes. The primary users of the\n> + * IPC objects are pipeline handlers and Image Processing Algorithms components.\n> + * A pipeline handler would act as an IPC master and send messages for an IPA to\n> + * process and react to.\n\nNote that this should be transparent for pipeline handlers, all handled\nin the IPA proxy. How about not specifying this class to pipeline\nhandlers and IPAs, and replacing this first paragraph as follows to keep\nit generic ?\n\n * The Unix socket IPC allows bidirectional communication between two processes\n * through unnamed Unix sockets. It implements datagram-based communication,\n * transporting entire payloads with guaranteed ordering.\n\n> + *\n> + * The IPC design is asynchronous, a message is queued to a receiver which gets\n> + * notified that a message is ready to be consumed by a signal. The queuer of\n> + * the message gets no notification when a message is delivers nor processed.\n\ns/delivers/delivered/\n\n> + * If such interactions are needed a protocol specific to the users use-case\n> + * should be implemented on top of the IPC objects.\n> + *\n> + * The IPC design can transmit messages in any direction and the only difference\n> + * from a master and slave operation is how they are created and attached to one\n> + * another. After the two parts are setup the operation to send and receive\n> + * messages are the same for both.\n\nI would expand this paragraph to explain how the sockets are setup. How\nabout the following text ?\n\n * Establishment of an IPC channel is asymmetrical. The side that initiates\n * communication first instantiates a local side socket and creates the channel\n * with create(). The method returns a file descriptor for the remote side of\n * the channel, which is passed to the remote process through an out-of-band\n * communication method. The remote side then instantiates a socket, and binds\n * it to the other side by passing the file descriptor to bind(). At that point\n * the channel is operation and communication is bidirectional and symmmetrical.\n\n> + */\n> +\n> +IPCUnixSocket::IPCUnixSocket()\n> +\t: fd_(-1), master_(false), notifier_(nullptr)\n> +{\n> +}\n> +\n> +/**\n> + * \\brief Create an new IPC channel\n> + *\n> + * Create a new IPC channel. Returned on success is a file descriptor which\n> + * needs to be passed to the slave process and used when attaching to the IPC\n> + * channel using attach().\n\n * This method creates a new IPC channel. The socket instance is bound to the\n * local side of the channel, and the method returns a file descriptor bound to\n * the remote side. The caller is responsible for passing the file descriptor to\n * the remote process, where it can be used with IPCUnixSocket::bind() to bind\n * the remote side socket.\n\n> + *\n> + * \\return A file descriptor on success, negative error code on failure\n> + */\n> +int IPCUnixSocket::create()\n> +{\n> +\tint sockets[2];\n> +\tint ret;\n> +\n> +\tret = socketpair(AF_UNIX, SOCK_DGRAM, 0, sockets);\n> +\tif (ret) {\n> +\t\tret = -errno;\n> +\t\tLOG(IPCUnixSocket, Error)\n> +\t\t\t<< \"Failed to create socket pair: \" << strerror(-ret);\n> +\t\treturn ret;\n> +\t}\n> +\n> +\tfd_ = sockets[0];\n> +\tmaster_ = true;\n> +\n> +\tret = configure();\n> +\tif (ret)\n> +\t\treturn ret;\n> +\n> +\treturn sockets[1];\n> +}\n> +\n> +/**\n> + * \\brief Attach to an existing IPC channel\n\ns/Attach/Bind/\n\n> + * \\param[in] fd File descriptor\n> + *\n> + * Attach to an existing IPC channel. The \\a fd argument is the file descriptor\n> + * returned from create() in the master process and passed to the slave process\n> + * to be able to establish the IPC channel.\n\n * This method binds the socket instance to an existing IPC channel identified\n * by the file descriptor \\fd. The file descriptor is obtained from the\n * IPCUnixSocket::create() method.\n\n> + *\n> + * \\return 0 on success or a negative error code otherwise\n> + */\n> +int IPCUnixSocket::attach(int fd)\n> +{\n> +\tfd_ = fd;\n> +\n> +\treturn configure();\n> +}\n> +\n> +/**\n> + * \\brief Close the IPC channel\n> + *\n> + * Close the IPC channel, no communication is possible after close() have been\n> + * called.\n\nI would drop the first part of the sentence as it just repeats the\nbrief.\n\ns/have been/has been/\n\n> + */\n> +void IPCUnixSocket::close()\n> +{\n> +\tdelete notifier_;\n\n\tnotifier_ = nullptr;\n\n> +\n> +\tif (fd_ == -1)\n> +\t\treturn;\n> +\n> +\t::close(fd_);\n> +\n> +\tfd_ = -1;\n> +}\n> +\n> +/**\n> + * \\brief Send a message payload\n> + * \\param[in] payload Message payload to send\n> + *\n> + * Queues the message payload for transmission to the other end of the IPC\n\n\"Queue\" or \"This method queues\".\n\n> + * channel.\n\nAnd I would add \"This method returns immediately, before the message is\ndelivered to the remote side.\"\n\n> + *\n> + * \\return 0 on success or a negative error code otherwise\n> + */\n> +int IPCUnixSocket::send(Payload &payload)\n> +{\n> +\tHeader hdr;\n> +\tint ret;\n> +\n> +\tif (fd_ < 0)\n> +\t\treturn -ENOTCONN;\n> +\n> +\thdr.data = payload.data.size();\n> +\thdr.fds = payload.fds.size();\n> +\n> +\tret = ::send(fd_, &hdr, sizeof(hdr), 0);\n> +\tif (ret < 0) {\n> +\t\tret = -errno;\n> +\t\tLOG(IPCUnixSocket, Error)\n> +\t\t\t<< \"Failed to send: \" << strerror(-ret);\n> +\t\treturn ret;\n> +\t}\n> +\n> +\tret = sendData(payload.data.data(), hdr.data, payload.fds.data(), hdr.fds);\n> +\tif (ret)\n> +\t\treturn ret;\n> +\n> +\treturn 0;\n\nJust\n\n\treturn sendData(...);\n\n> +}\n> +\n> +/**\n> + * \\var IPCUnixSocket::payloadReceived\n> + * \\brief A Signal emitted when a message payload is received\n> + */\n> +\n> +int IPCUnixSocket::configure()\n> +{\n> +\tnotifier_ = new EventNotifier(fd_, EventNotifier::Read);\n> +\tnotifier_->activated.connect(this, &IPCUnixSocket::dataNotifier);\n> +\n> +\treturn 0;\n\nIf this function never fails I would make it void.\n\nI would also pass the fd to this function and store it in fd_ here. This\nwill make sure that both the fd_ and the notifier_ are set together,\navoiding buggy states where one would be valid and the other wouldn't.\n\n> +}\n> +\n> +int IPCUnixSocket::sendData(void *buffer, size_t length, const int32_t *fds, unsigned int num)\n> +{\n> +\tstruct iovec iov[1];\n> +\tiov[0].iov_base = buffer;\n> +\tiov[0].iov_len = length;\n> +\n> +\tchar buf[CMSG_SPACE(num * sizeof(uint32_t))];\n> +\tmemset(buf, 0, sizeof(buf));\n> +\n> +\tstruct cmsghdr *cmsg = (struct cmsghdr *)buf;\n> +\tcmsg->cmsg_len = CMSG_LEN(num * sizeof(uint32_t));\n> +\tcmsg->cmsg_level = SOL_SOCKET;\n> +\tcmsg->cmsg_type = SCM_RIGHTS;\n> +\n> +\tstruct msghdr msg;\n> +\tmsg.msg_name = nullptr;\n> +\tmsg.msg_namelen = 0;\n> +\tmsg.msg_iov = iov;\n> +\tmsg.msg_iovlen = 1;\n> +\tmsg.msg_control = cmsg;\n> +\tmsg.msg_controllen = cmsg->cmsg_len;\n> +\tmsg.msg_flags = 0;\n> +\tmemcpy(CMSG_DATA(cmsg), fds, num * sizeof(uint32_t));\n> +\n> +\tif (sendmsg(fd_, &msg, 0) < 0) {\n> +\t\tint ret = -errno;\n> +\t\tLOG(IPCUnixSocket, Error)\n> +\t\t\t<< \"Failed to sendmsg: \" << strerror(-ret);\n> +\t\treturn ret;\n> +\t}\n> +\n> +\treturn 0;\n> +}\n> +\n> +int IPCUnixSocket::recv()\n> +{\n> +\tPayload payload;\n> +\tHeader hdr;\n> +\tint ret;\n> +\n> +\tif (fd_ < 0)\n> +\t\treturn -ENOTCONN;\n> +\n> +\tret = ::recv(fd_, &hdr, sizeof(hdr), 0);\n> +\tif (ret < 0) {\n> +\t\tret = -errno;\n> +\t\tLOG(IPCUnixSocket, Error)\n> +\t\t\t<< \"Failed to recv header: \" << strerror(-ret);\n> +\t\treturn ret;\n> +\t}\n> +\n> +\tpayload.data.resize(hdr.data);\n> +\tpayload.fds.resize(hdr.fds);\n> +\n> +\tret = recvData(payload.data.data(), hdr.data, payload.fds.data(), hdr.fds);\n> +\tif (ret)\n> +\t\treturn ret;\n> +\n> +\tpayloadReceived.emit(payload);\n> +\n> +\treturn 0;\n> +}\n> +\n> +int IPCUnixSocket::recvData(void *buffer, size_t length, int32_t *fds, unsigned int num)\n> +{\n> +\tstruct iovec iov[1];\n> +\tiov[0].iov_base = buffer;\n> +\tiov[0].iov_len = length;\n> +\n> +\tchar buf[CMSG_SPACE(num * sizeof(uint32_t))];\n> +\tmemset(buf, 0, sizeof(buf));\n> +\n> +\tstruct cmsghdr *cmsg = (struct cmsghdr *)buf;\n> +\tcmsg->cmsg_len = CMSG_LEN(num * sizeof(uint32_t));\n> +\tcmsg->cmsg_level = SOL_SOCKET;\n> +\tcmsg->cmsg_type = SCM_RIGHTS;\n> +\n> +\tstruct msghdr msg;\n> +\tmsg.msg_name = nullptr;\n> +\tmsg.msg_namelen = 0;\n> +\tmsg.msg_iov = iov;\n> +\tmsg.msg_iovlen = 1;\n> +\tmsg.msg_control = cmsg;\n> +\tmsg.msg_controllen = cmsg->cmsg_len;\n> +\tmsg.msg_flags = 0;\n> +\n> +\tif (recvmsg(fd_, &msg, 0) < 0) {\n> +\t\tint ret = -errno;\n> +\t\tLOG(IPCUnixSocket, Error)\n> +\t\t\t<< \"Failed to recvmsg: \" << strerror(-ret);\n> +\t\treturn ret;\n> +\t}\n> +\n> +\tmemcpy(fds, CMSG_DATA(cmsg), num * sizeof(uint32_t));\n> +\n> +\treturn 0;\n> +}\n> +\n> +void IPCUnixSocket::dataNotifier(EventNotifier *notifier)\n> +{\n> +\trecv();\n> +}\n> +\n> +} /* namespace libcamera */\n> diff --git a/src/libcamera/meson.build b/src/libcamera/meson.build\n> index 985aa7e8ab0eb6ce..45bd9d1793aa0b19 100644\n> --- a/src/libcamera/meson.build\n> +++ b/src/libcamera/meson.build\n> @@ -13,6 +13,7 @@ libcamera_sources = files([\n> 'ipa_interface.cpp',\n> 'ipa_manager.cpp',\n> 'ipa_module.cpp',\n> + 'ipc_unixsocket.cpp',\n> 'log.cpp',\n> 'media_device.cpp',\n> 'media_object.cpp',\n> @@ -38,6 +39,7 @@ libcamera_headers = files([\n> 'include/formats.h',\n> 'include/ipa_manager.h',\n> 'include/ipa_module.h',\n> + 'include/ipc_unixsocket.h',\n> 'include/log.h',\n> 'include/media_device.h',\n> 'include/media_object.h',","headers":{"Return-Path":"","Received":["from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 10F8960BC7\n\tfor ;\n\tFri, 28 Jun 2019 14:03:46 +0200 (CEST)","from pendragon.ideasonboard.com\n\t(dfj612yhrgyx302h3jwwy-3.rev.dnainternet.fi\n\t[IPv6:2001:14ba:21f5:5b00:ce28:277f:58d7:3ca4])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 808DD2C6;\n\tFri, 28 Jun 2019 14:03:45 +0200 (CEST)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1561723425;\n\tbh=/S708TZ8T6JhDbYd51kCK6iOeFsnCtNs5PBkWgcqfLE=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=UM8m/+SjQznoCUEdREmYfbrgoTYmOgUeGUSFwOHF7kpuz7CgmIG1KXxhZGSQ68q2B\n\toCT8b/PbvY1e2GFgdHhvcbyboGvyo5LOvj5KxRkniMEDgGOiVI8prb8fEfiKitYYYW\n\tpxoFdd7QVHHN4+KEcQPK1iBwz6+WRfNT8j2SqtLg=","Date":"Fri, 28 Jun 2019 15:03:26 +0300","From":"Laurent Pinchart ","To":"Niklas =?utf-8?q?S=C3=B6derlund?= ","Cc":"libcamera-devel@lists.libcamera.org","Message-ID":"<20190628120326.GC4779@pendragon.ideasonboard.com>","References":"<20190627020955.6166-1-niklas.soderlund@ragnatech.se>\n\t<20190627020955.6166-2-niklas.soderlund@ragnatech.se>","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Disposition":"inline","Content-Transfer-Encoding":"8bit","In-Reply-To":"<20190627020955.6166-2-niklas.soderlund@ragnatech.se>","User-Agent":"Mutt/1.10.1 (2018-07-13)","Subject":"Re: [libcamera-devel] [PATCH 1/2] libcamera: ipc: unix: Add a IPC\n\tmechanism based on Unix sockets","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","X-List-Received-Date":"Fri, 28 Jun 2019 12:03:46 -0000"}},{"id":2058,"web_url":"https://patchwork.libcamera.org/comment/2058/","msgid":"<20190630135552.GD20789@bigcity.dyn.berto.se>","date":"2019-06-30T13:55:52","subject":"Re: [libcamera-devel] [PATCH 1/2] libcamera: ipc: unix: Add a IPC\n\tmechanism based on Unix sockets","submitter":{"id":5,"url":"https://patchwork.libcamera.org/api/people/5/","name":"Niklas Söderlund","email":"niklas.soderlund@ragnatech.se"},"content":"Hi Laurent,\n\nThanks for your feedback. As always a special thanks to your comments on \nthe documentation!\n\nOn 2019-06-28 15:03:26 +0300, Laurent Pinchart wrote:\n\n> > +\tint sendData(void *buffer, size_t length, const int32_t *fds, \n> > unsigned int num);\n> \n> How about passing the Payload reference to this function ? It would make\n> its signature simpler, and the code more readable in my opinion.\n> \n> > +\n> > +\tint recv();\n> > +\tint recvData(void *buffer, size_t length, int32_t *fds, unsigned int num);\n> \n> Same comment here.\n\nI kind of like this interface as the translation from C++ objects is \nhandled in send() and receive() and {send,recv}Data() can deal with the \njust transmitting data. I will keep this for v2 while addressing all \nother comments in this review.","headers":{"Return-Path":"","Received":["from mail-lf1-x143.google.com (mail-lf1-x143.google.com\n\t[IPv6:2a00:1450:4864:20::143])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 1546260BC0\n\tfor ;\n\tSun, 30 Jun 2019 15:55:55 +0200 (CEST)","by mail-lf1-x143.google.com with SMTP id q26so6982208lfc.3\n\tfor ;\n\tSun, 30 Jun 2019 06:55:55 -0700 (PDT)","from localhost (customer-145-14-112-32.stosn.net. [145.14.112.32])\n\tby smtp.gmail.com with ESMTPSA id\n\t2sm2593047lju.52.2019.06.30.06.55.53\n\t(version=TLS1_3 cipher=AEAD-AES256-GCM-SHA384 bits=256/256);\n\tSun, 30 Jun 2019 06:55:53 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=ragnatech-se.20150623.gappssmtp.com; s=20150623;\n\th=date:from:to:cc:subject:message-id:references:mime-version\n\t:content-disposition:content-transfer-encoding:in-reply-to\n\t:user-agent; bh=UXI7NqEBnLH2gDuRua0IcsQ71vOkVY4VckxhV0wbDCM=;\n\tb=JGEiUUGjuJ9W9+IC5WDk+iqhSlTgz6pFOCniTt1HUGdpWpXyV0HQLaShk+IaykXewB\n\t9suedoWUaAY2eMYmXevTyCr6Pp2eTguShRqCHcATr7YEE/zyjFKO3yRyq29IB//am9vt\n\tZ7P5FP8myS5rPNqKIGWJgMGsBKVmd9F9UQjzMuqQcmW1IlcFXZ1WawgDkrMtPckTazMn\n\tfARhf01ak75VwvzkdJT7d7lUvURALOcyXR0mSr77cJ25DvsD0Fb+fgkBn8vUVN8vI0rV\n\tK6GJL9ufTWtNr/+U3PVc/8aLFN5kRwwUBFzikeHD4GoNG4bDd6IXqLkeLvIZQscR5Hig\n\tWeIg==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:content-transfer-encoding\n\t:in-reply-to:user-agent;\n\tbh=UXI7NqEBnLH2gDuRua0IcsQ71vOkVY4VckxhV0wbDCM=;\n\tb=IZV4aKz7lHQPImsbqoiwmSDWzkc0/nuQpfXhzq7Fg4E2lbDvtK+NDL9WFUer6eiEsR\n\tbrYg5IAS0MN5T2FIqnxXcd2emlcm7HtvBuxQkS5PHROdyShP5o971L6PiqriKK+LDRbC\n\tBSh0mr/RFYH3Z8jdMhWabnf+qDXbP5JMJZt6JVyCseL4/Wap/ASCB0nznkjUJsC8Eo+7\n\tE/4nraOUG0DYQoOJtfpXQdnXLFSyWqo9UP7dlkacoTs6QYzEpHHpP5CyPw0PFAuxZBHD\n\tuIaZUT0e4+LktMp7peOw2FBmpIMxjuNPeGAQ/h2zbeKbny+Gbj7BjO8Pg/YG0kJeHlYz\n\t6Gfg==","X-Gm-Message-State":"APjAAAVW9EyD6Kd+0mo2SasCmqk1XOZR662RazTHl+r0lKGX9usbmULK\n\tc7OL+BsdtBTOKAB3bTTe2gmy0A30y6Y=","X-Google-Smtp-Source":"APXvYqydfxyP39g+SsrVr5BdsGKh7idd2Hb11UCWgFqTcTN07c+yoenpvmDsKmEFjWhJISJeL5rx4g==","X-Received":"by 2002:a19:8c06:: with SMTP id o6mr9392427lfd.176.1561902954556;\n\tSun, 30 Jun 2019 06:55:54 -0700 (PDT)","Date":"Sun, 30 Jun 2019 15:55:52 +0200","From":"Niklas =?iso-8859-1?q?S=F6derlund?= ","To":"Laurent Pinchart ","Cc":"libcamera-devel@lists.libcamera.org","Message-ID":"<20190630135552.GD20789@bigcity.dyn.berto.se>","References":"<20190627020955.6166-1-niklas.soderlund@ragnatech.se>\n\t<20190627020955.6166-2-niklas.soderlund@ragnatech.se>\n\t<20190628120326.GC4779@pendragon.ideasonboard.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=iso-8859-1","Content-Disposition":"inline","Content-Transfer-Encoding":"8bit","In-Reply-To":"<20190628120326.GC4779@pendragon.ideasonboard.com>","User-Agent":"Mutt/1.12.1 (2019-06-15)","Subject":"Re: [libcamera-devel] [PATCH 1/2] libcamera: ipc: unix: Add a IPC\n\tmechanism based on Unix sockets","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","X-List-Received-Date":"Sun, 30 Jun 2019 13:55:55 -0000"}}]