Differential D56534

geom manuals: Use ISO/IEC byte units
ClosedPublic
Actions

Authored by ziaee on Mon, Apr 20, 5:38 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Apr 25, 6:34 AM

	Unknown Object (File)
	Sat, Apr 25, 6:02 AM

	Unknown Object (File)
	Sat, Apr 25, 5:28 AM

	Unknown Object (File)
	Sat, Apr 25, 5:10 AM

	Unknown Object (File)
	Sat, Apr 25, 5:03 AM

	Unknown Object (File)
	Sat, Apr 25, 4:27 AM

	Unknown Object (File)
	Fri, Apr 24, 7:57 PM

	Unknown Object (File)
	Fri, Apr 24, 6:07 AM

Subscribers

Details

Reviewers

Group Reviewers

Commits

rG7ac7604b9141: geom manuals: Clarify units
rG975e3605ebb1: geom manuals: Clarify units

Summary

The gpart manual says that sizes are specified in blocks, unless an SI
unit suffix is provided. This confuses new operators because GEOM uses
binary bytes, a large difference at modern storage pool sizes. Rewrite
suffixes in all GEOM manuals to consistently clarify this, matching what
we and the rest of the industry have been doing in other documentation.

MFC after: 3 days
Reported by: bbaovanc <bbaovanc@bbaovanc.com>

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 72381
Build 69264: arc lint + arc unit

Event Timeline

ziaee created this revision.Mon, Apr 20, 5:38 PM

Herald added a subscriber: imp. · View Herald TranscriptMon, Apr 20, 5:38 PM

ziaee requested review of this revision.Mon, Apr 20, 5:38 PM

Harbormaster completed remote builds in B72379: Diff 175928.Mon, Apr 20, 5:38 PM

drop the IEC, it's technically an anachronism, and "binary byte sizes"
isn't any less clear.

Harbormaster completed remote builds in B72380: Diff 175933.Mon, Apr 20, 5:45 PM

missed a spot, thanks bbaovanc!

Harbormaster completed remote builds in B72381: Diff 175934.Mon, Apr 20, 5:51 PM

missed another spot

Harbormaster completed remote builds in B72382: Diff 175935.Mon, Apr 20, 5:52 PM

use non-breaking spaces for unit sizes, unless we already use a hyphen on that page.

Harbormaster completed remote builds in B72386: Diff 175942.Mon, Apr 20, 8:52 PM

LGTM

This revision is now accepted and ready to land.Mon, Apr 20, 8:56 PM

Closed by commit rG975e3605ebb1: geom manuals: Clarify units (authored by ziaee). · Explain WhyMon, Apr 20, 9:24 PM

This revision was automatically updated to reflect the committed changes.

ziaee added a commit: rG975e3605ebb1: geom manuals: Clarify units.

ziaee added a commit: rG7ac7604b9141: geom manuals: Clarify units.Thu, Apr 23, 8:29 PM

Revision Contents
Changeset List

Path

Size

lib/

geom/

cache/

4 lines

eli/

2 lines

part/

42 lines

sbin/

ggate/

ggatec/

6 lines

ggated/

4 lines

Diff 175934

lib/geom/cache/gcache.8

Loading...

lib/geom/eli/geli.8

Loading...

lib/geom/part/gpart.8

Loading...

sbin/ggate/ggatec/ggatec.8

Loading...

sbin/ggate/ggated/ggated.8

Loading...