[PATCH 16/16] libcamera: software_isp: Remove stats-sharing TODO item
Milan Zamazal
mzamazal at redhat.com
Mon Aug 12 13:50:05 CEST 2024
Shared per-frame statistics buffers are implemented, we can remove
TODO #2.
Signed-off-by: Milan Zamazal <mzamazal at redhat.com>
---
src/libcamera/software_isp/TODO | 27 ---------------------------
1 file changed, 27 deletions(-)
diff --git a/src/libcamera/software_isp/TODO b/src/libcamera/software_isp/TODO
index db1115dd..b83c3ba3 100644
--- a/src/libcamera/software_isp/TODO
+++ b/src/libcamera/software_isp/TODO
@@ -1,30 +1,3 @@
-2. Reconsider stats sharing
-
->>> +void SwStatsCpu::finishFrame(void)
->>> +{
->>> + *sharedStats_ = stats_;
->>
->> Is it more efficient to copy the stats instead of operating directly on
->> the shared memory ?
->
-> I inherited doing things this way from Andrey. I kept this because
-> we don't really have any synchronization with the IPA reading this.
->
-> So the idea is to only touch this when the next set of statistics
-> is ready since we don't know when the IPA is done with accessing
-> the previous set of statistics ...
->
-> This is both something which seems mostly a theoretic problem,
-> yet also definitely something which I think we need to fix.
->
-> Maybe use a ringbuffer of stats buffers and pass the index into
-> the ringbuffer to the emit signal ?
-
-That would match how we deal with hardware ISPs, and I think that's a
-good idea. It will help decoupling the processing side from the IPA.
-
----
-
3. Remove statsReady signal
> class SwStatsCpu
--
2.44.1
More information about the libcamera-devel
mailing list