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

https://reviews.freebsd.org/D15124 //RaidZ expansion #8437

Closed
tronder88 opened this issue Feb 20, 2019 · 5 comments
Closed

https://reviews.freebsd.org/D15124 //RaidZ expansion #8437

tronder88 opened this issue Feb 20, 2019 · 5 comments
Labels
Status: Inactive Not being actively updated Type: Feature Feature request or new feature

Comments

@tronder88
Copy link

Describe the problem you're observing

"Lack of progress" for ZFS RaidZ Expansion

I have been following the RaidZ project from Matthew Ahrens with great interest, how ever, the following pre-alpha code has been idle for the last year as far as I can tell.

Adding this issue, so that the great community for ZFSonLinux can pick up the thread and see what is possible.

@tronder88 tronder88 changed the title https://reviews.freebsd.org/D15124 RaidZ expansion https://reviews.freebsd.org/D15124 //RaidZ expansion Feb 20, 2019
@allanjude
Copy link
Contributor

additionally, the performance is questionable, with the indirection table that is generated - it'll be somewhat similar to dedup.

RAID-Z expansion doesn't use an indirection table. It only needs to keep track of how far through the vdev it is, so it can tell if a requested DVA is before or after the rewrite point, to decide which geometry to use to perform the read.

The trick RAID-Z expansion uses, is that by 'reflowing' the data, like if you were changing the column width of some text in a text editor, the 10th word is still the 10th word, it is just that it is now on line 2 instead of line 3. So the DVA addresses in the blockpointers don't actually change, the data just moves to the address it is supposed to be at, which is now closer to the start of the vdev since we have added an additional column.

Maybe the graphics explain it better than I am. See Matt's slides:
http://www.open-zfs.org/w/images/6/68/RAIDZ_Expansion_v2.pdf

@owlshrimp
Copy link

So development of this feature is currently halted beyond an experimental proof-of-concept?

@allanjude
Copy link
Contributor

So development of this feature is currently halted beyond an experimental proof-of-concept?

An updated preview is available here: #8853

@emaste
Copy link

emaste commented Jan 24, 2024

This issue should be closed; RAID-Z expansion was merged in 5caeef0

@behlendorf
Copy link
Contributor

Thanks for pointing out this issue. Closed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Inactive Not being actively updated Type: Feature Feature request or new feature
Projects
None yet
Development

No branches or pull requests

6 participants
@behlendorf @emaste @allanjude @tronder88 @owlshrimp and others