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

Retry on SDIO Write errors. #211

Closed
wants to merge 1 commit into from
Closed

Retry on SDIO Write errors. #211

wants to merge 1 commit into from

Conversation

nsloss
Copy link

@nsloss nsloss commented Nov 20, 2024

This fixes issues I experinced on a BlueSCSI-v2 with wifi PowerBook Edition.

Retrying the writing of the block resulted in no further write errors reported to NetBSD-current 2024-11-20.

The patch in issue #210 was garbled by the web form. This is the patch in it's complete form.

Best regards,

Nat

This fixes issues I experinced on a BlueSCSI-v2 with wifi PowerBook Edition.

Retrying the writing of the block resulted in no further write errors reported
to NetBSD-current 2024-11-20.
@erichelgeson
Copy link
Contributor

Friendly ping - hopefully you saw my comment on your issue - if you could test that scenario.

@nsloss
Copy link
Author

nsloss commented Dec 5, 2024

I can confirm so fat 2024-11-17 is working just fine.

There are no errors reported in NetBSD and so far. I've read/written about 500MB.

Many thanks,

I'll keep testing for a day or two, but otherwise this issue can be closed.

Best regards,

Nat

@nsloss
Copy link
Author

nsloss commented Dec 6, 2024

closing this issue and pull request as the latest version 2024-11-17 is working great.

Many thanks to all that looked at this.

Best regards,

Nat

@nsloss nsloss closed this Dec 6, 2024
@erichelgeson
Copy link
Contributor

Glad to hear! Thanks for digging in anyways - if you ever have any other issue please let us know.

If you had some time to do a little write up on how you're using BlueSCSI in NetBSD on your powebook I'd be very interested in reading that!

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