Page MenuHomeFreeBSD
Differential D57991

linuxkpi: Move LINUXKPI_PROT_VALID higher
Needs ReviewPublic
Actions

Authored by andrew on Jul 1 2026, 10:10 AM.
Tags
Referenced Files
Unknown Object (File)
Sun, Oct 4, 4:19 AM
Unknown Object (File)
Fri, Oct 2, 9:19 PM
Unknown Object (File)
Thu, Oct 1, 10:05 PM
Unknown Object (File)
Thu, Oct 1, 2:09 AM
Unknown Object (File)
Wed, Sep 30, 10:42 PM
Unknown Object (File)
Mon, Sep 28, 3:46 AM
Unknown Object (File)
Sat, Sep 26, 10:31 AM
Unknown Object (File)
Fri, Sep 11, 3:20 PM
View All 63 Files
Subscribers
ashafer
bz
dumbbell
emaste
imp
linuxkpi

Details

Reviewers
None
Group Reviewers
arm64
linuxkpi
Summary

Move LINUXKPI_PROT_VALID and LINUXKPI_CACHE_MODE_SHIFT out of the
bottom 8 bits. This allows the entire vm_prot_t to be stored in the
LinuxKPI pgprot_t.

This will be needed when we add more flags to vm_prot_t, e.g. for MTE
on arm64.

Sponsored by: Arm Ltd

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 74453
Build 71336: arc lint + arc unit

Event Timeline

andrew created this revision.Jul 1 2026, 10:10 AM
Herald added subscribers: linuxkpi, emaste, imp. · View Herald TranscriptJul 1 2026, 10:10 AM
andrew requested review of this revision.Jul 1 2026, 10:10 AM
Harbormaster completed remote builds in B74453: Diff 181163.Jul 1 2026, 10:10 AM
andrew added a parent revision: D57990: amr64: Switch from PV_MTE_TAGGED to PV_MTE_CLEAN.Jul 1 2026, 10:10 AM
andrew added a child revision: D57992: arm64: Add mte_enabled to see if MTE is enabled.
andrew added a reviewer: linuxkpi.Jul 1 2026, 10:11 AM
emaste added a project: linuxkpi.Aug 17 2026, 4:59 PM
andrew added a comment.Thu, Sep 10, 1:15 PM

Is anyone who knows about LinuxKPI able to check this? I don't have anything that uses it, but need to move the value for MTE.

bz added subscribers: ashafer, dumbbell, bz.Thu, Sep 10, 3:55 PM

This came from 1ea4c85781e94 and was revised in ebf854802d83a.

@ashafer has an open review ( https://reviews.freebsd.org/D59436 ) and I was trying to understand some of this recently.
If you can give me a few days I'll try to fully wrap my head around this.

I assume @ashafer and @dumbbell can test this for nvidia and i915/amdgpu.

ashafer added a comment.Fri, Sep 11, 2:04 PM

This seems reasonable to me

bz added a comment.Thu, Sep 17, 9:23 PM

I still do not get how this is supposed to fully work but I assume the shortcut from before remains right. I'd say for me you can add:

OKed: bz

Revision Contents
Changeset List

PathSize
sys/
compat/
linuxkpi/
common/
include/
linux/
6 lines

Diff 181163

View Options

sys/compat/linuxkpi/common/include/linux/page.h

Loading...