WARNS=6 cleanup.

Submitted-by: Sepherosa Ziehau <sepherosa@softhome.net>
With-some-modification-by: Matt Dillon

WARNS=6 cleanup.

Submitted-by: Peter Schuller <peter.schuller@infidyne.com>

WARNS cleanup of array initializer.

General WARNS cleanups: staticize functions.  Adjust strdup()
calls to check for allocation failures.

Submitted-by: Peter Schuller <peter.schuller@infidyne.com>

WARNS=6 cleanup.

Submitted-by: Peter Schuller <peter.schuller@infidyne.com>

WARNS=6.

Split pcm into the generic framework (pcm) and the sound cards (snd).
Add support for choosing single devices by the same name as the corresponding
module.

E.g. device snd_ich gives the AC97 support, device "snd_sb8" gives the
SoundBlaster 8 support.

Remove VoxWare related entries, don't list drivers support by NEWPCM.
Just call the section "Sound drivers". Give the example for PNP first,
it is much more common know.

Remove reminders of VoxWare.

The vnode recycler was not handling a hardlinking case that is capable
of tying up a large number of vnodes and namecache records in the directory
topology.   Extracting a large CVS repository, where the 'CVS/Root'
file is massively hardlinked, can result in the vnode recycler failing.

Reported-by: Mike Moore <sansapparel@gmail.com>

- Clearly state that errno is set

Ok: joerg@, hsu@

- Cosmetic

Move the processing of flags out of the loop. The exception is M_WAITOK,
which decides wether VM_ALLOC_NORMAL is used or not. This gets cleared
or set each time.

Move the processing of malloc flags out of the loop. The exception
is the M_WAITOK, since the interpretation changing when the current
thread is preempting or not.  This can change with the second run of
the loop, so evaluate it each time.

- warn() -> warnx(). The global variable errno will not be set.

Move unused version into Attic. Remove DragonFly_Stable tag afterwards.

FreeBSD-Date: 2005/03/31 11:12:45
Author: harti

Move the MAX macro to a util.h so it becomes usable in other modules.

Suggested by:   Max Okumoto <okumoto@ucsd.edu>

Date: 2005/03/30 15:18:58
Author: harti

Make the structure for handling the input stack local to the parse
module. The only module accessing it (the current line number) was the
condition module, so pass the current line number as a function argument.
Centralize the pushing of new input sources into one function
ParsePushInput() and rename the function handling the popping from ParseEOF()
to ParsePopInput(). Make the entire thing a little bit clearer, by holding
the current input source in the top element of the stack instead of
using extra variables for this. Use a type-safe intrusive list for the
input stack.

FreeBSD-Date: 2005/03/23 16:28:57
Author: harti

Style: de-lispify by removing extra parantheses in conditional expressions.

The previous patch had conflicts with our version. This one
handles the stuff we had to do by hand.

Style: make indentation 8 characters.

FreeBSD-Date: 2005/03/23 16:14:41
Author: harti

Style: make indentation 8 characters.

Reduce white space diffs between FreeBSD.

FreeBSD-Date: 2005/03/23 12:56:15
Author: harti

Make paths an explicite datatype instead of using the generic Lst.
A Path is now a TAILQ of PathElements each of which just points to
a reference counted directory. Rename all functions dealing with Paths
from the Dir_ prefix to a Path_ prefix.

FreeBSD-Date: 2005/03/22 12:59:02
Author: harti

Style: <space><tab> -> <tab>

FreeBSD-Date: 2005/03/22 12:54:46
Author: harti

Rename struct Path to struct Dir and move it into the C-file. It
is not used outside.

FreeBSD-Date: 2005/03/22 12:40:24
FreeBSD-Date: 2005/03/22 12:52:00

Author: harti

Remove Lst_Find() and Lst_FindFrom() now that they aren't needed anymore.

FreeBSD-Date: 2005/03/21 11:44:57
Author: harti

Replace calls to Lst_Find with either appropriate LST_FOREACH macros
plus predicate inlining or a special purpose function with takes
and returns the correct types.

FreeBSD-Date: 2005/03/22 12:38:55
Author: harti

Remove the last two instances of Lst_Find() calls.

Move the setting of sopt->sopt_dir down into the lower level sogetopt()
and sosetopt() procedures.  NFS was calling the low level procedures without
setting sopt_dir which prevented TCP_NODELAY from being set for NFS TCP
mounts, resulting in poor performance.

fix bug that creeped in in last commit

Noticed-by: Andrew Atrens <atrens at nortelnetworks.com>

Back out part of last commit, optind has to be initialised to 1.

Make it O(strlen(s) + strlen(charset)) like strcspn.

Submitted-by: Andreas Hauser <andy-dragonfly@splashground.de>

Fix non-BPF declaration of bpf_ptap, it was out-of-sync with the header.

Reported-by: tcslv@cox.net

A kludge to always give the driver a second chance to attach the cbb device.

Similar to the following part of FreeBSD rev 1.66 by imp:
  o Remove bogus checks when reallocating memory for the registers.  They
    weren't needed and turned out to be completely bogus.

  This lets me load/unload pccard with a pccard in a slot and have the
  child correctly detach/attach.  This should help people that have wi
  in their kernel, but that kldload cbb and pccard, for example.

Fix typo.

Declare prototypes for all functions.

const correctness

Avoid discarding constness.

Remove extern for functions, line up function names, remove option names.
Move _GLOBAL_OFFSET_TABLE define up to the other global variables.

Remove tcb_size and flag argument for _rtld_allocate_tls,
remove tcb_size argument for _rtld_free_tls. The size is defined
by the architecture's struct tls_tcb layout and the behaviour
of RTLD_ALLOC_TLS_FREE_OLD is the only possible reaction for
old_tcb != NULL which doesn't leak memory.

Fixed a bug that broke building docs on FreeBSD.  The mistake was
that ParseModifier() consumes characters so it can not be moved
below the following tests.

Fix warnings, use ISO prototype.

GNU getopt resets itself partially when the application sets optind to 0.
A good example in our tree is CVS. Copy this behaviour and put optind into
BSS.

New strcspn implementation, which is O(strln(str) + strlen(chars))
and not O(strlen(str) * strlen(chars)) as before.

Submitted-by: Andreas Hauser <andy-dragonfly@splashground.de>

Cleanup the TLS implementation:
- Define the TCB layout in machine/tls.h, define wether tcb_self exists or
  not.
- Define the layout of the DTV, not used yet.
- Define the TLS variant
- Define common access functions (tls_get_tcb, tls_get_curthread, tls_set_tcb)
  and the RTLD functions.

Merge from vendor branch HEIMDAL:
Remove the telnet program from the heimdal source as it contains
security problems.  It was not connected to the build and wasn't
being utilized anyway.

Remove the telnet program from the heimdal source as it contains
security problems.  It was not connected to the build and wasn't
being utilized anyway.

Remove the telnet program from the heimdal source as it contains
security problems.  It was not connected to the build and wasn't
being utilized anyway.

Remove some uses of the SCARG macro.

Remove PC98 keymaps.

Give syscons its own malloc_type and remove some unnecessary casts.

Suggested and submitted by: Andreas Hauser <andy@splashground.de>

Keep a hint for the last packet in the singly-linked list of packets
in a sockbuf in order to convert the cost of append operations from
O(n) to O(1).

Separate M_NULLOK from M_RNOWAIT.

Correct a pair of buffer overflows in the telnet(1) command:

CAN-2005-0468) A heap buffer overflow in env_opt_add() and related
functions.

(CAN-2005-0469) A global uninitialized data section buffer overflow in
slc_add_reply() and related functions.

As a result of these vulnerabilities, it may be possible for a malicious
telnet server or active network attacker to cause telnet(1) to execute
arbitrary code with the privileges of the user running it.

These fixes are based in part on patches
Submitted by:   Solar Designer <solar@openwall.com>
With calibration with the FreeBSD security officer: Jacques Vidrine <nectar@freebsd.org>

Merge from vendor branch HEIMDAL:
Correct a pair of buffer overflows in the telnet(1) command:

CAN-2005-0468) A heap buffer overflow in env_opt_add() and related
functions.

(CAN-2005-0469) A global uninitialized data section buffer overflow in
slc_add_reply() and related functions.

As a result of these vulnerabilities, it may be possible for a malicious
telnet server or active network attacker to cause telnet(1) to execute
arbitrary code with the privileges of the user running it.

These fixes are based in part on patches
Submitted by:   Solar Designer <solar@openwall.com>
With calibration with the FreeBSD security officer: Jacques Vidrine <nectar@freebsd.org>

Correct a pair of buffer overflows in the telnet(1) command:

CAN-2005-0468) A heap buffer overflow in env_opt_add() and related
functions.

(CAN-2005-0469) A global uninitialized data section buffer overflow in
slc_add_reply() and related functions.

As a result of these vulnerabilities, it may be possible for a malicious
telnet server or active network attacker to cause telnet(1) to execute
arbitrary code with the privileges of the user running it.

These fixes are based in part on patches
Submitted by:   Solar Designer <solar@openwall.com>
With calibration with the FreeBSD security officer: Jacques Vidrine <nectar@freebsd.org>

test

Remove the pre-NEWPCM sound drivers and the speaker-based emulations.
In detail the devices css, gus, gusxvi, mpu, mss, opl, pas, sb, snd,
speaker, sscape, sscape_mss, trix and uart are no longer supported.

They are ISA-only devices and the more common devices are supported by
NEWPCM too. speaker and pca collide with the timer use of the default
kernel and have been broken for a while now.

Remove redundant panic.

Cleanup and retool portions of the TLS support and make sure that
non-threaded binaries remain compatible with older kernels.

* Removes the tcb alignment argument.  Instead an alignment #define is
  placed in sys/tls.h

* Gets rid of the Variant I code (we can add it in later, it just gets
  in the way).

* Retools the Variant II code to support %gs:OFFSET (negative offset)
  AND %gs:0 relative accesses, supporting both -mtls-direct-seg-refs and
  -mno-tls-direct-seg-refs.

* Changes the Elf_Addr array for the TCB into a real structure in
  sys/tls.h

* Retains the DTV methodology.

* Retains the TCB methodology, but note that the area 'after' the tcb
  is now available for future use (at least with Variant I removed).
  Frankly I'm not sure we would ever want to support having the
  'data' area after the TCB instead of before the TCB, at least not
  for i386.

* Do not try to call sys_set_tls_area() if there is no TLS storage.
  i.e. so non-threaded binaries remain compatible with old kernels.

  The TLS is only initialized if (a) There is thread local storage, aka
  gcc3's __thread keyword, or (b) The program is linked against a threading
  library.  The threading library makes the appropriate calls to set up the
  TLS.

Don't hide the library build's ar/ranlib lines, or shared library link
command sequences.

Add another syscall test, and test cvs commits after loginfo/commitinfo/config
updates.

test 2

loginfo/commitinfo test

Temporarily back out the last change due to time zone display issues
(e.g. 'PST' not being displayed).  It will be recommitted when the issues
have been fixed.

The old ntpdate displayed the clock error by default.  Change rdate to do
the same thing.

Do a better job distributing RPC requests on the NFS server side.  This
applies to both TCP and UDP mounts.

Previously when multiple requests were received all at once only one would
be immediately distributed to an idle NFSD, causing the requests to be
serialized on the server if no other nfsd's were active at the time and if
no other event occurs to wake up other idle nfsd's.

Now nfsrv_getstream() returns a count of the number of RPCs that were parsed
and multiple NFSDs will be woken up to process them.

Some fine-tuning is still required.  In particular, it is actually less
efficient for the server to wakeup multiple NFSd (at least on a UP machine)
if the requests do not entail I/O.  However, since this only effects client
operations that are issued in parallel (e.g. a make -j build on the client)
I believe the positive aspects outweight the negative ones.

Fix a bug where the main system clock stops operating.  We were using a field
in the globaldata structure called gd_nextclock to determine whether a systimer
was queued for a cpu.  This field indicated the next pending timeout for the
systimer on the cpu.  However, at 8254 rates this 32 bit field has a chance of
calculating to 0 once every 1/18 second, which disables the systimer dispatch
on that cpu and can cause the system clocks to stop operating.

Remove the field and replace with a TAILQ_FIRST queue check instead.

The machine dependant part of the systimer implementations needs a better
way to figure out which cpus to dispatch to.

Reported-by: justin@shiningsilence.com

- Correct usage of gethostname(3)

- WARNS 6

Submitted by: H}kon Schad Bergsaker <hakonsb@student.matnat.uio.no>

${CC}'s here were supposed be used as preprocessor, add -E.

Reported-by: Martin P. Hellwig <mhellwig (a) xs4all.nl>

Sync with the kernel to make ncptrace work again.  CINV_PARENT no longer
exists.

Display additional information about a vnode's object.

Fix a server-side TCP NFS mount stream sequencing problem which could result
in portions of the TCP stream being parsed out of order, corrupting it and
causing the NFS server to drop the connection.

The TCP protocol stack upcalls into nfsrv_rcv().  The various nfsd threads
also call nfsrv_rcv().  A short blocking condition in the parsing code plus
the blocking lock on the sockbuf (when called via an nfsd) could result in
data from the socket being incorporated into the nfs socket's mbuf chain
out of order.

Reported-by: Andreas Hauser <andy@splashground.de>

License typo.

Adjust man page sections.

tcb is now managed by rtld, caching it is not correct,
otherwise we may get stale copy of tls data from previous
dead thread, caching should be done in rtld if needed.

Have the server complain if the NFS rpc stream from a client gets corrupted,
so we at least know why the tcp link is reconnecting.  The client already
complains if the NFS rpc stream from a server gets corrupted).

Import man pages for pf(4), pflog(4) and pfsync(4) devices.

These are not the latest versions, but those that seem to suit
best with our version of pf.

Reminded-By: "Alexander 'alxl' Lobachov" <alxl@alxl.info>
Obtained-From: OpenBSD

Undo last commit.  This was not my hisenbug. :-P

Remove an assertion in bundirty() that requires the buffer to not be on
a queue.  There is a code path in brelse() where the buffer may be put on
a queue prior to calling bundirty().

Reported-by: David Rhodus <sdrhodus@gmail.com>

Additional note to last commit.  GCC-3.4 improperly generates a warning
and an int $5 for unpromoted conversions (somebody really scewed up in GCC
land if they thought that that was a reasonable solution!)... It really
should generate a fatal error for this case since the traditional varadic
macros were able to handle the case.

Use the correct type in va_arg call, char is promoted to int before calling
and has to be fetched as such.

Pass exact number of threads to thr_umtx_wake.

Eliminate a redundant variable assignment.

Noticed by: Noritoshi Demizu <demizu@dd.iij4u.or.jp>

We can only do upper-layer protocol length checks on the first fragment.

Correct the th_off check against ip_len.  The check in ip_demux occurs
before ip_len is adjusted so we have to add iphlen into the equation.  We
forgot to do this when the code was originally moved from tcp_input to
ip_demux (in tcp_input the check occurs after ip_len is adjusted).

This fixes a panic assertion in tcp_input when a mangled packet is received.

Reported-by: Joe Talbott <josepht@cstone.net>

Fixed reference of memory after free.

Because destroy_all_dev() checks the mask/match against the device's si_udev,
which is a field combining both major and minor numbers, we must mask off the
major bits (to retain only the minor bits) from the 'mask' variable in order
to allow a generic -1 to be passed as the mask.  Otherwise we will not match
anything.

Revamp a good chunk of the documentation to try to make the major/minor
number masking issues clear.

Bug-found-by: Chuck Tuffli <chuck_tuffli@agilent.com>

Allow an info.size field of -1 to indicate that the descriptor should map
the entire address space, requires for direct-tls %gs:OFFSET support (where
the OFFSET is negative).  Otherwise the descriptor only maps the specified
range (typically just the TCB area created by the thread library) and %gs:0
must be used to obtain a pointer from which negative relative memory accesses
can be made.

Reverse the order of dragonfly and FreeBSD since it says right after
it is in alphabetical order.

Submitted-by: Craig Dooley <xlnxminusx@gmail.com>

Don't activate -fstrict-aliasing by default, not even with -O2+.

Fix handling of deallocation of dynamic TLS, the previous code could
free random memory after a DSO with TLS segment was unloaded.

Don't restart a timeout wait, one can periodically send signal to
thread and interval is less than the timeout, then the thread will
never be timeout.

Use rtld's TLS interface to allocate tcb.

Add support for TLS.

Obtained from : FreeBSD

Start working on the full-duplex journaling feature, where the target can
acknowledge the sequence space to prevent information loss if a journaling
stream is interrupted.  Implement a skeleton for the receiver thread.

Delete journals associated with a mount point that is undergoing an unmount.
(reported-by: Fabian <fabian.duelli@bluewin.ch>)

Do not allow the journaling descriptor to be a regular file on the same
filesystem that is being journaled.  This doesn't catch loops, but it
should catch the most common foot shooting case.

Reported-by: Fabian <fabian.duelli@bluewin.ch>

Fix a minor bug in pass*) generation.  'units' was not properly optioned.

Submitted-by: Joerg Anslik <joerg@anslik.de>

Fixed indent for structure

FreeBSD-Date: 2005/03/22 08:00:02
Author: harti