Page MenuHomeFreeBSD

stand: Move i386_devdesc to a union
ClosedPublic

Authored by imp on Jul 27 2022, 3:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 7, 6:22 AM
Unknown Object (File)
Mon, Nov 25, 12:35 PM
Unknown Object (File)
Sat, Nov 16, 3:57 PM
Unknown Object (File)
Sat, Nov 16, 2:50 PM
Unknown Object (File)
Sat, Nov 16, 2:24 PM
Unknown Object (File)
Sat, Nov 16, 1:02 PM
Unknown Object (File)
Nov 8 2024, 4:36 AM
Unknown Object (File)
Nov 1 2024, 6:48 PM
Subscribers

Details

Summary

Rather than have the magic, hand-crafted fields that have to align with
fields in other structures at the end of i386_devdesc, make it into
anonymous union and adjust the code accordingly. This is safer and
similar to what CAM does.

Sponsored by: Netflix

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 46622
Build 43511: arc lint + arc unit

Event Timeline

imp requested review of this revision.Jul 27 2022, 3:39 PM

I like it... I'm not really a fan of the naming, but I can't exactly come up with a better suggestion so take that for the very little that it's worth.

This revision is now accepted and ready to land.Jul 27 2022, 5:50 PM

Let's try 'disk' instead of ddd and 'zfs' instad of zdd.

This revision now requires review to proceed.Jul 28 2022, 2:38 PM

That seems intuitive enough, thanks! :-)

This revision is now accepted and ready to land.Jul 28 2022, 3:18 PM
This revision now requires review to proceed.Jul 28 2022, 4:52 PM
This revision was not accepted when it landed; it landed in state Needs Review.Aug 11 2022, 4:27 PM
This revision was automatically updated to reflect the committed changes.