{"id":18323,"url":"https://patchwork.libcamera.org/api/covers/18323/?format=json","web_url":"https://patchwork.libcamera.org/cover/18323/","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":"<20230302081349.3334290-1-chenghaoyang@google.com>","date":"2023-03-02T08:13:46","name":"[libcamera-devel,v3,0/3] Add HeapAllocator","submitter":{"id":117,"url":"https://patchwork.libcamera.org/api/people/117/?format=json","name":"Cheng-Hao Yang","email":"chenghaoyang@chromium.org"},"mbox":"https://patchwork.libcamera.org/cover/18323/mbox/","series":[{"id":3780,"url":"https://patchwork.libcamera.org/api/series/3780/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=3780","date":"2023-03-02T08:13:46","name":"Add HeapAllocator","version":3,"mbox":"https://patchwork.libcamera.org/series/3780/mbox/"}],"comments":"https://patchwork.libcamera.org/api/covers/18323/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 17555BF415\n\tfor ;\n\tThu, 2 Mar 2023 08:13:58 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 95E76626C1;\n\tThu, 2 Mar 2023 09:13:57 +0100 (CET)","from mail-pf1-x435.google.com (mail-pf1-x435.google.com\n\t[IPv6:2607:f8b0:4864:20::435])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 9EB3A61EDF\n\tfor ;\n\tThu, 2 Mar 2023 09:13:56 +0100 (CET)","by mail-pf1-x435.google.com with SMTP id cp12so8827907pfb.5\n\tfor ;\n\tThu, 02 Mar 2023 00:13:56 -0800 (PST)","from chenghaoyang-low.c.googlers.com.com\n\t(112.157.221.35.bc.googleusercontent.com. [35.221.157.112])\n\tby smtp.gmail.com with ESMTPSA id\n\tp9-20020aa78609000000b005a8bdc18453sm9179755pfn.35.2023.03.02.00.13.53\n\t(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n\tThu, 02 Mar 2023 00:13:54 -0800 (PST)"],"DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1677744837;\n\tbh=93X9ynrFJ4UbYfUioUZ9gR+tSIlYWQB27sNjARBPr3s=;\n\th=To:Date:Subject:List-Id:List-Unsubscribe:List-Archive:List-Post:\n\tList-Help:List-Subscribe:From:Reply-To:Cc:From;\n\tb=VBc32ReMw/4SH3lOPSwgyqRE5xvI5eDtRnEcOqQ3f0sK7vaRkJkFiMkP3ccrFqFlw\n\tYyxVZeEQ48hvKoKzLOEbGPwogg05Z94zBSPX6K0+kfXJ8pqe4yk0++ROAD/ttMWbw1\n\trMQnaNQlsBtLUUnKZlSYrGeuuWJQxoM6S22tr9/bC9FxPGS/7cWOjaN8SvA8VXVUax\n\teSfeG69jMYmZWJ/RX7yZC4wFMzXmU9BDUkrAZyeMjvcBCCt80e+oOBAX5Tr7NuBqUc\n\tZ+xRkoELKl5Uuvvsps9KpOWuCBC4J5ehzVZZWDyUH7OhO6Fr5/ZKgc2ncQelaZdjZI\n\tdQCzS9VcRXWhg==","v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org;\n\ts=google; \n\th=content-transfer-encoding:mime-version:message-id:date:subject:cc\n\t:to:from:from:to:cc:subject:date:message-id:reply-to;\n\tbh=fqSDexSw2izQs27vhNavnPMDZrsDuDeJ/7ud7dO4OXI=;\n\tb=bE4h7PdpSQKp5ChO0xHwmbIJpciaddMDtXXY7bI4Sj61s2+eNQzmBp30HK38w8EOHa\n\t0U42nWlKZu/TAH6ZI6IIECd8mdFDI0iEWEr9aedJduS67UBRPoiRYmS4+H/9Q2fvpenS\n\tzEH4DJcNc8NdTYm2prq5TwSwnMscpd0A+EpIU="],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key; \n\tunprotected) header.d=chromium.org\n\theader.i=@chromium.org header.b=\"bE4h7Pdp\"; \n\tdkim-atps=neutral","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20210112;\n\th=content-transfer-encoding:mime-version:message-id:date:subject:cc\n\t:to:from:x-gm-message-state:from:to:cc:subject:date:message-id\n\t:reply-to;\n\tbh=fqSDexSw2izQs27vhNavnPMDZrsDuDeJ/7ud7dO4OXI=;\n\tb=cEYo/IPiUD7s4WNv8oZ7OkoSIj6OfmoxsbKeMSSpDj2jyPVv/kYsptV/n305q6ifnT\n\taXnMSoC1Rykf6V17VHAH8+dPDzOSI8F7c4q+CwbZUOfDYsN4C4XyyDYIAbp8zKDoHZfd\n\taz12cnraWedOMzkIXeCriUIs6N2RjOIXdLNmnVs9QJXVpAXxTtKsUkGfGVwdS27jBuHN\n\t1oPBKAD6PwW2HzCUy9op7mw+u4jiTt6Hd6Xiwlw422BvC1p4g5b67/4pcLvrIwwomM31\n\tb/anHA2ShSRu8mrGT9u/GNSjY1bTV54N+jx2mmZUJbItBJWtJcKvcfFECFDg/0MPJfeN\n\tXc7g==","X-Gm-Message-State":"AO0yUKVIAIgo/pBVn7sN2Qg4eEkylpBYl6aJbVH8szOfZaD+klaGc806\n\tz2gPX4I9D+nvJZ4+CQu7TxMJd/Qjm1Y2rRhM","X-Google-Smtp-Source":"AK7set/AYk04xs/2s+gsQJEH1pYzDsMUQ1ZvvwhL1czw94ZgCcT17KN8ooK4PDEKRltBKghuhSIAqw==","X-Received":"by 2002:aa7:98de:0:b0:592:fc9f:8ae5 with SMTP id\n\te30-20020aa798de000000b00592fc9f8ae5mr8019155pfm.23.1677744834840; \n\tThu, 02 Mar 2023 00:13:54 -0800 (PST)","X-Google-Original-From":"Harvey Yang ","To":"libcamera-devel@lists.libcamera.org","Date":"Thu, 2 Mar 2023 08:13:46 +0000","Message-Id":"<20230302081349.3334290-1-chenghaoyang@google.com>","X-Mailer":"git-send-email 2.39.2.722.g9855ee24e9-goog","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","Subject":"[libcamera-devel] [PATCH v3 0/3] Add HeapAllocator","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","From":"Harvey Yang via libcamera-devel ","Reply-To":"Harvey Yang ","Cc":"Harvey Yang ","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "},"content":"Hi all,\n\nThe series of patches added HeapAllocator, which uses DmaHeap (moved\nfrom Raspberry Pi pipeline handler) and UdmaHeap to allocate heap\nbuffers.\n\nFixed the c'tor of DmaHeap, where I incorrectly mixed some udma heap\ncode.\n\nI've only build-test it, and I'll need to test it on my virtual pipeline\nhandler patches, but please take a look and let me know if it makes\nsense. Thanks!\n\nBR,\nHarvey\n\nHarvey Yang (3):\n libcamera: Move DmaHeap to HeapAllocator as a base class\n libcamera: Add UdmaHeap if DmaHeap is not valid\n libcamera: Add exportFrameBuffers in HeapAllocator\n\n include/libcamera/dma_heap.h | 20 +++\n include/libcamera/heap.h | 27 ++++\n include/libcamera/heap_allocator.h | 40 ++++++\n include/libcamera/meson.build | 4 +\n include/libcamera/udma_heap.h | 20 +++\n .../dma_heaps.cpp => dma_heap.cpp} | 35 +++---\n src/libcamera/heap_allocator.cpp | 90 ++++++++++++++\n src/libcamera/meson.build | 3 +\n .../pipeline/raspberrypi/dma_heaps.h | 32 -----\n .../pipeline/raspberrypi/meson.build | 1 -\n .../pipeline/raspberrypi/raspberrypi.cpp | 10 +-\n src/libcamera/udma_heap.cpp | 117 ++++++++++++++++++\n 12 files changed, 342 insertions(+), 57 deletions(-)\n create mode 100644 include/libcamera/dma_heap.h\n create mode 100644 include/libcamera/heap.h\n create mode 100644 include/libcamera/heap_allocator.h\n create mode 100644 include/libcamera/udma_heap.h\n rename src/libcamera/{pipeline/raspberrypi/dma_heaps.cpp => dma_heap.cpp} (69%)\n create mode 100644 src/libcamera/heap_allocator.cpp\n delete mode 100644 src/libcamera/pipeline/raspberrypi/dma_heaps.h\n create mode 100644 src/libcamera/udma_heap.cpp"}