Home | Search | boringssl - Builders
Login

Builder linux32 Build 8922328485889369840 Canonical Ubuntu

Overview

Success

Input

Revision 55b9acda9969a04751cacfcf235e6b8da5ce982b

Infra

Steps and Logs

Show:
  1. ( 152 ms ) setup_build

    running recipe: “boringssl”

  2. ( 6 ms ) uname
  3. ( 1 secs ) bot_update

    [21GB/295GB used (7%)]

  4. ( 7 secs ) gclient runhooks
  5. ( 38 ms ) clean
  6. ( 37 ms ) mkdir
  7. ( 6 secs ) cmake
  8. ( 32 secs ) ninja
  9. ( 266 ms ) check filenames
  10. ( 35 secs ) unit tests
  11. ( 29 secs ) ssl tests
  12. ( 0 ) recipe result

Timing

Create Tuesday, 05-Feb-19 21:18:22 UTC
Start Tuesday, 05-Feb-19 21:19:44 UTC
End Tuesday, 05-Feb-19 21:21:47 UTC
Pending 1 mins 21 secs
Execution 2 mins 2 secs

Tags

KeyValue
builder linux32
buildset commit/git/55b9acda9969a04751cacfcf235e6b8da5ce982b
buildset commit/gitiles/boringssl.googlesource.com/boringssl/+/55b9acda9969a04751cacfcf235e6b8da5ce982b
scheduler_invocation_id 9087758548159328464
scheduler_job_id boringssl/linux32
user_agent luci-scheduler

Input Properties

NameValue
$recipe_engine/runtime { "is_experimental": false, "is_luci": true }
branch "refs/heads/master"
buildername "linux32"
repository "https://boringssl.googlesource.com/boringssl.git"
revision "55b9acda9969a04751cacfcf235e6b8da5ce982b"

Output Properties

NameValue
$recipe_engine/buildbucket { "build": { "builder": { "bucket": "ci", "builder": "linux32", "project": "boringssl" }, "createTime": "2019-02-05T21:18:22.675174Z", "createdBy": "user:luci-scheduler@appspot.gserviceaccount.com", "id": "8922328485889369840", "infra": { "buildbucket": { "serviceConfigRevision": "ef156230f5cd7f8c23f689fa4dc7f92742cd904d" }, "recipe": { "cipdPackage": "infra/recipe_bundles/chromium.googlesource.com/chromium/tools/build", "name": "boringssl" }, "swarming": { "hostname": "chromium-swarm.appspot.com" } }, "input": { "gitilesCommit": { "host": "boringssl.googlesource.com", "id": "55b9acda9969a04751cacfcf235e6b8da5ce982b", "project": "boringssl", "ref": "refs/heads/master" } }, "tags": [ { "key": "builder", "value": "linux32" }, { "key": "buildset", "value": "commit/git/55b9acda9969a04751cacfcf235e6b8da5ce982b" }, { "key": "buildset", "value": "commit/gitiles/boringssl.googlesource.com/boringssl/+/55b9acda9969a04751cacfcf235e6b8da5ce982b" }, { "key": "scheduler_invocation_id", "value": "9087758548159328464" }, { "key": "scheduler_job_id", "value": "boringssl/linux32" }, { "key": "user_agent", "value": "luci-scheduler" } ] } }
$recipe_engine/path { "cache_dir": "/b/s/w/ir/cache", "temp_dir": "/b/s/w/ir/tmp/rt" }
$recipe_engine/runtime { "is_experimental": false, "is_luci": true }
bot_id "gce-trusty-e833d7b0-us-west1-a-nkkd"
branch "refs/heads/master"
buildbucket { "build": { "bucket": "luci.boringssl.ci", "created_by": "user:luci-scheduler@appspot.gserviceaccount.com", "created_ts": 1549401502675174, "id": "8922328485889369840", "project": "boringssl", "tags": [ "builder:linux32", "buildset:commit/git/55b9acda9969a04751cacfcf235e6b8da5ce982b", "buildset:commit/gitiles/boringssl.googlesource.com/boringssl/+/55b9acda9969a04751cacfcf235e6b8da5ce982b", "gitiles_ref:refs/heads/master", "scheduler_invocation_id:9087758548159328464", "scheduler_job_id:boringssl/linux32", "user_agent:luci-scheduler" ] }, "hostname": "cr-buildbucket.appspot.com" }
buildername "linux32"
got_revision "55b9acda9969a04751cacfcf235e6b8da5ce982b"
path_config "generic"
recipe "boringssl"
repository "https://boringssl.googlesource.com/boringssl.git"
revision "55b9acda9969a04751cacfcf235e6b8da5ce982b"

All Changes

  1. Fix ABI error in bn_mul_mont on aarch64.

    Changed by David Benjamin - davidbenohnoyoudont@google.com
    Changed at Tuesday, 05-Feb-19 21:17:54 UTC
    Repository https://boringssl.googlesource.com/boringssl
    Branch
    Revision 55b9acda9969a04751cacfcf235e6b8da5ce982b

    Comments

    Fix ABI error in bn_mul_mont on aarch64.
    
    This was caught by an aarch64 ABI tester. aarch64 has the same
    considerations around small arguments as x86_64 does. The aarch64
    version of bn_mul_mont does not mask off the upper words of the
    argument.
    
    The x86_64 version does, so size_t is, strictly speaking, wrong for
    aarch64, but bn_mul_mont already has an implicit size limit to support
    its internal alloca, so this doesn't really make things worse than
    before.
    
    Change-Id: I39bffc8fdb2287e45a2d1f0d1b4bd5532bbf3868
    Reviewed-on: https://boringssl-review.googlesource.com/c/34804
    Commit-Queue: David Benjamin <davidben@google.com>
    Reviewed-by: Adam Langley <agl@google.com>
    

    Changed files

    • crypto/fipsmodule/bn/asm/armv8-mont.pl
    • crypto/fipsmodule/bn/asm/x86_64-mont.pl
    • crypto/fipsmodule/bn/internal.h