Page MenuHomeFreeBSD

geom_vinum: add gone_in deprecation notice
Needs ReviewPublic

Authored by emaste on Mar 29 2021, 5:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sep 24 2024, 3:30 PM
Unknown Object (File)
Sep 18 2024, 12:28 PM
Unknown Object (File)
Sep 9 2024, 12:17 AM
Unknown Object (File)
Sep 6 2024, 10:15 PM
Unknown Object (File)
Sep 6 2024, 7:42 PM
Unknown Object (File)
Aug 28 2024, 8:59 PM
Unknown Object (File)
Aug 25 2024, 11:20 PM
Unknown Object (File)
Aug 19 2024, 10:52 AM
Subscribers
None

Details

Reviewers
imp
jhb
Summary

Fixes: b218441ac074 ("gvinum: add deprecation notice")

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

emaste created this revision.

Followon to D29424

I'm not sure where the best place to put this is. At the end of gv_drive_tasted may be a bit spammy, but has a higher likelihood of being noticed when e.g. adding a disk etc. I could just put it in gv_init instead.

Have you tried testing it? Maybe try to 'gvinum create' across two memory disks or ZVOLs to see if the warning works? You could maybe then do 'gvinum detach' and 'gvinum attach' to replicate boot-time probing. That is what I did when testing deprecation warnings for GELI algorithms I deprecated for 13. I don't have a good opinion on where the warning should go, and if it works in testing I'm fine with whatever works.

I will make sure it works either way, but am unsure about the tradeoff between making sure the message is noticed and being overly spammy. At one end we can emit the message once during startup, at the other for every activity on a disk.

For GELI I took the strategy of spamming a message each time a new volume was created with a deprecated algorithm via geli init, and a message each time an existing volume created a new provider (so if you booted with 5 GELI volumes and each used a deprecated algorithm, you got 5 warnings IIRC). The geli changes for this were in R10:5c420aae3b18027809507dc9142182d4290897bf. Unlike some other warnings I added for IPsec, I didn't bother with rate limiting.