Differential D5412

Handle cloudabi in kdump.
ClosedPublic
Actions

Authored by jhb on Feb 23 2016, 8:19 PM.

Details

Reviewers

Commits

rS296042: Add simple support for CloudABI processes to kdump(1).

Summary

Add simple support for CloudABI processes to kdump(1).

This does not decode arguments to system calls but should properly
decode system call names and error return values.

Test Plan

I have no cloudabi binaries to test with, so would appreciate some testing.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb updated this revision to Diff 13660.Feb 23 2016, 8:19 PM

jhb retitled this revision from to Handle cloudabi in kdump..

jhb updated this object.

jhb edited the test plan for this revision. (Show Details)

jhb added a reviewer: ed.

Herald added a subscriber: imp. · View Herald TranscriptFeb 23 2016, 8:19 PM

Hi John,

That change looks all right in my opinion. On this page there are some instructions how to set up a cross compilation toolchain for CloudABI. At the bottom of the page there are even some notes on how to run the entire C library test suite: https://nuxi.nl/doc/freebsd/.

Still, if you want to quickly test this change, here you can find a very simple ELF executable that prints the string "Hello, John!" (SHA256: 8099083e40bef7cef909a47b740e3f37bb625e8093db5f7d7f8e0e0d0ff47411): http://80386.nl/pub/hello-john

This revision is now accepted and ready to land.Feb 24 2016, 8:38 AM

Closed by commit rS296042: Add simple support for CloudABI processes to kdump(1). (authored by jhb). · Explain WhyFeb 26 2016, 11:06 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

usr.bin/

kdump/

kdump.c

7 lines

Diff 13774

View Options

head/usr.bin/kdump/kdump.c

Handle cloudabi in kdump.ClosedPublicActions