Mark td_generation as volatile.

Retire vslock() and vsunlock() with extreme prejudice.

Discussed with: pete

 - Further simplify ffs_sync().  The vnode lock is required for UFS_UPDATE()
   so make the code slightly more uniform.  The vnode lock is acquired in
   all cases and now the only difference between VCHR and other is we
   call UFS_UPDATE instead of VOP_FSYNC().

 - In ffs_update() assert that either the vnode lock or the XLOCK is held.

Fix a security problem in sysctl() the long way round.

Use pre-emption detection to avoid the need for wiring a userland buffer
when copying opaque data structures.

sysctl_wire_old_buffer() is now a no-op. Other consumers of this
API should use pre-emption detection to notice update collisions.

vslock() and vsunlock() should no longer be called by any code
and should be retired in subsequent commits.

Discussed with: pete, phk
MFC after: 1 week

Add a pre-emption counter, td_generation, so that threads can notice
when they have been pre-empted by other threads. This is bumped from
within mi_switch() every time a context switch takes place.

Discussed with: pete

Add a sentence forgotten in the previous commit.

MFi386: revisions 1.572, 1.573 and 1.574.

MFi386: revision 1.205

Fold the vslock() and vsunlock() calls in this file with #if 0's; they will
go away in due course. Involuntary pre-emption means that we can't count
on wiring of pages alone for consistency when performing a SYSCTL_OUT()
bigger than PAGE_SIZE.

Discussed with: pete, phk

New release note: SA-03:18.

New errata:  SA-03:14, SA-03:17, SA-03:18.

Include <sys/mutex.h>.  Don't depend on namespace pollution in <sys/vnode.h>.

Fixed a nearby style bug.  The include of vcoda.h used angle brackets and
was not used.

 - Check the XLOCK before inspecting v_data.
 - Slightly rewrite the fsync loop to be more lock friendly.  We must
   acquire the vnode interlock before dropping the mnt lock.  We must
   also check XLOCK to prevent vclean() races.
 - Use LK_INTERLOCK in the vget() in ffs_sync to further prevent vclean()
   races.
 - Use a local variable to store the results of the nvp == TAILQ_NEXT
   test so that we do not access the vp after we've vrele()d it.
 - Add an XXX comment about UFS_UPDATE() not being protected by any lock
   here.  I suspect that it should need the VOP lock.

 - Apply a big giant lock around the namecache.  This has been sitting in
   my tree since BSDcon.

 - Fix an XXX.  Check the error of vn_lock() in vflush().  Don't specify
   LK_RETRY either, we don't want this vnode if it turns into another.
 - Remove the code that checks the mount point after acquiring the lock
   we are guaranteed to either fail or get the vnode that we wanted.

Assert that the containing vm object's lock is held in
vm_page_set_invalid().

 - Skip over xvp if XLOCK is set.

 - Remove an incorrect XXX comment.  This code does respect the XLOCK since
   it uses vget() which will fail if the identity changes.

 - Check the XLOCK before we inspect the vnode.

 - We don't need to cache_purge() in nfs_reclaim(), vclean() does it for us.

 - Check the XLOCK prior to inspecting v_data.

 - Check XLOCK prior to accessing v_data.

 - File systems that wish to inspect the vnode contents or their private
   v_data field before calling vget/vn_lock must check VI_XLOCK manually to
   be sure that v_data is still valid.  Implement this check in two places
   here.

Correct a typo on line 552 of revision 1.92 which was breaking GENERIC:-
_FreeBSD_version should be __FreeBSD_version.

Remove magic numbers surrounding locking state in the sysctl module, and
replace them with more meaningful defines.

 - Rename vcanrecycle() to vtryrecycle() to reflect its new role.
 - In vtryrecycle() try to vgonel the vnode if all of the previous checks
   passed.  We won't vgonel if someone has either acquired a hold or usecount
   or started the vgone process elsewhere.  This is because we may have been
   removed from the free list while we were inspecting the vnode for
   recycling.
 - The VI_TRYLOCK stops two threads from entering getnewvnode() and recycling
   the same vnode.  To further reduce the likelyhood of this event, requeue
   the vnode on the tail of the list prior to calling vtryrecycle().  We can
   not actually remove the vnode from the list until we know that it's
   going to be recycled because other interlock holders may see the VI_FREE
   flag and try to remove it from the free list.
 - Kill a bogus XXX comment.  If XLOCK is set we shouldn't wait for it
   regardless of MNT_WAIT because the vnode does not actually belong to
   this filesystem.

 - Don't cache_purge() in getnewvnode.  It's done in vclean().  With this
   purge, the purge in vclean, and the filesystems purge, we had 3 purges
   per vnode.
 - Move the insmntque(vp, 0) to vclean() so that we may remove it from the
   two vgone() functions and reduce the number of lock operations required.

 - Don't cache_purge() in cd9660_reclaim.  vclean() does it for us so
   this is redundant.

 - Don't cache_purge() in ufs_reclaim.  vclean() does it for us so
   this is redundant.

 - Don't cache_purge() in ext2_reclaim.  vclean() does it for us so
   this is redundant.

 - Don't cache_purge() in *_reclaim routines.  vclean() does it for us so
   this is redundant.

Update the page_req classes VM_ALLOC_NOOBJ and VM_ALLOC_ZERO.

Suggested by: alc

 - Solve a LOR with the sync_mtx by using the VI_ONWORKLST flag to determine
   whether or not the sync failed.  This could potentially get set between
   the time that we VOP_UNLOCK and VI_LOCK() but the race would harmelssly
   lead to the sync being delayed by an extra 30 seconds.  If we do not move
   the vnode it could cause an endless loop if it continues to fail to sync.
 - Use vhold and vdrop to stop the vnode from changing identities while we
   have it unlocked.  Other internal vfs lists are likely to follow this
   scheme.

Don't bother setting a page table page's valid field.  It is unused and
not setting it is consistent with other uses of VM_ALLOC_NOOBJ pages.

 - Move the xlock 'locking' code into vx_lock() and vx_unlock().
 - Create a new function, vgonechrl(), which performs vgone for an in-use
   character device.  Move the code from vflush() that did this into
   vgonechrl().
 - Hold the xlock across the entirety of vgonel() and vgonechrl() so that
   at no point will an invalid vnode exist on any list without XLOCK set.
 - Move the xlock code out of vclean() now that it is in the vgone*()
   functions.

Synchronize access to a vm page's valid field using the containing
vm object's lock.

Eliminate some unnecessary uses of the vm page queues lock around the
vm page's valid field.  This field is being synchronized using the
containing vm object's lock.

Make it easier to run this code on RELENG_4.

Submitted by: luoqi

Fix the apm problem for real.  We leave the first 4K page for the bios to
work in, but we had it mapped read-only.  While this has always been the
case, the PG_PS enable hack hid it and the apm bios code ended up taking
advantage of it.

Assert that the containing vm object's lock is held in
vm_page_zero_invalid().

Make it easier to run this code on RELENG_4.

Submitted by: luoqi

Synchronize access to a vm page's valid field using the containing
vm object's lock.

Synchronize access to a vm page's valid field using the containing
vm object's lock.

 - Extend the scope the vm object lock to cover calls to
   vm_page_is_valid().
 - Assert that the lock on the containing vm object is held in
   vm_page_is_valid().

Synchronize access to a vm page's valid field using the containing
vm object's lock.

Retired the "most" and "installmost" targets -- they just
do not have a chance to work nowadays as we have a lot of
internal libraries in lib/.

Discussed with: marcel, wollman

any -> ? for new entry (to allow time for people to upgrade their pccardd)

Ooops.  Committed sin number 1: updating the code w/o updating the comments.
Update the comments too.

I've been burned about half a dozen times by the old PAO syntax for
'any' interrupt.  There's no reason not to be liberal here and accept
the PAO syntax.

MFC After: 2 weeks

 - In sched_sync() test our preconditions prior to dropping the sync_mtx.
   This is so that we may grab the interlock while still holding the
   sync_mtx.  We have to VI_TRYLOCK() because in all other cases the lock
   order runs the other way.
 - If we don't meet any of the preconditions, reinsert the vp into the
   list for the next second.
 - We don't need to panic if we fail to sync here because each FSYNC
   function handles this case.  Removing this redundant code also
   simplifies locking.

 - Consistently set sopt_dir.

Pointed out by: pete@isilon.com

 - Change a lame iterative algorithm to a constant time algorithm.  Remove
   the XXX that complains about it as well.

Submitted by: ThomasWuerfl@gmx.de

 - Set the sopt_dir member of the sockopt structure, otherwise, this parameter
   will not actually be set even though we're calling sosetopt.  sosetopt
   calls down to a single ctloutput function if the name or level is
   implemented by a specific protocol.

Submitted by: pete@isilon.com

 - Don't use vrecycle() call vgonel() directly after grabing the vnode
   interlock.  We do this so that we still hold the interlock when we lock
   the vnode later.  This prevents races with the mnt vnode list.

 - The VI assert in getdirtybuf() is only valid if we're not on a VCHR
   vnode.  VCHR vnodes don't do background writes.

Reported by: kan

missing word

Assorted minor fixes, mostly style(9):
- PID should be pid_t, not int;
- sort #include's and local variables;
- don't overuse initializers;
- use warn(3) instead of perror(3) consistently;
- amplify the comment on signals.

 - In a Giantless world, the vn_lock() in vcanrecycle() could legitimately
   fail.  Remove the panic from that case and document why it might fail.
 - Document the reason for calling cache_purge() on a newly created vnode.
 - In insmntque() order the operations so that we can call mtx_unlock()
   one fewer times.  This makes the code somewhat clearer as well.
 - Add XXX comments in sched_sync() and vflush().
 - In vget(), do not sleep while waiting for XLOCK to clear if LK_NOWAIT is
   set.
 - In vclean() we don't need to acquire a lock around a single TAILQ_FIRST
   call.  It's ok if we race here, the vinvalbuf will just do nothing.
 - Increase the scope of the lock in vgonel() to reduce the number of lock
   operations that are performed.

reference ntpd(8)

Don't dump core from the time(1) process itself
if the child process has exited on a signal
whose default action is to dump core.

 - If we are called with LK_NOWAIT in vn_lock() we may be holding a mutex
   and should not sleep while waiting for XLOCK to clear.  Care needs to be
   taken in functions that use this capability to avoid spinning.

 - Document more of the vnode locking strategy.

 - Increase the scope of the interlock in ffs_reload().  Acquire it before
   we release the mntvnode_mtx.
 - Call vgonel() directly instead of going through vrecycle() since we own
   the interlock now.
 - Remove a few cases where we locked the interlock just so that we could
   call VOP_UNLOCK with interlock held.

 - Fix an unlocked call to GETATTR by slightly shuffling the code in
   ffs_snapshot() around.
 - Acquire the interlock before releasing the mntvnode_mtx.  Use the
   interlock to protect v_usecount access.

 - Use the UMA_ZONE_VM flag on the fakepg and object zones to prevent
   vm recursion and LORs.  This may be necessary for other zones created in
   the vm but this needs to be verified.

 - Use the VI_LOCK macro in two places where we directly called mtx_lock()
   before.  Direct calls indicated places that needed review and these have
   now been reviewed.

 - Properly acquire the vnode interlock before releasing the
   mntvnode_mtx.
 - Use a local variable to store the results of the test to see if the
   next vnode on the mount list has changed.  This is so that we no longer
   acess the vnode after we vput() it.

 - Acquire the vnode interlock prior to dropping the mntvnode_mtx.
 - Make a note of the lack of XLOCK protection in this code.  We would access
   a vnode while it is changing identities without Giant.

 - Make proper use of the mntvnode_mtx.  We do not need the loop label
   because we do not drop the mntvnode_mtx.  If this code had ever executed
   and hit the loop condition it would have spun forever.

 - The proper test is CPU_ENABLE_SSE and not CPU_ENABLED_SSE.  This
   effectively disabled the sse2_pagezero() code.

Spotted by: bde

 - Acquire the vnode interlock prior to droping the mntvnode_mtx.  This does
   not eliminate races where the vnode could be reclaimed and end up with
   a NULL v_data pointer but Giant is protecting us from that at the moment.

 - Remove a mp_fixme() and some locks that weren't necessary.  I now
   understand how this works.

Synchronize access to a page's valid field by using the lock from its
containing object.

 - Remove the backtrace() call from the *_vinvalbuf() functions.  Thanks to a
   stack trace supplied by phk, I now understand what's going on here.  The
   check for VI_XLOCK stops us from calling vinvalbuf once the vnode has been
   partially torn down in vclean().  It is not clear that this would cause
   a problem.  Document this in nfs_bio.c, which is where the other two
   filesystems copied this code from.

Check for write errors; report and exit with error value.

Emulate bugs in the old PSE code so that apm works again.

I do not yet understand why, but apm *depended* on the fact that the old
PSE code caused the first 1MB of ram to be mapped read/write because it
was in the same 4MB page as the kernel text+data+bss blob.

If anybody ever tried DISABLE_PSE before, apm would not work.

If your cpu did not have PSE, apm would not work there either (eg: 486).

This bug has been around for a Very Long Time.

The Pentium-4-fix commits did not emulate this unintended side effect of
the PSE post-early-boot fixup, and thus apm blew up.  I've added a hack to
emulate the bug until either apm is fixed or we set fire to our bridges.

This is bad though because it gives kernel mode code the opportunity
to accidently write to the first few megs of the general page pool
which is remapped at KERNBASE.  It needs to be fixed properly.

Locking for updates to routing table entries.  Each rtentry gets a mutex
that covers updates to the contents.  Note this is separate from holding
a reference and/or locking the routing table itself.

Other/related changes:

o rtredirect loses the final parameter by which an rtentry reference
  may be returned; this was never used and added unwarranted complexity
  for locking.
o minor style cleanups to routing code (e.g. ansi-fy function decls)
o remove the logic to bump the refcnt on the parent of cloned routes,
  we assume the parent will remain as long as the clone; doing this avoids
  a circularity in locking during delete
o convert some timeouts to MPSAFE callouts

Notes:

1. rt_mtx in struct rtentry is guarded by #ifdef _KERNEL as user-level
   applications cannot/do-no know about mutex's.  Doing this requires
   that the mutex be the last element in the structure.  A better solution
   is to introduce an externalized version of struct rtentry but this is
   a major task because of the intertwining of rtentry and other data
   structures that are visible to user applications.
2. There are known LOR's that are expected to go away with forthcoming
   work to eliminate many held references.  If not these will be resolved
   prior to release.
3. ATM changes are untested.

Sponsored by: FreeBSD Foundation
Obtained from: BSD/OS (partly)

Handle a 'const' parameter without using __DECONST().

MFC after: 14 days

add a stub for bpfattach2 so bpf is not required with the 802.11
module or related drivers

Spotted by: Dan Lukes <dan@obluda.cz>

Remove the pmap_prefault(9) page as it is no longer required.

Update the pmap(9) documentation to reflect the movement of pmap_prefault()
to the machine-independent VM layer, as per alc's recent commit.
Add a definition for the new pmap_is_prefaultable() helper function.

Migrate pmap_prefault() into the machine-independent virtual memory layer.

A small helper function pmap_is_prefaultable() is added.  This function
encapsulate the few lines of pmap_prefault() that actually vary from
machine to machine.  Note: pmap_is_prefaultable() and pmap_mincore() have
much in common.  Going forward, it's worth considering their merger.

hookup ctlinput for fast ipsec versions of esp+ah protocols

Supported by: FreeBSD Foundation

While make has been fixed to grok this construct, the new make hasn't
been widely deploy and that's causing us a lot of pain.  Back out the
last commit for a few weeks so that we can lessen the support load in
current@ asking why they can't build kernels anymore.  Instructions in
UPDATING have been updated, but this should be more effective.

Revert the reverting: November 1st, 2003

Default ntpd to write a "driftfile" in /var/db/ntpd.drift.

A "driftfile" caches the oscillator offset estimate from boot to boot,
having this means faster and less bumpy time synchronization.  Will
be overridden by any value in the config file.

Make the -q option DTRT in the compat mode.

PR: 48210

remove unused variable

Supported by: FreeBSD Foundation

remove unneeded include of route.h

Supported by: FreeBSD Foundation

remove include of route.h now that ip_dummynet.h no longer exposes
data structures that have an embedded struct route

Sponsored by: FreeBSD Foundation

place some kernel-specific data structures under #ifdef _KERNEL

Sponsored by: FreeBSD Foundation

Removed the ancient .LIBS setting that causes non-existent
libraries to be reported as up-to-date.

Before:

# make -f /dev/null nonexistent.a
`nonexistent.a' is up to date.

After:

# make -f /dev/null nonexistent.a
make: don't know how to make nonexistent.a. Stop

PR: bin/44137 (part of)

Fix typo.

Spotted by: "Bjoern A. Zeeb" <bzeeb@zabbadoz.net>

Make PAGE_SIZE and related quantities signed on sparc64.  (They are signed
quantities on every other architecture.)  This change is required in order
to move pmap_prefault() out of the pmap and into the machine-independent
layer.

Only install .k5login if people have not set NO_KERBEROS.

Style fixes: shell script indentation, if/then style, plus a lot
of comments in C code (capitalized, end with full stop).

Submitted by: schweikh

Create a driver that will compile without warnings on -CURRENT
More changes are probably needed to create a correct driver.

Reviewed by: markm
Approved by: schweikh (mentor)

More checking if kenv actually know something.

Guard against test(1)-unfriendly outputs from kenv.

Fixed the "To build a kernel" section to mention -DALWAYS_CHECK_MAKE
as the new sys/conf/kmod.mk depends on a make(1) bugfix now.

Reported by: current

When direct dispatching an netisr (net.isr.enable=1), if there are already
any queued packets for the isr, process those packets before the newly
submitted packet, maintaining ordering of all packets being delivered
to the netisr.  Remove the bypass counter since we don't bypass anymore.
Leave the comment about possible problems and options since later
performance optimization may change the strategy for addressing ordering
problems here.

Specifically, this maintains the strong isr ordering guarantee; additional
parallelism and lower latency may be possible by moving to weaker
guarantees (per-interface, for example).  We will probably at some point
also want to remove the one instance netisr dispatch limit currently
enforced by a mutex, but it's not clear that's 100% safe yet, even in
the netperf branch.

Reviewed by: sam, others