Create Task
Maniphest T334

Check memory leaks
Open, HighPublic
Actions

Description

Valgrind shows a lot at the moment, because no memory manager is implemented atm

BAndiT1983 created this task.Mar 21 2015, 2:16 PM
BAndiT1983 updated the task description. (Show Details)
BAndiT1983 raised the priority of this task from to Needs Triage.
BAndiT1983 added a project: Open Cine.
BAndiT1983 moved this task to Core on the Open Cine board.
BAndiT1983 added a subscriber: BAndiT1983.
BAndiT1983 triaged this task as High priority.Feb 10 2016, 3:56 PM
BAndiT1983 added a comment.Feb 10 2016, 4:06 PM

Current RAM usage is too high when 2 CDNG frames are loaded, ~210MB. For 2432w * 1366h * 2 bytes * 3 channels should occupy around 20MB, but 40 are occupied at the moment. Image buffers will be reviewed to isolate the problem, also some parts will be stripped for test to determine minimal memory usage (Release/Debug).

BAndiT1983 added a comment.EditedFeb 12 2016, 1:18 PM

Removed some leaks, also memcpy() will be moved out of OCImage, as it was only added for quick tests. OCImage will hold only pointers to red/green/blue data arrays, image loader will move the data to memory pool (static allocation).