Subject: Re: CVS commit: src/sys/arch/arm/arm
To: Steve Woodford <scw@netbsd.org>
From: Ben Harris <bjh21@NetBSD.org>
List: source-changes
Date: 10/08/2006 17:48:42
On Sun, 8 Oct 2006, Steve Woodford wrote:
> On Saturday 07 October 2006 15:57, Ben Harris wrote:
>
> > Modified Files:
> > src/sys/arch/arm/arm: in_cksum_arm.S
> >
> > Log Message:
> > in L_cksumdata, stash R14 on the stack before doing anything. This is
> > helpful on acorn26, since kernel-mode page faults corrupt R14. Doing
> > this seems to help with the occasional crashes I've seen in this
> > routine.
>
> Could you change this to a compile-time switch please? Load/Store
> multiple, particularly if conditional, are slow on some flavours of CPU.
OK, I've done that (and made it use STR/LDR while I was there).
--
Ben Harris <bjh21@NetBSD.org>
Portmaster, NetBSD/acorn26 <http://www.NetBSD.org/Ports/acorn26/>