From: Junio C Hamano <gitster@pobox.com>
Date: Mon, 07 Jul 2025 17:09:43 GMT
Subject: Re: [PATCH v5] builtin/gc: correct total_ram calculation with HAVE_BSD_SYSCTL
Message-ID: <xmqqldp03p2g.fsf@gitster.g>
In-Reply-To: <20250707164518.6600-1-carenas@gmail.com>

Carlo Marcelo Arenas Belón <carenas@gmail.com> writes:

> The calls to sysctl() assume a 64-bit memory size for the variable
> holding the value, but the actual size depends on the key name and
> platform, at least for HW_PHYSMEM.
>
> Detect any mismatched reads, and retry with a shorter variable
> when needed.
>
> Signed-off-by: Carlo Marcelo Arenas Belón <carenas@gmail.com>
> ---
>  builtin/gc.c | 13 ++++++++++---
>  1 file changed, 10 insertions(+), 3 deletions(-)

Thanks.  Let's declare victory and merge this to 'next'.