[odc] Daily src changes for 2007-10-06

ODC auto at squish.net
Sun Oct 7 07:00:01 BST 2007


OpenBSD src changes summary for 2007-10-06
==========================================

distrib/sets                            sys/arch/alpha/isa
sys/arch/alpha/tc                       sys/arch/arm/arm
sys/arch/hppa/dev                       sys/arch/hppa64/conf
sys/arch/hppa64/hppa64                  sys/arch/hppa64/include
sys/arch/landisk/landisk                sys/arch/mvme68k/dev
sys/arch/mvme88k/dev                    sys/arch/mvme88k/mvme88k
sys/arch/mvmeppc/mvmeppc                sys/arch/sh/sh
sys/arch/zaurus/dev                     sys/ddb
sys/dev                                 sys/dev/ic
sys/dev/isa                             sys/dev/mii
sys/dev/pci                             sys/dev/usb
sys/isofs/udf                           sys/msdosfs
sys/netatalk                            sys/netinet
sys/netnatm                             sys/ntfs
sys/ufs/ext2fs                          usr.bin/cvs
usr.bin/make                            usr.bin/pcc
usr.bin/sudo                            usr.sbin/dhcpd
usr.sbin/pkg_add                        

== distrib =========================================================== 01/04 ==

  http://www.openbsd.org/cgi-bin/cvsweb/src/distrib

sets

  ~ lists/comp/mi                         

  > sync (deraadt@)

== sys =============================================================== 02/04 ==

  http://www.openbsd.org/cgi-bin/cvsweb/src/sys

arch/alpha/isa

  ~ isadma_bounce.c                       

  > Some archs used memset() rather than bzero(). So duplicate diff
  > previously applied to other archs deleting a memset() this time. e.g.
  > -	    if ((mapstore = malloc(mapsize, M_DEVBUF,
  > -		(flags & BUS_DMA_NOWAIT) ? M_NOWAIT : M_WAITOK)) == NULL)
  > +	    if ((mapstore = malloc(mapsize, M_DEVBUF, (flags & BUS_DMA_NOWAIT)
  > ?
  > +		(M_NOWAIT | M_ZERO) : (M_WAITOK | M_ZERO))) == NULL)
  > return (ENOMEM);
  > -	    memset(mapstore, 0, mapsize); (krw@)

arch/alpha/tc

  ~ tc_dma_3000_500.c                     

  > Simpliest memset(,0,) -> M_ZERO changes. One (caddr *) cast removal,
  > otherwise just adding M_ZERO to malloc() and removing the immediately
  > adjacent memset(,0,). (krw@)

arch/arm/arm

  ~ bus_dma.c                             

  > Some archs used memset() rather than bzero(). So duplicate diff
  > previously applied to other archs deleting a memset() this time. e.g.
  > -	    if ((mapstore = malloc(mapsize, M_DEVBUF,
  > -		(flags & BUS_DMA_NOWAIT) ? M_NOWAIT : M_WAITOK)) == NULL)
  > +	    if ((mapstore = malloc(mapsize, M_DEVBUF, (flags & BUS_DMA_NOWAIT)
  > ?
  > +		(M_NOWAIT | M_ZERO) : (M_WAITOK | M_ZERO))) == NULL)
  > return (ENOMEM);
  > -	    memset(mapstore, 0, mapsize); (krw@)

  ~ mem.c                                 

  > Simpliest memset(,0,) -> M_ZERO changes. One (caddr *) cast removal,
  > otherwise just adding M_ZERO to malloc() and removing the immediately
  > adjacent memset(,0,). (krw@)

arch/hppa/dev

  ~ apic.c                                ~ astro.c

  > Simpliest memset(,0,) -> M_ZERO changes. One (caddr *) cast removal,
  > otherwise just adding M_ZERO to malloc() and removing the immediately
  > adjacent memset(,0,). (krw@)

arch/hppa64/conf

  ~ files.hppa64                          

  > Simple single-processor only mutex implementation; cloned from hppa.
  > (kettenis@)

arch/hppa64/hppa64

  ~ disksubr.c                            

  > Remove unused variable. (kettenis@)

  + mutex.c                               

  > Simple single-processor only mutex implementation; cloned from hppa.
  > (kettenis@)

arch/hppa64/include

  + mutex.h                               

  > Simple single-processor only mutex implementation; cloned from hppa.
  > (kettenis@)

  + lock.h                                

  > Provide <machine/lock.h> on hppa64 too. (kettenis@)

arch/landisk/landisk

  ~ bus_dma.c                             

  > Some archs used memset() rather than bzero(). So duplicate diff
  > previously applied to other archs deleting a memset() this time. e.g.
  > -	    if ((mapstore = malloc(mapsize, M_DEVBUF,
  > -		(flags & BUS_DMA_NOWAIT) ? M_NOWAIT : M_WAITOK)) == NULL)
  > +	    if ((mapstore = malloc(mapsize, M_DEVBUF, (flags & BUS_DMA_NOWAIT)
  > ?
  > +		(M_NOWAIT | M_ZERO) : (M_WAITOK | M_ZERO))) == NULL)
  > return (ENOMEM);
  > -	    memset(mapstore, 0, mapsize); (krw@)

arch/mvme68k/dev

  ~ vs.c                                  

  > Oops. Forgot to do FREE -> free when I did MALLOC -> malloc. (krw@)

arch/mvme88k/dev

  ~ vs.c                                  

  > Oops. Forgot to do FREE -> free when I did MALLOC -> malloc. (krw@)

arch/mvme88k/mvme88k

  ~ bus_dma.c                             

  > Some archs used memset() rather than bzero(). So duplicate diff
  > previously applied to other archs deleting a memset() this time. e.g.
  > -	    if ((mapstore = malloc(mapsize, M_DEVBUF,
  > -		(flags & BUS_DMA_NOWAIT) ? M_NOWAIT : M_WAITOK)) == NULL)
  > +	    if ((mapstore = malloc(mapsize, M_DEVBUF, (flags & BUS_DMA_NOWAIT)
  > ?
  > +		(M_NOWAIT | M_ZERO) : (M_WAITOK | M_ZERO))) == NULL)
  > return (ENOMEM);
  > -	    memset(mapstore, 0, mapsize); (krw@)

arch/mvmeppc/mvmeppc

  ~ bus_dma.c                             

  > Some archs used memset() rather than bzero(). So duplicate diff
  > previously applied to other archs deleting a memset() this time. e.g.
  > -	    if ((mapstore = malloc(mapsize, M_DEVBUF,
  > -		(flags & BUS_DMA_NOWAIT) ? M_NOWAIT : M_WAITOK)) == NULL)
  > +	    if ((mapstore = malloc(mapsize, M_DEVBUF, (flags & BUS_DMA_NOWAIT)
  > ?
  > +		(M_NOWAIT | M_ZERO) : (M_WAITOK | M_ZERO))) == NULL)
  > return (ENOMEM);
  > -	    memset(mapstore, 0, mapsize); (krw@)

arch/sh/sh

  ~ mem.c                                 

  > Simpliest memset(,0,) -> M_ZERO changes. One (caddr *) cast removal,
  > otherwise just adding M_ZERO to malloc() and removing the immediately
  > adjacent memset(,0,). (krw@)

arch/zaurus/dev

  ~ zaurus_flash.c                        

  > Oops. Forgot to do FREE -> free when I did MALLOC -> malloc. (krw@)

ddb

  ~ db_elf.c                              

  > Use db_printf instead of printf, to prevent problems with locking
  > kprintf_mutex in the early stages of booting.
  > ok miod@ (a while ago) (kettenis@)

dev

  ~ sequencer.c                           

  > Simpliest memset(,0,) -> M_ZERO changes. One (caddr *) cast removal,
  > otherwise just adding M_ZERO to malloc() and removing the immediately
  > adjacent memset(,0,). (krw@)

dev/ic

  ~ if_wi_hostap.c                        

  > Oops. Forgot to do FREE -> free when I did MALLOC -> malloc. (krw@)

dev/isa

  ~ if_ex.c                               

  > Rename bus_space macros. ISA_GET/PUT -> CSR_READ/WRITE
  > Tested by todd@
  > ok dlg@ (brad@)

dev/mii

  ~ eephy.c                               

  > Oops, committed this with a typo, which made it do not quite the right
  > thing. (kettenis@)

dev/pci

  ~ if_cas.c                              

  > NPBFILTER -> NBPFILTER in comments.
  > ok kettenis@ (jsg@)

  ~ agp_amd.c                             ~ agp_i810.c
  ~ agp_intel.c                           ~ agp_via.c
  ~ cz.c                                  

  > Simpliest memset(,0,) -> M_ZERO changes. One (caddr *) cast removal,
  > otherwise just adding M_ZERO to malloc() and removing the immediately
  > adjacent memset(,0,). (krw@)

dev/usb

  ~ if_atu.c                              

  > Another NPBFILTER comment typo.
  > From Jung <moorang at gmail.com> (jsg@)

  ~ umidi.c                               ~ usb_subr.c

  > Simpliest memset(,0,) -> M_ZERO changes. One (caddr *) cast removal,
  > otherwise just adding M_ZERO to malloc() and removing the immediately
  > adjacent memset(,0,). (krw@)

isofs/udf

  ~ udf_vfsops.c                          

  > Oops. Forgot to do FREE -> free when I did MALLOC -> malloc. (krw@)

msdosfs

  ~ msdosfs_denode.c                      

  > Oops. Forgot to do FREE -> free when I did MALLOC -> malloc. (krw@)

netatalk

  ~ ddp_usrreq.c                          

  > Oops. Forgot to do FREE -> free when I did MALLOC -> malloc. (krw@)

netinet

  ~ ip_esp.c                              ~ ip_ipcomp.c

  > Oops. Forgot to do FREE -> free when I did MALLOC -> malloc. (krw@)

netnatm

  ~ natm_pcb.c                            

  > Oops. Forgot to do FREE -> free when I did MALLOC -> malloc. (krw@)

ntfs

  ~ ntfs_subr.c                           

  > Oops. Forgot to do FREE -> free when I did MALLOC -> malloc. (krw@)

ufs/ext2fs

  ~ ext2fs_vfsops.c                       

  > Simpliest memset(,0,) -> M_ZERO changes. One (caddr *) cast removal,
  > otherwise just adding M_ZERO to malloc() and removing the immediately
  > adjacent memset(,0,). (krw@)

== usr.bin =========================================================== 03/04 ==

  http://www.openbsd.org/cgi-bin/cvsweb/src/usr.bin

cvs

  ~ date.y                                

  > replace abort() by return (-1)
  > from ray@ suggestion
  > "looks good" ray@ (chl@)

make

  ~ job.c                                 

  > replace realloc/memset/pointer-arithmetic with recalloc; ok millert@ ray@
  > espie@ (cnst@)

pcc

  ~ arch/x86/local.c                      

  > _Bool is also a type, don't discrimnate against it; ok ragge@ (otto@)

  ~ arch/x86/local.c                      

  > From master repo:
  > > Do not try to branch-optimize pointer comparisions, found by Otto M.
  > (ragge@)

  ~ cc/ccom/trees.c                       

  > From master repo:
  > > Compalin if user is applying sizeof to bitfield, from Stefan Kempf.
  > (ragge@)

sudo

  ~ sudoers                               

  > add WRKOBJDIR to default env_keep to help 'make fake' in ports.
  > ok millert@ (sthen@)

== usr.sbin ========================================================== 04/04 ==

  http://www.openbsd.org/cgi-bin/cvsweb/src/usr.sbin

dhcpd

  ~ pfutils.c                             

  > Allow the various pf table actions to operate independently. For example,
  > "-L leased" wouldn't work without "-A abandoned".
  > testing help from merdely
  > "i like" deraadt (ckuethe@)

pkg_add

  ~ OpenBSD/SharedLibs.pm                 

  > finish loop as soon as the ldconfig search directories have been found.
  > ok espie@ (steven@)

===============================================================================


More information about the odc mailing list