6885 CTF Everywhere Part 1
Review Request #1379 — Created Jan. 18, 2019 and submitted
Information | |
---|---|
rm | |
illumos-gate | |
master | |
6885 | |
5931ecc... | |
Reviewers | |
general | |
6885 CTF Everywhere Part 1
6886 Want ctfdiff
6887 ctfdump should be written in terms of libctf
6888 ctfmerge should be implemented in terms of libctf
6889 ctfconvert should be implemented in terms of libctf
6890 Want general workq 6891 Want general mergeq
6892 ctf_add_encoded assigns() incorrect byte size to types
6893 ctf_add_{struct,union,enum} can reuse forwards
6894 ctf_add_{struct,union,enum} occasionally forget to dirty the ctf_file_t
6895 ctf_add_member could better handle bitfields
6896 ctf_type_size() reports wrong size for forwards
6897 Want libctf ctf_kind_name() function
6898 Want libctf function to set struct/union size
Portions contributed by: John Levon <john.levon@joyent.com>
Portions contributed by: Richard Lowe <richlowe@richlowe.net>
Using ctfdump/ctfdiff to compare and contrast older CTF tools, extensive use at Joyent over past several years.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 2 (+14891 -1358) |
-
Format string mismatches, etc.
-
usr/src/cmd/ctfdiff/ctfdiff.c (Diff revision 1) Type of
iidx
is ulong_t, so please use%lu
conversion specifier. -
-
-
-
usr/src/cmd/ctfdump/ctfdump.c (Diff revision 1) These lines should be:
len = snprintf(NULL, 0, " [%lu] %ld", g_stats.cs_ndata, id); ctfdump_printf(CTFDUMP_OBJECTS, " [%lu] %ld %*s%s (%lu)\n",
-
usr/src/cmd/ctfdump/ctfdump.c (Diff revision 1) " [%lu] %s (%lu) returns: %ld args: (", g_stats.cs_nfuncs, name,
-
usr/src/cmd/ctfdump/ctfdump.c (Diff revision 1) ctfdump_printf(CTFDUMP_FUNCTIONS, "%ld%s", g_fargc[i],
-
usr/src/cmd/ctfdump/ctfdump.c (Diff revision 1) ctfdump_printf(CTFDUMP_LABELS, " %5ld %s\n", li->ctb_typeidx, name);
-
usr/src/cmd/ctfdump/ctfdump.c (Diff revision 1) ctfdump_printf(CTFDUMP_TYPES, "\t%s type=%ld off=%lu\n", member, type,
-
-
-
-
usr/src/cmd/ctfdump/ctfdump.c (Diff revision 1) ctfdump_printf(CTFDUMP_TYPES, "%s refers to %ld", name,
-
usr/src/cmd/ctfdump/ctfdump.c (Diff revision 1) ctfdump_printf(CTFDUMP_TYPES, "%s contents: %ld, index: %ld",
-
-
-
usr/src/cmd/ctfdump/ctfdump.c (Diff revision 1) ctfdump_printf(CTFDUMP_TYPES, "%s (%zu bytes)\n", name, size);
-
usr/src/cmd/ctfdump/ctfdump.c (Diff revision 1) ctfdump_printf(CTFDUMP_TYPES, "typedef %s refers to %ld", name,
-
usr/src/cmd/ctfdump/ctfdump.c (Diff revision 1) ctfdump_printf(CTFDUMP_TYPES, "%s refers to %ld", name,
-
usr/src/cmd/ctfdump/ctfdump.c (Diff revision 1) ctfdump_printf(CTFDUMP_TYPES, "%s refers to %ld", name,
-
usr/src/cmd/ctfdump/ctfdump.c (Diff revision 1) ctfdump_printf(CTFDUMP_TYPES, "%s refers to %ld", name,
-
-
-
-
usr/src/cmd/ctfmerge/ctfmerge.c (Diff revision 1) ctfmerge_fatal("failed to read symbol table entry %lu "
-
usr/src/cmd/ctfmerge/ctfmerge.c (Diff revision 1) ctfmerge_fatal("failed to set parallelism to %u: %s\n",
-
usr/src/common/ctf/ctf_create.c (Diff revision 1) ctf_dprintf("lbloff: %u\nobjtoff: %u\nfuncoff: %u\n" "typeoff: %u\nstroff: %u\nstrlen: %u\n",
-
usr/src/common/ctf/ctf_create.c (Diff revision 1) If cast to
int
is unnecessary, then these lines could be:ctf_dprintf("bad contents for array: %ld\n", arp->ctr_contents);
-
usr/src/common/ctf/ctf_create.c (Diff revision 1) ctf_dprintf("bad index for array: %ld\n", arp->ctr_index);
-
-
-
usr/src/lib/libctf/common/ctf_dwarf.c (Diff revision 1) See in /usr/src/lib/libdwarf/common/libdwarf.h:
/* To enable printing with printf regardless of the
actual underlying data type, we define the DW_PR_xxx macros. */ctf_dprintf("dwmap: %p %" DW_PR_DUx "->%ld\n", dwmap, off, id);
-
usr/src/lib/libctf/common/ctf_dwarf.c (Diff revision 1) "failed to get die from offset %" DW_PR_DUu ": %s\n",
-
usr/src/lib/libctf/common/ctf_dwarf.c (Diff revision 1) "unknown ELF data encoding: %u", ehdr.e_ident[EI_DATA]);
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 3 (+14866 -1362) |
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 4 (+14867 -1362) |
Change Summary:
Rebase, etc.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 5 (+14867 -1362) |
Change Summary:
Sorry, got the wires crossed slightly.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 6 (+14867 -1362) |