stand/geli: Allocate saved_keys instead of making it static
AcceptedPublic
Actions

Authored by imp on Sep 14 2022, 7:50 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Aug 19, 1:20 PM

	Unknown Object (File)
	Sun, Aug 9, 10:13 AM

	Unknown Object (File)
	Sat, Aug 8, 12:40 AM

	Unknown Object (File)
	Tue, Aug 4, 8:41 AM

	Unknown Object (File)
	Sun, Aug 2, 5:14 AM

	Unknown Object (File)
	Jul 15 2026, 7:57 PM

	Unknown Object (File)
	Jul 12 2026, 5:36 PM

	Unknown Object (File)
	Jul 1 2026, 7:50 PM

View All Files

Subscribers

dab

Details

Reviewers

manu
kevans
tsoome

Summary

The saved_keys buffer is 4k and was allocated in bss. Dynamically
allocate it at runtime. This saves 4k of space in the lower 640k.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 47392
Build 44279: arc lint + arc unit

Event Timeline

imp created this revision.Sep 14 2022, 7:50 PM

Herald added a reviewer: manu. · View Herald TranscriptSep 14 2022, 7:50 PM

Herald added a subscriber: dab. · View Herald Transcript

imp requested review of this revision.Sep 14 2022, 7:50 PM

Harbormaster completed remote builds in B47392: Diff 110568.Sep 14 2022, 7:50 PM

imp added a child revision: D36577: stand/elf: Only support swapping headers on powerpc..Sep 14 2022, 7:53 PM

imp added reviewers: kevans, tsoome.

imp added a parent revision: D36575: stand: Allocate bootinfo rather than have it be static.

tsoome accepted this revision.Sep 14 2022, 8:29 PM

This revision is now accepted and ready to land.Sep 14 2022, 8:29 PM

Revision Contents
Changeset List

Path

Size

stand/

libsa/

geli/

geliboot.c

12 lines

Diff 110568

View Options

stand/libsa/geli/geliboot.c

stand/geli: Allocate saved_keys instead of making it staticAcceptedPublicActions