sbin/hammer: ignore negative length for history dump

- Use default value 32 when a negative value was given for
  history dump.

sbin/hammer: check strtol()/strtoll() results to avoid irrelevant history
queries

- Check errno for possible overflow/underflow by strtol()/strtoll() in
  order to avoid ioctl with irrelevant offset/length.

- Examples
  # hammer -vvv history@11111111111111111111111111111111,111 ./out
  Result too large: @11111111111111111111111111111111
  # hammer -vvv history@111,11111111111111111111111111111111 ./out
  Result too large: ,11111111111111111111111111111111
  # hammer -vvv history@-11111111111111111111111111111111,111 ./out
  Result too large: @-11111111111111111111111111111111
  # hammer -vvv history@111,-11111111111111111111111111111111 ./out
  Result too large: ,-11111111111111111111111111111111

sbin/hammer: trivial cleanup

- commit 00b46268 reverting 417cb1b1 brought back unnecessary blank line
  at the end of the file that was once removed in 417cb1b1, so remove it
  again.

sbin/hammer: close file descriptor on history iteration

- Make hammer history command close(fd) each time it opens path@@0x`tid`
  for hist.count times.

sbin/hammer: trivial coding style fix

- Trivial coding style fix to a commit 24dd5805.

- Add parenthesis as hammer code mostly prefers "return(retval);"
  to "return retval;".

libhammer - Preparation for snapshot handling & fixes

Trivial changes:
- Function renaming to actually match what they are
  describing.
- Clearer variable naming.

Additions:
- Structures to hold snapshot data.
- Functions to retrieve snapshots from metadata for PFSes.
- A file for version compatibility functions.
- A function to return the canonical access path of a
  PFS which is suitable for opening.
- Wrappers to operate per PFS snapshot list.

Changes & fixes:
- Use the new functions to gather all snaps of a PFS.
- Adapt hammer program for the changes.
- Check before free'ing in libhammer_free_fsinfo()
- Display the number of snapshots for unmounted PFSes.

Still missing:
- Manpages changes.

sys/kern: Adjust some function declaration vs. definition mismatches.

All these functions are declared static already, so no functional change.

vinum(8): Remove unused macro.

If it is ever needed, it can be done with offsetof().

natacontrol(8): Break line better and reduce parentheses.

Add cryptodev to our default configuration file.

While here, change wrong 'pseudo-device' to 'device'.

contrib/opie: Sync with FreeBSD, but not much to see here.

A couple of bugfixes:

* Fix the overflow check in challenge handling (off by 1).

* Store all hashes in little endian format before folding to 64 bits, per
  RFC 2289. SHA1 code is big endian, so a bswap32() is needed.

* Create /etc/opiekeys with 0600, not 0644.

* Increase OPIE_HASHNAME_MAX for SHA1.

* Fix a buffer overflow in an unused file.

* Remove two unused (and empty) files.

The rest is just cosmetics.

acpi/cstate: Fix comment

crunchide(1): fix offset for x86_64, restores WORLD_LDVER

The e_shoff offset for x86_64 is 40, not 44.  FreeBSD figured this out
in January.  The gold linker figured out there was a problem with the
crunch binaries when I tried WORLD_LDVER=ld.gold and it broke on initrd.
The ld.bfd linker didn't squawk.  With the FreeBSD patch, buildworld
completes using the gold linker once again.

While here, tidy up a bit.

keyserv(8): Fix sizeof() of a pointer.

Commit 748a243b13dd295c04c9e06923a801157f6943e5 tried to fix this, but
did it wrongly, because xdrproc_t is a function pointer type and the
sizeof will still give 8.

The correct way is to use keybuf which is the type and has the correct
size (aka HEXKEYBYTES). It is done like this in key_set_1_svc_prog()
already.

keyserv(8): Raise WARNS to 2 and fix warnings.

Revert "hammer - Fix 'history' directive"

This reverts commit 417cb1b104fd140d4e31775d8873ad473216f7e4 which
broke the behaviour described in the manpage.

Spotted-by: tkusumi

unbreak world, take 2

The previous style works in the world makefiles but not here.
Try to fix the unbreak by checking if .MAKE.BUILT.BY is defined before
attempting to use it.

unbreak world

Upgrading from a world with the older bmake that didn't know
.MAKE.BUILT.BY results in a broken world, immediately.  Change the check
from "&&" to two separate checks to avoid the malformed condition.

sys.mk: Add _GLIBCXX_USE_CXX11_ABI=0 definition in some cases

When the world is built by gcc47, and sys.mk isn't being used to build
the system, the CXXFLAGS will have -D_GLIBCXX_USE_CXX11_ABI=0 added to
it to force gcc50 to use the c++ ABI that gcc47 used when building the
system libraries.

If the world was built with gcc50, the system libraries were built
against the new c++ ABI so this flag would not be added in that case.

Due to new SYSBUILD definition in Makefile.inc1, this flag is never
added during the build of the world.

memtemp/e5: Don't attach if CLTT is not set by BIOS in chn_temp_cfg

Tested-by: mneumann@

make(1): Create custom variable .MAKE.BUILT.BY

The value of .MAKE.BUILT.BY is normally either "gcc47" or "gcc50"
depending on which base compiler built the DragonFly world.  It could
also be any valid CCVER for a dports compiler such as "clang35".

This capability is needed to handle the c++ ABI change that has been
introduced by gcc50.  The problem hits when gcc50 is used to build when
the system libraries have been built by gcc47.  The sys.mk file will
need .MAKE.BUILT.BY to adjust c++ flags appropriately.

memtemp/e5: Setup hiwat and lowat before sensor task register.

memtemp/e5: Send devctl notify, if DIMM is too hot, and set sensor status

- There is no need to save DIMM external id in dimm softc; use a stack
  variable instead.
- White space cleanup.

sys/vfs/hammer: minor zone cleanup

- hammer_generate_undo() takes either zone 1 or zone 2 address
  (hammer generates undos for both volume metadata and buffers).

gcc50/libstdc++: Use generated symbol map instead

For both gcc47 and gcc50, we've been using pregenerate symbol map for
libstdc++ due to the apparently complexity of generating it.  That's
already come back to bite on gcc50 as several symbols were modified and
added in the last couple of weeks.  It was time to bite the bullet and
figure out how to generate the symbol map.  It turns out that it really
wasn't that hard.

This *should* fix some dports errors (tbc).

Update the pciconf(8) database.

February 14, 2015 snapshot from http://pciids.sourceforge.net/

sys/vfs/hammer2: fix typos

- Fix trivial typos in comments.

sbin/hammer: cleanups

- Minor cleanups in addition to 24dd5805.

- hunk1 - no need to dereference indirect pointer **bufferp via direct
  pointer *buffer in here anymore.
- hunk2 - all the rest of the code use **bufferp instead of **bufp.

- from Submit #2783

sbin/hammer: add get_ondisk()

- Cleanup get_buffer_data() and get_node() using a new inline function
  get_ondisk(). These two actually do the same thing except for the way
  they release an existing buffer (--ref and eventually free it).

- from Submit #2783

systat - Improve pftop output

* Fix bugs where ICMP states were display bandwidth as '<huge>' .

* Fix bugs where the RB tree compare code would sometimes collapse
  distinct states, creating confusion.

* Greatly improve readability by reformatting addresses and ports,
  in particular truncating IPV6 addresses to fit better.

* Reduce jumping around of display elements as bandwidth values change.
  Instead of sorting by the current bandwidth the display is sorted
  by the 1/8 decay bandwidth.

drm: Add linux/ctype.h

drm: Add DMI_MATCH and DMI_EXACT_MATCH

Obtained-from: OpenBSD

drm: Update linux/kernel.h

* Add clamp_t and DIV_ROUND_UP_ULL(), both obtained from NetBSD

* Add WARN_ON_SMP()

* Add max3()

drm: Add get_seconds()

drm: Add linux/string.h

Obtained-from: FreeBSD's OFED subsystem

drm: Add linux/printk.h

drm: Add linux/spinlock.h

gcc50: use DF stddef.h by adding max_align_t

C11 and C++11 need the max_align_t type definition.  In order to get it,
the gcc version was installed at /usr/libdata/gcc50/stddef.h.  We'd
rather the base gcc use only system-provided headers, so the max_align_t
structure has been added to /usr/include/stddef.h

patch-from: swildner

drm: Update linux/workqueue.h

Obtained-from: FreeBSD's OFED subsystem

drm: Update various Linux headers

chflagsat: fix typo in prototype

add chflagsat system call

nrelease - Revive the gui build

Pkgsrc is long gone. Update Makefile to use dports.

Add option to override dports git host

By specifying GITHOST_DPORTS you can override the default git host for
fetching the dports.git repository:

  cd /usr && make dports-create GITHOST_DPORTS=localhost

ecc/memtemp/e5: Correct E5 v3 DID <-> SLOT

Revert the change in cb830af (revert PCI slot numbers) and instead
modify the PCI device ids. Channel 0/1 was swapped with channel 2/3.

gcc50: Switch to SVN over weekly snapshots for latest

A dports bulk run revealed at least 3 major issues, all of which have
already been fixed upstream.  Two were found in 8 February snapshot, but
the third still hasn't been release.  Since the snapshots are just
the packaged Subversion repo without the .svn directory, I'm switching
over to SVN directly until the prerelease.

With here, disable extra diagnostic checks which slow the compiler down
considerably.

Merge branch 'vendor/GCC50'

Update gcc-50 to SVN version 220677

Last Changed Date: 2015-02-13 08:56:14 +0100 (Fri, 13 Feb 2015)

ecc/e5: Reorganize bootverbose prints a little bit

ecc/memtemp: Add support for Intel E5 v3

memtemp: Add support for Intel Core Haswell and E3 v3 cpus

Tested-by: dillon@ (no sensors available though)

installer: Don't create a separate /usr pfs

* /usr as a separate filesystem from / doesn't really make sense
  anymore in this day and age. Both / and /usr binaries are dynamically
  linked and belong to the same DragonFly base system.

* What's more, it's dangerous. With a separate /usr filesystem, the
  rescue static binaries in /usr/share/initrd can't be accessed without
  /usr mounted and in some situations /usr can't be mounted anymore with
  the regular mount* binaries...

gcc50: Look for ld-elf.so.2 on the root slice always

A couple of us could not could not even get the shell after rebuilding
world with the new compiler.  The problem was that several programs such
as sh, cd, mount*, etc need to run before /usr is mounted when /usr is on
a different slice (those programs are dynamically linked these days)  I
didn't consider this when I decided to leave the interpreter at the default
value which is "/usr/libexec/ld-elf.so.2".  That file is now symlink to
/libexec/ld-elf.so.2.  sh, cd, etc, could not access the dynamic linker
because the mount command failed due to also needing the dynamic linker.

Let's rectify that bad decision now before any other systems get badly
messed up.

test commit

make upgrade: Expand the list a bit for gcc44 removal.

ecc/e3: Split it into two drivers, coremctl(4) and ecc(4)

coremctl(4), which maps the MCHBAR, is now the parent of ecc(4) for
Intel E3 cpus.  This paves way to add Intel Core/E3 support to
memtemp(4).

Tested-by: dillon@ on i3/Haswell and E3/v3

emx: Allow 2 TX rings for i217 and i218

Tested-by: dillon@ on i217

gcc50/stddef.h: Remove __FreeBSD only conditions

In a very special case of DragonFly masquerading as FreeBSD within dports,
using the stock stddef.h that comes with gcc always results in failure.

The masquerading is done by passing __FreeBSD definition through cflags,
and this causes the compiler to attempt to include headers that do not
exist on DragonFly.  The gcc compilers in dports already have this fix.

This commit simply removes all clauses that pertain to FreeBSD and not
DragonFly.

Removal of unused gcc 4.4 sources and makefiles

It's been confirmed that a gcc50-built world and kernel boots and
functions as expected.  With that good news, it is safe to remove
the gcc 4.4 sources and makefiles.  It wasn't building and this will
make the cloning of a shallow repo faster.

gcc50/csu: Skip depends step to avoid possible race

I've witnessed buildworld break once on muscles at gcc50/csu, on the
depends step.  I can reproduce the error manually by doing:

  > cd gnu/lib/gcc50/csu
  > wmake clean cleandepend
  > wmake depend

It fails to locate insn-constants.h which should be avaiable due to
and include flag.  The depends is just checking if all the headers are
in place.  Skipping the depends step may prevent this breakage.  I have
successfully built world with this patch so it doesn't do harm.

gcc50: Build with no-avx like gcc47

This check will also match newer gcc compilers from dports.

Requested-by: zrj (irc)

UPDATING: Initial entry discussing gcc 4.4 & 5.0

This entry may need updating before release, particularly if the release
version of GCC 5.0 is brought in and especially if gcc 5.0 becomes
designated as the primary compiler.

Bump __DragonFly_version to mark introduction of gcc50

ecc/memtemp/e5 - Correct E5 v3 PCI slot numbers

The PCI slot numbers for E5 v3 IMC as introduced with commit 0c543cdd were
wrong, at least according to the hardware I am testing with (E5 1620).

After this commit and this patch from sephe [1] I can get ecc(4) and
memtemp(4) working on the E5 1600 v3 series:

ecc0: <Intel E5 v3 ECC node0 channel0> [tentative] at device 21.2 on pci255
ecc0: DDR4 ecc0: native DDR
ecc0: DIMM0 8GB, 1x4, density 4GB
ecc0: DIMM0 rank0, corrected error threshold 32767
ecc0: <Intel E5 v3 ECC node0 channel0> [attached!] at device 21.2 on pci255

memtemp0: <Intel E5 v3 memory thermal sensor node0 channel0> [tentative]
           at device 21.0 on pci255
memtemp0: <Intel E5 v3 memory thermal sensor node0 channel0> [attached!]
           at device 21.0 on pci255

[1]: http://leaf.dragonflybsd.org/~sephe/e5_v3_memtemp_ecc.diff

rwhod(8): Fix the warning and raise WARNS back up to 6.

'next' is only initialized and referenced (in separate if()'s) when
'quiet_mode' is not set. Unfortunately gcc does not realize this and
warns about 'next' being maybe uninitialized.

Fix this by initializing 'next', with no functional change.

pppd(8): Include libpcap's config.h for HAVE_* in pcap-int.h.

pflogd(8): Fix the warning and raise WARNS back up to 5.

The issue here is that pcap-int.h (from libpcap) is included without
also including libpcap's config.h. This results in HAVE_STRLCPY not
being set which makes visible the strlcpy() compatibility macro in
pcap-int.h where gcc ultimately warns.

Fix by including libpcap's config.h so the pcap-int.h header knows
that we have strlcpy().

While here, fix a wrong comparison of if_exists()'s result with -1
as if_exists() only returns 0 or 1 (as correctly pointed out by grim7
in the IRC channel).

ix: Fix and simplify link state bit testing logic

Noticed-by: joris@

games: Fix real bugs on three games (found by gcc 5.0)

I had NO_GAMES set so I missed these bugs yesterday.  Adventure, Fortune,
and Mille all had bugs in them  (the latter had two).

Now gcc-5.0 can build the entire world and kernel.
I still need to verify that a gcc-5.0 built world and kernel is bootable
and works though.

ecc/memtemp/e5: Prepare for E5 v3 support

kern ath drivers: Fix loops bounds (real bug x2) + source

The ar9300 source had a couple of logic ambiguities causing
[-Werror=logical-not-parenthesis] warnings.  The other two used the
wrong variable for the upper bounds of the loop.  I assume it was
correct once but the code changed, and the author neglected to use the
next edges max value.

kern: Fix [-Werror=aggressive-loop-optimizations]

These were basic upper bound checking errors on loops

kernel: Fix -Wmaybe-unitialized

Likely there was never a bug here, so explicitly initializing won't
make a difference other than make gcc 5.0 happy.

kern: Fix [-Werror=logical-not-parenthesis]

Since the whole point of this error is that logic order can be
ambiguous, I'm not 100% sure confident in these fixes, but they are
required to get the kernel to build with gcc 5.0.

gcc50: local mod to support kernel printf formats

The diff from gcc47 patched without issues (only line offsets).

kern: Make sure gcc47 - gcc50 don't check for set-not-used

kern: Address set-but-unused warnings (mostly)

On of these is an [-Werror=logical-not-parenthesis] error.
This is as far as I can get without implementing %r format again.

rwhod(8): Lower WARNS to 2 (masks -Wmaybe-uninitialized)

pflogd(8): lower WARNS and fix logic errors

The contrib files trigger a new -Wunused-value warning, so lower WARNS
to 3 to suppress that.  Also, a [logical-not-parenthesis] error was
triggering [-Werror=bool-compare].  Fixing it fixed the compare too.

bsd.sys.mk: Teach it about -Wunused-value (new with gcc4.9)

keyserv(8): Fix size argument on debugging output

This was another size of pointer problem.

truss(1): remove inline directive to fix [-Werror=inline]

GCC 5.0 thinks inlining the usage function will fail and only cause the
program to get bigger.  Minor, but I'll following GCC's advice.

opiekey (contrib): Fix serious (?) [-Wsizeof-pointer-memaccess]

On a password verification function of opiekey, the amount of bytes
allocated to a secret was 4 bytes of i386 and 8 bytes of x86-64.  Memset
was using the size of the pointer to allocate memory.

Change 3 instances of this to size OPIE_SECRET_MAX + 1 to fix it.
FYI this vendor branch is like 16 years old...

sysctl(8): fix [-Werror=logical-not-parentheses] error

rtsol(8): Fix loop bug (aggressive-loop-optimizations)

The index was being checked for validity after the array was accessed,
putting the check first makes gcc 5.0 happy.

newfs_hammer(8): fix [-Wsizeof-pointer-memaccess]

The device type comparison only worked by accident on x86-64 and not
reliability on i386 because the sizeof function returned the size of
the address rather than the size of the vol->type string.  Fix by
specifying the size of "DEVICE" to strncmp instead.

natacontrol(8): Fix [-Werror=logical-not-parentheses] error

devd(8): Lower WARNS from 5 to 2 (has unused local typedefs)

GCC 5.0 has detected local unused typedefs, so lower the warnings so
that compiler can build it.

bsd.sys.mk: support -Wunused-local-typedef

Add support to bsd.sys.mk for a new warning, unused-local-typedef.  This
warning showed up in gcc 4.8 and is included in -Wall.  Hopefully I did
this correctly, but if not, swildner will soon correct it. :)

Fix static archive libstdc++.a for gcc 5.0

The libstdc++ libaries for gcc 5.0 are built differently than they have
been in the past, and closely mirror what the vendor build system does.
Three "convenience" librarys are built first: c++98, c++11, and supc++.
Both static and PIC versions are built.  The final library combines the
three components along with about 10 unique objects.

It worked fine for the shared version (libstd.so.9) but the static
version was unreadable.  I fixed it by creating a dedicated "ar" script
to dictate what gets assembled in the new static archive.  It would
have been a sweet solution has "ar" not had an incredible limitation:
The script can not have consecutive "+" characters on any line.  To
work around this, I had to:
  1) rename the 3 component libraries (all had ++ in the name)
  2) rename the output library (libstdc++.a => libstdcxx.a)
  3) copy all the cc files with "++" in their names to a new filename.
     This affected about 3-4 files.
  4) move the output library back to the proper name of libstdc++.a

What a pain those hacks were, but it works and gcc-5.0 can now build
itself.  It still can't complete world due to Werror trigging on new
warnings, but it's a good fix.

bsd.lib.mk: Allow for possibility to override static lib target

The upcoming fix for gcc 5.0 libstdc++.a requires a dedicated static
library target.  Up until now this hasn't been possible without bmake
screaming about duplicate targets (although it does work).  Add a simple
"if not target defined" check to bsd.lib.mk to avoid these warnings.

Add two more files to contrib/gcc-5.0/README.DELETED

These were removed from the vendor branch to avoid the makefiles
accidentally picking up the generic version instead of the dfly version.
They were libstdc++ source files.

Merge branch 'vendor/GCC50'

Remove two generic configure files for libstdc++

They are superceded by DragonFly-specific files

if_rum - Reapply ratectl fix from 12ae2d804aa2cd27a8e9884f125d484784e76e75.

Hook gcc 5.0 (pre-release) in build, unhook gcc 4.4

Effectively replace GCC 4.4 with the 1 February 2015 snapshot of the
pre-release GCC 5.0.  This compiler is in pretty good shape, but it is
not perfect.  It cannot fully build world if WORLD_CCCVER is set to
GCC50.  This compiler import is a work in progress.

* The old compiler has been unhooked.
* The gcc44 sources may be removed soon
* Obviously NO_GCC44 doesn't work anymore
* NO_GCC50 has been created and should work
* NO_CXX was never documented and has been removed
* NO_OBJC was not deemed worthwhile and it has been removed
* man pages have been updated accordingly
* default make.conf has been updated accordingly
* Per bsd.sys.mk, gcc50 behaves just like gcc47
* Actually, all CCVER starting with "gcc" now behave like gcc47
* The proper CCVER value is "gcc50".  If "gcc5" is specified, the
  dports compiler lang/gcc5 will be used.

Iterating: THIS IS AN EXPERIMENTAL COMPILER!  It's not ready for
prime-time yet.  Do NOT set WORLD_CCVER to "gcc50".

Add README's to new vendor/GCC50 branch and one local mod

Since GCC 5.0 will be the first release to officially support DragonFly,
there are very few local modifications planned, three.  At this time,
I'm only adding the first which defines the default search directories
for gcc libraries and programs.

Merge branch 'vendor/GCC50' - gcc 5.0 snapshot 1 FEB 2015

Import pre-release gcc-5.0 to new vendor branch

gcc50: Bring in makefiles but leave them unhooked

The pre-release version of GCC 5.0 will be brought into base soon.  This
commit brings in the makefiles that build it, but they are not hooked
into the build in any way.  Before that happens, the vendor branch needs
to be merged into base, and dozens for files need to be simultaneously
changed.  GCC 4.4 will be unhooked at the same time.

The GCC releases provide pre-generated man pages, but the pre-releases do
not.  To support this, a switch has been added that installs man pages
from these directories.  When GCC 5.0 is released, those man pages will
be removed and the man pages will be installed from contrib/gcc-5.0