Re: [PATCH v4 4/4] Makefile/coccicheck: set SPATCH_BATCH_SIZE to 8
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?
Show 22 quoted lines
>
> 1. https://lore.kernel.org/git/YEIGzXMDax83cwAx@coredump.intra.peff.net/
>
> Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
> ---
> 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
>
René Scharfe. <l.s.r@web.de>