GC local variable size.

GC strsize.

Separate error handling path from normal return to avoid
compiler warnings about unused variables.

Remove extra token after #endif introduced in last commit.

Remove unneeded packed attributes.

Verified-by: md5

- Fix a few grammar 'nits'.

Submitted by: Jonathon McKitrick <jcm@FreeBSD-uk.eu.org>

Remove unnecessary packed attribute.

Verified-by: md5

install altq headers

altq include dir

Harti moved my comment.

Removed period from end of sentence, to reduce diffs between FreeBSD us from
getting too large.

Add ALTQ support to pfctl(8). Fix a small bug introduced earlier
in the HTONL --> htonl conversion.

Import ALTQ support from KAME. This is based on the FreeBSD 4 snapshot.
This includes neither the ALTQ3 compat code nor the !DragonFly defines.
The macros have been replaced with inline functions in net/ifq_var.h.

This also renames pkthdr.pf_flags as it is intended as general flag bit.
Currently supported are ppp(4), sppp(4), tun(4) and wi(4), more drivers
are coming later.

Reviewed-by: corecode, dillon, hsu
Comments-from: hmp

Remove unused variable.

Reduced diff between FreeBSD.

- Fix some spelling mistakes.

Submitted by: Jonathon McKitrick <jcm@freebsd-uk.eu.org>

- Improve the debugging functionality in battd.
- In the last commit, I removed a check on the percentage. This
  was wrong since we need to test whether the default can be
  supported also (most likely is).
- Make sure we check whether time is supported by /dev/apm when
  using the -T option.

Renamed variable 'l' to 'ln'

Lst_Append returns void, so the other case of the ? statement
should also be void.

Author: harti
Taken-from: FreeBSD

Initialise the status variable. W_SETTERMSIG uses its value.
Author: harti
Taken-from: FreeBSD

More cleaning:

- Remove unnecessary header

Add the 'All rights reserved.'. It's not entirely clear if this is still
necessary by Law or not.

Temporarily create new scope and reindent section which should
become its own function.

Fix comment indention.

Move assignment of variables {freePtr, dynamic, start} closer to
return statements.  This helps clarify what variables are used to
communicate between largely seperated code.

Move some variables at top of Var_Parse() into local scope.
Makes the code more clean, and in prep for splitting up this
900 line function.

Remove stale local variable ifq.

Use ifp->if_snd directly. Use IFQ_SET_MAXLEN.

Don't assign &ifp->if_snd to a temporary variable, it's type will change
soon. Use ifp directly for BPF_MTAP.

Directly use ifp->if_snd, it's type will change soon.

Add -DIN_TARGET_LIB to prevent libobjc from depending on in-tree GCC code.
This removes the undefined x86_field_alignment from the DSO.

Noticed-by: eirikn

Cleanup white space (removed spc between tabs)

Connect our 1:1 threading library to the build.

Changed renamed macros *_BRACKET to *_BRACE.

Untangle VarFind and, while rewriting most of the function, fix the
intendation. (A large part of the function was already at intendation 8).
Cool! much cleaner.

Taken-from: FreeBSD
Author: harti

Move VarCreate() and VarDestroy() close to top of file, and add white
space at begining of some functions.

Removed uneeded white space.

- Move Error case to top of elseif chain.

- Do not initialize startc, at top of function, but where a know
  valid value can be assigned.

- Split out error case where input to Var_Parse() is "$", only a
  dollar sign.
- Make default case an explicit error instead of falling out of the
  switch statment into another error case.

Fix a case that can prevent the vnlru_proc vnode recycler from operating.
The vnode recycler is not allowed to recycle 'internal' directory nodes
in the namecache topology.  i.e. if the path A/B/C/D is cached, then the
recycler is not allowed to remove A, B, or C.  The recycler checks
this condition by checking the vnode's v_holdcnt.  The recycler also tries
to recycle vnodes with little or no cached data rather then the vnodes
with a great deal of cached data.

However, the buffer cache now uses v_holdcnt as well.  If the buffer cache
is large enough and maxvnodes is small enough the recycler can wind up not
finding any vnodes to recycle.

The fix is clean out the buffers with the candidate vnode and then make a
final v_holdcnt check.

Reported-by: Gary Allan <dragonfly@gallan.plus.com>

Eliminate conditional check for initialized 'fp' on error in kern_sendfile().

Now that 'so_pcb' is properly declared as a 'void *', remove a layer of
indirection and directly use 'so->so_pcb' in place of 'sotoinpcb(so)'.

Set so->so_pcb directly to NULL to avoid lvalue cast.

Include pci_private.h to get pci_class_to_string.

- convert to bus_space macros
- netgraph cleanup
- bus_alloc_resource_any
- if_printf insteed of printf("%s...", ifp_if_xname);
- use M_WAITOK in arc_init, it's called only from arc_attach

Mostly-obtained-from: FreeBSD

Remove call to readdir() that was missed durning the libc namespace
cleanup.  We now make use of _readidir_unlocked().

First reported by : Joerg Anslik <joerg@anslik.de>
Bug hunter and fix provided by : YONETANI Tomokazu <qhwt+dfly@les.ath.cx>

- Removed declaration of VarModify() since it is defined before it is used.
  Done to sync with FreeBSD.

Change whitespace to match with FreeBSD.  But to reduce diffs, but
we really need to do a complete file sweep and clean them all up.

- Add brace around switch statement.
- Cleanup white space in var.h

Up the initial interrupt configuration hook delay to 20 seconds before
reporting a potential problem.

Add support for pure kernel thread stack frames.  Pure kernel threads do not
have a PCB, they simply push the register state (depending on the situation)
onto the stack and save the stack pointer.

More white space diff reduction with FreeBSD.

gdb-6 uses /dev/kmem exclusively for kernel addresses when gdb'ing a live
kernel, but the globaldata mapping is outside the bounds of kernel_map.
Make sure that the globaldata mapping is visible to it.

- Reduce white space diffs with FreeBSD.

- Reduce white space diffs after harti commited my change to FreeBSD.
- Removed extra scope.

- Remove two unnecessary headers

- Use a temp variable to simplify long condition in if-statement.

- Eliminate duplicate code.

- cleanup Var_Parse() by removing negtive logic in if statement.

Don't cast lvavalues.

- Remove redundent code. Why check for isupper before a
  case statement where you only have upper case values.

- Define new typedef VarModifyProc and use it to clarify the code.

Moved WARNS?=6 out of subdir Makefiles in to src/bin/Makefile.inc.
Subdir that need lower values should set WARNS instead.

Move WARNS?=6 into src/bin/Makefile.inc instead of having it set
in all the subdir Makefiles.  Subdirectories that fail WARNS=6
should have lower values to override.

Add missing */.

Noticed-by: corecode

Remove the conditionalized FreeBSD 5 code. Keep the capability assignment,
even if we currently don't use it.

During init time, we can savely allocate the mbuf cluster with
MB_WAIT. Pass the flag to em_get_buf to allow this.

Add a macro SYSCTL_SET_CHILDREN. Use it to avoid lhs cast.

- Add support for DT_FLAGS.
- Define various things from the most recent ELF spec.

rtld.c, 1.83
rtld.h, 1.28
map_object.c, 1.14
sys/elf_common.h, 1.12

Do initialise fp to NULL explicitly, the last comment wasn't enough.
The first conditional can fail and the variable points to some random
place.

- Remove mode argument from open().

Raise WARNS to 6:
- Move the print() function above the display() function so that it can
  be inlined by the compiler.
- Un-nest extern declarations; put them into the shared header file.
- Initialize some local variables.

- Reduce diffs from FreeBSD.

- make a temporary copy of the name because Var_Subst insists on
  writing into the string

- Pass NULL to brk_string() when we don't need the number
  of result strings.

Readd the copystr for f_mntfromname of root. It wasn't meant to be
deleted, but didn't change normal system behaviour. In case of an
unclean root filesystem, it prevented the remounting.

- Fix syntax error.  That I added. :-P

- In main.c use NULL instead of 0
- In str.c check if brk_string() is passed a null pointer in
  store_argc.  If so then don't return argc.

Taken-from: FreeBSD

Stop caring about GCC versions between 2.5 and 2.7, they are
historic artefacts.
Remove a double define of __func__ from the !LINT case.
Define __func__ to NULL for GCC before 2.7 and non ISO C99 compilers.
Since all GCC versions above 2.95 support __restrict, include them
in the check too. Remove the weak defines for !__STDC__, we don't
support such a setting.
If _C99_SOURCE is set, don't include POSIX / BSD / XSI extensions.

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

- Constify Var_Delete()
- And more white space work

- Removed unneeded variable 'internal'.

- Revert some white space changes, to keep us in sync with FreeBSD.
- Move Buf_Clear() to end of file (why?) to keep us in sycn with FreeBSD.

Raise WARNS to 6:
- ANSIfy function arguments.

Raise WARNS to 6.

More white space insertion.  Why does putting a blank line at the
begining of functions without variables help readablity?

Put back spaces at begining of functions that have no variables. Silly
but it keeps us in sync with FreeBSD.

Raise WARNS to 6:
- Constify some function arguments and a variable.

The last commit was not sufficient.  Rework the code a bit to make it
sufficient.

Reported-by: EM <EM1897@aol.com>

Back out constify of Parse_File().  It is assigned to curFile.fname
and you cannot const this because it's passed to free() later on.

Revert the last device_print_child() change, it was too confusing to
display the device name after the attach because recursion can insert a
lot of other messages in there.

Instead, if -v is specified, generate a line before and after the attach
suitably marked [tentative] or [attached!].  This also has the advantage
of bracketing any recursion that occurs.

Keep the bootverbose printing of the device chain but get rid of the "ATTACH "
on the front.  It's obvious what it is and the attach message follows
directly now.

s/ich_arg/ich_desc/

One of the last things the system does before it tries to mount root is
wait for interrupt-driven configuration hooks to finish.  This is one of
the common places where the system locks up due to e.g. blown interrupt
routing.

Wait up to 60 seconds for interrupt driven hooks to complete.  Generate
a warning every 10 seconds listing the hooks that have not yet completed.
Name all the hook structures to make the listing meaningful.  After 60
seconds, give up and continue booting.  The system might actually
be useable enough to aid in debugging depending on which device failed.

Annotate the class byte with a class name in the bootverbose pci "found->"
bus logging code.  Numbers are fine, but when there are thousands of them
on the screen you need a bit of human readable context.

Temporarily disable non-working Path MTU discovery pending real fix.

Do not depend on existence of _end symbol in obj_from_addr, use
obj->mapbase and obj->mapsize instead.

rtld.c, 1.92
Obtained-from: FreeBSD

If we change obj_rtld.path after initialising __progname, make sure we
change __progname to point at the new storage otherwise it ends up
pointing at freed memory which leads to confusing garbled error messages.

rtld.c, 1.95
Obtained-from: FreeBSD

m_clalloc() was improperly assuming that an mcl malloc would always succeed
when in fact it is allowed not to (because M_NULLOK is specified).

Reported-by: EM <EM1897@aol.com>

give out a little more information in case of a missing dependency

rtld.c, 1.96
Obtained-from: FreeBSD

Remove -DFREEBSD_ELF.

Verified-by: md5

No need to zero fill memory, mmapped anonymously. Kernel will
return pre-zeroed pages itself.

i386/reloc.c, 1.12
Obtained-from: FreeBSD