Repository navigation
fix: fall back to GIO on destination open failure for MTP copies - #1923
Merged
Merged
Conversation
When copying files to an MTP device mounted via GVfs FUSE, compio's io_uring backend fails to open the destination file with O_CREAT|O_EXCL. The existing gio_file_copy fallback only triggered on write_at failures, so the error propagated before GIO was ever tried. Now, if the destination open fails via compio, we fall back to gio_file_copy (which uses POSIX I/O through GIO), matching the same fallback pattern already used for write_at Unsupported errors.
mmstick
reviewed
Jul 15, 2026
mmstick
reviewed
Jul 15, 2026
mmstick
reviewed
Jul 15, 2026
Use .with_context() instead of anyhow::anyhow! so the original io::Error is preserved in the error chain, matching reviewer feedback on pop-os#1923.
mmstick
previously approved these changes
Jul 15, 2026
jacobgkau
previously approved these changes
Jul 15, 2026
jacobgkau
left a comment
Member
There was a problem hiding this comment.
Like Michael, I also wasn't able to recreate any MTP issues with my own Android devices, but I'm not seeing any regressions with MTP, either.
Standard regression testing passed:
Checklist
Basic navigation
- Middle-click opens directory in a new tab (not focused).
- Open two scrollable tabs. Scroll one tab, then switch to the other tab; it should not have scrolled.
- Hover over the top item in the folder, then scroll down until it's out of view (while still hovered). On scrolling back up (with the mouse in a different position), the item should not have the hover highlight.
- Right-click an item in the sidebar. No visual change should occur with the rest of the items.
- Remove an item from the sidebar, then re-pin it.
File operations
- Right-click -> Create a new folder, then enter it.
- Right-click in the empty folder -> Create a new file.
- Navigate to the parent folder, create another new file, then drag it into the created folder.
- Files can be renamed.
- Files can be opened with non-default apps & browsing store for new apps works.
- Normal right-click shows
Move to trashoption. - Shift right-click, and right-click followed by Shift, both show
Permanently deleteoption.
Advanced navigation & view settings
- Image and video thumbnails generate & display in local folders.
- Gallery preview shows with Spacebar.
- Details pane shows with Ctrl+Spacebar.
- Zoom in/out and reset to default zoom work.
- Ctrl+1 and Ctrl+2 switch between list and icon view.
- Ctrl+H shows/hides hidden files.
- Directories can be sorted at top or inline.
- Settings -> Theme works.
- Settings -> Type to Search affects behavior as designed.
- Single-click to open setting takes effect.
- Recents in sidebar setting works.
- Sorting options work.
- Cutting, copying, and pasting files works.
- F5 reloads current directory.
- Left sidebar can be collapsed and expanded.
External filesystems
- Add a network drive (e.g. SFTP) and navigate into it.
- Plug in a USB drive; able to mount, browse, and eject.
Integrations
- Desktop icons display as expected
- Drag-and-drop into Firefox works
Member
|
It does look like the |
Add .map_err(Into::into) to convert anyhow::Error to Box<dyn Error> matching the function's return type.
mmstick
approved these changes
Jul 15, 2026
jacobgkau
approved these changes
Jul 15, 2026
jacobgkau
left a comment
Member
There was a problem hiding this comment.
CI's working now. Testing looks the same as before.
Regression testing passed again:
Checklist
Basic navigation
- Middle-click opens directory in a new tab (not focused).
- Open two scrollable tabs. Scroll one tab, then switch to the other tab; it should not have scrolled.
- Hover over the top item in the folder, then scroll down until it's out of view (while still hovered). On scrolling back up (with the mouse in a different position), the item should not have the hover highlight.
- Right-click an item in the sidebar. No visual change should occur with the rest of the items.
- Remove an item from the sidebar, then re-pin it.
File operations
- Right-click -> Create a new folder, then enter it.
- Right-click in the empty folder -> Create a new file.
- Navigate to the parent folder, create another new file, then drag it into the created folder.
- Files can be renamed.
- Files can be opened with non-default apps & browsing store for new apps works.
- Normal right-click shows
Move to trashoption. - Shift right-click, and right-click followed by Shift, both show
Permanently deleteoption.
Advanced navigation & view settings
- Image and video thumbnails generate & display in local folders.
- Gallery preview shows with Spacebar.
- Details pane shows with Ctrl+Spacebar.
- Zoom in/out and reset to default zoom work.
- Ctrl+1 and Ctrl+2 switch between list and icon view.
- Ctrl+H shows/hides hidden files.
- Directories can be sorted at top or inline.
- Settings -> Theme works.
- Settings -> Type to Search affects behavior as designed.
- Single-click to open setting takes effect.
- Recents in sidebar setting works.
- Sorting options work.
- Cutting, copying, and pasting files works.
- F5 reloads current directory.
- Left sidebar can be collapsed and expanded.
External filesystems
- Add a network drive (e.g. SFTP) and navigate into it.
- Plug in a USB drive; able to mount, browse, and eject.
Integrations
- Desktop icons display as expected
- Drag-and-drop into Firefox works
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
When copying files to an MTP device mounted via GVfs FUSE, compio's io_uring backend fails to open the destination file with O_CREAT|O_EXCL. The existing gio_file_copy fallback only triggered on write_at failures, so the error propagated before GIO was ever tried.
Now, if the destination open fails via compio, we fall back to gio_file_copy (which uses POSIX I/O through GIO), matching the same fallback pattern already used for write_at Unsupported errors.
Fixes #1782