reduce memory significantly for single tile RGB encoding

iron261 / openjpeg

Automatically exported from code.google.com/p/openjpeg

Other

0 stars 0 forks source link

reduce memory significantly for single tile RGB encoding #375

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

For t1 encoder, use tile data as input rather than creating a new buffer for 
each code block and copying into that buffer

For single tile encoding, simply re-use image component data as input, rather 
than allocating two copies of each component and copying into the tile 
component.

Please see attached patch file.

Original issue reported on code.google.com by boxe...@gmail.com on 14 Jul 2014 at 1:18

Attachments:

0001-for-t1-encoder-use-tile-data-rather-than-creating-a-.patch

GoogleCodeExporter commented 9 years ago

Original comment by antonin on 17 Jul 2014 at 7:29

Changed state: Accepted
Added labels: Priority-High
Removed labels: Priority-Medium

GoogleCodeExporter commented 9 years ago

Here's the patch for r2903

Not yet tested against the whole test suite.
On data/input/nonregression/Cevennes1.bmp, memory allocations are lowered by 
about 30% :
Total allocations reported by Instrument tool on MacOS : 248,83MB before & 
176,72MB after
Maximum Heap usage reported by valgrind massif : 233,10MB before & 161,00MB 
after

Original comment by m.darb...@gmail.com on 17 Oct 2014 at 9:56

Attachments:

issue375.patch

GoogleCodeExporter commented 9 years ago

Previous patch buggy against the whole test suite (opj_j2k_write_tile).
This one corrects this.

Original comment by m.darb...@gmail.com on 18 Oct 2014 at 5:19

Attachments:

issue375.patch

GoogleCodeExporter commented 9 years ago

Thanks for testing my baby !

Original comment by boxe...@gmail.com on 18 Oct 2014 at 8:14

GoogleCodeExporter commented 9 years ago

Original comment by antonin on 22 Oct 2014 at 12:47

Added labels: Type-Enhancement
Removed labels: Type-Defect

GoogleCodeExporter commented 9 years ago

Original comment by antonin on 22 Oct 2014 at 11:05

Changed state: Verified

GoogleCodeExporter commented 9 years ago

Attach patch against r2918
Still no regressions with Test Suite

Original comment by m.darb...@gmail.com on 23 Oct 2014 at 5:55

Attachments:

issue375-r2918.patch

GoogleCodeExporter commented 9 years ago

Attach patch against r2922

Original comment by m.darb...@gmail.com on 30 Oct 2014 at 10:51

Attachments:

issue375-r2922.patch

GoogleCodeExporter commented 9 years ago

This issue was closed by revision r2924.

Original comment by antonin on 3 Nov 2014 at 2:51

Changed state: Fixed

GoogleCodeExporter commented 9 years ago

Yay!!!!

Original comment by boxe...@gmail.com on 6 Nov 2014 at 7:58