{"id":12868,"url":"https://patchwork.libcamera.org/api/covers/12868/?format=json","web_url":"https://patchwork.libcamera.org/cover/12868/","project":{"id":1,"url":"https://patchwork.libcamera.org/api/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":"<20210708105405.147172-1-umang.jain@ideasonboard.com>","date":"2021-07-08T10:54:03","name":"[libcamera-devel,v2,0/2] utils: ipc: proxy: Always reset ControlSerializer during IPA configure","submitter":{"id":86,"url":"https://patchwork.libcamera.org/api/people/86/?format=json","name":"Umang Jain","email":"umang.jain@ideasonboard.com"},"mbox":"https://patchwork.libcamera.org/cover/12868/mbox/","series":[{"id":2218,"url":"https://patchwork.libcamera.org/api/series/2218/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=2218","date":"2021-07-08T10:54:03","name":"utils: ipc: proxy: Always reset ControlSerializer during IPA configure","version":2,"mbox":"https://patchwork.libcamera.org/series/2218/mbox/"}],"comments":"https://patchwork.libcamera.org/api/covers/12868/comments/","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 49FD4BD794\n\tfor ;\n\tThu, 8 Jul 2021 10:54:18 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 333906851F;\n\tThu, 8 Jul 2021 12:54:17 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 592F568506\n\tfor ;\n\tThu, 8 Jul 2021 12:54:16 +0200 (CEST)","from perceval.ideasonboard.com (unknown [103.251.226.59])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 41663E7;\n\tThu, 8 Jul 2021 12:54:14 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"XMECubnD\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1625741656;\n\tbh=7dn0dkV/Klle2fEWC71xk2dvgvdoyJyYYu2nEsaGAZ4=;\n\th=From:To:Cc:Subject:Date:From;\n\tb=XMECubnDO7/q8ZUOniWfIDMIE9D33dMIDXUE9dS3Hgi+ZaEBa9408g97KF+tqPCwL\n\tjyps6B1x9Lr4eEsQG6yrQDF6eTjko++WRYpP4YojNf587JYBnAFA/ZlA14wDgXpRuc\n\tEv055fOHacmkuScCFDz/RGf6QK4ScJQi/OEsFxFE=","From":"Umang Jain ","To":"libcamera-devel@lists.libcamera.org","Date":"Thu, 8 Jul 2021 16:24:03 +0530","Message-Id":"<20210708105405.147172-1-umang.jain@ideasonboard.com>","X-Mailer":"git-send-email 2.31.1","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","Subject":"[libcamera-devel] [PATCH v2 0/2] utils: ipc: proxy: Always reset\n\tControlSerializer during IPA configure","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","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "},"content":"This is a small patch which seems to fix issue highlighted by:\nhttps://bugs.libcamera.org/show_bug.cgi?id=58\n\n1/2 is a DNI, which can help for logging in isolated IPA mode.\n\nHowever, while testing this series with CTS, a new issue has been\nidentified related to open/lingering fds. For what I can see, it is\nunrelated to this fix and is currently tracked:\nhttps://bugs.libcamera.org/show_bug.cgi?id=63\n\nV2 changes:\n- Incorporate Paul's suggestion to /only/ reset() in isolated mode.\n\nUmang Jain (2):\n DNI: Enforce IPA isolation and set log file path\n utils: ipc: proxy: Reset ControlSerializer during IPA configure\n\n src/ipa/ipu3/ipu3.cpp | 2 ++\n src/libcamera/ipa_manager.cpp | 1 +\n .../generators/libcamera_templates/module_ipa_proxy.cpp.tmpl | 3 +++\n 3 files changed, 6 insertions(+)"}