Repository navigation
ata: recover SATAPI ports and revalidate optical media - #129
Merged
Merged
Conversation
The firmware pairs its host timing with the drive's DMA mode. Probe each drive without sending SET FEATURES or programming vendor timing registers, then compare a PIO and a DMA read of the same sectors once at boot. A failed check, a missing BAR4, or a runtime bus-master error drops the drive to PIO; ATA retries the same request in PIO, ATAPI resumes from the completed offset, and a stuck BSY resets the channel. ATAPI capacity probing follows REQUEST SENSE retries and maps a missing medium to -ENOMEDIUM. Signed-off-by: MicroFish <2609948707@qq.com> Co-Authored-By: Claude Code <noreply@anthropic.com>
A failed packet leaves the port's PxCI bit set and stalls every later command. Restart the port engine after a task-file error or timeout, and poll READ CAPACITY with REQUEST SENSE retries after a media change so a new disc is picked up after unit attention or becoming ready. Signed-off-by: MicroFish <2609948707@qq.com> Co-Authored-By: Claude Code <noreply@anthropic.com>
Add an optional blockdev_ops.revalidate hook and call it from devtmpfs when a block node is opened, refreshing the node geometry after a disc change. Reads and writes on an empty ATAPI drive return -ENOMEDIUM, and the block open path propagates the refresh error so a mount reports the drive as having no medium. Signed-off-by: MicroFish <2609948707@qq.com> Co-Authored-By: Claude Code <noreply@anthropic.com>
FengHeting
force-pushed
the
atapi-hotplug
branch
from
November 4, 2026 19:53
a29926e to
643056f
Compare
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.
Summary
Stacked on the
pata-dmabranch; retarget todeveloponce that PR is merged.blockdev_opsgains an optionalrevalidatehook.devtmpfscalls it when a block node is opened, so the node size and block size follow the inserted disc.srnumber with capacity 0; reads and writes return-ENOMEDIUM, and the block-open path propagates the refresh error so a mount reports no medium.Testing
make -j$(nproc)passes.blockdev-change-mediumonide1-cd1: eject/insert updates capacity, mounts after insertion succeed, and reads on an empty drive report no medium.