There is no longer a KNOWN_DEFECTS file, so don't try to install it.

Remove unused variable.

Detected by: Coverity (ID#704)

In stange circumstances we may end up being the last reference to a
session in tprintf().   SESSRELE() needs to properly dispose of the
sessions mutex.

Add sessrele() which does the proper cleanup and have SESSRELE() call it.

Use SESSRELE also in pgdelete().

Found by: Coverity (ID:526)

Expand and refine a few sections for future reference

Delete all aix ports

Regenerate for 9.3.1

bmake changes to handle the move of dns/sec and related files

This commit was generated by cvs2svn to compensate for changes in r143734,
which included commits to RCS files with non-trunk default branches.

This commit was generated by cvs2svn to compensate for changes in r143731,
which included commits to RCS files with non-trunk default branches.

Vendor import of BIND 9.3.1

Bring back some of the ioctl junk that was removed in rev 1.59 as a
i386-only kernel option, ASR_COMPAT, and under BURN_BRIDGES.
It is really ugly, but raidutils depends on it.

Discussed with: scottl

Use PCIR_BARS instead of PCIR_MAPS.

Introduce a new function, pmap_wb_page(), which check all userland mappings for
a given page and, if the pmap is the current pmap, write back the associated
cache line.
Use pmap_wb_page in pmap_qenter() instead of inconditionally
write back/invalidating the data cache.

Consumers of nfslockdans() seems to think it should return 0 on success, so
make it so.

Spell 'fedisableexcept' correctly.

Remove comments relevant only to pc98 as there are no amd64 pc98 machines.

Customize this for the alpha by removing pc98 defines (unused on alpha)
as well as saying that the alpha is wired up in a certain way.

Plug memory leak.

Submitted by: Ted Unangst
Found by: Coverity Prevent analysis tool
Approved by: phk
MFC after: 3 days

Eleminate 3 PC98 ifdefs:
o Use IP_NPX in preference to hard coded value to write 0 to clear busy#
o Use md macro for a full reset of the npx
o Use IRQ_NPX in preference to hard coded value for each platform.

# The other two ifdefs in this file are hard to remove

Define IRQ_NPX for the irq used for the npx.  Define macro for a full
reset of of npx, as appropriate for the platform.

correct comparison for null ptr

Noticed by: Coverity Prevent analysis tool

avoid potential array index by -1

Noticed by: Coverity Prevent analysis tool

eliminate use after free in debug code

Noticed by: Coverity Prevent analysis tool

kill dead code

Noticed by: Coverity Prevent analysis tool

bus_alloc_resource must obey the same rules as
rman_resource_resournce_bound wrt end parameter.  The end parameter
here was the same as the start.  However, it should be start + count -
1, so make it that instead.

Make it clear nve needs mii, and shorten long comment line.

- Replace fe[gs]etmask() with feenableexcept(), fedisableexcept(), and
  fegetexcept().
- Add additional tests for the above.
- Output a separate success message for each of the 8 components of
  this set of tests.

Document feenableexcept(), fedisableexcept(), and fegetexcept().

Replace fegetmask() and fesetmask() with feenableexcept(),
fedisableexcept(), and fegetexcept().  These two sets of routines
provide the same functionality.  I implemented the former as an
undocumented internal interface to make the regression test easier to
write.  However, fe(enable|disable|get)except() is already part of
glibc, and I would like to avoid gratuitous differences.  The only
major flaw in the glibc API is that there's no good way to report
errors on processors that don't support all the unmasked exceptions.

Unload and destroy the TX DMA maps before destroying the DMA tag
they're attached to, not after.

Spotted by: Coverity via sam

Now that there are no users of Lst_ForEach and Lst_ForEachFrom are left
delete these two macros and all the associated stuff.

Replace Lst_ForEach by LST_FOREACH.

Replace another bunch of Lst_ForEachs by LST_FOREACHs and simplify code.

Use vfs_hash.

Add two arguments to the vfs_hash() KPI so that filesystems which do
not have unique hashes (NFS) can also use it.

Remove unused file

Fix a memoryleak in case of failed root filesystem mount.

Spotted by:     Coverity via sam

Dike out another kvm indiscretion.

Start natd(8) before loading firewall rules, to give the
ipdivert.ko module a chance to load.

MFp4: use the function to fix the packet header length instead of rolling
our own...

Remove inode fields previously used for private inode hash tables.

Fix a comment.

Get rid of another bunch of Lst_ForEach in favour of LST_FOREACH and
simplify code accordingly.

MFp4: print a more useful error when we don't have a /dev to mount devfs
on..

MFp4:  add in making fiq's work by coping to the correct page incase we have
the vectors relocated high..

fix up white space, I had a simple comment fix, but I might as well do the
rest while I'm here...

Add mnt_hashseed to struct mount and initialize it witn PRNG bits, use
it to get better hashing in vfs_hash.

In case of an insert collision in vfs_hash_insert(), put the loosing vnode
on a special list so that vfs_hash_remove() can just assume that it is on
a list.

Drop the VI_HASHED flag.

XXX: unnecessary pointer in inode.

Don't store the disk cdev in all inodes.

Don't hold a reference to the disk vnode for each inode.

Don't store the disk cdev in all inodes, it's only used for debugging
printfs.

plug resource leak

Noticed by: Coverity Prevent analysis tool

correct bounds check

Noticed by: Coverity Prevent analysis tool

Enable ehci by default on i386 and amd64. It had got to the stage
where having this disabled was actually hurting us, since so many
BIOSes include legacy USB emulation that takes control of all usb
ports and only the ehci driver knows how to disable it.

Chase location of scd.c and bump .Dd.

Submitted by: markus
MFC after: 3 days

make bus_dmamem_alloc always allocate a new map like we are suppose to..
This was found when I tried to run the usb code on my arm board...

Approved by: cognet

Clarify that the end argument meaning for rman_reserve_resource_bound
is the highest acceptable value for the ending of the resource being
allocated.  One could also believe that it is the highest starting
value of the resource.  The code definitely expects the former, but I
could find no documentation of this apart from TFSC.

If bus_generic_susped returns an error, devlist is not freed.  Free it.

Submitted by: Ted Unangst (using the Coverity Prevent analysis tool)

Don't hold a reference to the disk vnode for each inode.

Eliminate cdev and vnode pointer to the disk from the inodes,
the mount holds everything we need.

Eliminate cdev pointer in inodes, they're not used or needed.

The cdev could have been pulled out of the mountpoint cheaper back
when it was used anyway.

Don't hold a reference on the disk vnode for each inode.

Sometimes, when asked to return region A..C, we'd return A+N..C+N
instead of failing.

When looking for a region to allocate, we used to check to see if the
start address was < end.  In the case where A..B is allocated already,
and one wants to allocate A..C (B < C), then this test would
improperly fail (which means we'd examine that region as a possible
one), and we'd return the region B+1..C+(B-A+1) rather than NULL.
Since C+(B-A+1) is necessarily larger than C (end argument), this is
incorrect behavior for rman_reserve_resource_bound().

The fix is to exclude those regions where r->r_start + count - 1 > end
rather than r->r_start > end.  This bug has been in this code for a
very long time.  I believe that all other tests against end are
correctly done.

This is why sio0 generated a message about interrupts not being
enabled properly for the device.  When fdc had a bug that allocated
from 0x3f7 to 0x3fb, sio0 was then given 0x3fc-0x404 rather than the
0x3f8-0x3ff that it wanted.  Now when fdc has the same bug, sio0 fails
to allocate its ports, which is the proper behavior.  Since the probe
failed, we never saw the messed up resources reported.

I suspect that there are other places in the tree that have weird
looping or other odd work arounds to try to cope with the observed
weirdness this bug can introduce.  These workarounds should be located
and eliminated.

Minor debug write fix to match the above test done as well.

'nice' by: mdodd
Sponsored by: timing solutions (http://www.timing.com/)

Fix a debugging printf.  The order of start/end was inconsistant with
all the other start/end debugs, causing momentary confusion when the
output was examined.

Improve the vfs_hash() API: vput() the unneeded vnode centrally to
avoid replicating the vput in all the filesystems.

Remove OBE comment about AMD64 memory model.

Ensure GCC does not use FP registers in integer code.
I think all we really need is -fno-sse2.
I really don't like cluttering up the compiler invocation,
but this bigger hammer will fix reported problems for now.

Remove fpsetsticky().  This was added for SysV compatibility, but due
to mistakes from day 1, it has always had semantics inconsistent with
SVR4 and its successors.  In particular, given argument M:

- On Solaris and FreeBSD/{alpha,sparc64}, it clobbers the old flags
  and *sets* the new flag word to M.  (NetBSD, too?)
- On FreeBSD/{amd64,i386}, it *clears* the flags that are specified in M
  and leaves the remaining flags unchanged (modulo a small bug on amd64.)
- On FreeBSD/ia64, it is not implemented.

There is no way to fix fpsetsticky() to DTRT for both old FreeBSD apps
and apps ported from other operating systems, so the best approach
seems to be to kill the function and fix any apps that break.  I
couldn't find any ports that use it, and any such ports would already
be broken on FreeBSD/ia64 and Linux anyway.

By the way, the routine has always been undocumented in FreeBSD,
except for an MLINK to a manpage that doesn't describe it.  This
manpage has stated since 5.3-RELEASE that the functions it describes
are deprecated, so that must mean that functions that it is *supposed*
to describe but doesn't are even *more* deprecated.  ;-)

Note that fpresetsticky() has been retained on FreeBSD/i386.  As far
as I can tell, no other operating systems or ports of FreeBSD
implement it, so there's nothing for it to be inconsistent with.

PR: 75862
Suggested by: bde

Ups. Revert the last commits. These have been committed by accident.

modifier_M: instead of going through the string twice to compute the
size of the buffer we need, just allocate the possible maximum.

Patch: 7.117

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

fix arm wrt to busdma...

also wrap the two macros that have bare if's w/ do {} while(0) so that
my epe driver doesn't get a warning about braces around confused else...

time_t is signed, so s/uintmax_t/intmax_t/g.  Has anyone got an extra pointy
hat for me?  I'm running out of them.

Spotted by: des
Pointy hat to: philip

Style: Move a variable from a local scope up to the begin of the function.
Rename result variable so common code becomes more visible.
Rename freePtr to freeResult to make clear what pointer must be freed.

Patch: 7.116, 7.116a

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

 - Now that there are no external users of vfree() make it static.
 - Move VSHOULDBUSY, VSHOULDFREE, and VTRYRECYCLE into vfs_subr.c so
   no one else attempts to grow a dependency on them.
 - Now that objects with pages hold the vnode we don't have to do unlocked
   checks for the page count in the vm object in VSHOULDFREE.  These three
   macros could simply check for holdcnt state transitions to determine
   whether the vnode is on the free list already, but the extra safety
   the flag affords us is probably worth the minimal cost.
 - The leafonly sysctl and code have been dead for several years now,
   remove the sysctl and the code that employed it from vtryrecycle().
 - vtryrecycle() also no longer has to check the object's page count as
   the object holds the vnode until it reaches 0.

Sponsored by: Isilon Systems, Inc.

Dike out unwarranted si_udev fondling.

Get rid of a number of Lst_ForEach calls in favour of LST_FOREACH
and simplify the printing functions.

VarParseLong: move the detection of the modifier separator ':' into
the loop. Add a comment why the 'consumed' variable is updated.
Rename lengthPtr to consumed.

Patch: 7.115

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

Dike out unwarranted reference to si_udev.

Yet another version of passing the last line to ParseFinishLine().
It turns out that some ports use the obscure feature of spreading
a dependency block across multiple include files. While this seems
bad style, allow it for now and call said function only at end of
all input to process the really last line of everything.

 - In vm_page_insert() hold the backing vnode when the first page
   is inserted.
 - In vm_page_remove() drop the backing vnode when the last page
   is removed.
 - Don't check the vnode to see if it must be reclaimed on every
   call to vm_page_free_toq() as we only check it now when it is
   actually required.  This saves us two lock operations per call.

Sponsored by: Isilon Systems, Inc.

 - Remove the now unused LK_THISLAYER.

Fix a debug message to print a usable device name rather than useless
major+minor tupple.

 - Assume that all lower filesystems now support proper locking.  Assert
   that they set v->v_vnlock.  This is true for all filesystems in the
   tree.
 - Remove all uses of LK_THISLAYER.  If the lower layer is locked, the
   null layer is locked.  We only use vget() to get a reference now.
   null essentially does no locking.  This fixes LOOKUP_SHARED with
   nullfs.
 - Remove the special LK_DRAIN considerations, I do not believe this is
   needed now as LK_DRAIN doesn't destroy the lower vnode's lock, and
   it's hardly used anymore.
 - Add one well commented hack to prevent the lowervp from going away
   while we're in it's VOP_LOCK routine.  This can only happen if we're
   forcibly unmounted while some callers are waiting in the lock.  In
   this case the lowervp could be recycled after we drop our last ref
   in null_reclaim().  Prevent this with a vhold().

 - Expose vholdl() so it may be used outside of vfs_subr.c

Remove findcdev().

Polish.

Disable two users of findcdev.  They do the wrong thing now and will
need to be fixed.  In both cases the API should be reengineered to do
something (more) sensible.

o Add missed language doc subdirectories, sort.

PR: docs/78859
Submitted by: skv

Neuter the duplicated disk-device magic code for now.  Somebody with
serious linux-clue is necessary to fix this properly.

Prepend underscore to bus_dmamap_{unload|sync} in line with
recent busdma changes.

Include <sys/signalvar.h> for trapsignal prototype.

Long overdue sync-up with ATA code

Rename cdev->si_udev to cdev->si_drv0 to reflect the new nature of
the field.

 - We have to transfer lockers after reseting our vnlock pointer.

Sponsored by: Isilon Systems, Inc.

Don't export major,minor, instead export tty name.

Clean up forward struct decls.

forward declare struct disk.

Cast time_t to uintmax_t so they can be printed as %ju without breaking on
various architectures.  This should appease tinderbox on alpha and not break
it anywhere else.

Suggested by: harti

Print devtoname() instead of minor().

Fix typo: pointers are not boolean in style(9).

Move devtoname() prototype to systm.h to reduce #include pollution,
it is (or should be) used in many printf() calls.

 - transferlockers() requires the interlock to be SMP safe.

Sponsored by: Isilon Systems, Inc.