Skip to content
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

Disable test large_file on 32-bit rr. #3562

Merged
merged 1 commit into from
Jul 17, 2023

Conversation

bernhardu
Copy link
Contributor

Because 32-bit rr is compiled without -D_FILE_OFFSET_BITS=64. Therefore the open of the large file fails with EOVERFLOW.

[FATAL src/record_syscall.cc:5531:process_execve() errno: EOVERFLOW]
(task 2019470 (rec:2019470) at time 14)
-> Assertion `fd.is_open()' failed to hold.


My attempts to either use -D_FILE_OFFSET_BITS=64 or open64/stat64 were leading to a row of changes involving size_t/off_t or struct stat, which I guess are not desired for the fading 32-bit architecture?

Because 32-bit rr is compiled without `-D_FILE_OFFSET_BITS=64`.
Therefore the open of the large file fails with EOVERFLOW.

 [FATAL src/record_syscall.cc:5531:process_execve() errno: EOVERFLOW]
  (task 2019470 (rec:2019470) at time 14)
  -> Assertion `fd.is_open()' failed to hold.
@rocallahan
Copy link
Collaborator

I would probably take a fix for this, but this is fine too.

@rocallahan rocallahan merged commit 973d0d7 into rr-debugger:master Jul 17, 2023
1 check passed
@bernhardu bernhardu deleted the skip-large-file-32bit-rr branch March 20, 2024 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants