Obtained from: illumos
Details
Details
Diff Detail
Diff Detail
- Repository
- rG FreeBSD src repository
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
Since the "serial" taken from the configuration is directly copied to become the virtio-blk device's ident, should we call this "ident" simply? Looks good to me otherwise.
EDIT: I missed the fact that it was obtained from Illumos.
Comment Actions
I think the goal was to also be similar to the name used for the AHCI and NVMe device models which both use "ser" for this.
Comment Actions
Right - it was added as ser to match the others. Accepting the alternative of serial is there because that's what some illumos distributions had before the configuration management refactoring.
Comment Actions
Apologies for getting to this late, but I tested this with ser, serial, and with no options. Seemed to work as expected