Re: CVS: cvs.openbsd.org: src

[Tom Cosgrove <tom.cosgrove@arches-consulting.com>]

A recent-ish log message from miod@ has prompted me to send in a
CVS diff I use on my own projects, as I was getting tired of seeing
log messages such as (this is miod@'s original message):

   More cleaning of the exception handling code, and collateral damage:
   - always give C routines invoked by the assembly code some breathing
     room on the stack
   - merge error and reset exception handlers -- gets us rid of
     error_fault() and error_reset().
   - remove all references to SR0 and "threads" inherited from Mach. In
     fact, we do not use SR0 at all now.
   - only use double load and stores instructions when we are 200%
     sure we are accessing a correctly aligned area. I am not fond of
     unaligned kernel accesses, and forcing every pgb to be aligned on
     an 8 byte boundary is gross.

come through email as

   More cleaning of the exception handling code, and collateral damage:
   - always give C routines invoked by the assembly code some breathing
   room on the stack
   - merge error and reset exception handlers -- gets us rid of
   error_fault() and error_reset().
   - remove all references to SR0 and "threads" inherited from Mach. In
   fact, we do not use SR0 at all now.
   - only use double load and stores instructions when we are 200%
   sure we are accessing a correctly aligned area. I am not fond of
   unaligned kernel accesses, and forcing every pgb to be aligned on
   an 8 byte boundary is gross.


It's a simple change to log_accum2 in CVSROOT/ :

--- log_accum2.orig     Mon Sep 29 10:53:32 2003
+++ log_accum2  Mon Sep 29 10:54:36 2003
@@ -319,8 +319,8 @@
     if (/^Log Message/)    { $state = $STATE_LOG;     next; }
     if (/^Revision\/Branch/) { /^[^:]+:\s*(.*)/; $branch = $+; next; }

-    s/^[ \t\n]+//;             # delete leading whitespace
-    s/[ \t\n]+$//;             # delete trailing whitespace
+    s/^[ \t\n]+// if $state != $STATE_LOG ;    # delete leading whitespace
+    s/[ \t\n]+$//;                             # delete trailing whitespace

     if ($state == $STATE_CHANGED) { push(@changed_files, split); }
     if ($state == $STATE_ADDED)   { push(@added_files,   split); }


Thanks

Tom


>>> Miod Vallat 17-Sep-03 23:22 >>>
>
> CVSROOT:	/cvs
> Module name:	src
> Changes by:	miod@cvs.openbsd.org	2003/09/17 16:22:32
>
> Modified files:
> 	sys/arch/mvme88k/include: trap.h 
> 	sys/arch/mvme88k/mvme88k: eh.S trap.c 
>
> Log message:
> More cleaning of the exception handling code, and collateral damage:
> - always give C routines invoked by the assembly code some breathing room on
> the stack
> - merge error and reset exception handlers -- gets us rid of error_fault()
> and error_reset().
> - remove all references to SR0 and "threads" inherited from Mach. In fact,
> we do not use SR0 at all now.
> - only use double load and stores instructions when we are 200% sure we are
> accessing a correctly aligned area. I am not fond of unaligned kernel
> accesses, and forcing every pgb to be aligned on an 8 byte boundary is
> gross.


>>> "Todd C. Miller" 31-Jul-03 03:53 >>>
>
> CVSROOT:	/cvs
> Module name:	src
> Changes by:	millert@cvs.openbsd.org	2003/07/30 20:53:57
>
> Modified files:
> 	usr.bin/diff   : diffreg.c 
>
> Log message:
> o correct exit code when comparing stdin and stdin (a noop)
> o after copying to a temp file, lseek() to the beginning so the
> inline cmp routine works.
> Fixes an exit code issue when comparing against stdin.