{"id":25159,"url":"https://patchwork.libcamera.org/api/covers/25159/?format=json","web_url":"https://patchwork.libcamera.org/cover/25159/","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":"<20251124-cam-control-override-v1-0-dfc3a2f3feee@ideasonboard.com>","date":"2025-11-24T16:43:43","name":"[0/2] libcamera: ipc: ControlLists without valid idmap break IPC","submitter":{"id":143,"url":"https://patchwork.libcamera.org/api/people/143/?format=json","name":"Jacopo Mondi","email":"jacopo.mondi@ideasonboard.com"},"mbox":"https://patchwork.libcamera.org/cover/25159/mbox/","series":[{"id":5609,"url":"https://patchwork.libcamera.org/api/series/5609/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=5609","date":"2025-11-24T16:43:43","name":"libcamera: ipc: ControlLists without valid idmap break IPC","version":1,"mbox":"https://patchwork.libcamera.org/series/5609/mbox/"}],"comments":"https://patchwork.libcamera.org/api/covers/25159/comments/","headers":{"Return-Path":"<libcamera-devel-bounces@lists.libcamera.org>","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 03BC0C32E0\n\tfor <parsemail@patchwork.libcamera.org>;\n\tMon, 24 Nov 2025 16:44:05 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id DF84360A86;\n\tMon, 24 Nov 2025 17:44:03 +0100 (CET)","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 AE7D76096B\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon, 24 Nov 2025 17:44:02 +0100 (CET)","from [192.168.0.172] (mob-5-90-137-64.net.vodafone.it\n\t[5.90.137.64])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 0F3BB741;\n\tMon, 24 Nov 2025 17:41:53 +0100 (CET)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"bjv8ku3t\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1764002514;\n\tbh=TINOl6HxpoxWlzHuxUK1c/B1l75OEV1ef7T4J4TzxCE=;\n\th=From:Subject:Date:To:Cc:From;\n\tb=bjv8ku3tjQwzxIKiHDfrtwBYmrIKnlJ5vRIiCAE0RXGG0I3Zqa5YzsrBS51JpaAuL\n\tTi/1l/o1U9soJcWqJJna3Txx2oJIEaiQ3gOnwpYTAjIEgowMOPY9FPw+YlLgLDBdAK\n\tD5LZ+1QGZxv8RIhYk/qPum6TZxM6FT8etya/i4s8=","From":"Jacopo Mondi <jacopo.mondi@ideasonboard.com>","Subject":"[PATCH 0/2] libcamera: ipc: ControlLists without valid idmap break\n\tIPC","Date":"Mon, 24 Nov 2025 17:43:43 +0100","Message-Id":"<20251124-cam-control-override-v1-0-dfc3a2f3feee@ideasonboard.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"7bit","X-B4-Tracking":"v=1; b=H4sIAD+LJGkC/x3MSwqAMAwA0atI1gaa+gOvIi60Rg1oK6mIIN7d4\n\tvItZh6IrMIR2uwB5UuiBJ9AeQZuHfzCKFMyWGMrIluiG3Z0wZ8aNgwXq8rEWJGpi2Z0diQDKT2\n\tUZ7n/bde/7wevKI7CZgAAAA==","X-Change-ID":"20251124-cam-control-override-510637bc2b10","To":"libcamera-devel@lists.libcamera.org","Cc":"Jacopo Mondi <jacopo.mondi@ideasonboard.com>","X-Mailer":"b4 0.14.2","X-Developer-Signature":"v=1; a=openpgp-sha256; l=995;\n\ti=jacopo.mondi@ideasonboard.com; h=from:subject:message-id;\n\tbh=TINOl6HxpoxWlzHuxUK1c/B1l75OEV1ef7T4J4TzxCE=;\n\tb=owEBbQKS/ZANAwAKAXI0Bo8WoVY8AcsmYgBpJItR4/fGR4jbzb0pj4No1bYST3r881N3GEDiE\n\tbOZ5/rrZK6JAjMEAAEKAB0WIQS1xD1IgJogio9YOMByNAaPFqFWPAUCaSSLUQAKCRByNAaPFqFW\n\tPBqGD/95oYx1Kq3E4fUckjNPzvXzw5LNaRKXTp+2c8RDsvvE7wwLPN6vbuRpIc3+l36e6hHNg8u\n\teYoh5fhIRh3XWp/vIglZTFQFHsxu18o6CvGO1pBbIuvx5VeeUdfTKyZiJp6jaj+qg49Pl2mh4Ta\n\tIAcgqxNBvgRUndrzo4oZ5kUCu4RL9XyOtBVL9rxMkA1fAlzZ/DHoNtAkoekQ5Dmy4YZ6PK8NHEG\n\tpMe7irgp184ywycTv1hFOIKH8wfNFLt1FrIAIpO7WIgLMPWlBE4HrPbIUAjnAyvsSaH+Jb84xot\n\tW6m7O1F5i+6izq1zby7AdA2lacQinjvaaZia7NZyhdBWc7ow68SkSFGyHKFFXHTs+EYZRWVqB8n\n\tw4+YLkEzKYab1ZyYbxQokl7UrICuXS4H5lQQU4zhTX/JuPDva78oOx138y0YzLhoGfPcOiO7b6u\n\tTZx580W5QUdqGNMOiNQwUWDMbdad3jFbsz1O+95QqSekf2HJ8vuEP0ViuVTAbDMdUjY/yjaf5c5\n\tL6cQjOwjonXEAZtUjBXOh/2HqSc/uEYZNe/34lvKCbxncDjcFaBMEU/JSCGSvmE7+uZCHvZEnSu\n\tYEZTllj+prs82mPcePqCHQkomvpqrsIaiCFyxYTrvjsY95V1L2ufUWRFAEwvAgKdCZE8UYi34Fk\n\tJ803hmROQ+zn0yg==","X-Developer-Key":"i=jacopo.mondi@ideasonboard.com; a=openpgp;\n\tfpr=72392EDC88144A65C701EA9BA5826A2587AD026B","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"<libcamera-devel.lists.libcamera.org>","List-Unsubscribe":"<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>","List-Archive":"<https://lists.libcamera.org/pipermail/libcamera-devel/>","List-Post":"<mailto:libcamera-devel@lists.libcamera.org>","List-Help":"<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>","List-Subscribe":"<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"},"content":"https://gitlab.freedesktop.org/camera/libcamera/-/issues/295\n\nThis is first fix for the most visible issue, but I guess the real\nfix should be designing an API for the Request class that doesn't\nallow applications to overwrite the ControlList associated with a\nRequest. What do people think ?\n\nAlso, an RFC patch for something that seems trivial to me, but I suspect\nif it was -that- trivial, it would have been done like this from the\nvery beginning.\n\nSigned-off-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>\n---\nJacopo Mondi (2):\n      apps: cam: Do not override Request::controls()\n      [RFC] libcamera: request: Create control list with Camera info map\n\n src/apps/cam/camera_session.cpp | 2 +-\n src/libcamera/request.cpp       | 2 +-\n 2 files changed, 2 insertions(+), 2 deletions(-)\n---\nbase-commit: 5a33bc10e9d3bc3a7cb1ead66c7ec0a413083d91\nchange-id: 20251124-cam-control-override-510637bc2b10\n\nBest regards,"}