22:18 John Baldwin (jhb) 

devel/gdb: Upgrade to 14.1

- Notable upstream fixes for FreeBSD include better support for
  multiprocessing as well as support for AVX and other extended
  registers in core dumps on newer AMD CPUs.

- Use all-gdb as the build target to reduce build time.

- sim can no longer be excluded during the tarball as a couple of
  source files in GDB now use headers from sim/ unconditionally.
  Instead, use --disable-sim to skip building the simulators.

Reviewed by:	pizzamig
Sponsored by:	DARPA
Differential Revision:	https://reviews.freebsd.org/D43053

    af0d94a

20:46 John Baldwin (jhb) 

devel/gdb: Update to 11.1.

This also pulls in some updates to libcxx-gdbpy to add pretty printers
for std::deque<>, std::stack<>, and std::unordered_map<>

Reviewed by:	pizzamig (maintainer)
Differential Revision:	https://reviews.freebsd.org/D32000

    242d369

14:31 Mathieu Arnold (mat) 

all: Remove all other $FreeBSD keywords.

    135fdee

17:23 jhb 

Upgrade to 8.2.

- Remove patches for upstream commits in 8.2.
- Add an upstream patch to include NT_PROCSTAT_AUXV, NT_PROCSTAT_PS_STRINGS,
  and NT_PROCSTAT_VMMAP notes in core dumps generated by 'gcore'.
- Update kgdb for changes in 8.2.
- Add 'USES=gettext-runtime'

Reviewed by:	pizzamig (maintainer)
Differential Revision:	https://reviews.freebsd.org/D17085

 

17:28 jhb 

Fixes for kgdb including initial FreeBSD/arm support.

- Improve fallback handling for the read of pcb_size.

  If for some reason kgdb has only the kernel's debug info, a read of
  pcb_size will simply return 0, resulting in an exception when unwinding
  the stack of a thread that was on a CPU at the time the dump was
  created.  In this case, fall back to using sizeof(struct pcb).
- Always supply a value for the PC register on x86.

  A thread's PCB may have been swapped out, and a dummy value ensures that
  we don't get an exception while enumerating threads in a dump.
- Mark all 64-bit address bits as valid for aarch64 kernels.

  ARMv8 supports pointer tagging in which case the upper 8 bits of VAs are
  ignored, but FreeBSD kernels are linked such that they depend on those
  upper 8 bits being set to 1.  GDB was stripping those bits by default
  causing KVAs to be mapped to invalid addresses.
- Rework kgdb -w support to only open /dev/mem writable.

  Don't depend on the gdb global 'write_files' option which tries to
  rewrite the kernel binary on close which can crash.  Instead, add an
  optional '-w' flag to 'target vmcore' which opens the vmcore with
  O_RDWR instead of O_RDONLY.  Change the kgdb '-w' option to set this
  flag in the nested 'target vmcore' command rather than passing the
  global '-w' option to gdb.
- Garbage collect old code for reading dumptid.
- First cut at FreeBSD/arm kernel support.

  Tested on live kernel on my RPi, but not against a crashdump, and only
  trapframes from userland, not a nested trap.

Submitted by:	markj (1, 2)
Approved by:	kan
Differential Revision:	https://reviews.freebsd.org/D16013

 

14:33 pizzamig 

devel/gdb: Update to version 8.1

The official annoucement is available here:
https://www.gnu.org/software/gdb/download/ANNOUNCEMENT

Moreover:
* support for 'info proc' is added (jhb@)
* update kgdb to use gdb 8.1

Reviewed by:	jhb@
Differential Revision:	https://reviews.freebsd.org/D14148

 

14:49 pizzamig 

devel/gdb: Update to gdb 8.0.1

Resolve tons of warnings.
Fix the type of literal string as char const *
Fix Makefile's variables order and adding the license file
Rework patches to make portlint happy

Reviewed by:	jhb
Approved by:	lme (mentor)
Differential Revision:	https://reviews.freebsd.org/D12525

 

16:08 tijl 

Update to 7.12.

PR:		214927
Submitted by:	luca.pizzamiglio@gmail.com (maintainer)

 

17:36 jhb 

Various fixes to kgdb.  Port revision bumped since kgdb is enabled by
default.
- kgdb -n now permits any valid string to be used instead of only
  permitting numbers.  In particular, this allows 'kgdb -n last' to be
  used to open the most recent vmcore.
- kgdb will now try to determine the list of kernel modules even if the
  kernel binary does not include debug symbols.  This works fine in kernels
  that have the changes in r290728.
- Mark trapframes as "signal trampoline frames".  GDB assumes that "normal"
  frames will never call into a NULL PC.  Instead, it assumes that calling a
  NULL PC will result in an exception (and thus a signal being posted
  resulting in a signal frame).  A trap for a NULL function pointer would
  thus stop unwinding once it hit the frame with a NULL PC.  Marking the
  trapframes as a signal frame tells GDB it is ok to unwind past a NULL PC.
  One side effect is that frames in the asm handler now display as "signal
  handler called" instead of the raw line in assembly.  Perhaps at some
  point it would be nice to mark these up the way ddb does with the trap
  number, etc. but GDB's stack code doesn't support custom frame printers.

PR:		206044
Reviewed by:	luca.pizzamiglio@gmail.com (maintainer)
Approved by:	koobs

 

18:52 jhb 

Add a new KGDB option to the devel/gdb port.  This adds a forward port
of the kernel-specific bits of kgdb to recent gdb.  It only supports
amd64, i386, powerpc, powerpc64, and sparc64.

PR:		203299
Reviewed by:	bdrewery
Glanced at by:	emaste, jilles, luca.pizzamiglio@gmail.com (maintainer)
Differential Revision:	https://reviews.freebsd.org/D3727