-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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.
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
Releasing GIL in cryptographic operations #11585
Comments
Not really. I thought we did for things like scrypt, but I don't see it, so
I guess not. I'd be open to taking PRs for these though
…On Wed, Sep 11, 2024, 9:19 AM Jakub Kahovec ***@***.***> wrote:
I'd like to ask if GIL gets released when cryptographic operations are
being performed as I can't find it in the code, i.e
https://github.com/pyca/cryptography/blob/60913069bb27d788c57687840a8b1b54904e9139/src/rust/src/backend/kdf.rs#L10
—
Reply to this email directly, view it on GitHub
<#11585>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAAGBA76QMRPU377PPVYV3ZWA7NLAVCNFSM6AAAAABOA6WQYWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGUYTSNZTGI3TCMI>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Ok, I was kind of puzzled by this comment |
That predates a lot of reactors, including moving to rust. Cffi used to
implicitly drop the GIL during all calls to openssl
…On Wed, Sep 11, 2024, 10:48 AM Jakub Kahovec ***@***.***> wrote:
Ok, I was kind of puzzled by this comment
#6896 (comment)
<#6896 (comment)>
—
Reply to this email directly, view it on GitHub
<#11585 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAAGBBAWICSB3OOGVB5DELZWBJ2DAVCNFSM6AAAAABOA6WQYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBTHA4TGMRVGY>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I'd like to ask if GIL gets released when cryptographic operations are being performed as I can't find it in the code, i.e
cryptography/src/rust/src/backend/kdf.rs
Line 10 in 6091306
The text was updated successfully, but these errors were encountered: