Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Atomics #21260

Closed
wants to merge 24 commits into from
Closed

Atomics #21260

wants to merge 24 commits into from

Conversation

paulidale
Copy link
Contributor

@paulidale paulidale commented Jun 22, 2023 •
edited by arapov

  • documentation is added or updated
  • tests are added or updated

fixes openssl/project#51

@paulidale paulidale added branch: master Merge to master branch approval: review pending This pull request needs review by a committer approval: otc review pending This pull request needs review by an OTC member triaged: refactor The issue/pr requests/implements refactoring tests: present The PR has suitable tests present labels Jun 22, 2023
@paulidale paulidale self-assigned this Jun 22, 2023
@paulidale paulidale requested a review from hlandau June 22, 2023 00:54
@paulidale paulidale mentioned this pull request Jun 22, 2023
Closed
@github-actions github-actions bot added the severity: fips change The pull request changes FIPS provider sources label Jun 22, 2023
@paulidale paulidale force-pushed the atomics branch 2 times, most recently from bef9253 to 3f87f8c Compare June 22, 2023 01:14
@paulidale
refcnt: convert references counts to a structure
1476292 
The structure includes the count and optionally a lock.
The count has the appropriate qualifiers.
@paulidale paulidale force-pushed the atomics branch 3 times, most recently from 94625ae to 23c0cc5 Compare June 22, 2023 02:03
kroeckx
kroeckx reviewed Jul 1, 2023
View reviewed changes
ssl/ssl_lib.c Show resolved Hide resolved
kroeckx
kroeckx reviewed Jul 1, 2023
View reviewed changes
ssl/ssl_sess.c Show resolved Hide resolved
@paulidale
Copy link
Contributor Author

I've done the fixes in #21341

MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
refcnt: convert references counts to a structure
7431d70 
The structure includes the count and optionally a lock.
The count has the appropriate qualifiers.

Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
engine: update to structure based atomics
c3be046 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
bio: update to structure based atomics
9966665 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
dh: update to structure based atomics
0460618 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
dsa: update to structure based atomics
4c7c5e9 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
dso: update to structure based atomics
8e04cde 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
asn1: update to structure based atomics
d390260 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
ec: update to structure based atomics
56bea6e 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
ecx: update to structure based atomics
5801d2a 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
provider: update to structure based atomics
a656213 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
rsa: update to structure based atomics
b43d299 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
store: update to structure based atomics
bc923c1 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
test: update to structure based atomics
b70fe49 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
tls: update to structure based atomics
77115de 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
quic: update to structure based atomics
44cf01f 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
encoder: update to structure based atomics
0b8d91c 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
prov(legacy): update to structure based atomics
30c8176 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
x509: update to structure based atomics
ed3cc8c 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
MrE-Fog pushed a commit to MrE-Fog/opensslJSQ that referenced this pull request Jul 3, 2023
@paulidale @MrE-Fog
evp: update to structure based atomics
bf0faf6 
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#21260)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kroeckx kroeckx kroeckx left review comments

@t8m t8m t8m approved these changes

@mattcaswell mattcaswell mattcaswell approved these changes

@hlandau hlandau Awaiting requested review from hlandau

@beldmit beldmit Awaiting requested review from beldmit

Assignees

@paulidale paulidale

Labels
approval: done This pull request has the required number of approvals branch: master Merge to master branch severity: fips change The pull request changes FIPS provider sources tests: present The PR has suitable tests present triaged: refactor The issue/pr requests/implements refactoring
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Convert reference counts to a structure
6 participants
@paulidale @beldmit @t8m @kroeckx @slontis @mattcaswell