[odc] Daily src changes for 2007-07-20

[ODC]

OpenBSD src changes summary for 2007-07-20
==========================================

lib/libpthread                          share/man
sys/arch/hppa/conf                      sys/arch/hppa/dev
sys/arch/hppa/hppa                      sys/arch/hppa/include
sys/arch/i386/i386                      sys/crypto
sys/dev/usb                             sys/kern
sys/netinet                             sys/sys
usr.bin/cdio                            usr.bin/make

== lib =============================================================== 01/04 ==

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

libpthread

  ~ uthread/pthread_private.h             ~ uthread/uthread_init.c
  ~ uthread/uthread_spec.c                

  > Initialize the locks in key_table.	On hppa _SPINLOCK_LOCKED is 0, so an
  > uninitialized lock ends up in a locked state.  This lead to a deadlock if
  > we
  > called pthread_key_create().
  > ok marc@, kurt@ (kettenis@)

== share ============================================================= 02/04 ==

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

man

  ~ man4/man4.hppa/lcd.4                  

  > Mention that the LCD can be made to "blink" by setting machdep.led_blink.
  > (kettenis@)

== sys =============================================================== 03/04 ==

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

arch/hppa/conf

  ~ GENERIC                               

  > Enable lcd(4). (kettenis@)

arch/hppa/dev

  ~ lcd.c                                 

  > Make lcd(4) "blink" when machdep.led_blink is set. (kettenis@)

arch/hppa/hppa

  ~ process_machdep.c                     

  > Use HPPA_PC_PRIV_USER instead of hardcoded 3.
  > requested by miod@ (kettenis@)

  ~ locore.S                              

  > PDC I/O space lives at 0xf0f0000000, not 0xf000000000, at least on astr(4)
  > based systems. (kettenis@)

  ~ machdep.c                             

  > Give hppa the blink_led API from sparc64, and the associated
  > machdep.led_blink
  > sysctl. (kettenis@)

arch/hppa/include

  ~ cpu.h                                 

  > Give hppa the blink_led API from sparc64, and the associated
  > machdep.led_blink
  > sysctl. (kettenis@)

arch/i386/i386

  ~ machdep.c                             

  > Oh, look, code that begins like this:
  > if (cpu_class != CPUCLASS_386) {
  > Since we don't support 386 now, unconditionally execute the block.
  > ok tom toby (mk@)

  ~ pmapae.c                              

  > More code that tests for CPUCLASS_386 that can go away.  Pointed out by
  > Charles Longeau, thanks.
  > Fix an indentation nit while there.
  > ok toby (mk@)

crypto

  ~ sha1.c                                

  > unbreak landisk kernel sha1 by working around a compiler bug;
  > also brings kernel sha1 more in line with userland;
  > discussed with miod@ and millert@; (otto@)

dev/usb

  ~ uhci.c                                

  > Don't show debug output when an interrupt is not for us.
  > From Marc Winiger <mw at msys.ch>
  > ok dlg, claudio. (mbalmer@)

kern

  ~ uipc_mbuf.c                           

  > Remove the MFREE() macro and replace it with a call to m_free().
  > Also remove the _MEXTREMOVE macro which was only used by MFREE.
  > This time with the uipc_mbuf.c change that I missed last time. (claudio@)

netinet

  ~ igmp.c                                ~ in.c
  ~ in_var.h                              ~ ip_output.c

  > Remove inm_ifp from struct in_multi -- caching struct ifnet is dangerous
  > because interfaces may disappear without notice causing use after free
  > bugs.
  > Instead use the inm_ia->ia_ifp as a hint, struct in_ifaddr correctly tracks
  > removals of interfaces and invalidates ia_ifp in such cases.
  > looks good henning@ markus@ (claudio@)

sys

  ~ mbuf.h                                

  > Remove the MFREE() macro and replace it with a call to m_free().
  > Also remove the _MEXTREMOVE macro which was only used by MFREE.
  > This time with the uipc_mbuf.c change that I missed last time. (claudio@)

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

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

cdio

  ~ cdio.1                                

  > - document the "play" default a little more clearly
  > - document the difference between "play" and "cdplay" more clearly
  > ok mjc (jmc@)

make

  ~ var.c                                 

  > reorder stuff in var.c so that the next patch is more readable. (espie@)

  ~ for.c                                 ~ parse.c
  ~ var.c                                 ~ var.h
  ~ varmodifiers.c                        ~ varmodifiers.h

  > big clean-up patch:
  > - rename a few functions in var.c to names that make more sense.
  > - introduce Var_Deletei because it makes more sense.
  > - rewrite .for loops to use a local LoopVar construct to avoid looking up
  > more
  > stuff.
  > - reformat var.c to near K&R
  > - rewrite most comments in var.c
  > shown to a few people. millert@ gave me his okay since it passes through
  > ports correctly. (espie@)

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