indent-style-note-20060227

Document our official indent style.

rmtsys-up-limits-20060225

FIXES 27095

raise limits of constants in rmtsys

vnop-remove-revised-lock-order-20060225

FIXES 27176

drop the tdc lock even rearlier

windows-ioctl-20060224

FIXES 27225

properly compute the length of the ioctl output buffer for each data
segment

windows-freelance-default-20060224

change freelance default to on.  this matches what the installers do.

aclprocs-protos-20060223

Prototypes for aclprocs.
Also some other prototypes for viced.

softsig-thread-hack-not-on-nbsd30-20060223

based on comment from gendalia@iastate.edu

viced-cleanup-20060222

Prototypes, dead code removal, and other minor cleanups

rxkad-krb5-improvements-20060222

correct precedence of && and || in conditional to determine when
tkt_DecodeTicket5() should be called.

optimize order of ticket property evaluation to delay call to get_key()
which will require that a lock be obtained until after we know that
all of the other checks will succeed.

viced-log-get-key-failure-20060221

log afsconf_GetKey() failure

server-rxmaxmtu-option-20060221

add -rxmaxmtu <bytes> to allow setting the max send and receive RX MTU size

fbsd61-20060221

Param file for FreeBSD 6.1. Builds, untested.

cache-autotune-20060220

put in defaults so an old afsd will still start ok.

viced-client-cps-race-fix-20060220

remove a race condition between h_TossStuff_r and SRXAFS_FlushCPS by
changing h_ID2Client to return the client with a refcount and then
release the refCount with PutClient.

when the CPS list is non-NULL and the viceid is ANONYMOUSID be sure
to set the prlist_val to NULL if the prlist_len is being set to 0
since all of the tests are performed on prlist_val

rx-dpf-20060217

typo

====================
This delta was composed from multiple commits as part of the CVS->Git migration.
The checkin message with each commit was inconsistent.
The following are the additional commit messages.
====================

conditionalize rx_intentionallyDroppedOnReadPer100

flushrec-prototype-20060217

add a prototype to shut up gcc4

vnode-ref-verification-20060217

verify we got a ref when we call vnode_ref

rx-dpf-20060216

when dpf is in use, log error codes from abort packets and perform
ntoh translation of addresses and port numbers.

qremove-set-null-20060217

when we remove an object from a queue null its list pointers

flushvolume-vhashq-consistency-20060217

avoid having vflushed vnodes in play

test-heimdal-20060217

Add a configure test for heimdal and use it to build asetkey or not.
Thanks to Derek Atkins for his helpful suggestions.

hpux11-store-fd-sonner-20060217

FIXES 23112

based on report from   jbuehler@spirentcom.com

darwin-x86-lwp-fix-20060217

make lwp work on darwin x86

windows-notes-20060216

1.5.0 development build

darwin-lost-wakeups-20060213

eliminate the apparent lost wakeup problem in tiger

viced-has-as-member-locking-20060217

FIXES 26967

A crucial line was missing from DELTA viced-fix-check-rights-race-20060213

windows-optimize-find-uid-20060216

Rename smb_GetUser to smb_GetUserFromVCP and add smb_GetUserFromUID.

Use calls to smb_GetUserFromUID to reduce the number of times that
the smb_user_t object is obtained and released during the processing
of an smb call.

windows-server-ping-20060216

add logging of server addresses, status and capabilities

prevent multiple threads from pinging the same server simultaneously.
instead, threads block and then use the result of the ping that was
already taking place.

windows-daemon-check-registry-20060216

Add registry values to allow the defaults for daemon checks to be altered
at startup.  Log values to afsd_init.log.

daemonCheckDownInterval
daemonCheckUpInterval
daemonCheckVolInterval
daemonCheckCBInterval
daemonCheckLockInterval
daemonCheckTokenInterval

windows-lock-testing-20060216

turn on the support for backing byte range lock allocations with
afs file server locks.

windows-rx-wsaeconnreset-20060216

Deal with the fact that Windows returns WSAECONNRESET after an ICMP
port not reachable packet was returned in response to a packet previously
sent.

linux-inode-i-mutex-20060214

new (2.6.16) kernerls have i_mutex instead of i_sem

====================
This delta was composed from multiple commits as part of the CVS->Git migration.
The checkin message with each commit was inconsistent.
The following are the additional commit messages.
====================

make i_sem become i_mutex

darwin-event-cleanup-20060215

clean up event resources added for deadlock avoidance

asetkey-macos-20060215

make the same fix we do on aklog w.r.t com_err

darwin-ifnet-flags-20060215

darwin (8.0) already has this

windows-reg-fix-20060215

cleanup the back connection hostname code

darwin80-mkdir-error-leak-fix-20060215

avoid a potential refcount decrement "leak" (well, the opposite, really) when a mkdir fails

macos-dynroot-start-anyway-20060214

start afsd if -dynroot specified

macos-afsdb-20060214

enable afsdb by default

lookup-checkcode-bulkstats-20060214

do error code analysis on bulkstat results also

dead-code-and-prototyes-20060214

Remove dead code and add prototypes.

windows-build-sys_name-20060214

Add the sys_name to the dest and obj directories in order to allow
both 32-bit and 64-bit builds to be built from the same directory
tree.

windows-notes-20060214

updates

remove-recursive-vfs-getcontext-20060213

remove unused code

solaris-afs-nfsrdwr-avoid-reading-past-end-of-file-20060213

FIXES 25747

avoid a panic when we attempt a read past the end of the file

revert-bkg-disable-var-20060213

remove the bkg daemon disabling

viced-fix-check-rights-race-20060213

FIXES 25869

acl_CheckRights() was generating core dumps because the client's prlist
was being invalidated by h_TossStuff_r() as part of freeing the associated
host.  This patch fixes the problem in probably redundant ways.
A missing hold on the host is now obtained.  refCounts on the client
are now preserved as part of GetClient()/PutClient().  Read/Write locks
are now obtained on the client when accessing/manipulating the prlist.

This patch adds the client parameter to GetVolumePackage() and
PutVolumePackage() and adds the new function PutClient().

This patch removes the need for the client X had conn Y stolen by X messages.
This is replaced by "deleted client X already had conn Y stolen by X.  If
we see this there is definitely a logic problem as the client should not
be deleted at this point.  We a no longer allowing the race condition that
would have resulted in the original message.

In addition, this patch fixes a problem with UAEs caused by the
inappropriate translation of VICE error into UAEs when there is overlap
between VICE error values and those in sys/errno.h.

windows-findacl-optimize-20060213

don't remove an entry from the queue and add it to the head if it already
is the head.

windows-new-server-ping-20060213

when adding a new server, mark it down to start.
this will allow the Ping to use the short timeout when checking the real
status.

windows-off-to-the-races-20060211

several race conditions were introduced over the last couple of weeks.
let's fix them.

windows-vcp-fidsp-empty-20060210

protect against the case in which the vcp->fidsp list is empty
which de-queuing smb_fid_t objects

viced-probe-uuid-on-old-connection-20060210

When we have found an existing host via Uuid, send the probe to the
connection associated with the old host instead of the new one.  If
you send it to the new connection, we will fail to detect that the
old host is no longer responding and hence that it should be removed.
This should solve the 55 second delay problem seen from clients with
Uuids that move addresses or sit behind a NAT.

viced-log-typo-20060210

missing % in format string causes crash at log level 125

viced-debug-port-numbers-20060210

fix port number printing to not print byteswapped port numbers

windows-afslogon-20060209

replace sizeof(opt) with sizeof(LogonOption_t)

add more debug statements

viced-host-nat-20060207

When we discover a new address for an existing host we can take the
opportunity to cleanup the hash table and the interface list if the
new address differs from an existing address only by the port number.
In that case we know the client is communicating to us from behind a
NAT and the old addr/port number combination is no longer going to
be of use to us.

viced-more-dbg-improvements-20060207

convert the rest of the addresses output in hex to dotted notation

no-global-sunlock-20060206

Don't use global_sunlock on OpenBSD unless running smp kernel

windows-winqual-20060206

Fix two bugs reported via Windows Crash Reporting:

 * Freelance initialization is somehow broken allowing the number of
   locally defined mountpoints to become negative.  Due to the use of
   != instead of < in loops, it is possible to read/write unallocated
   memory.

 * GetTextualSid() was not properly handling a NULL output buffer
   pointer as an indication that the desired size should be returned.

darwin-avoid-bogus-name-caching-20060205

FIXES 26225

avoid the possibility of creating bogus cached named vnodes

windows-mountpoints-20060205

 * if the mountpointstring is empty, return No Such Path; do not panic

windows-afslogon-20060205

 * AFS_Logoff_Event must destroy tokens even if integrated logon is not
   being used unless LogoffPreserveTokens is non-zero

windows-afslogon-20060204

 * fix LogoffPreserveTokens to work in the correct direction

 * modify behavior to avoid domain controller queries when integrated
   logon is disabled or the logon account is local

windows-todays-bugs-20060203

* cell names must be treated as case insensitive

* smb logoff messages do not mean the user is logging off the machine.
  do not use as an indication that tokens should be destroyed

* use the correct lock and avoid a deadlock when handling dead VCs

viced-log-add-ports-20060202

* add port numbers to Host addresses so we can distinguish clients behind
  NATs

* make messages consistent with regards to case and punctuation of
  "Host x.x.x.x:y"

* convert hex addresses to dotted notation

windows-more-cleanup-20060201

missed one case where the vcpCounter wrapped fids at 0

bkg-disable-var-20060202

don't extern this

windows-more-cleanups-20060201

* remove all references to 'dead_vcp'; cleanup smb_vc_t's as soon as
  we know they are dead

* add mx holds across the cm_cell_t updates

* add cm_FindSCacheParent() and remove duplicate code elsewhere

* add mx holds across scp->flags updates

* add cm_CleanFile()

* clear CM_SCACHEFLAG_CALLBACK when discarding callbacks

* fix smb fid wrapping.  wrap at 0xFFFF instead of 0 because 0xFFFF
  is -1 which is INVALID_HANDLE

* add missing mx holds around vcp->flags updates

aix-prototyping-fixes-20060201

FIXES 26102

fix some prototypes and enable prototype enforcement for aix

osi-audit-locking-fix-20060201

FIXES 25955

fix aix builds

getsomespace-r-rewrite-20060201

rewrite of GetSomeSpace_r to get rid of the brokenness we had before.

also make GetCapabilities work like other rpc stubs which don't need callbacks

windows-smb_fid_t-mx-20060130

*Includes the changes from STABLE14-windows-smb_fid_t-audit-20060125
 as well:

"Further testing revealed that some smb_vc_t objects could not be freed
because the associated smb_fid_t objects never reached a zero refcount.
Additional auditing uncovered cases in which there were holds not being
released and others in which they were released to many times.  This
patch fixes the problems and improves auditability by modifying the
behavior of the smb_IoctlXXX() functions to not release a reference
that was obtained by the caller.  Now the caller releases the reference."

Fixup token deletion logic

Surround all references to smb_fid_t flags and other references
by obtaining and releasing the 'mx' lock.

merge-up-fixes-rxtcp-20060130

Merge in fixes to up from the rxtcp branch.

simple-darwin-disable-bkg-20060128

avoid potential hang by disabling bkg daemons on 10.4 for now

newvcache-dont-spin-20060128

make things so newvcache won't spin

reclaim-defered-flush-fixes-20060128

fix side effect of reclaim lookaside

up-cleanup-20060126

FIXES 25863

subsumes todd's fix

osi-panic-revert-to-kandr-20060126

FIXES 25924

avoid prototype issue for now

linux-vcache-extra-ref-unneeded-20060126

FIXES 25829

the dynamic/system inode version of the linux client already keeps an
extra reference when the inode is on the vcache list. the inode/vnode
should not go away while we are trying to flush the dcache children.

remove-spurious-var-20060126

Remove unused local variable "lock".

cast-pointer-20060126

Cast pointer to long for printf.
"%p" would be a better fix, but is it supported on all platforms?

macos-avoid-reclaim-hang-20060124

FIXES 25686

based on chaskiel's suggestion, implement a lookaside reclaim list

more-rxglock-cleanup-20060126

FIXES 23321

more rx glock cleanup. eval for pullup after 1.4.1

linux26-build-update-20060126

FIXES 25879

KBUILD_BASENAME will need to be set in 2.6.15 and later for configure to work usefully

ukernel-rx-knet-deref-20060126

FIXES 25927

pass correct parameter to avoid type mismatch

sol10-vnode-cleanup-20060126

Because we use a private pool of vnodes, it's up to us to trigger cleanup.
here we do so.

regen-no-doc-noise-20060125

Don't say we're building man pages if we're not.

man-page-html-20060124

Initial cut at an HTML conversion of the POD reference pages.  Requires
Pod::Simple be installed (version 3.0 or later, probably).  Also fix a POD
formatting bug in the afs(1) man page noticed while testing HTML output.

macos-avoid-reclaim-hang-20060124

FIXES 25686

based on chaskiel's suggestion, implement a lookaside reclaim list

release-avoid-negative-time-20060124

avoid any possibility of time going negative when releasing incrementally

implement-vpeek-for-listvol-20060124

roll back for listonevolume (only)

usebuffer-macros-20060124

FIXES 25819

get rid of usebuffers macro usage warning

windows-continued-refcount-cleanup-20060124

* re-enable LogoffTokenTransferTimeout and LogoffTokenTransfer.
  Tokens are now destroyed at logoff based upon the values specified
  here.  Default is ON and 120 seconds.  Setting this to OFF will
  result in tokens never being destroyed.  This will leak memory.

* protect global queues with mutexes and avoid a variety of race
  conditions.

====================
This delta was composed from multiple commits as part of the CVS->Git migration.
The checkin message with each commit was inconsistent.
The following are the additional commit messages.
====================

rollback unintended commit

viced-grow-h_maxSlots-20060123

and jhutz@cs.cmu.edu

The size of h_maxSlots as computed based upon the value of
MAX_FILESERVER_THREAD is too small.   It is possible for h_lwpIndex
(in the pthread case) to have produce the same Slot value for multiple
threads.  In that case it is possible for the following to occur:

       X: h_Hold_r(host)
       X: H_UNLOCK
       ...
       E: H_LOCK
       E: Check h_Held_r(host); it is true, so don't hold
       E: H_UNLOCK
       X: H_LOCK
       X: h_Release_r(host)
       X: h_TossStuff_r(host)  [called by h_Release_r]
       X: H_UNLOCK
       E: CheckHost(host)

The end result would be a crash due to a reference to a null field
in the host processed by CheckHost.

This patch is a bit of a hack in that it solves the problem by increasing
the number of slots for threads and does not determine the correct number
of threads the process should be allowed to produce.

check-host-optimize-20060123

There should never be a case where h_Enumerate calls CheckHost with a
host whose callback_rxcon that is NULL.  However, due to a bug it ended up
being the case that it happened.  An examination of the CheckHost code
showed that the check for the HOSTDELETED flag should take place before
a reference to callback_rxcon is obtained.  If HOSTDELETED were set, the
code would simply release the reference immediately.  However, the process
of obtaining and releasing the reference required dropping and obtaining
locks that would be a performance hit.

fs-getfid-20060123

add a fs subcommand to get a fid

windows-smb-logoff-or-abandonment-detection-20060121

This patch:

(1) removes the rest of the dead logoff code that was originally
    stripped of any meaning by DELTA
    winnt-win2000-win98-afs-client-updates-20010623.

(2) gives new meaning to smb_TokenTransfer and smb_TokenTransferTimeout.
    these variables now control how long a smb_username_t and its
    associated cm_user_t and its cm_cellinfo_t (including tokens)
    will be preserved after a logoff.

(3) adds logic to detect logoff conditions

(4) adds cm_CheckVCs().  This function probes the SMB client with a ECHO
    response to determine if the associated SMB virtual circuit is still
    valid.   This is executed once every five minutes by smb_Daemon()
    and whenever the machine's IP addresses change.  This allows
    abandoned VCs to be detected and the associated user credentials,
    file handles, and locks to be cleaned up.  This will also prevent
    the exhaustion of the limited number of SMB sessions.

====================
This delta was composed from multiple commits as part of the CVS->Git migration.
The checkin message with each commit was inconsistent.
The following are the additional commit messages.
====================

and remember to mark the session dead so it can be re-used

h_gethost_r-hold-fix-20060121

fix hold errors

host-cpu-20060120

Introduce @HOST_CPU@ and use it to find correct headers for FreeBSD build.

windows-notes-20060120

more notes for 1.4.1-rc5