Age | Commit message (Collapse) | Author |
|
ok krw@
|
|
reduce meat of editor_add() && editor_modify() to single if statements
checking for != 0 from all get_* functions. No functional change.
|
|
called during -E initialization. Thus it no longer needs the
'resolve' parameter.
|
|
new offset being in a free area.
|
|
in a semi-global. Call it whenever the count is needed rather than
trying to make sure it is called anytime the value might change.
Eliminates a bunch of now unnecessary *freep parameters and calls to
editor_countfree().
|
|
|
|
a partition can be without causing overlap. Use it to eliminate the
need to check overlap in get_size() and to make the (c)hange command
message more meaningful, i.e. mention max size allowed not the amount
of total free space.
|
|
get_offset() when the user enters an invalid value. Just abort the
partition operation and return to the main prompt. Prevents users
getting trapped because they don't know about ^D. Requested by
deraadt@.
Tweak an error message to remove ambiguity about what 'between x and
y' means.
|
|
allow editing the fsize/bsize FFS values for FS_UNUSED partitions.
ok (for at least the first bit) miod@ weingart@ beck@
|
|
editor_modify(). No functional change.
|
|
details of the free space it calculates. Gives an easy way to see all
the chunks of free sectors on the disk.
ok beck@
|
|
|
|
This removes the last need for has_overlap() to touch the free space
counter. So eliminate that parameter.
|
|
|
|
-E, zero intervening partitions as well as the specified partition so
no partitions are inadvertantly activated.
|
|
error.
1) Don't accept a partition offset in used space.
2) Make the default size the space available from the offset
to the end of the free chunk the offset is in.
3) Make (m)odify ask the questions in the same order as
the (a)dd command. i.e. filesystem type after offset/size.
4) Make it more obvious new partitions default into the largest
chunk of free space.
5) Nuke another now pointless function - next_offset().
"Looks reasonable" millert@
|
|
|
|
get_size() anyway. Allows elimination of make_contiguous() since we
no longer allow overlaps to develop.
|
|
than hand rolling another partition walking loop. No functional change.
|
|
work anyway and ignores this value. So just set it to 1 and nuke
get_cpg(), shrinking and simplifying.
ok otto@ millert@
|
|
has been removed.
ok krw
|
|
ending_sector' tests and one superfluous editor_countfree(). Rectify a
few comments.
|
|
fields.
|
|
ending_sector, so no need to check for this condition after calling
get_size().
|
|
to existing FS_BOOT and FS_UNUSED partitions via -E, tighten up error
verbiage.
Feedback from jsing@.
ok millert@.
|
|
another, valid, set while adding a partition. Lost in r1.119 when we
switched to DL_SETP* defines.
|
|
code. Makes the size prompt/helpstring/looping the same for
(a)dd/(m)odify/(c)hange.
|
|
the variable 'c' to 'partno' in editor_delete() so all editor_*
functions are even more consistant. No functional change.
|
|
counter. Call edit_countfree() instead.
Feedback from otto@.
ok millert@ "Go for it" deraadt@
|
|
there is no point in allowing the edit mode commands
(a)add/(n)ame/(m)odify/(d)elete/(c)hange to pretend to fiddle with it.
Some already checked for and avoided 'c', but do it in a consistant
manner in all cases.
ok millert@ otto@ marco@
|
|
The only visible change is allowing only one chance to specify a valid
partition letter before returning to the main edit prompt.
"This looks OK to me" millert@
|
|
consistant. No functional change.
|
|
already is used to implement 'z'. Vocal support for both commands so
keep both for now.
'sounds good' deraadt@.
|
|
the partition size with DL_SETPSIZE(pp, *freep).
|
|
ok jmc@
|
|
temporarily restoring default behaviour. This is not 100% ideal.
But this fixes editor handling in mail... bah, it is really unfortunate
that got broken
- refactor the restoration code as well, to make it simpler
ok ray
|
|
|
|
|
|
|
|
|
|
|
|
noone cares to see anymore (for those architectures where this is
important, disklabel(8) continues to enfoce the cyl-locked policy)
ok otto
|
|
'b' command, subtract the overlapping space from the free space.
fixes weird free space calculation on hppa, where converted lif
labels start at sector 1, but the 'b' area skips the first cylinder.
ok millert@
|
|
don't say "can't read master boot record". Just display an empty
disklabel. Really yucky code that needs a lot of tough love, but
marginally less confusing until then.
ok deraadt@
|
|
converted to v1 format before being used.
ok deraadt@ otto@
|
|
|
|
bit D_VENDOR if they face the Sun cylinder / start-at-zero semantics. Other
disklabels found on these architectures do not have these issues, and do
not need to be constrained. checked by otto
|
|
|
|
the result was occasional disklabel trashing as a result of disklabel -r,
and perhaps other requests to the command; ok krw
|
|
Also, on systems with sun labels, don't allow the user to create a
partition with fewer than a cylinder's worth of sectors.
OK otto@
|