Back to build
Raw log | Switch to full mode | Login

python -u /b/s/w/ir/kitchen-checkout/depot_tools/recipes/recipe_modules/bot_update/resources/bot_update.py --spec-path /b/s/w/ir/tmp/t/tmpIDM2XU --patch_root boringssl --revision_mapping_file /b/s/w/ir/tmp/t/tmpKWKGxR.json --git-cache-dir /b/s/w/ir/cache/git --cleanup-dir /b/s/w/ir/k/recipe_cleanup/bot_update --output_json /b/s/w/ir/tmp/t/tmp2Btmw1.json --patch_ref https://boringssl.googlesource.com/boringssl@refs/changes/25/34925/1 --revision boringssl@HEAD in dir /b/s/w/ir/k: allow_subannotations: False base_name: 'bot_update' cmd: ['python', '-u', '/b/s/w/ir/kitchen-checkout/depot_tools/recipes/recipe_modules/bot_update/resources/bot_update.py', '--spec-path', '/b/s/w/ir/tmp/t/tmpIDM2XU', '--patch_root', 'boringssl', '--revision_mapping_file', '/b/s/w/ir/tmp/t/tmpKWKGxR.json', '--git-cache-dir', '/b/s/w/ir/cache/git', '--cleanup-dir', '/b/s/w/ir/k/recipe_cleanup/bot_update', '--output_json', '/b/s/w/ir/tmp/t/tmp2Btmw1.json', '--patch_ref', 'https://boringssl.googlesource.com/boringssl@refs/changes/25/34925/1', '--revision', 'boringssl@HEAD'] env: {} env_prefixes: {} env_suffixes: {'PATH': '/b/s/w/ir/kitchen-checkout/depot_tools'} infra_step: True name: 'bot_update' nest_level: 0 ok_ret: frozenset([0, 87, 88]) step_test_data: <lambda>(...) trigger_specs: () full environment: BOTO_CONFIG: /b/s/w/ir/tmp/gsutil_task/.boto BUILDBUCKET_EXPERIMENTAL: FALSE CIPD_CACHE_DIR: /b/s/cipd_cache/cache CIPD_PROTOCOL: v2 DEVSHELL_CLIENT_PORT: 46061 DOCKER_CONFIG: /b/s/w/ir/tmp/docker_cfg_task DOCKER_TMPDIR: /b/s/w/ir/tmp/docker_tmp_task GIT_CONFIG_NOSYSTEM: 1 GIT_TERMINAL_PROMPT: 0 HOME: /home/chrome-bot INFRA_GIT_WRAPPER_HOME: /b/s/w/ir/tmp/git_home_task LOGDOG_COORDINATOR_HOST: logs.chromium.org LOGDOG_STREAM_PREFIX: buildbucket/cr-buildbucket.appspot.com/8920957154349143952 LOGDOG_STREAM_PROJECT: boringssl LOGDOG_STREAM_SERVER_PATH: unix:/b/s/w/ir/tmp/ld.sock LOGNAME: chrome-bot LUCI_CONTEXT: /b/s/w/itOTeUPn/luci_context.200088045 MAC_CHROMIUM_TMPDIR: /b/s/w/ir/tmp/t MAIL: /var/mail/chrome-bot NO_GCE_CHECK: False PATH: /b/s/w/ir/cipd_bin_packages:/b/s/w/ir/cipd_bin_packages/bin:/b/s/cipd_cache/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/snap/bin:/b/s/w/ir/kitchen-checkout/depot_tools PWD: /b/s/w/ir/k PYTHONIOENCODING: UTF-8 PYTHONUNBUFFERED: 1 SHELL: /bin/bash SHLVL: 1 SUDO_COMMAND: /usr/bin/python /b/s/swarming_bot.zip start_bot SUDO_GID: 0 SUDO_UID: 0 SUDO_USER: root SWARMING_BOT_ID: gce-trusty-e833d7b0-us-east1-b-6s9p SWARMING_EXTERNAL_BOT_SETUP: 1 SWARMING_HEADLESS: 1 SWARMING_SERVER: https://chromium-swarm.appspot.com SWARMING_TASK_ID: 43264bc0670c4d11 TEMP: /b/s/w/ir/tmp/t TEMPDIR: /b/s/w/ir/tmp/t TERM: linux TMP: /b/s/w/ir/tmp/t TMPDIR: /b/s/w/ir/tmp/t USER: chrome-bot USERNAME: chrome-bot VPYTHON_VIRTUALENV_ROOT: /b/s/w/ir/cache/vpython _: /b/s/w/ir/cipd_bin_packages/vpython Debugging info: CURRENT_DIR: '/b/s/w/ir/k' DEPOT_TOOLS_DIR: '/b/s/w/ir/kitchen-checkout/depot_tools' THIS_DIR: '/b/s/w/ir/kitchen-checkout/depot_tools/recipes/recipe_modules/bot_update/resources' Gclient Solutions ================= boringssl (https://boringssl.googlesource.com/boringssl.git) Dependencies file is util/bot/DEPS Managed mode is OFF Custom Variables: checkout_clang = True ===Running /b/s/w/ir/cipd_bin_packages/bin/python -u /b/s/w/ir/kitchen-checkout/depot_tools/gclient.py recurse -v git update-index --refresh === In directory: /b/s/w/ir/k WARNING: Your metrics.cfg file was invalid or nonexistent. A new one will be created. Can't find .gclient_entries You need to run gclient sync at least once to use 'recurse'. This is because .gclient_entries needs to exist and be up to date. ===Failed in 0.0 mins of /b/s/w/ir/cipd_bin_packages/bin/python -u /b/s/w/ir/kitchen-checkout/depot_tools/gclient.py recurse -v git update-index --refresh === WARNING: Failed to update git indices. Revisions: ['boringssl@HEAD'] Fetching Git checkout at boringssl@HEAD Using Python version: 2.7.14+chromium14 (df075345c7-dirty:df075345c7, Oct 3 2017, 17:29:00) [GCC 4.8.4] Checking git version... ===Running git version === In directory: /b/s/w/ir/k git version 2.17.1.chromium15 / Infra wrapper (infra/tools/git/linux-amd64 @ Ekj2vguJrYM9lu187ZJAw2m8VRwjNNH_QfZgu6h5cRMC) ===Succeeded in 0.0 mins of git version === Using git version 2.17.1.chromium15 / Infra wrapper (infra/tools/git/linux-amd64 @ Ekj2vguJrYM9lu187ZJAw2m8VRwjNNH_QfZgu6h5cRMC) Fetching Git checkout ===Running /b/s/w/ir/cipd_bin_packages/bin/python -u /b/s/w/ir/kitchen-checkout/depot_tools/git_cache.py populate --ignore_locks -v --cache-dir /b/s/w/ir/cache/git https://boringssl.googlesource.com/boringssl.git --reset-fetch-config === In directory: /b/s/w/ir/k /b/s/w/ir/cache/git/boringssl.googlesource.com-boringssl has 1 .pack files, re-bootstrapping if >50 running "git config --unset-all remote.origin.fetch" in "/b/s/w/ir/cache/git/boringssl.googlesource.com-boringssl" running "git config gc.autodetach 0" in "/b/s/w/ir/cache/git/boringssl.googlesource.com-boringssl" running "git config core.deltaBaseCacheLimit 2g" in "/b/s/w/ir/cache/git/boringssl.googlesource.com-boringssl" running "git config remote.origin.url https://boringssl.googlesource.com/boringssl.git" in "/b/s/w/ir/cache/git/boringssl.googlesource.com-boringssl" running "git config --replace-all remote.origin.fetch +refs/heads/*:refs/heads/* \+refs/heads/\*:.*" in "/b/s/w/ir/cache/git/boringssl.googlesource.com-boringssl" Fetching +refs/heads/*:refs/heads/* running "git fetch -v --progress origin +refs/heads/*:refs/heads/*" in "/b/s/w/ir/cache/git/boringssl.googlesource.com-boringssl" From https://boringssl.googlesource.com/a/boringssl = [up to date] 2214 -> 2214 = [up to date] 2272 -> 2272 = [up to date] 2311 -> 2311 = [up to date] 2357 -> 2357 = [up to date] 2490 -> 2490 = [up to date] 2564 -> 2564 = [up to date] 2623 -> 2623 = [up to date] 2661 -> 2661 = [up to date] 2704 -> 2704 = [up to date] 2883 -> 2883 = [up to date] 2924 -> 2924 = [up to date] 2987 -> 2987 = [up to date] 3029 -> 3029 = [up to date] 3071 -> 3071 = [up to date] 3112 -> 3112 = [up to date] 3202 -> 3202 = [up to date] 3239 -> 3239 = [up to date] 3282 -> 3282 = [up to date] 3359 -> 3359 = [up to date] 3538 -> 3538 = [up to date] chromium-stable -> chromium-stable = [up to date] chromium-stable-with-bazel -> chromium-stable-with-bazel = [up to date] fips-20180730 -> fips-20180730 = [up to date] infra/config -> infra/config = [up to date] master -> master = [up to date] master-with-bazel -> master-with-bazel fetch +refs/heads/*:refs/heads/* took 0.0 minutes ===Succeeded in 0.0 mins of /b/s/w/ir/cipd_bin_packages/bin/python -u /b/s/w/ir/kitchen-checkout/depot_tools/git_cache.py populate --ignore_locks -v --cache-dir /b/s/w/ir/cache/git https://boringssl.googlesource.com/boringssl.git --reset-fetch-config === ===Running /b/s/w/ir/cipd_bin_packages/bin/python -u /b/s/w/ir/kitchen-checkout/depot_tools/git_cache.py exists --quiet --cache-dir /b/s/w/ir/cache/git https://boringssl.googlesource.com/boringssl.git === In directory: /b/s/w/ir/k /b/s/w/ir/cache/git/boringssl.googlesource.com-boringssl ===Succeeded in 0.0 mins of /b/s/w/ir/cipd_bin_packages/bin/python -u /b/s/w/ir/kitchen-checkout/depot_tools/git_cache.py exists --quiet --cache-dir /b/s/w/ir/cache/git https://boringssl.googlesource.com/boringssl.git === ===Running git clone --no-checkout --local --shared /b/s/w/ir/cache/git/boringssl.googlesource.com-boringssl /b/s/w/ir/k/boringssl === In directory: /b/s/w/ir/k Cloning into '/b/s/w/ir/k/boringssl'... done. ===Succeeded in 0.0 mins of git clone --no-checkout --local --shared /b/s/w/ir/cache/git/boringssl.googlesource.com-boringssl /b/s/w/ir/k/boringssl === ===Running git config gc.auto 0 === In directory: /b/s/w/ir/k/boringssl ===Succeeded in 0.0 mins of git config gc.auto 0 === ===Running git config gc.autodetach 0 === In directory: /b/s/w/ir/k/boringssl ===Succeeded in 0.0 mins of git config gc.autodetach 0 === ===Running git config gc.autopacklimit 0 === In directory: /b/s/w/ir/k/boringssl ===Succeeded in 0.0 mins of git config gc.autopacklimit 0 === ===Running git remote set-url --push origin https://boringssl.googlesource.com/boringssl.git === In directory: /b/s/w/ir/k/boringssl ===Succeeded in 0.0 mins of git remote set-url --push origin https://boringssl.googlesource.com/boringssl.git === ===Running git checkout --force origin/HEAD -- === In directory: /b/s/w/ir/k/boringssl Note: checking out 'origin/HEAD'. You are in 'detached HEAD' state. You can look around, make experimental changes and commit them, and you can discard any commits you make in this state without impacting any branches by performing another checkout. If you want to create a new branch to retain commits you create, you may do so (now or later) by using -b with the checkout command again. Example: git checkout -b <new-branch-name> HEAD is now at 104306f58 Remove STRICT_ALIGNMENT code from modes. ===Succeeded in 0.0 mins of git checkout --force origin/HEAD -- === ===Running git clean -dff === In directory: /b/s/w/ir/k/boringssl ===Succeeded in 0.0 mins of git clean -dff === ===Running git log --format=%H --max-count=1 === In directory: /b/s/w/ir/k/boringssl 104306f587751f34852838915fb61ce5551c2332 ===Succeeded in 0.0 mins of git log --format=%H --max-count=1 === ===Running git config --global user.name === In directory: /b/s/w/ir/k boringssl-try-builder ===Succeeded in 0.0 mins of git config --global user.name === ===Running git config --global user.email === In directory: /b/s/w/ir/k boringssl-try-builder@chops-service-accounts.iam.gserviceaccount.com ===Succeeded in 0.0 mins of git config --global user.email === ===Running /b/s/w/ir/cipd_bin_packages/bin/python -u /b/s/w/ir/kitchen-checkout/depot_tools/gclient.py sync --verbose --reset --force --ignore_locks --output-json /b/s/w/ir/tmp/t/tmpqMI3E9.json --nohooks --noprehooks --delete_unversioned_trees --revision boringssl@unmanaged --patch-ref https://boringssl.googlesource.com/boringssl@refs/changes/25/34925/1 === In directory: /b/s/w/ir/k Looking for .gclient starting from /b/s/w/ir/k Loaded .gclient config in /b/s/w/ir/k: solutions = [ { 'custom_vars': { 'checkout_clang': 'True'}, 'deps_file': 'util/bot/DEPS', 'managed': False, 'name': 'boringssl', 'url': 'https://boringssl.googlesource.com/boringssl.git'}] cache_dir = r"/b/s/w/ir/cache/git" target_os_only=False gclient(743) ParseDepsFile:Found recursedeps "{'boringssl/util/bot/android_tools': 'util/bot/DEPS'}". boringssl (Elapsed: 0:00:02) ---------------------------------------- [0:00:00] Started. _____ boringssl at refs/remotes/origin/master [0:00:00] ________ unmanaged solution; skipping boringssl [0:00:00] ===Applying patch ref=== [0:00:00] Patch ref is 'https://boringssl.googlesource.com/boringssl.git' @ 'refs/changes/25/34925/1'. Target branch for patch is 'refs/remotes/origin/master'. Current HEAD is '104306f587751f34852838915fb61ce5551c2332'. Current dir is '/b/s/w/ir/k/boringssl' [0:00:02] Merge base of refs/remotes/origin/master and 6f22d55d28af355a34cfa8b744f4942f14dd14f9 is 104306f587751f34852838915fb61ce5551c2332 [0:00:02] Finished. ---------------------------------------- boringssl/util/bot/libFuzzer (Elapsed: 0:00:00) ---------------------------------------- [0:00:02] Started. _____ boringssl/util/bot/libFuzzer at 2a53098584c48af50aec3fb51febe5e651489774 [0:00:02] running "git cat-file -e 2a53098584c48af50aec3fb51febe5e651489774^{commit}" in "/b/s/w/ir/cache/git/chromium.googlesource.com-chromium-llvm--project-compiler--rt-lib-fuzzer" skipping mirror update, it has rev=2a53098584c48af50aec3fb51febe5e651489774 already ________ running 'git -c core.deltaBaseCacheLimit=2g clone --no-checkout --progress --shared --verbose /b/s/w/ir/cache/git/chromium.googlesource.com-chromium-llvm--project-compiler--rt-lib-fuzzer /b/s/w/ir/k/boringssl/util/bot/_gclient_libFuzzer_86jaIr' in '/b/s/w/ir/k' [0:00:02] Cloning into '/b/s/w/ir/k/boringssl/util/bot/_gclient_libFuzzer_86jaIr'... [0:00:02] done. ________ running 'git -c core.deltaBaseCacheLimit=2g fetch origin --prune --verbose' in '/b/s/w/ir/k/boringssl/util/bot/libFuzzer' [0:00:02] From /b/s/w/ir/cache/git/chromium.googlesource.com-chromium-llvm--project-compiler--rt-lib-fuzzer [0:00:02] = [up to date] master -> origin/master [0:00:02] Checked out 2a53098584c48af50aec3fb51febe5e651489774 to a detached HEAD. Before making any commits in this repo, you should use 'git checkout <branch>' to switch to an existing branch or use 'git checkout origin -b <branch>' to create a new branch for your work. [0:00:02] Finished. ---------------------------------------- boringssl/util/bot/gyp (Elapsed: 0:00:00) ---------------------------------------- [0:00:02] Started. _____ boringssl/util/bot/gyp at d61a9397e668fa9843c4aa7da9e79460fe590bfb [0:00:02] running "git cat-file -e d61a9397e668fa9843c4aa7da9e79460fe590bfb^{commit}" in "/b/s/w/ir/cache/git/chromium.googlesource.com-external-gyp" skipping mirror update, it has rev=d61a9397e668fa9843c4aa7da9e79460fe590bfb already ________ running 'git -c core.deltaBaseCacheLimit=2g clone --no-checkout --progress --shared --verbose /b/s/w/ir/cache/git/chromium.googlesource.com-external-gyp /b/s/w/ir/k/boringssl/util/bot/_gclient_gyp_PWCXBY' in '/b/s/w/ir/k' [0:00:02] Cloning into '/b/s/w/ir/k/boringssl/util/bot/_gclient_gyp_PWCXBY'... [0:00:02] done. ________ running 'git -c core.deltaBaseCacheLimit=2g fetch origin --prune --verbose' in '/b/s/w/ir/k/boringssl/util/bot/gyp' [0:00:02] From /b/s/w/ir/cache/git/chromium.googlesource.com-external-gyp [0:00:02] = [up to date] master -> origin/master [0:00:02] = [up to date] infra/config -> origin/infra/config [0:00:02] = [up to date] md-pages -> origin/md-pages [0:00:02] Checked out d61a9397e668fa9843c4aa7da9e79460fe590bfb to a detached HEAD. Before making any commits in this repo, you should use 'git checkout <branch>' to switch to an existing branch or use 'git checkout origin -b <branch>' to create a new branch for your work. [0:00:02] Finished. ---------------------------------------- ===Succeeded in 0.0 mins of /b/s/w/ir/cipd_bin_packages/bin/python -u /b/s/w/ir/kitchen-checkout/depot_tools/gclient.py sync --verbose --reset --force --ignore_locks --output-json /b/s/w/ir/tmp/t/tmpqMI3E9.json --nohooks --noprehooks --delete_unversioned_trees --revision boringssl@unmanaged --patch-ref https://boringssl.googlesource.com/boringssl@refs/changes/25/34925/1 === ===Running git ls-files .DEPS.git === In directory: boringssl ===Succeeded in 0.0 mins of git ls-files .DEPS.git === ===Running git rev-parse HEAD === In directory: boringssl/ 104306f587751f34852838915fb61ce5551c2332 ===Succeeded in 0.0 mins of git rev-parse HEAD === ===Running git log --format=%B -n1 HEAD === In directory: boringssl/ Remove STRICT_ALIGNMENT code from modes. STRICT_ALIGNMENT is a remnant of OpenSSL code would cast pointers to size_t* and load more than one byte at a time. Not all architectures support unaligned access, so it did an alignment check and only enterred this path if aligned or the underlying architecture didn't care. This is UB. Unaligned casts in C are undefined on all architectures, so we switch these to memcpy some time ago. Compilers can optimize memcpy to the unaligned accesses we wanted. That left our modes logic as: - If STRICT_ALIGNMENT is 1 and things are unaligned, work byte-by-byte. - Otherwise, use the memcpy-based word-by-word code, which now works independent of STRICT_ALIGNMENT. Remove the first check to simplify things. On x86, x86_64, and aarch64, STRICT_ALIGNMENT is zero and this is a no-op. ARM is more complex. Per [0], ARMv7 and up support unaligned access. ARMv5 do not. ARMv6 does, but can run in a mode where it looks more like ARMv5. For ARMv7 and up, STRICT_ALIGNMENT should have been zero, but was one. Thus this change should be an improvement for ARMv7 (right now unaligned inputs lose bsaes-armv7). The Android NDK does not even support the pre-ARMv7 ABI anymore[1]. Nonetheless, Cronet still supports ARMv6 as a library. It builds with -march=armv6 which GCC interprets as supporting unaligned access, so it too did not want this code. For completeness, should anyone still care about ARMv5 or be building with an overly permissive -march flag, GCC does appear unable to inline the memcpy calls. However, GCC also does not interpret (uintptr_t)ptr % sizeof(size_t) as an alignment assertion, so such consumers have already been paying for the memcpy here and throughout the library. In general, C's arcane pointer rules mean we must resort to memcpy often, so, realistically, we must require that the compiler optimize memcpy well. [0] https://medium.com/@iLevex/the-curious-case-of-unaligned-access-on-arm-5dd0ebe24965 [1] https://developer.android.com/ndk/guides/abis#armeabi Change-Id: I3c7dea562adaeb663032e395499e69530dd8e145 Reviewed-on: https://boringssl-review.googlesource.com/c/34873 Reviewed-by: Adam Langley <agl@google.com> ===Succeeded in 0.0 mins of git log --format=%B -n1 HEAD === ===Running /b/s/w/ir/cipd_bin_packages/bin/python -u /b/s/w/ir/kitchen-checkout/depot_tools/gclient.py revinfo -a === In directory: /b/s/w/ir/k boringssl: https://boringssl.googlesource.com/boringssl.git@104306f587751f34852838915fb61ce5551c2332 boringssl/util/bot/gyp: https://chromium.googlesource.com/external/gyp.git@d61a9397e668fa9843c4aa7da9e79460fe590bfb boringssl/util/bot/libFuzzer: https://chromium.googlesource.com/chromium/llvm-project/compiler-rt/lib/fuzzer.git@2a53098584c48af50aec3fb51febe5e651489774 ===Succeeded in 0.0 mins of /b/s/w/ir/cipd_bin_packages/bin/python -u /b/s/w/ir/kitchen-checkout/depot_tools/gclient.py revinfo -a ===