Skip to content

Commit

Permalink
Bug 1574643 - NSS changes for haclv2 r=jcj,kjacobs
Browse files Browse the repository at this point in the history
This patch contains the changes in NSS, necessary to pick up HACL*v2 in D55413. It has a couple of TODOs:
* The chacha20 saw verification fails for some reason; it's disabled pending Bug 1604130.
* The hacl task on CI requires Bug 1593647 to get fixed.

Depends on D55413.

Differential Revision: https://phabricator.services.mozilla.com/D55414

--HG--
extra : moz-landing-system : lando
  • Loading branch information
franziskuskiefer committed Dec 18, 2019
1 parent 330f742 commit 7c5abde
Show file tree
Hide file tree
Showing 20 changed files with 197 additions and 493 deletions.
4 changes: 2 additions & 2 deletions automation/saw/chacha20.saw
Expand Up @@ -34,7 +34,7 @@ let SpecChaCha20 n = do {
};

print "Proving equality for a single block...";
time (llvm_verify m "Hacl_Chacha20_chacha20" [] (SpecChaCha20 64));
time (llvm_verify m "Hacl_Chacha20_chacha20_encrypt" [] (SpecChaCha20 64));

print "Proving equality for multiple blocks...";
time (llvm_verify m "Hacl_Chacha20_chacha20" [] (SpecChaCha20 256));
time (llvm_verify m "Hacl_Chacha20_chacha20_encrypt" [] (SpecChaCha20 256));
4 changes: 4 additions & 0 deletions automation/taskcluster/docker-builds/Dockerfile
Expand Up @@ -34,9 +34,13 @@ RUN apt-get update \
pkg-config \
valgrind \
zlib1g-dev \
clang-format-3.9 \
&& rm -rf /var/lib/apt/lists/* \
&& apt-get autoremove -y && apt-get clean -y

RUN update-alternatives --install /usr/bin/clang-format \
clang-format $(which clang-format-3.9) 10

# Latest version of abigail-tools
RUN apt-get update \
&& apt-get install -y --no-install-recommends automake libtool libxml2-dev \
Expand Down

This file was deleted.

31 changes: 0 additions & 31 deletions automation/taskcluster/docker-hacl/Dockerfile

This file was deleted.

20 changes: 0 additions & 20 deletions automation/taskcluster/docker-hacl/bin/checkout.sh

This file was deleted.

15 changes: 0 additions & 15 deletions automation/taskcluster/docker-hacl/license.txt

This file was deleted.

25 changes: 0 additions & 25 deletions automation/taskcluster/docker-hacl/setup-user.sh

This file was deleted.

34 changes: 0 additions & 34 deletions automation/taskcluster/docker-hacl/setup.sh

This file was deleted.

39 changes: 21 additions & 18 deletions automation/taskcluster/graph/src/extend.js
Expand Up @@ -41,11 +41,6 @@ const FUZZ_IMAGE_32 = {
path: "automation/taskcluster/docker-fuzz32"
};

const HACL_GEN_IMAGE = {
name: "hacl",
path: "automation/taskcluster/docker-hacl"
};

const SAW_IMAGE = {
name: "saw",
path: "automation/taskcluster/docker-saw"
Expand Down Expand Up @@ -1026,6 +1021,10 @@ function scheduleTests(task_build, task_cert, test_base) {
NSS_DISABLE_SSSE3: "1"
}, group: "Cipher"
}));
queue.scheduleTask(merge(cert_base_long, {
name: "Cipher tests", symbol: "NoSSE4.1", tests: "cipher",
env: {NSS_DISABLE_SSE4_1: "1"}, group: "Cipher"
}));
queue.scheduleTask(merge(cert_base, {
name: "EC tests", symbol: "EC", tests: "ec"
}));
Expand Down Expand Up @@ -1154,7 +1153,7 @@ async function scheduleTools() {
queue.scheduleTask(merge(base, {
symbol: "hacl",
name: "hacl",
image: HACL_GEN_IMAGE,
image: LINUX_BUILDS_IMAGE,
command: [
"/bin/bash",
"-c",
Expand Down Expand Up @@ -1200,18 +1199,22 @@ async function scheduleTools() {
]
}));

queue.scheduleTask(merge(base, {
parent: task_saw,
symbol: "ChaCha20",
group: "SAW",
name: "chacha20.saw",
image: SAW_IMAGE,
command: [
"/bin/bash",
"-c",
"bin/checkout.sh && nss/automation/taskcluster/scripts/run_saw.sh chacha20"
]
}));
// TODO: The ChaCha20 saw verification is currently disabled because the new
// HACL 32-bit code can't be verified by saw right now to the best of
// my knowledge.
// Bug 1604130
// queue.scheduleTask(merge(base, {
// parent: task_saw,
// symbol: "ChaCha20",
// group: "SAW",
// name: "chacha20.saw",
// image: SAW_IMAGE,
// command: [
// "/bin/bash",
// "-c",
// "bin/checkout.sh && nss/automation/taskcluster/scripts/run_saw.sh chacha20"
// ]
// }));

queue.scheduleTask(merge(base, {
parent: task_saw,
Expand Down

0 comments on commit 7c5abde

Please sign in to comment.