[PATCH v2 0/4] libcamera: Address soft ISP file seal TODO item

[Hans de Goede]

Hi,

On 7/31/24 3:59 PM, Laurent Pinchart wrote:
> Hello,
> 
> This small series addresses one TODO item of the soft ISP, namely shrink
> and grow sealing of the memfd underlying SharedMem objects.
> 
> Patch 1/4 starts by sharing the memfd handling code between the
> DmaBufAllocator and SharedMem classes by moving it to a new MemFd helper
> class. Patch 2/4 then applies a suggestion from Nicolas to improve code
> readability. Patch 3/4 addresses the TODO item, and patch 4/4 finally
> drops it from the TODO file.
> 
> Could someone test this with the soft ISP to make sure I haven't broken
> anything ?

I have given this a test on an IPU6 system and everything still works:

Tested-by: Hans de Goede <hdegoede at redhat.com>

Regards,

Hans




> 
> Laurent Pinchart (4):
>   libcamera: base: Add MemFd helper class
>   libcamera: base: memfd: Handle uClibc compatibility with function
>     wrapper
>   libcamera: shared_mem_object: Prevent memfd from shrinking or growing
>   libcamera: software_isp: Remove file seal TODO item
> 
>  include/libcamera/base/memfd.h      |  34 ++++++++
>  include/libcamera/base/meson.build  |   1 +
>  src/libcamera/base/memfd.cpp        | 123 ++++++++++++++++++++++++++++
>  src/libcamera/base/meson.build      |   1 +
>  src/libcamera/dma_buf_allocator.cpp |  46 +----------
>  src/libcamera/shared_mem_object.cpp |  22 ++---
>  src/libcamera/software_isp/TODO     |  19 -----
>  7 files changed, 169 insertions(+), 77 deletions(-)
>  create mode 100644 include/libcamera/base/memfd.h
>  create mode 100644 src/libcamera/base/memfd.cpp
> 
> 
> base-commit: 98b01768397f982bd177b55e9bc67002b645b4d0