[PATCH 0/1] Fix update hook perf regression in next

Hello everyone,

This fixes a performance regression I introduced in next by removing the "exit early" check for hooks which output over a sideband, during the conversion to the new hook API.

That was unintentional and these hooks should continue to exit early if no hook is found, to avoid unnecessarily spinning un/down async threads which no-op and just add overhead.

Reported by Patrick at [1] and independently root caused and confirmed by Peff who fixed it in a very similar manner [2].

Pushed to GitHub [3] and succesfully ran the CI [4].

1: https://lore.kernel.org/git/aaWeSu-d1FMz_sW8@pks.im/T/#m4a1e62b3149825ef03f9b5b48f478933abc521cd 2: https://lore.kernel.org/git/aaWeSu-d1FMz_sW8@pks.im/T/#me0d2655bb53f5ca8fc8f31e5726ecf4d2971fa11 3: https://github.com/10ne1/git/tree/refs/heads/dev/aratiu/update-regression-fix 4: https://github.com/10ne1/git/actions/runs/22590151068

Adrian Ratiu (1):
  builtin/receive-pack: avoid spinning no-op sideband_async threads

 builtin/receive-pack.c | 12 ++++++++++++
 1 file changed, 12 insertions(+)