Age | Commit message (Collapse) | Author |
|
Reduce #ifdef'ing within the control logic to make it clearer that there
are no essential differences in behaviour between the platforms.
Make installboot(8) write startup.nsh to enable simpler and more consistent
code in install.md.
Input and OK kettenis@ deraadt@
|
|
The code is common to EFI platforms, not specific to armv7.
Suggested by kettenis@
|
|
|
|
3rd (variadic) mode_t parameter is irrelevant. Many developers in the past
have passed mode_t (0, 044, 0644, or such), which might lead future people
to copy this broken idiom, and perhaps even believe this parameter has some
meaning or implication or application. Delete them all.
This comes out of a conversation where tb@ noticed that a strange (but
intentional) pledge behaviour is to always knock-out high-bits from
mode_t on a number of system calls as a safety factor, and his bewilderment
that this appeared to be happening against valid modes (at least visually),
but no sorry, they are all irrelevant junk. They could all be 0xdeafbeef.
ok millert
|
|
the bootloader. This is a no-op on architectures where such a filesystem
isn't needed.
ok krw@, deraadt@
|
|
ok deraadt@
|
|
|
|
media length check to allow EFI GPT partitions to be smaller that
the entire disk.
Consistently use GPTSECTOR instead of randomly tossing in some
literal '1's.
ok kettenis@
|
|
ok krw@, deraadt@
|
|
later.
ok kettenis gkoehler
|
|
While here, make messages more correct.
|
|
ok visa kettenis
|
|
ok kettenis
|
|
ok deraadt@
|
|
special case scripting in install.md.
(macppc still requires manual steps for HFS bootmode)
tested by krw, visa, gkoehler
|
|
sparc64 installboot(8) on softraid(4) with too large files, e.g. unstripped
builds, fails poorly with "installboot: softraid installboot failed".
This is due to the BIOCINSTALLBOOT ioctl(2) returing the default EINVAL
rather than using softraid's sr_error() interface properly; additionally,
installboot does not check for such message from the bio(4) layer.
Make the kernel generate "boot block too large" and "boot loader too large"
messages for softraid devices and have installboot act upon them analogous
to bioctl(8), by adapting its bio_status() into the new sr_status() helper.
Input, reminder to look at bioctl, same kernel diff from, OK jsing
|
|
back to a 4 byte add instruction.
We do not know *why* though, so if somebody likes a challenge...
Lots of help from semarie@ who has a few systems showing the issue.
ok deraadt@
|
|
Sorry for the inconvenience. With help from semarie@.
|
|
To do this, installboot(8) patches an extra value into biosboot(8).
Code originally from Pedro Martelletto with a twist from myself and
kettenis@; ok jsing@ kettenis@
|
|
|
|
invention; and I have had little to do with the i386_installboot.[ch] files.
This was mixed up in the giant reshuffle a few years back. ok jsing@
|
|
|
|
value < 0. errno is only updated in this case. Change all (most?)
callers of syscalls to follow this better, and let's see if this strictness
helps us in the future.
|
|
reason.
Noticed by deraadt@
|
|
the disk. Even if the OpenBSD area starts with FS_UNUSED or FS_BOOT
partitions.
ok deraadt@ as part of larger diff
|
|
|
|
It was added to support Vax. Vax is dead Jim.
ok deraadt@
|
|
This was relying on realpath(3) working for paths that don't exist
yet, which will be changing soon.
Use a combination of dirname(3), realpath(3), and basename(3) to
construct a sane path while still ensuring that the target directory
exists.
with martijn
ok martijn, deraadt
|
|
ok jasper@, jca@, deraadt@
|
|
On i386, clang puts the char array at an odd address in .bss.
ok deraadt@
|
|
ok tom@
|
|
Requested & ok jsing@
|
|
intentional functional change.
Diff from Miod.
ok millert@ deraadt@
|
|
searching for a linux partition type, now it looks for a fat type.
ok kettenis@
|
|
creating more copies.
ok deraadt@, jsing@
|
|
this isn't the kernel.
requested by deraadt@
|
|
hppa reverse-stack gives us a valuable test case, but most developers don't
have a 2nd one to proceed further with this.
ok kettenis
|
|
Pointed out by David Carlier.
|
|
lots of agreement.
|
|
Reported by Sigi Rudzio. Thanks!
"go for it" deraadt@
|
|
|
|
ok millert@ a while ago.
|
|
returning 0 or 1 since only boolean checks of the result are done.
No need for EINVAL as a return value.
|
|
non-static) by passing disk size as 2nd parameter instead of the
different structures holding the disk size info. The fifth copy of
gpt_chk_mbr() in fdisk is a little specialer. No functional change.
|
|
declarations of gpt_chk_mbr() to match prototypes in efidev.c and
softraid.c.
|
|
- in fsck_msdos, '-f' option is a no-op
- newfs_msdos does not need '-t msdos'
OK krw@
|
|
|
|
using GPT info. Otherwise GPT remnants might become revenants.
|
|
|
|
is needed.
|