Re: [PATCH] erofs-utils: lib: name worker threads erofs_compress

Gao Xiang
2026-03-16T17:16:07Z

Hi Nithurshen,

On 2026/3/17 01:11, Nithurshen Karthikeyan wrote:
> Hi Xiang,
>

...

>>
>> Why not just calling erofs_compressor, since those worker are really
>> compressor.
>
> I initially tested using "erofs_compressor", but the OS thread naming
> API (`prctl` with `PR_SET_NAME`) has a strict 16-byte limit,
> which includes the null terminator.
>
> Because "erofs_compressor" is exactly 16 characters long, the null
> byte pushes it to 17 bytes. As a result, the kernel truncates it, and
> it actually shows up as "erofs_compresso" in `ps` and `top`.
>
> To keep the output looking clean and intentional, I chose to use
> "erofs_compress" (14 chars) instead. Please let me know if you are
> okay with this, or if you'd prefer a different abbreviation.

Ok, or just call it as `erofscompressor`?
`erofs_compress` is really awkward to me.

Thanks,
Gao Xiang