Skip to content

Comments

Fix error in the loop of ECDH in speed.c#1024

Closed
agrandi wants to merge 1 commit intoopenssl:masterfrom
agrandi:fix-speed-ecdh
Closed

Fix error in the loop of ECDH in speed.c#1024
agrandi wants to merge 1 commit intoopenssl:masterfrom
agrandi:fix-speed-ecdh

Conversation

@agrandi
Copy link
Contributor

@agrandi agrandi commented May 4, 2016

The tests were incorrectly repeated multiple times when using the option -async_jobs

For example, with async_jobs = 2 all the tests were repeated twice:

> apps/openssl speed -engine dasync -elapsed -async_jobs 2 ecd
engine "dasync" set.
You have chosen to measure elapsed time instead of user CPU time.
Doing 160 bit  ecdh's for 1s: 4464 160-bit ECDH ops in 1.00s
Doing 160 bit  ecdh's for 1s: 4464 160-bit ECDH ops in 1.00s
Doing 192 bit  ecdh's for 1s: 3685 192-bit ECDH ops in 1.00s
Doing 192 bit  ecdh's for 1s: 3677 192-bit ECDH ops in 1.00s
Doing 224 bit  ecdh's for 1s: 2712 224-bit ECDH ops in 1.00s
Doing 224 bit  ecdh's for 1s: 2714 224-bit ECDH ops in 1.00s

The tests was incorrectly repeated multiple times when using the
async_jobs options
@agrandi agrandi changed the title Fix error in the loop of ECDH Fix error in the loop of ECDH in speed.c May 4, 2016
@mattcaswell
Copy link
Member

+1

@kroeckx kroeckx added the approval: done This pull request has the required number of approvals label May 7, 2016
@kroeckx
Copy link
Member

kroeckx commented May 7, 2016

+1

@mattcaswell
Copy link
Member

Merged. Thanks.

@mattcaswell mattcaswell closed this May 9, 2016
@agrandi
Copy link
Contributor Author

agrandi commented May 9, 2016

Thanks a lot @mattcaswell and @kroeckx !!

@kroeckx kroeckx mentioned this pull request Apr 1, 2020
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants