-
-
Notifications
You must be signed in to change notification settings - Fork 9.8k
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
[NFCI] Move Keccak rhotates tables to rodata #21440
Conversation
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.
Okay with calling this trivial.
If you agree, please amend the commit to include a line CLA: trivial
at the end.
rhotates tables are placed to .text section which confuses tools such as BOLT. Move them to rodata to unbreak and avoid polluting icache/iTLB with data. CLA: trivial
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.
Yes, it's trivial
This pull request is ready to merge |
Merged to all three branches. Thanks for the fix. |
rhotates tables are placed to .text section which confuses tools such as BOLT. Move them to rodata to unbreak and avoid polluting icache/iTLB with data. CLA: trivial Reviewed-by: Hugo Landau <hlandau@openssl.org> Reviewed-by: Paul Yang <kaishen.yy@antfin.com> Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from #21440)
rhotates tables are placed to .text section which confuses tools such as BOLT. Move them to rodata to unbreak and avoid polluting icache/iTLB with data. CLA: trivial Reviewed-by: Hugo Landau <hlandau@openssl.org> Reviewed-by: Paul Yang <kaishen.yy@antfin.com> Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from #21440) (cherry picked from commit 2fd82c2)
rhotates tables are placed to .text section which confuses tools such as BOLT. Move them to rodata to unbreak and avoid polluting icache/iTLB with data. CLA: trivial Reviewed-by: Hugo Landau <hlandau@openssl.org> Reviewed-by: Paul Yang <kaishen.yy@antfin.com> Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from #21440) (cherry picked from commit 2fd82c2)
rhotates tables are added to .text section which confuses tools such as BOLT. Move them to rodata to unbreak and avoid polluting icache/iTLB with data. Fix in OpenSSL openssl/openssl#21440
rhotates tables are added to .text section which confuses tools such as BOLT. Move them to rodata to unbreak and avoid polluting icache/iTLB with data. Fix in OpenSSL openssl/openssl#21440
rhotates tables are placed to .text section which confuses tools such as BOLT. Move them to rodata to unbreak and avoid polluting icache/iTLB with data. Sync with XKCP upstream: XKCP/XKCP#137 Similar fix in OpenSSL: openssl/openssl#21440 1. Update patch file using the steps in scripts/copy_from_xkcp/README 2. Apply the updated patch with scripts/copy_from_xkcp/package.sh
rhotates tables are placed to .text section which confuses tools such as BOLT. Move them to rodata to unbreak and avoid polluting icache/iTLB with data. 1. Update patch file using the steps in scripts/copy_from_xkcp/README 2. Apply the updated patch with scripts/copy_from_xkcp/package.sh Sync with XKCP upstream: XKCP/XKCP#137 Similar fix in OpenSSL: openssl/openssl#21440 Redo of open-quantum-safe#1508
rhotates tables are placed to .text section which confuses tools such as BOLT. Move them to rodata to unbreak and avoid polluting icache/iTLB with data. 1. Update patch file using the steps in scripts/copy_from_xkcp/README 2. Apply the updated patch with scripts/copy_from_xkcp/package.sh Sync with XKCP upstream: XKCP/XKCP#137 Similar fix in OpenSSL: openssl/openssl#21440 Redo of open-quantum-safe#1508 Signed-off-by: Amir Ayupov <aaupov@fb.com>
rhotates tables are placed to .text section which confuses tools such as BOLT. Move them to rodata to unbreak and avoid polluting icache/iTLB with data. 1. Update patch file using the steps in scripts/copy_from_xkcp/README 2. Apply the updated patch with scripts/copy_from_xkcp/package.sh Sync with XKCP upstream: XKCP/XKCP#137 Similar fix in OpenSSL: openssl/openssl#21440 Redo of #1508 Signed-off-by: Amir Ayupov <aaupov@fb.com>
rhotates tables are placed to .text section which confuses tools such as BOLT. Move them to rodata to unbreak and avoid polluting icache/iTLB with data. 1. Update patch file using the steps in scripts/copy_from_xkcp/README 2. Apply the updated patch with scripts/copy_from_xkcp/package.sh Sync with XKCP upstream: XKCP/XKCP#137 Similar fix in OpenSSL: openssl/openssl#21440 Redo of open-quantum-safe#1508 Signed-off-by: Amir Ayupov <aaupov@fb.com> Signed-off-by: Eddy Kim <Eddy.M.Kim@outlook.com>
rhotates tables are placed to .text section which confuses tools such as BOLT.
Move them to rodata to unbreak and avoid polluting icache/iTLB with data.