Skip to content

Revert "zfs: unlock for 5.15"#146310

Merged
Mic92 merged 1 commit intoNixOS:masterfrom
TredwellGit:zfs
Nov 16, 2021
Merged

Revert "zfs: unlock for 5.15"#146310
Mic92 merged 1 commit intoNixOS:masterfrom
TredwellGit:zfs

Conversation

@TredwellGit
Copy link
Copy Markdown
Member

Motivation for this change

This reverts commit 86e057a.

Things done

@TredwellGit TredwellGit mentioned this pull request Nov 16, 2021
12 tasks
@TredwellGit TredwellGit requested review from Mic92 and hmenke and removed request for hmenke November 16, 2021 22:14
@jonringer
Copy link
Copy Markdown
Contributor

After having my zfs partition borked from the SEEK_HOLE issue, I'm apprehensive about pushing the bounds of zfs. I would like to be more on the side of caution. Especially since data integrity is the first priority for anything.

@Mic92 Mic92 merged commit 089f00c into NixOS:master Nov 16, 2021
@Mic92
Copy link
Copy Markdown
Member

Mic92 commented Nov 16, 2021

After having my zfs partition borked from the SEEK_HOLE issue, I'm apprehensive about pushing the bounds of zfs. I would like to be more on the side of caution. Especially since data integrity is the first priority for anything.

That was a zfs bug and not a compatibility issue: openzfs/zfs#11900

@TredwellGit TredwellGit deleted the zfs branch November 16, 2021 22:33
@jonringer
Copy link
Copy Markdown
Contributor

After having my zfs partition borked from the SEEK_HOLE issue, I'm apprehensive about pushing the bounds of zfs. I would like to be more on the side of caution. Especially since data integrity is the first priority for anything.

That was a zfs bug and not a compatibility issue: openzfs/zfs#11900

Although I absolutely agree, it still makes me more risk adverse.

@jansol
Copy link
Copy Markdown
Contributor

jansol commented Dec 11, 2021

You win some, you lose some I guess.

This in combination with #147446 means that I woke up the other day to not getting a graphical login screen. Turns out that thanks to 5.14 being EOL I'm now stuck on 5.10, where amdgpu does not yet have any support for my card at all (RDNA2).

@Mic92
Copy link
Copy Markdown
Member

Mic92 commented Dec 11, 2021

Well. You can just apply Mic92@b31e0be
I used for some weeks now without issues.

@KenMacD
Copy link
Copy Markdown
Contributor

KenMacD commented Dec 16, 2021

@TredwellGit 2.1.2 is out, and lists 5.15 support 🥺

@TredwellGit
Copy link
Copy Markdown
Member Author

#150923

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.

5 participants