On Fri, Dec 03 2021, Han Xin wrote: > + unsigned long bufsize = dry_run ? 4096 : size; > + void *buf = xmallocz(bufsize); It's probably nothing, but in your CL you note that you changed another hardcoding from 4k to 8k, should this one still be 4k? It's probably fine, just wondering...