Skip to content

nati_x86_64_defconfig: Enable CORE_DUMP_DEFAULT_ELF_HEADERS#279

Merged
gratian merged 1 commit into
ni:nilrt/master/6.12from
chaitu236:dev/cvadrevu/612_CORE_DUMP_DEFAULT_ELF_HEADERS
Jun 18, 2026
Merged

nati_x86_64_defconfig: Enable CORE_DUMP_DEFAULT_ELF_HEADERS#279
gratian merged 1 commit into
ni:nilrt/master/6.12from
chaitu236:dev/cvadrevu/612_CORE_DUMP_DEFAULT_ELF_HEADERS

Conversation

@chaitu236

@chaitu236 chaitu236 commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Enable CORE_DUMP_DEFAULT_ELF_HEADERS so core files contain build-id which is useful during debugging.

It is possible to configure this dynamically this seems like a useful setting so thought it'd be better to bake into kernel.

Testing

  • Built kernel
  • Booted on an VM and verified that core files generated with gcore <pid> contain build-ids.
# eu-unstrip -n --core core.2131
0x400000+0xc000 ee19d71a236f5eef544d31932c4b493a9c6a9bd9@0x400378 . - /bin/sleep.coreutils
0x7f5665b7f000+0x2000 f44a0666d2349d5fbd935f441399e7654b858dd2@0x7f5665b7f570 . - linux-vdso.so.1
0x7f5665b81000+0x365d8 0d169f0bcd80370ed631ac306e3f791f944b9e9d@0x7f5665b812f8 /lib/ld-linux-x86-64.so.2 - ld-linux-x86-64.so.2
0x7f566598e000+0x1e5db0 4e6ba54d4a1fa41a8c20234938fff8fe8cf549cb@0x7f566598e3a0 /lib/libc.so.6 - libc.so.6

Without this change, this is what it looked like

# eu-unstrip -n --core core.32613
0x7f85fd796000+0x2000 5fbfd551ef8c8e65f543acbd558a8b372a31a485@0x7f85fd796588 . - linux-vdso.so

Note

I noticed about 2MB increase in core file size.
core file for /bin/sleep increased from 420KB to 2.5MB and for niauth_daemon, it went from 388MB to 390MB.

bzImage size increased from 7087104 bytes to 7095296 bytes (about 8KB).

I'll create PRs for 6.18 and 4.14 if this change is acceptable.

Enable CORE_DUMP_DEFAULT_ELF_HEADERS so core files contain build-id
which is useful during debugging.

Signed-off-by: Chaitanya Vadrevu <chaitanya.vadrevu@emerson.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the nati_x86_64_defconfig to enable CORE_DUMP_DEFAULT_ELF_HEADERS so generated ELF core dumps include partial ELF headers (including build-id information), improving post-mortem/debug symbol resolution.

Changes:

  • Removes the explicit # CONFIG_CORE_DUMP_DEFAULT_ELF_HEADERS is not set line from the defconfig, allowing the Kconfig default (default y when BINFMT_ELF && ELF_CORE) to take effect.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@gratian gratian merged commit 28f912c into ni:nilrt/master/6.12 Jun 18, 2026
1 check passed
@chaitu236 chaitu236 deleted the dev/cvadrevu/612_CORE_DUMP_DEFAULT_ELF_HEADERS branch June 18, 2026 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants