Re: [PATCH] index-pack: reduce memory footprint a bit

From: Duy Nguyen <pclouds@gmail.com>
Date: Feb 11, 2015, 13:10 UTC
Message-ID: <CACsJy8CsGKK7Dt4aHHg=GQ9Y2h8bBPBVWnHwqu=CRDkYzzyGkg@mail.gmail.com>
In-Reply-To: <CAHKF-AsF=8n0zxmbYfEKnBgOAnjqW_Psw1eNsZtyMjDchyt5zA@mail.gmail.com>

On Wed, Feb 11, 2015 at 8:01 PM, matthew sporleder <msporleder@gmail.com> wrote:

Show 20 quoted lines

> On Tue, Feb 10, 2015 at 12:49 PM, Junio C Hamano <gitster@pobox.com> wrote:
>> matthew sporleder <msporleder@gmail.com> writes:
>>
>>> I'm having trouble getting this new patch to apply.
>>
>> Apply the first one, replace all object_entry_extra with
>> object_stat, replace all objects_extra with obj_stat and amend the
>> first one.  Then apply this one.
>
> I got this to work and had a good experience but got this from an arm user:
>
> Cloning into 'NetBSD-src-git'...
> remote: Counting objects: 3484984, done.
> remote: Compressing objects: 100% (636083/636083), done.
> error: index-pack died of signal 10
> fatal: index-pack failed
>       125.84 real         0.13 user         0.49 sys
>
> Core was generated by `git'.
> Program terminated with signal SIGBUS, Bus error.

It might be the effect of __attribute__((packed)). Maybe you could try again without that in builtin/index-pack.c. Also could you run gdb and do

p sizeof(*ofs_deltas)

? No need to actually run it from gdb.

-- 
Duy