Opened 4 years ago

Last modified 14 months ago

#820 assigned Bug / Defect

Compress manual doesn't list lz4-v2

Reported by: jiquera Owned by: David Sommerseth
Priority: minor Milestone: release 2.4.9
Component: Documentation Version: OpenVPN 2.4.0 (Community Ed)
Severity: Not set (select this one, unless your'e a OpenVPN developer) Keywords:
Cc: plaisthos


Please fix the entry in the manual and explain the difference and the preferred setting. Right know the sample scrips are using lz4-v2 and the manual only mentions lz4.

Related to that... why do you support both? the difference is pretty much negligible. (one byte at most per packet)

Change History (6)

comment:1 Changed 4 years ago by Gert Döring

Cc: plaisthos added

the difference is packet data alignment, which is better with the -v2 compression algorithms - which reflects into "less CPU usage, less power drain on mobile devices, possibly higher throughput".

And you're right, we need to document this.

comment:2 Changed 4 years ago by Gert Döring

Milestone: release 2.4.1

comment:3 Changed 4 years ago by jiquera

yeah you explained it the other day over IRC ;) which finally made me understand the code. This is basically the follow up: document it :)

comment:4 Changed 3 years ago by mmokrejs

Why is this trivial change not yet present in 2.4.3 manpage?

comment:5 Changed 15 months ago by Gert Döring

Milestone: release 2.4.1release 2.4.9
Owner: set to David Sommerseth
Status: newassigned

@dazo: while you're at moving the manpage over to rst, can you include lz4-v2?

comment:6 Changed 14 months ago by tincantech

This is a low hanging fruit, I will submit a patch.

Note: See TracTickets for help on using tickets.