tarfs.5: Provide some guidance on zstd frame sizes
ClosedPublic
Actions

Authored by markj on Apr 4 2024, 4:30 PM.

Details

Reviewers

emaste
des
kib

Commits

rG753de7251761: tarfs.5: Provide some guidance on zstd frame sizes
rGfd56d2472586: tarfs.5: Provide some guidance on zstd frame sizes

Summary

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Apr 4 2024, 4:30 PM

Herald added a subscriber: imp. · View Herald TranscriptApr 4 2024, 4:30 PM

markj requested review of this revision.Apr 4 2024, 4:30 PM

Harbormaster completed remote builds in B56915: Diff 136540.Apr 4 2024, 4:30 PM

This LGTM thanks

This revision is now accepted and ready to land.Apr 4 2024, 5:19 PM

des added inline comments.Apr 5 2024, 11:19 AM

share/man/man5/tarfs.5
83	Smaller frames also require more memory since we keep an index of them.

Note that smaller frames imply more kernel memory usage.

This revision now requires review to proceed.Apr 5 2024, 12:10 PM

Harbormaster completed remote builds in B56925: Diff 136563.Apr 5 2024, 12:11 PM

emaste accepted this revision.Apr 5 2024, 12:21 PM

This revision is now accepted and ready to land.Apr 5 2024, 12:21 PM

des accepted this revision.Apr 5 2024, 3:25 PM

Closed by commit rGfd56d2472586: tarfs.5: Provide some guidance on zstd frame sizes (authored by markj). · Explain WhyApr 5 2024, 3:44 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rGfd56d2472586: tarfs.5: Provide some guidance on zstd frame sizes.

markj added a commit: rG753de7251761: tarfs.5: Provide some guidance on zstd frame sizes.Apr 22 2024, 1:49 PM

Revision Contents
Changeset List

Path

Size

share/

man/

man5/

tarfs.5

9 lines

Diff 136575

View Options

share/man/man5/tarfs.5

This file was added.

				#ifndef VERIFY_H
				#define VERIFY_H

				int verify(const char*);

				#endif /* VERIFY_H */

tarfs.5: Provide some guidance on zstd frame sizesClosedPublicActions