[Tremor] Implementing Tremor on low-end ARM
Nicholas Vinen
hb at x256.org
Tue Nov 25 11:48:52 PST 2008
Andy Grundman wrote:
> I tested your patch with one of my test files with a 100K artwork
> comment, it's a -q9 file, and fails:
>
> STATIC MEMORY ERROR: tmem.bufdata exhausted (9216/8192)
>
> It also fails on a newly encoded -q9 file without any comments:
>
> STATIC MEMORY ERROR: tmem.dectable exhausted (15844/14602)
>
> This file is here: http://bugs.slimdevices.com/attachment.cgi?id=3261
> _______________________________________________
> Tremor mailing list
> Tremor at xiph.org
> http://lists.xiph.org/mailman/listinfo/tremor
>
It's because different allocations take up different amounts of space as
the quality level changes. My latest patch allocates pretty much
everything out of a single piece of memory, so if one thing grows and
another shrinks it doesn't fail like that. Try this one:
http://x256.org/~hb/Tremor3.patch
I'm still a bit uncertain about the comment truncation code; it works on
all my test files now (1k, 2k, 3k, 4k, 5k and 40k comments) but I'm
really not sure that it won't break on some other size. Unfortunately I
find the way the code parses the file rather mysterious and complex and
I have too many other things to work out now to work it all out. I think
the general approach is sound - ignore and discard the latter pages of
large comment fields as they're being read in - but I've implemented it
in a rather clumsy way.
Nicholas
More information about the Tremor
mailing list