Page MenuHomeFreeBSD
Differential D35215

libsysdecode: Add method for decoding linux signal names
ClosedPublic
Actions

Authored by dchagin on May 15 2022, 4:48 PM.
Tags
None
Referenced Files
F166941970: D35215.id107231.diff
Mon, Aug 17, 10:56 PM
F166910527: D35215.id106480.diff
Mon, Aug 17, 5:20 PM
F166909599: D35215.id107231.diff
Mon, Aug 17, 5:10 PM
F166896992: D35215.id106015.diff
Mon, Aug 17, 2:49 PM
F166878368: D35215.id106015.diff
Mon, Aug 17, 11:18 AM
Unknown Object (File)
Sun, Aug 16, 9:14 PM
Unknown Object (File)
Sun, Aug 16, 8:46 PM
Unknown Object (File)
Sun, Aug 16, 8:15 PM
View All Files
Subscribers
imp

Details

Reviewers
jhb
emaste
Commits
rG1705937d854b: libsysdecode: Add method for decoding Linux signal names
rGf85848fc6175: libsysdecode: Add method for decoding Linux signal names

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 45771
Build 42659: arc lint + arc unit

Event Timeline

dchagin created this revision.May 15 2022, 4:48 PM
Herald added a subscriber: imp. · View Herald TranscriptMay 15 2022, 4:48 PM
dchagin requested review of this revision.May 15 2022, 4:48 PM
Harbormaster completed remote builds in B45584: Diff 106015.May 15 2022, 4:48 PM
dchagin added reviewers: jhb, emaste.May 15 2022, 5:08 PM
dchagin changed the repository for this revision from rS FreeBSD src repository - subversion to rG FreeBSD src repository.
dchagin updated this revision to Diff 106079.May 16 2022, 6:24 PM

Put linux code under ifdef

Harbormaster completed remote builds in B45595: Diff 106079.May 16 2022, 6:24 PM
emaste added a comment.May 16 2022, 6:59 PM

Do you think it's worth #defining a LIBSYSDECODE_LINUXULATOR or such? I presume we'll add riscv and/or ppc at some point.

dchagin added a comment.May 16 2022, 7:10 PM
In D35215#798456, @emaste wrote:

Do you think it's worth #defining a LIBSYSDECODE_LINUXULATOR or such? I presume we'll add riscv and/or ppc at some point.

indeed, I'll put it into the sysdecode.h like
#if defined(amd64) || defined(i386) || defined(aarch64)
#define LINUX_ABI_PRESENT
#endif /* amd64 || i386 || aarch64 */

dchagin updated this revision to Diff 106480.May 30 2022, 5:13 PM

moved to linux.c

Harbormaster completed remote builds in B45771: Diff 106480.May 30 2022, 5:13 PM
dchagin added parent revisions: D35353: libsysdecode: For future use extract common code to a separate files, D35354: libsysdecode: Add preliminary support for decoding Linux syscalls.May 30 2022, 5:24 PM
dchagin added a comment.May 30 2022, 5:59 PM

btw, for SIGRT0 and SIGRT1 I temporarily use glibc names, because it's convenient for me right now, someday it can be replaced

jhb added a comment.May 31 2022, 4:34 PM

I suspect btw you should probably add a sysdecode_linux.3 manpage or the like as part of this series.

jhb accepted this revision.May 31 2022, 4:34 PM
This revision is now accepted and ready to land.May 31 2022, 4:34 PM
Closed by commit rGf85848fc6175: libsysdecode: Add method for decoding Linux signal names (authored by dchagin). · Explain WhyJun 22 2022, 11:41 AM
This revision was automatically updated to reflect the committed changes.
dchagin added a commit: rGf85848fc6175: libsysdecode: Add method for decoding Linux signal names.
dchagin added a commit: rG1705937d854b: libsysdecode: Add method for decoding Linux signal names.Jul 6 2022, 11:05 AM

Revision Contents
Changeset List

PathSize
lib/
libsysdecode/
79 lines
1 line

Diff 106480

View Options

lib/libsysdecode/linux.c

Loading...
View Options

lib/libsysdecode/sysdecode.h

Loading...