cvs commit: src/sys/dev/usb ehci.c if_udav.c src/sys/fs/hpfs hpfs_subr.h src/sys/fs/ntfs ntfs_subr.c src/sys/kern kern_lock.c src/sys/sys buf.h lockmgr.h

[Attilio Rao]

attilio     2008-01-08 23:48:31 UTC

  FreeBSD src repository

  Modified files:
    sys/dev/usb          ehci.c if_udav.c 
    sys/fs/hpfs          hpfs_subr.h 
    sys/fs/ntfs          ntfs_subr.c 
    sys/kern             kern_lock.c 
    sys/sys              buf.h lockmgr.h 
  Log:
  Remove explicit calling of lockmgr() with the NULL argument.
  Now, lockmgr() function can only be called passing curthread and the
  KASSERT() is upgraded according with this.
  
  In order to support on-the-fly owner switching, the new function
  lockmgr_disown() has been introduced and gets used in BUF_KERNPROC().
  KPI, so, results changed and FreeBSD version will be bumped soon.
  Differently from previous code, we assume idle thread cannot try to
  acquire the lockmgr as it cannot sleep, so loose the relative check[1]
  in BUF_KERNPROC().
  
  Tested by: kris
  
  [1] kib asked for a KASSERT in the lockmgr_disown() about this
  condition, but after thinking at it, as this is a well known general
  rule, I found it not really necessary.
  
  Revision  Changes    Path
  1.57      +2 -2      src/sys/dev/usb/ehci.c
  1.34      +2 -2      src/sys/dev/usb/if_udav.c
  1.4       +4 -2      src/sys/fs/hpfs/hpfs_subr.h
  1.43      +8 -8      src/sys/fs/ntfs/ntfs_subr.c
  1.117     +42 -23    src/sys/kern/kern_lock.c
  1.197     +1 -4      src/sys/sys/buf.h
  1.56      +1 -0      src/sys/sys/lockmgr.h