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/>