Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Tamp (low-memory intended for embedded targets) #135

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

BrianPugh
Copy link

Tamp is a low-memory, DEFLATE-inspired lossless compression library intended for embedded targets. Tamp is intended for situations where previous heatshrink was used. Tamp offers higher compression ratios, better tooling, better API, small firmware, and a small memory footprint (barely larger than the window buffer). Tamp has an easy to install CLI, python library, and C implementation.

The design priorities (in order) of Tamp is:

  1. Low memory usage.
  2. Good compression ratios.
  3. Small firmware size.

Here's an example output from running the following on a M1 macbook air; typical use-case is level 10:

$ ./lzbench -t16,16 -etamp silesia.tar
lzbench 1.8 (64-bit MacOS)  (null)
Assembled by P.Skibinski

Compressor name         Compress. Decompress. Compr. size  Ratio Filename
memcpy                  29634 MB/s 29552 MB/s   211975168 100.00 silesia.tar
tamp 1.3.1 -8            13.0 MB/s   188 MB/s   114873328  54.19 silesia.tar
tamp 1.3.1 -9            10.2 MB/s   196 MB/s   107632159  50.78 silesia.tar
tamp 1.3.1 -10           6.24 MB/s   202 MB/s   102660646  48.43 silesia.tar
tamp 1.3.1 -11           3.67 MB/s   206 MB/s    99280285  46.84 silesia.tar
tamp 1.3.1 -12           2.16 MB/s   216 MB/s    95567672  45.08 silesia.tar
tamp 1.3.1 -13           1.23 MB/s   227 MB/s    93114331  43.93 silesia.tar
tamp 1.3.1 -14           0.71 MB/s   236 MB/s    91469264  43.15 silesia.tar
tamp 1.3.1 -15           0.40 MB/s   243 MB/s    90653607  42.77 silesia.tar
done... (cIters=1 dIters=1 cTime=16.0 dTime=16.0 chunkSize=1706MB cSpeed=0MB)

@tansy
Copy link
Contributor

tansy commented Mar 10, 2024

It cannot decompress it's own 'files'.
Also levels below 8 crash.

@BrianPugh
Copy link
Author

It cannot decompress it's own 'files'.

Can you elaborate? What issues are you seeing?

Also levels below 8 crash.

This is intentional, in Tamp levels below 8 are invalid. This is why I have the range begins at 8 in lzbench.h

@tansy
Copy link
Contributor

tansy commented Mar 10, 2024

It seems to not be able to correctly decompress compressed data. Lzbench check correctness of decompressed result and if it differs from original then it indicates error.
In simple terms - your decompressor does not decompress its own compressed stream into original state.

$ lzbench-tamp -etamp reymont 
lzbench 1.8 (32-bit Linux)

Compressor name         Compress. Decompress. Compr. size  Ratio Filename
tamp 1.3.1 -8            5.12 MB/s      ERROR     3281840  49.52 reymont
tamp 1.3.1 -9            3.38 MB/s      ERROR     3014779  45.49 reymont
tamp 1.3.1 -10           2.14 MB/s      ERROR     2847981  42.97 reymont
^C

@BrianPugh
Copy link
Author

I know it's not a great response, but it works fine on my M1 macos machine :D

I'll try and replicate when I get my hands on my linux box in a week.

@BrianPugh
Copy link
Author

I just ran this on a 64 bit linux machine without issues:

$ ./lzbench -t16,16 -etamp silesia.tar
lzbench 1.8 (64-bit Linux)  Intel(R) Core(TM) i5-8500 CPU @ 3.00GHz
Assembled by P.Skibinski

Compressor name         Compress. Decompress. Compr. size  Ratio Filename
memcpy                  13128 MB/s 13567 MB/s   211975168 100.00 silesia.tar
tamp 1.3.1 -8            9.19 MB/s   101 MB/s   114873328  54.19 silesia.tar
tamp 1.3.1 -9            5.82 MB/s   104 MB/s   107632159  50.78 silesia.tar
tamp 1.3.1 -10           3.53 MB/s   109 MB/s   102660646  48.43 silesia.tar

I then cross-compiled it for 32-bit:

$ ./lzbench -t16,16 -etamp reymont
lzbench 1.8 (32-bit Linux)  Intel(R) Core(TM) i5-8500 CPU @ 3.00GHz
Assembled by P.Skibinski

Compressor name         Compress. Decompress. Compr. size  Ratio Filename
memcpy                  15503 MB/s 15493 MB/s     6627202 100.00 reymont
tamp 1.3.1 -8            9.16 MB/s      ERROR     3281840  49.52 reymont
tamp 1.3.1 -9            6.23 MB/s      ERROR     3014779  45.49 reymont
tamp 1.3.1 -10           4.00 MB/s      ERROR     2847981  42.97 reymont

Investigating what the cause could be.

@BrianPugh
Copy link
Author

@tansy this should be fixed now by 2a9d7d9. The issue was that I was pointing at the reported size int64_t with a size_t *. This isn't an issue if the int64_t size is initialized to 0, but it was never explicitly initialized, so the upper 4 bytes were just whatever garbage was on the stack. Because the code was using a size_t * pointer, only the 4 lower bytes were being updated.

@tansy
Copy link
Contributor

tansy commented Mar 24, 2024

Yes, it works now.

@tansy
Copy link
Contributor

tansy commented Mar 24, 2024

This is intentional, in Tamp levels below 8 are invalid

What's the rationale behind that?

@BrianPugh
Copy link
Author

With tamp, the compression level directly corresponds to the window size. Tamp's header uses 3 bits to represent the window size:

Number of bits, minus 8, used to represent the size of the shifting window. e.g. A 12-bit window is encoded as the number 4, 0b100. This means the smallest window is 256 bytes, and largest is 32768.

For the API, it was decided that the user should just provide values in range [8, 15] instead of [0, 7] as those values are more meaningful.

@tansy
Copy link
Contributor

tansy commented Mar 26, 2024

It may be more meaningful to you but not neccesarily to average user, who doesn't (even) know what the sliding window is.

@BrianPugh
Copy link
Author

BrianPugh commented Mar 26, 2024

Tamp doesn't perform any allocations, so the user must provide the window buffer. It is much more natural to do:

TampCompressor compressor;
const WINDOW_BITS = 10;
window_buffer = malloc(1 << WINDOW_BITS);
TampConf conf = {
    .window = WINDOW_BITS,  // this will change in example below
    .literal=8,
}
tamp_compressor_init(&compressor, &conf, window_buffer);

rather than

    .window = WINDOW_BITS - 8,  // magical number 8

In this PR, we could change the range expressed by lzbench to something like [1, 8] by adding a constant, but that seems unnecessarily confusing to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants