-
-
Notifications
You must be signed in to change notification settings - Fork 9.9k
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
Changes to resolve the symbol conflict because of gf_mul #21421
Conversation
Since these changes are just renaming internal functions, can we consider this trivial and not need the CLA check. |
Please see https://www.openssl.org/policies/cla.html how to mark the commit with CLA: trivial - it needs to be written in the commit message body on a separate line. |
@@ -100,6 +100,6 @@ void gf_mulw_unsigned(gf_s * RESTRICT cs, const gf as, uint32_t b) | |||
|
|||
void gf_sqr(gf_s * RESTRICT cs, const gf as) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please rename also this one?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will follow the same standard, ossl_gf_sqr
Please add the As this is pure rename of 3 symbols I think it is acceptable as CLA: trivial. |
CLA: trivial Changed names of internal functions to resolve symbol conflict when Openssl is used with intel/ISA-L.
Added CLA: Trivial to the git commit message |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK with CLA: trivial
@kroeckx please confirm you're OK with CLA: trivial for this. |
24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set. Please review the updates and set the label manually. |
Merged. Thanks for the contribution. |
CLA: trivial Changed names of internal functions to resolve symbol conflict when Openssl is used with intel/ISA-L. Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Kurt Roeckx <kurt@roeckx.be> Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from #21421)
Would we cherry pick it to 3.0 and 3.1 branch? |
Please submit a PR against 3.1 branch. |
CLA: trivial Changed names of internal functions to resolve symbol conflict when Openssl is used with intel/ISA-L. Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Kurt Roeckx <kurt@roeckx.be> Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from openssl#21421)
CLA: trivial Changed names of internal functions to resolve symbol conflict when Openssl is used with intel/ISA-L. Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Kurt Roeckx <kurt@roeckx.be> Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from openssl#21421)
CLA: trivial
This PR is a result of discussion on resolving
gf_mul
symbol conflict when using intel/ISA-L and Openssl, #21302.