Age | Commit message (Collapse) | Author |
|
Careful second audit by millert
|
|
instead of binary operators for comparison.
OK krw@
|
|
|
|
DEV_BSIZE blocks. Counts of possibly other-sized blocks (e.g. disk
sector addresses) are u_int64_t. The values stored in disklabels are
counts of possibly other-sized blocks and hence should be handled with
u_int64_t variables.
Start enforcing this rule. No intended functional change.
Rule strongly suggested by deraadt@
|
|
No doubt quiets one or more of the fringe compilers out there. Use
variable 'part' rather than recalculating same value in the subsequent
printf().
From dhill. ok otto@
|
|
test period; i think 3 years ago the last bugs fell out.
ok otto beck others
|
|
ok jsing
|
|
(2008), when '-r' support was removed. Spotted by jsg@ with the
assistance of clang.
|
|
There's nothing special about -c and -d requiring them to be
separated from other flags or specified as mutually exclusive. So
just show a single block of flags. Some flag combos are silly
but harmless.
Move all -B options inside NUMBOOT #if, eliminating confusing
duplication of -w and -R usage lines. And some string
declaration dancing.
Since we only have NUMBOOT = 1, no need to try and finesse
output for different values of NUMBOOT.
Line shit up.
|
|
it and the associated 'bootxx' variable. Update man page accordingly.
|
|
'N' command line options.
ok matthew@
|
|
info from the physical disk after 'disklabel -e' writes the label
to disk. Saves having to reboot or do other contortions to initialize
these fields. Toss in same treatment for total sectors so all fields
forcibly set by kernel have current values.
0'ing of the bounds noted by matthew@.
'lovely' deraadt@
|
|
|
|
in disklabel(8)
pointed out by jmc@
|
|
|
|
information to the specified file. After every label write, read the
label to get current UID info.
Some internal code cleanup with no intended functional change.
ok deraadt@
|
|
|
|
naming.
ok krw@
|
|
a disklabel UID. Based on a diff from josh@elsasser.org. Resolves PR6471.
ok krw@
|
|
uid_print() function.
ok krw@
|
|
|
|
a more useful error message for invalid ones. ok deraadt
|
|
"total sectors" to "boundstart" and "boundend" in the list of fields
that are left alone during a RESTORE operation.
ok deraadt@
|
|
This ensures that all physical fields are filled in with current
values. Lack of values (boundstart, boundend in particular) noted
by ray@.
ok deraadt@
|
|
the value 0 to one function call. Use 0 there, eliminate the
variable and the pointless verbose verbiage that always displayed
the same value.
ok deraadt@ matthew@
|
|
ok krw@ jsing@
|
|
gets rid of #include <sys/dkio.h> in sys/ioctl.h and adds #include
<sys/dkio.h> to the places that actually want and use the disk
ioctls.
this became an issue when krw@'s X build failed when he was testing
a change to dkio.h.
tested by krw@
help from and ok miod@
|
|
sscanf().
ok marco@
|
|
a new uid instead.
ok krw@ marco@
|
|
ok deraadt@ krw@
|
|
identifier, allowing the disk to be identified without relying on the
device name.
ok deraadt@ krw@ beck@ marco@ todd@
|
|
unmaintainable). these days, people use source. these id's do not provide
any benefit, and do hurt the small install media
(the 33,000 line diff is essentially mechanical)
ok with the idea millert, ok dms
|
|
is used. discussed and ok krw@
|
|
bounds information, ie. the zone of the disk that OpenBSD can use. Have each
pre-disklabel parser (MBR, DPME, or per-arch MD disklabel parsers) figure out
this area and pass it up to userland. Then, delete all the same disk parsing
code from disklabel(8) since the kernel passes it up. Lots and lots of - signs
in the disklabel(8) code.
Tested on as many platforms as possible, the fallout will be repaired as time
goes on. To test, use disklabel -d <drive> and validate that the bounds do not
overlap any boot blocks. This same information is used by disklabel -A...
OK for the concept from krw, miod, and drahn
|
|
Both cause partition sizes to be displayed using a human readable
format with the units automatically chosen by looking at the smallest
partition in the disk label. Remove forceable humanization in 'A'
code and use '-h' in install scripts.
Prodded & ok deraadt@, verbiage tweaks from jmc@
|
|
If we are humanizing then add a comment to 'total sectors' line
rather than replacing 'total sectors' with a new header.
Suggestion and ok deraadt@
|
|
for success. Fix a couple of bugs where errors in checklabel() got
lost.
Make Editor 'q' and 'x' commands exit with 0 (ok) rather than 1.
i.e. non-zero exit value now reserved for failure, not a decision
to leave the disklabel unchanged. This allows the install script
to use the exit value to catch failures to write a disklabel.
|
|
written, rather than every time editor_alloc() is called. Respect
'-n' for mountpoint info. Reduces surprise appearance of files when
you 'x' out of Editor.
"my sunglasses did not go dark" deraadt@
|
|
|
|
or without the '-f' option being present. Now you can see in Editor
mode or 'disklabel <dev>' where the partitions that exist will be
mounted in the current configuration.
Tweaks (and libc fixes) from, and ok deraadt@
|
|
offset/size information in MB if the disk is <10G, and in GB
otherwise. '-p' will override this default.
|
|
now show the planned mountpoint layout.
ok deraadt@
|
|
'lab'. So there is no need for the return value. Just use &lab
wherever it was used. No functional change.
Slightly different version read ok to blambert@
|
|
just display the auto-allocation plan otherwise. Don't overwrite any
spoofed partitions, insert auto allocated partitions where we can.
This is much more install script and weird architecture friendly. I'm
talking about you, macppc.
Many suggestions and code from otto@, feedback from many, cattle
prod applied by deraadt@.
ok deraadt@
"if it works for >8GB ... put it in" miod@
"to be clear, ok" todd@
|
|
and not DL_GETPSIZE().
|
|
boot loaders and update list of NUMBOOT>0 archs.
ok deraadt@
|
|
|
|
ok krw@
|
|
expectation that -R will read the output of disklabel. I suspect we
will find another way to do this; ok otto
|
|
display the partition info by default as is done in the E(ditor). The
physical info is now only displayed if '-v' is specified.
ok deraadt@
|