Page MenuHomeFreeBSD

loader: zfs toplevel vdev must have spa set.
ClosedPublic

Authored by tsoome on Nov 9 2016, 11:13 PM.
Tags
None
Referenced Files
F81657231: D8487.diff
Fri, Apr 19, 1:47 PM
Unknown Object (File)
Mon, Apr 8, 8:38 PM
Unknown Object (File)
Mon, Apr 8, 8:36 PM
Unknown Object (File)
Mar 14 2024, 7:53 AM
Unknown Object (File)
Feb 11 2024, 5:51 AM
Unknown Object (File)
Dec 29 2023, 5:03 AM
Unknown Object (File)
Dec 20 2023, 6:20 AM
Unknown Object (File)
Dec 19 2023, 3:19 AM

Details

Summary

With salted checksums the salt setup is using spa to store runtime salt,
and for the IO, the spa is often fetched from vdev structure.

Unfortunately the vdev->spa is set only for leaf devices, which leaves
mirror and raidz pool toplevel devices with vdev->spa unset, and this
does break salt based checksums and will cause system reset while booting
from such pool.

Test Plan

Tested with 2+1 raidz and zpool create -O checksum=skein.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tsoome retitled this revision from to loader: zfs toplevel vdev must have spa set..
tsoome updated this object.
tsoome edited the test plan for this revision. (Show Details)
tsoome added reviewers: allanjude, imp.

With this patch applied on top of svn head r308477, I built a custom memstick image, wiped the harddrives and reran my build guide steps including creation of the RAIDZ zpool with "-O checksum=skein" option and can confirm the system boots, so although I did not test without this patch applied, I think it's safe to say the patch fixed things given that there have been no other relevant changes in sys/boot between the revision I tested with previously (r307747) and r308477.

allanjude edited edge metadata.

approved for commit

Remember to set the MFC field

This revision is now accepted and ready to land.Nov 17 2016, 6:58 PM
imp edited edge metadata.
This revision was automatically updated to reflect the committed changes.