-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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
Swap file size on BE platform #92810
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/92810
Note: Links to docs will display an error until the docs builds have been completed. ❌ 1 FailuresAs of commit b1f42eb: NEW FAILURES - The following jobs have failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
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.
Thank you for the fix
@pytorchbot merge -f "ASAN is failing on master, this change only affects power" |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Fixes pytorch#92808 This PR fixes SIGSEGV on a big-endian machine when reading pickle data. The root cause is not to convert `size`, which is read from a file, from little-endian to big-endian while `size` is used in a method. The fix is to convert `size` on a big-endian machine instead of `nbytes`. I confirmed that the program in the issue works w/o SIGSEGV and the test passes, with this fix in master branch. ``` $ python test/test_autograd.py TestAutograd.test_pickle . ---------------------------------------------------------------------- Ran 1 test in 0.010s OK ``` Pull Request resolved: pytorch#92810 Approved by: https://github.com/malfet
Fixes pytorch#92808 This PR fixes SIGSEGV on a big-endian machine when reading pickle data. The root cause is not to convert `size`, which is read from a file, from little-endian to big-endian while `size` is used in a method. The fix is to convert `size` on a big-endian machine instead of `nbytes`. I confirmed that the program in the issue works w/o SIGSEGV and the test passes, with this fix in master branch. ``` $ python test/test_autograd.py TestAutograd.test_pickle . ---------------------------------------------------------------------- Ran 1 test in 0.010s OK ``` Pull Request resolved: pytorch#92810 Approved by: https://github.com/malfet
Fixes #92808
This PR fixes SIGSEGV on a big-endian machine when reading pickle data.
The root cause is not to convert
size
, which is read from a file, from little-endian to big-endian whilesize
is used in a method. The fix is to convertsize
on a big-endian machine instead ofnbytes
.I confirmed that the program in the issue works w/o SIGSEGV and the test passes, with this fix in master branch.