Details

Reviewers

Commits

rS293724: Enable warnings in EFI boot code

Summary

Set WARNS if not set for EFI boot code and fix the issues highlighted by setting it.

Most components are set to WARNS level 6 with few being left at lower levels due to the amount of changes needed to fix at higher levels.

Test Plan

make buildenv
cd sys/boot/efi
make clean
make

Once built boot using the created boot.efifat image under qemu.

Also:

make tinderbox JFLAG=-j24 MAKE_JUST_WORLDS=1

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

smh updated this revision to Diff 12069.Jan 9 2016, 3:49 AM

smh retitled this revision from to Enable warnings in EFI boot code.

smh updated this object.

smh edited the test plan for this revision. (Show Details)

smh added a reviewer: emaste.

Herald added subscribers: rpokala, imp. · View Herald TranscriptJan 9 2016, 3:49 AM

Restore removed blank line and correct spacing highlighted by phabricator view.

Additional spacing fix

emaste added inline comments.Jan 11 2016, 1:04 PM

sys/boot/efi/Makefile
5–6 ↗	(On Diff #12111)	This makefile doesn't build anything, so this will have no effect?
sys/boot/efi/boot1/Makefile
7–8 ↗	(On Diff #12111)	Seems WARNS conventionally goes after PROG= etc. in other Makefiles.
sys/boot/efi/include/efi_nii.h
29–30 ↗	(On Diff #12111)	Looks good, but I'd suggest removing the line continuation (even though it's then longer than 80 cols) to reduce diffs with other versions of this file. Here's NetBSD's, which seems to come via gnu-efi: http://cvsweb.netbsd.org/bsdweb.cgi/src/sys/external/bsd/gnu-efi/dist/inc/efi_nii.h?annotate=1.1.1.1&only_with_tag=MAIN #define EFI_NETWORK_INTERFACE_IDENTIFIER_PROTOCOL \ { 0xE18541CD, 0xF755, 0x4f73, {0x92, 0x8D, 0x64, 0x3C, 0x8A, 0x79, 0xB2, 0x29} } I'd say we should follow that formatting.

Address feedback from emaste:

Remove WARNS from sys/boot/efi/Makefile as it has no effect
Move WARNS below PROG= etc
Remove split of GUID / PROTOCOL defines even though they break style(9) 80 col limit, to reduce diff and match NetBSD.

smh marked 3 inline comments as done.Jan 11 2016, 1:35 PM

emaste mentioned this in D4713: loader bcache redesign experiment.Jan 11 2016, 1:46 PM

Spacing changes for GUID / PROTOCOL structs requested emaste

Fix ARM warnings highlighted by tinderbox.

Herald added a subscriber: andrew. · View Herald TranscriptJan 11 2016, 8:46 PM

smh edited the test plan for this revision. (Show Details)Jan 12 2016, 2:08 AM

Closed by commit rS293724: Enable warnings in EFI boot code (authored by smh). · Explain WhyJan 12 2016, 2:18 AM

This revision was automatically updated to reflect the committed changes.

Enable warnings in EFI boot codeClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 12140

head/sys/boot/arm64/libarm64/cache.c

head/sys/boot/common/load_elf.c

head/sys/boot/common/load_elf_obj.c

head/sys/boot/common/misc.c

head/sys/boot/common/module.c

head/sys/boot/common/part.c

head/sys/boot/common/self_reloc.c

head/sys/boot/common/ufsread.c

head/sys/boot/efi/boot1/Makefile

head/sys/boot/efi/boot1/boot1.c

head/sys/boot/efi/fdt/Makefile

head/sys/boot/efi/fdt/efi_fdt.c

head/sys/boot/efi/include/arm64/efibind.h

head/sys/boot/efi/include/efi_nii.h

head/sys/boot/efi/include/efiapi.h

head/sys/boot/efi/include/eficon.h

head/sys/boot/efi/include/eficonsctl.h

head/sys/boot/efi/include/efidevp.h

head/sys/boot/efi/include/efierr.h

head/sys/boot/efi/include/efifpswa.h

head/sys/boot/efi/include/efigop.h

head/sys/boot/efi/include/efilib.h

head/sys/boot/efi/include/efinet.h

head/sys/boot/efi/include/efipciio.h

head/sys/boot/efi/include/efiprot.h

head/sys/boot/efi/include/efipxebc.h

head/sys/boot/efi/include/efiser.h

head/sys/boot/efi/include/efiuga.h

head/sys/boot/efi/libefi/Makefile

head/sys/boot/efi/libefi/efi_console.c

head/sys/boot/efi/libefi/efipart.c

head/sys/boot/efi/libefi/libefi.c

head/sys/boot/efi/loader/Makefile

head/sys/boot/efi/loader/arch/amd64/elf64_freebsd.c

head/sys/boot/efi/loader/arch/amd64/framebuffer.c

head/sys/boot/efi/loader/arch/arm/exec.c

head/sys/boot/efi/loader/arch/arm64/exec.c

head/sys/boot/efi/loader/autoload.c

head/sys/boot/efi/loader/bootinfo.c

head/sys/boot/efi/loader/copy.c

head/sys/boot/efi/loader/devicename.c

head/sys/boot/efi/loader/loader_efi.h

head/sys/boot/efi/loader/main.c

head/sys/boot/i386/libi386/smbios.c

Enable warnings in EFI boot code
ClosedPublic
Actions

Revision Contents
Changeset List