Differential D54840

sys: Use __is_aligned and __align_down for some kstack alignment operations
AcceptedPublic
Actions

Authored by jhb on Fri, Jan 23, 6:45 PM.

Tags

None

Referenced Files

	F142949095: D54840.diff
	Sat, Jan 24, 9:42 PM

	Unknown Object (File)
	Sat, Jan 24, 10:42 AM

	Unknown Object (File)
	Sat, Jan 24, 4:18 AM

	Unknown Object (File)
	Sat, Jan 24, 2:07 AM

	Unknown Object (File)
	Sat, Jan 24, 1:23 AM

	Unknown Object (File)
	Fri, Jan 23, 10:23 PM

	Unknown Object (File)
	Fri, Jan 23, 9:24 PM

	Unknown Object (File)
	Fri, Jan 23, 9:24 PM

Subscribers

Details

Reviewers

jhibbits
adrian
fuz
kib

Group Reviewers

Summary

Effort: CHERI upstreaming
Sponsored by: AFRL, DARPA

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 70078
Build 66961: arc lint + arc unit

Event Timeline

jhb created this revision.Fri, Jan 23, 6:45 PM

Herald added a subscriber: imp. · View Herald TranscriptFri, Jan 23, 6:45 PM

jhb requested review of this revision.Fri, Jan 23, 6:45 PM

Herald added a reviewer: cheri. · View Herald TranscriptFri, Jan 23, 6:45 PM

Harbormaster completed remote builds in B70078: Diff 170283.Fri, Jan 23, 6:45 PM

jhb added a parent revision: D54839: powerpc: Fix alignment of initial PCB on kstack.Fri, Jan 23, 6:45 PM

jhb added a reviewer: kib.Fri, Jan 23, 6:46 PM

kib accepted this revision.Fri, Jan 23, 6:50 PM

This revision is now accepted and ready to land.Fri, Jan 23, 6:50 PM

Note: you can test this for ppc64 by using power8/power9 VMs:

https://wiki.freebsd.org/powerpc/QEMU/Pseries8

https://wiki.freebsd.org/powerpc/QEMU/Pseries9

(I'm still working on fixing up what's broken for ppc970 (g5) and powerpc 32 bit on g3/g4 CPUs in QEMU)

jhibbits accepted this revision.Sun, Jan 25, 3:07 AM

In D54840#1253684, @adrian wrote:

Note: you can test this for ppc64 by using power8/power9 VMs:

https://wiki.freebsd.org/powerpc/QEMU/Pseries8

https://wiki.freebsd.org/powerpc/QEMU/Pseries9

(I'm still working on fixing up what's broken for ppc970 (g5) and powerpc 32 bit on g3/g4 CPUs in QEMU)

I only really tested that this compiled for powerpc. These are some fixes I found while working on some larger tree-sweeps for CHERI that involve replacing vm_offset_t with void * in various APIs.

Revision Contents
Changeset List

Path

Size

sys/

amd64/

amd64/

3 lines

i386/

i386/

3 lines

powerpc/

powerpc/

4 lines

5 lines

4 lines

Diff 170283

sys/amd64/amd64/vm_machdep.c

Loading...

sys/i386/i386/vm_machdep.c

Loading...

sys/powerpc/powerpc/exec_machdep.c

Loading...

sys/powerpc/powerpc/machdep.c

Loading...

sys/powerpc/powerpc/vm_machdep.c

Loading...