linux-efi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ard Biesheuvel <ard.biesheuvel@linaro.org>
To: linux-efi@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	matt@codeblueprint.co.uk, mark.rutland@arm.com,
	will.deacon@arm.com, catalin.marinas@arm.com
Cc: leif.lindholm@linaro.org, Ard Biesheuvel <ard.biesheuvel@linaro.org>
Subject: [PATCH v3 0/2] efi/arm64: add vmlinux link to PE/COFF debug table
Date: Thu, 26 Jan 2017 19:24:44 +0000	[thread overview]
Message-ID: <1485458686-32544-1-git-send-email-ard.biesheuvel@linaro.org> (raw)

Much like GNU debug links, PE/COFF binaries can carry the path on the build
host to the binary containing the debugging symbols. Since the kernel binary
is loaded by UEFI at an a priori unknown offset in the 1:1 mapping of physical
memory, having this information is useful for the debugger automation to find
the file and the offset, and load the symbols automatically.

So if we have debugging symbols to begin with (CONFIG_DEBUG_INFO=y), add the
absolute path to vmlinux to the PE/COFF debug table.

v3: move the EFI debug table itself into the EFI header, which removes the
    need for externally visible symbols or changes to the linker script (#2)
    update commit log to explain that the debug support table is defined
    in the UEFI spec, and is thus not EDK2/PI specific

v2: rebase onto v4.10-rc
    use strip rather than objcopy for first pass (#1)
    move debug table inside #ifdef CONFIG_EFI region in head.S (#2)

Note that these patches can be merged independently, although they are
both required to get the functionality.

Ard Biesheuvel (2):
  efi: libstub: Preserve .debug sections after absolute relocation check
  efi: arm64: Add vmlinux debug link to the Image binary

 arch/arm64/kernel/Makefile            |  4 ++
 arch/arm64/kernel/head.S              | 46 +++++++++++++++++++-
 drivers/firmware/efi/libstub/Makefile | 24 ++++++----
 3 files changed, 65 insertions(+), 9 deletions(-)

-- 
2.7.4

             reply	other threads:[~2017-01-26 19:24 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-26 19:24 Ard Biesheuvel [this message]
2017-01-26 19:24 ` [PATCH v3 1/2] efi: libstub: Preserve .debug sections after absolute relocation check Ard Biesheuvel
     [not found] ` <1485458686-32544-1-git-send-email-ard.biesheuvel-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2017-01-26 19:24   ` [PATCH v3 2/2] efi: arm64: Add vmlinux debug link to the Image binary Ard Biesheuvel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1485458686-32544-1-git-send-email-ard.biesheuvel@linaro.org \
    --to=ard.biesheuvel@linaro.org \
    --cc=catalin.marinas@arm.com \
    --cc=leif.lindholm@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-efi@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=matt@codeblueprint.co.uk \
    --cc=will.deacon@arm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).