cvs commit: src/sbin/savecore savecore.c

[Nate Lawson]

On Wed, 8 Oct 2003, Bruce Evans wrote:
> On Wed, 8 Oct 2003, Marcel Moolenaar wrote:
> >   Modified files:
> >     sbin/savecore        savecore.c
> >   Log:
> >   Revision 1.61 changed the allocation of buffer 'buf' in DoFile() from
> >   the stack to the heap to work around a problem on ia64. Now, roughly
> >   16 months and two compiler updates later, it isn't an issue anymore
> >   in the sense that putting a 1M buffer on the stack just works and we
> >   don't actually need to work around anything anymore.
> >   However, since there's no advantage or need to put the buffer on the
> >   stack (again), this change merely removes the XXX comment describing
> >   that there's an explicit reason for the heap allocation. Hence, this
> >   change is a functional no-op.
>
> There is a negative advantage to putting the buffer on the stack.  It asks
> for a misaligned buffer.
>
> Bruce

That's why I'm glad he left the malloc() in.

-Nate