# Re: \[PATCH v4 4/4\] Makefile/coccicheck: set SPATCH\_BATCH\_SIZE to 8

- From: René Scharfe. \<hidden\>
- Date: 2021-03-22 18:06:16 
- List: git
- Page: https://ratatoskr.run/git/2021/03/7881378
- Thread: https://ratatoskr.run/git/2019/04/7809468/t/2.md

```text
Am 22.03.21 um 13:11 schrieb Ævar Arnfjörð Bjarmason:
> Change the "xargs -n X" limit for running spatch from 1 to 8, as
> suggested by Jeff King[1]. Now that we're not using --all-includes
> anymore this is going to take much less memory, so bumping the limit
> should be OK.

But --all-includes is still used, so does this patch still make sense?

>
> 1. https://lore.kernel.org/git/YEIGzXMDax83cwAx@coredump.intra.peff.net/
>
> Signed-off-by: Ævar Arnfjörð Bjarmason <redacted>
> ---
>  Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Makefile b/Makefile
> index eaac14275bb..543fe472482 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1204,7 +1204,7 @@ SPATCH_FLAGS = --all-includes --include-headers-for-types --patch .
>  # For the 'coccicheck' target; setting SPATCH_BATCH_SIZE higher will
>  # usually result in less CPU usage at the cost of higher peak memory.
>  # Setting it to 0 will feed all files in a single spatch invocation.
> -SPATCH_BATCH_SIZE = 1
> +SPATCH_BATCH_SIZE = 8
>
>  # For the 'coccicheck' target; SPATCH_XARGS can be used to manually
>  # tweak the xargs invocation. By default we invoke "xargs -n 1", and
>
```
