-
Notifications
You must be signed in to change notification settings - Fork 102
devices: set CacheType::Unsafe for macOS raw disks #297
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
Conversation
5738c9d
to
1ad4667
Compare
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.
LGTM. You mentioned in the issue that this problem persists with /dev/disk*
as well. Do you plan to address that in this PR as well?
@slp said
Besides, there's no practical reason to use |
1ad4667
to
a456e6a
Compare
Don't know what's with the failing checks now... they seem unrelated. |
a456e6a
to
9f3a706
Compare
Ugg... we've got hit again by another clippy update with new restrictions. I'll fix it in a different PR, and the we can rebase this one. |
I also noticed a crash in unit tests... |
Yes, and I can reproduce the failure here after updating to |
Signed-off-by: Jan Noha <[email protected]>
9f3a706
to
4a45522
Compare
Done |
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.
Code looks good and checks are now green, let's get it merged. Thanks @nohajc !
See #293