udevd(8): Reduce the scope of a yet unused variable.

newfs(8): Remove an unused variable.

Found-by: gcc47

mount{,_ufs}(8): Check for mountctl() error before printing buf.

kernel - Fix UP build for usched_dfly

* Fix UP build for usched_dfly (primarily for the 3.2 release, UP is being
  removed post-release).

rshd(8): Add an error check for getnameinfo().

kernel - Detect edge cases in ELF note handling

* Do more stringent tests on ELF note structures and headers to
  avoid possible panics on page-crossings.

* Patch constructed from examination of a similar fix made to FreeBSD
  cvs/1.227.  However, our patch is written a different way.

Reported-by: Lukasz Wojcik <lukasz.wojcik@zoho.com>

Revert a few additions of __unused that need some more thought.

We can postpone it until WARNS gets raised to 4 (at the moment, all
this stuff is WARNS = 2).

Revert some no longer needed local CFLAGS modifications.

crypto: Revert some changes in contributed code.

Move gcc47's -Wunused-but-set-{parameter,variable} to WARNS >= 4.

It's not really important stuff that they uncover.

Also, -Wunused-parameter is added at >= 4 too.

ndis_events(8): Check a read().

Found-by: gcc47

libstdc++47: Rework to prevent regeneration during install

The installincludes target regenerated files such as c++config.h during
the install phase.  It may be a minor bug as "bits/c++config.h:" would
trigger it but "c++config.h:" without the subdirectory wouldn't.  In any
case, generating it and two other headers to the local working bits
directory wasn't necessary as installincludes target installs them in an
accessible location.  Reworking the makefiles prevents a write operation
during installworld.

Reported-by: dillon

IPXrouted(8): Oops removed too much in the last commit.

<bsd.sys.mk>: Rename -W to -Wextra.

-Wextra is a newer name and it is a bit more descriptive.

While here, break a long line.

watchdogd(8): Silence a warning differently.

Also satisfies -Wunused-but-set-parameter.

Found-by: gcc47

kernel/x86_64: In pmap_enter(), entry is, in fact, used.

mptable(8): Remove an unused variable.

Found-by: gcc47

inetd(8): Fix initialization of se_rpc_{low,high}vers.

Found-by: gcc47

fwcontrol(8): f_cycle is only used in #if 0'd code, so #if 0 it too.

Found-by: gcc47

IPXrouted(8): Remove dead code, DragonFly has RTM_ADD.

ypxfr(8): Use the correct enum member for checking yp_errno.

Revert local CFLAGS, too.

Found-by: gcc47

yppoll(8): Comment out -h option.

The implementation of it is missing (see e.g. NetBSD or OpenBSD). Until
someone works on it, comment out the existing code to silence a warning
about 'hostname' being set but unused.

Found-by: gcc47

ldconfig(8): Remove -s from usage(), too.

make(1): Use an inline function instead of a macro for Lst_Valid().

This gives -Waddress something to test even if the address of a local
variable is passed.

Remove local CFLAGS and put back under -Werror.

Sync zoneinfo database with tzdata2012g from ftp://ftp.iana.org/tz/releases

* asia (Asia/Gaza, Asia/Hebron): Fall transition was Sep 21, not Sep 28.
    Thanks to Steffen Thorsen for this.

* australasia (Pacific/Apia): New DST rules for Western Samoa.
    Thanks to Nicholas Pereira for the heads-up and to Robert Elz
    for coding up the rules.

ndiscvt(8): Unbreak build.

compilers.conf: Add -Wno-unknown-warning-option for clang.

When compiling with clang, add -Wno-unknown-warning-option to silence
warnings about unknown warning options. We'll not maintain compatibility
for warning options in our wrapper.

Reported-by: Joris Giovannangeli <joris@giovannangeli.fr>

Remove more unused variables in userland.

Found-by: gcc47

snd/hda: Restore the magic register setting to enable cache snooping

It looks like certain chips ignore NOSNOOP settings in PCIe device
control configuration register.  Restore the magic register setting
to enable cache snooping.

Reported-by: many
Tested-by: Goetz Isenmann <info@goetz-isenmann.de>

vinum(8): Remove some unused variables.

Found-by: gcc47

growfs(8): Remove an unused variable.

Found-by: gcc47

rcs(1): Remove an unused variable.

Found-by: gcc47

loader(8): Remove an unused variable.

Found-by: gcc47

fsirand(8): Remove an unused variable.

Found-by: gcc47

ldconfig(8): Make -s a no-op.

The functionality of -s was removed in a369de1639fc. Also comment it out
in the manpage, but leave the option for compatibility.

Found-by: gcc47

ffsinfo(8): Remove an unused variable.

Found-by: gcc47

cryptdisks(8): The 'ok' variable is only needed in #ifdef DEBUG.

Found-by: gcc47

ccdconfig(8): Remove some unused variables.

Found-by: gcc47

camcontrol(8): Remove some unused variables.

Found-by: gcc47

usched_set.2: Mention usched(8).

usched(8): Add a minimal manual page.

usched(8): #if 0 out some (yet) unused code.

Found-by: gcc47

tcplay(3): The 'ok' variable is only needed in #ifdef DEBUG.

Found-by: gcc47

reboot(8): Remove an unused variable.

Found-by: gcc47

rcorder(8): Whitespace fix.

monitor(1): Check kevent().

Found-by: gcc47

dfregress(8): Check prop_dictionary_get_uint32()'s return value.

Consistent with the rest of the file.

Found-by: gcc47

brandelf(1): Make -v a no-op.

-v never did anything, so remove the verbose variable and comment it out
in the manpage, but leave the option for compatibility.

Found-by: gcc47

Add dragonfly.vim, for changing vim indentation rules to match style(9).

Assuming it works, haven't personally used it.

Taken-from: FreeBSD

Update the pciconf(8) database.

October 14, 2012 snapshot from http://pciids.sourceforge.net/

compilers.conf.5: Some more cleanup.

* Remove a reference to an old patch I did for building world with clang.
  The changes which are currently needed remain to be determined.

* Make the examples more generic and indicate that buildworld/kernel
  do not necessarily survive with the compilers listed. Also, stuff them
  into an EXAMPLES section.

* Add a comma.

Reported-by: Carsten Mattner <carstenmattner@gmail.com>

vkernel - Adjust parameters for vkernel test build

* Increase root partition from 512M to 2G to accomodate DEBUG_FLAGS=-g
  kernels.

* Increase default memory from 128M to 256M

* Create the /var/vkernel directory if necessary.

kernel - Fix sysclock_t comparison in usched code

* Fix a sysclock_t comparison that was attempting to detect an overflow
  by checking if an unsigned field was negative.  The field has
  to be cast to signed for the test to work as expected.

Reported-by: enjolras

HAMMER VFS - Use full-block undos for B-Tree nodes

* Use a full-block undos instead of small byte-range undos for
  B-Tree nodes.

  There is no bug here per-say, but we are trying to improve
  HAMMER's ability to recover after a power failure where a disk
  drive is writing to the platter at the time of the failure.
  While we have no control over any unrelated sectors that might
  get smashed by such an event, the concern is that sectors being
  specifically written might get corrupted beyond the ability for
  the byte-range undo to fix.

  The reason is that if, say, only 50 bytes in a B-Tree node need
  updating, HAMMER must still do a 16K physical write to disk, so
  really any portion of that 16K can wind up corrupt on a power
  failure, not just the 50 bytes being changed.

  So instead of writing a byte-ranged 50-byte UNDO we now write
  a 16K undo for B-Tree elements.

* The hope is that this will improve cases where HAMMER has become
  unmountable due to CRC errors in the B-Tree closer to the root of
  the B-Tree (which are usually updated to update only the mirror_tid)

Reported-by: sgeorge, marino

make device_probe_child() public

compilers.conf.5: Remove docs about *_VERSION, it no longer exists.

sleep.9: Really write PINTERLOCKED in the example.

Reported-by: vsrinivas

sleep.9: s/PINTERLOCK/PINTERLOCKED/

customcc/compilers.conf: Some adjustments and simplifications.

* pkgsrc's clang no longer needs special handling for C++ compilation.

* Use gcc44's cpp, gcc41 is gone.

* -dumpversion is now understool by both pkgsrc's and SVN trunk's
  clang, so no special handling needed for that either.

Based on an older patch I did with Juan Francisco a year ago.

Reported-by: Juan Francisco Cantero Hurtado <iam@juanfra.info>
               Joris Giovannangeli <joris@giovannangeli.fr>
Tested-by: Joris Giovannangeli <joris@giovannangeli.fr>
Dragonfly-bug: <http://bugs.dragonflybsd.org/issues/2205>

kernel: Some adjustments for the GCC 4.1 removal in Makefiles.

UPDATING: Some words about GCC 4.7.

compilers.conf.5: We only describe non-base compilers here.

"gcc47" is no longer handled via compilers.conf.

Makefile_upgrade.inc: Adjust modules path.

This can be helpful in cases where old modules for some reason
had not been cleaned when the location was still /boot/modules
and have ended up getting objcopy'd to /boot/kernel after the
location was changed.

kernel - Fix i386 pmap bug

* mpte will be NULL for KVM page table pages, there is no need to track
  the wire count.  Conditionalize-out the unwiring call.

* This is believed to be (this time for sure to the third power!) the
  PG_BUSY crash that vsrinivas was getting on i386 related to paging
  in a swapped kernel pipe buffer.

kernel - Make pcb_onfault more robust.

* Record the expected kernel stack pointer along with the pcb_onfault
  action.  Adjust the trap code to only take the action if the frame's
  stack pointer matches the recorded expected stack pointer.

  Otherwise this might be a recursive trap and we definitely do NOT want
  to execute the on-fault stuff in that situation.

* Prior to these changes recursive traps during uiomove()s could result
  in a kernel stack so corrupt that finding the actual cause of the panic
  becomes impossible.  This is believed to be making life difficult for us
  trying to track down a particular i386 panic.

* On x86-64 we had to increase the size of the pcb structure.  kgdb on
  kernel cores and live kernels will be effected (needs recompile).

top - Use RES by default instead of PRES

* The kernel support for PRES was removed during the SMP fine-grained
  locking work on the VM system last year so top was reporting 0K for
  everything.  Change top to go back to RES.

acpi/pstate: Be lenient about wrong # of processors in the power domain

Some stupid BIOSes seem to consider two logic CPUs (hyperthreading) as
one CPU.  Don't bail out; just print some error messages.

Reported-by: dillon@

kernel - Use atomic op for usched_global_cpumask

* Use atomic op for usched_global_cpumask in the cpu init code.  This
  code is serialized anyway, I think, but it is best to be safe.

* Add cpu_ccfence() to ensure that the compiler does not reorder the
  clearing of the mask in the cpusync code.

kernel - Remove debugging from i386 pmap

* Remove i386 pmap debugging kprintf.

kernel - Add MODULE_VERSION for if_ale

* Seems to solve boot-time lockup when if_ale is built into the kernel
  AND also specified in /boot/loader.conf

* Related issues now being reviewed by swildner

kernel - Probable fix to vm_fault: PG_BUSY panic on i386

* The info structure for the pmap_inval*() API is only initialized
  conditionally as an optimization.

* There was a case where the info structure was being used without
  first being initialized which matches reported panics (essentially
  a pipe buffer page in kernel memory is swapped out and the faulted back
  in during a uiomove).

Reported-by: vsrinivas, marino, several others

kernel - Fix missing LWKT token init in linprocfs

* Fix missing LWKT token init in linprocfs.  linprocfs_init() was not
  being called.

Reported-by: marino

Merge branch 'DragonFly_RELEASE_3_2' of ssh://crater.dragonflybsd.org/repository/git/dragonfly into DragonFly_RELEASE_3_2

Point the pkgsrc make targets at newest quarterly release.
I'm building packages to match now too.

<sys/param.h>: Fix wrongly merged comments on the release branch.

Branching 3.2/3.3.

kernel - Fix UP build for usched_dfly.c

* Fix UP build errors

Reported-by: multiple

vkernel - Properly initialize pool tokens.

 * There was no call to init_locks() in vkernel's initialization
   code, so provide one.

 * As pool tokens weren't initialized, the attempt to copy its
   t_desc in lwkt_getalltokens() resulted in a panic. This should
   be fixed now.

 * Add a KASSERT() so that uninitialized tokens will panic the system.

Suggested by: swildner

kmalloc.9: Document M_POWEROF2.

kmalloc: Use 'fls' to round up the size to the nearest power of 2

On average tests conducted on Intel i3, i7 and xeon-e3 in x86_64 mode,
fls version is 3 times faster than the simple loop version.

Submitted-by: vsrinivas@
Also M_POWEROF2 flag is used to do the nearest power of 2 size rounding
up, instead of a seperate function (was kmalloc_powerof2)

Suggested-by: sjg@, vsrinivas@

libgcc.a47: build with -fpic

The vendor makefile builds libgcc.a with the PIC flag.  The x86_64
platform needs it for packages that need need to link to libgcc.a

build - Unbreak world

* Fix issue w/ last commit

Submitted-by: marino

Use GCC's builtin offset_of as the first option

Other than C++ code, the GCC offsetof builtin was never getting used.
The version that did get expanded could not be used for array
declarations on GCC 4.5 and later because it wasn't considered a
constant expression by those later GCC compilers.

The first symptom was seen when building world with GCC 4.6, but the
the offsetof expansion was finally identified as the cause of the
problem while discussion PostgreSQL 9.1 build failures on DragonFly.
Since DragonFly was the only platform exibiting the behavior, the
problem was narrowed down to the offsetof macro or the compiler itself.
Fixing the macro allowed the unpatched pgsql to compile.

Thanks-to: Tom Lane

ifpoll: Utilize kmalloc_cachealign()

busdma: Panic if the kmalloc_powerof2() fails to meet the dma requirement

busdma: Utilize kmalloc_powerof2()

igb: Use kmalloc_cachealign to allocate rx ring, tx ring and msix data

kmalloc: Add kmalloc_powerof2() and kmalloc_cachealign()

kmalloc_powerof2()
    Ensures that the returned address will be power of 2 aligned.

kmalloc_cachealign()
    Ensures that the returned address will be cacheline size aligned.
    It is useful to allocate structs declared with __cachealign
    attribute.

These two function probably should _not_ be used on the hot code path
due to the computational cost to find the nearest power of 2 size.

igb: Cache align msix, rx and tx data

igb: Remove unused code

igb: Change polling(4) support to ifpoll support

For 82576, this means that all of the 16 RX rings could be enabled
and fully utilized; even in MSI-X mode, only 8 RX rings could be
used.

kernel/ixgbe: Remove double definition.

ixgbe: Define missing DEBUGOUT# macros.

DEBUGOUT[45] are not yet used but are being defined pre-emptively
to avoid future build breakage

Obtained-from: FreeBSD

kernel/hammer: Staticize hammer_skip_redo.

It's just used in hammer_recover.c.

re: Don't enable MSI by default

igb: Initialize if_name so functions used on if_init could use if_printf

Remove some more GCC 4.1 specific files via 'make upgrade'.

re(4): Use MSI if device supports it.

RT8168E supports MSI.

kernel - Acquire vm_token in pmap_remove_all() (3)

* Correct bug in last commit, resident_count adjustment has to be
  moved past the revalidation.

kernel - Acquire vm_token in pmap_remove_all() (2)

* Remove this, the token is already acquired in all pmap_remove_all()
  paths.

* Instead, try catching the problem in pmap_remove_all(m) by re-validating
  the pv_entry (essentially detecting a removal race).