• avoid #if

Address review's comments

Nice, it looks good to me.

In D23974#528509, @erj wrote:

I'm generally okay with this, but with the caveat that we can't test to make sure any future changes we make will work on anything other than amd64.

In D23974#527044, @erj wrote:

The driver is only intended to work on amd64; I can accept PowerPC64 but we don't want to support this driver on 32-bit architectures.

• Make ixl device available only on amd64 and PowerPC64
• Enable ixl by default on PowerPC64

LGTM

Address review's comments

In D23668#525565, @emaste wrote:

Yes, the similar path in aac.c has

			aac_remove_busy(cm);
 			aac_unmap_command(cm);

• enable aacraid module for powerpc
• add aacraid_be.c in aacraid module's src list

Replaced by D23844.

Replaced by D23844.

Looks good.
I just have some style related comments and 2 questions about the changes.

Address review's comments

Add man page for kvm_kerndisp

Adding Renato Riolino, as he is investigating failing PowerPC tests and this fixes some of them.

Nice, thanks for debugging this issue and documenting psl_userstatic / psl_userset.

• Address review comments

Address review comments

LGTM.
I've tested this by applying the QEMU patch over version 4.2.0 and everything worked fine.

Align src in bcopy/memmove

Fix ifunc resolver parameter name.

Fix ifunc resolver parameter name.

Fix diff, bypassing wrong arcanist copy detection.

• [PPC64] strcpy optimization
• Fix ifunc resolver parameter name
• Merge branch 'master' into luporl-strcpy

About the tests part, I've been using this patch for a couple months now (actually, "RTLD WIP3"), along with ELFv2 and libc optimization patches (D15118, D15368, D15369), and I have observed no issues so far.

In D22767#498741, @cem wrote:

There is some precedent for putting MD gdb code paths in, e.g., sys/powerpc/powerpc/gdb_machdep.c. The naming convention appears to be something like, gdb_cpu_do_offsets(). If you want to put the current gdb_do_offses code there, that would be cool. I would be fine leaving #ifdef __powerpc__ around the qOffsets handler and not implementing it everywhere, given it doesn't seem to be needed on other platforms. It is also fine with me to gdb_do_offset here, until/unless a second arch shows up needing to provide another implementation.

Move implementation of qOffsets to gdb_machdep.c

-Fix qOffsets for 32-bit PowerPC

In D22761#498280, @jhb wrote:

The kgdb bits actually live in github at https://github.com/bsdjhb/gdb/tree/kgdb-8.3 If it's not too hard to make a pull request against that, that would be nicer.

The other thing to consider after that is changing it to be cross-debug friendly. I can help with that. It means not using struct pcb but using a table describing the offsets of the registers from the pcb address.

For remote targets, symbol resolution can be fixed by implementing qOffsets packet handling (D22767).

In D21946#479486, @jhb wrote:

The relocation needs to be done a different way to support, e.g. remote debugging (e.g. attaching gdb to qemu's debug stub). Also, kgdb_main.c is just a simpler wrapper, and 'target vmcore /path/to/vmcore' in plain gdb should also work. I'll have to think about how this might be doable, but this approach is rather hacky. It also won't work for cross-debugging (examining a ppc vmcore on a non-ppc host which kgdb generally supports for other architectures). For PIE executables gdb relies on using the ELF auxv values to find the relocated binary.