mirrors/flatcar-scripts
1
0
Fork 0
mirror of https://github.com/flatcar/scripts.git synced 2025-09-24 23:21:17 +02:00
Code Issues Packages Projects Releases Wiki Activity

scripts: fall back to linkage symbols when debug info not present

Browse Source 
dump_syms bails if you give it a debug info path and the file there
does not have debug info, instead of falling back to dumping
public/linkage symbols (and more importantly, call frame info).
Give it the chance to redeem itself.

Also, instead of not ever uploading CFI for x86 modules, strip it
for any architecture but only when the file is above some
threshold.

BUG=chromium-os:22373 chromium-os:22741

Change-Id: Iad6981efec537868296a6713b1d9ca0cdb750d28
Reviewed-on: https://gerrit.chromium.org/gerrit/11443
Commit-Ready: Ken Mixter <kmixter@chromium.org>
Reviewed-by: Ken Mixter <kmixter@chromium.org>
Tested-by: Ken Mixter <kmixter@chromium.org>
This commit is contained in:
Ken Mixter 2011-11-09 23:51:27 +00:00 committed by Gerrit
parent 99fb742e5a
commit d8f9130863
2 changed files with 21 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
cros_generate_breakpad_symbols
View File

@ -1,5 +1,5 @@
#!/bin/bash
# Copyright (c) 2010 The Chromium OS Authors. All rights reserved.
# Copyright (c) 2011 The Chromium OS Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
@ -101,16 +101,22 @@ function dump_file() {
# Dump symbols as root in order to read all files.
if ! sudo "${DUMP_SYMS}" "${text_file}" "${debug_directory}" > "${SYM_FILE}" \
2> "${ERR_FILE}"; then
# A lot of files (like kernel files) contain no debug information, do
# not consider such occurrences as errors.
if grep -q "file contains no debugging information" "${ERR_FILE}"; then
warn "No symbols found for ${text_file}"
# Try dumping just the main file to get public-only symbols.
if ! sudo "${DUMP_SYMS}" "${text_file}" > "${SYM_FILE}" 2> "${ERR_FILE}";
then
# A lot of files (like kernel files) contain no debug information, do
# not consider such occurrences as errors.
if grep -q "file contains no debugging information" "${ERR_FILE}"; then
warn "No symbols found for ${text_file}"
return 1
fi
error "Unable to dump symbols for ${text_file}:"
cat "${ERR_FILE}"
ANY_ERRORS=1
return 1
else
warn "File ${text_file} did not have debug info, using linkage symbols"
fi
error "Unable to dump symbols for ${text_file}:"
cat "${ERR_FILE}"
ANY_ERRORS=1
return 1
fi
local file_id=$(head -1 ${SYM_FILE} | cut -d' ' -f4)
local module_name=$(head -1 ${SYM_FILE} | cut -d' ' -f5)

11
upload_symbols
View File

@ -39,7 +39,7 @@ DEFINE_string board "$DEFAULT_BOARD" "The board to build packages for."
DEFINE_string breakpad_root "" "Root directory for breakpad symbols."
DEFINE_boolean official_build ${FLAGS_FALSE} "Point to official symbol server."
DEFINE_boolean regenerate ${FLAGS_FALSE} "Regenerate all symbols."
DEFINE_boolean strip_x86_cfi ${FLAGS_TRUE} "Strip CFI data for x86."
DEFINE_integer strip_cfi 100000000 "Strip CFI data for files above this size."
DEFINE_boolean verbose ${FLAGS_FALSE} "Be verbose."
DEFINE_boolean yes ${FLAGS_FALSE} "Answer yes to all prompts."
@ -76,16 +76,17 @@ function upload_file() {
local symbol_file="$1"
local upload_url="$2"
local upload_file="${symbol_file}"
if [ ${FLAGS_strip_x86_cfi} -eq ${FLAGS_TRUE} ]; then
local arch=$(portageq-${FLAGS_board} envvar ARCH)
if [ "${arch}" == "x86" ]; then
if [ ${FLAGS_strip_cfi} -ne 0 ]; then
local symbol_size="$(stat -c%s ${symbol_file})"
if [ ${symbol_size} -gt ${FLAGS_strip_cfi} ]; then
# Call frame info is unnecessary for 32b x86 targets with frame
# pointer used (as all of ours have) and it accounts for over
# half the size of the symbols uploaded. To buy us more time
# for the server team to increase the current 160MB, remove this
# data unless the user requests it.
if [ ${FLAGS_verbose} -eq ${FLAGS_TRUE} ]; then
info "Stripping CFI for ${symbol_file} due to board ${FLAGS_board}."
info "Stripping CFI for ${symbol_file} due to size ${symbol_size} > \
${FLAGS_strip_cfi}."
fi
upload_file="${OUT_DIR}/stripped.sym"
sed '/^STACK CFI/d' < "${symbol_file}" > "${upload_file}"