While there, also simplify the creation of label paths a little bit
by requiring the / suffix for label directory prefixes (ld_dir renamed
to ld_dirprefix to indicate the change) and stop defining macros for
these when they are only used once.
Details
Details
test glabel behavior after change
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
@cem -- Note that D24968 needs to be amended due to conflict (I have it locally with an amendment to D25168 to make geli detach -l work again with aliases), I've uploaded these to https://github.com/delphij/freebsd/commits/scratch/geom .
Comment Actions
No worries, thanks for keeping it in mind.
This change looks good to me, sorry about the delay.
sys/geom/label/g_label.c | ||
---|---|---|
418โ421 โ | (On Diff #74224) | It might be nice to encode this in the g_labels[] table somehow instead. |
Comment Actions
ACK. I think this can be accomplished with a more intrusive change so let's do it in a later refactor instead.