smengcl commented on PR #6341:
URL: https://github.com/apache/ozone/pull/6341#issuecomment-2010879068

   > > @swamirishi we can merge this in once you specify the testing done for 
this.
   > 
   > I tested out this particular patch by building the code in an environment 
which supports avx2. I built the shared library and tried to load the library 
on another machine which doesn't support this instruction set without the flag 
PORTABLE=1 and with the flag PORTABLE=1. The library load failed with the error 
in the first run:
   > 
   > <img alt="Screen Shot 2024-03-20 at 15 15 19" width="856" 
src="https://private-user-images.githubusercontent.com/47532440/314694209-3132f5f0-17b6-4105-833c-76e09dc5e49f.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTA5NzYyNjMsIm5iZiI6MTcxMDk3NTk2MywicGF0aCI6Ii80NzUzMjQ0MC8zMTQ2OTQyMDktMzEzMmY1ZjAtMTdiNi00MTA1LTgzM2MtNzZlMDlkYzVlNDlmLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDAzMjAlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwMzIwVDIzMDYwM1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTMwNWZlOGM0YTI1NzQ4MzI4MDIxMzJhZTYxOTQ2YmZjODg4YzBjMzMxYmY5MDJmOWQ1Zjg2NzRjZWM3YmIyMTQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.bHgcArud9_06qCMH_vMyU6bmudP_VGshGIJwyxsI568";>
   > The second time the load succeeded.
   
   Yup fyi `PORTABLE` flag is used here in `CMakeLists.txt` to disable avx/avx2 
optimizations here:
   
   https://github.com/facebook/rocksdb/blob/v7.7.3/CMakeLists.txt#L252-L277


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to