KNF #includes

Remove some unnecessary ones while I'm here.

Trailing whitespace

Clean-up unnecessary inclusions of opt_coredump.h

Merge the [pgoyette-compat] branch

branches: 1.7.16; 1.7.18;
We never exec(2) with a kernel vmspace, so do not test for that, but instead
KASSERT() that we don't.
When calculating the load address for the interpreter (e.g. ld.elf_so),
we need to take into account wether the exec'd process will run with
topdown memory or bottom up. We can not use the current vmspace's flags
to test for that, as this happens too early. Luckily the execpack already
knows what the new state will be later, so instead of testing the current
vmspace, pass the info as additional argument to struct emul
e_vm_default_addr.
Fix all such functions and adopt all callers.

branches: 1.6.4;
Fill in netbsd32_vm_default_addr, and provide

- compat_13_netbsd32_sigreturn, and
- compat_16_netbsd32___sigreturn14

branches: 1.5.2; 1.5.4;
Simplify netbsd32_vm_default_addr (for now) to make it compilable.

whitespace

use the topdown hint.

Change the type of the 'cookie' that holds the state of the core dump file
from 'void *' to the actual type 'struct coredump_iostate *'.
In most of the code the contents of the structure are still unknown.
This just stops the wrong type of pointer being passed to the 'void *'
parameter.
I hope I've found everything, amd64 GENERIC and i386 GENERIC & ALL compile.

branches: 1.1.2; 1.1.4; 1.1.6; 1.1.8;
COMPAT_NETBSD32 support for ARM. Which is strange since ARM is already
32-bits. But the newer ARM ABI AAPCS changes the alignment of 64-bit
fields so structures need to copied in and out to deal with the alignment
change. This is a kludge but makes debugging of AAPCS support much easier.