nvmecontrol: Remove an incorrect use of PAGE_SIZE
ClosedPublic
Actions

Authored by andrew on Feb 17 2026, 5:46 PM.

Details

Reviewers

imp
jhb

Commits

rGdb2a34bc4528: nvmecontrol: Remove an incorrect use of PAGE_SIZE
rGbfb7c81f93b5: nvmecontrol: Remove an incorrect use of PAGE_SIZE

Summary

The mdts value is in terms of the nvme page size, not the host page
size. On many architectures these are both 4k, however on arm64 it is
possible to build a system with the host page size of 16k.

Use NVME_MPS_SHIFT to get the correct nvme page shift.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

andrew created this revision.Feb 17 2026, 5:46 PM

Herald added a subscriber: dab. · View Herald TranscriptFeb 17 2026, 5:46 PM

andrew requested review of this revision.Feb 17 2026, 5:46 PM

Harbormaster completed remote builds in B70734: Diff 172081.Feb 17 2026, 5:46 PM

Nice catch

This revision is now accepted and ready to land.Feb 17 2026, 7:34 PM

Closed by commit rGbfb7c81f93b5: nvmecontrol: Remove an incorrect use of PAGE_SIZE (authored by andrew). · Explain WhyFeb 18 2026, 3:16 PM

This revision was automatically updated to reflect the committed changes.

andrew added a commit: rGbfb7c81f93b5: nvmecontrol: Remove an incorrect use of PAGE_SIZE.

andrew added a commit: rGdb2a34bc4528: nvmecontrol: Remove an incorrect use of PAGE_SIZE.Wed, Apr 8, 2:02 PM

Revision Contents
Changeset List

Path

Size

sbin/

nvmecontrol/

identify_ext.c

2 lines

Diff 172159

View Options

sbin/nvmecontrol/identify_ext.c

nvmecontrol: Remove an incorrect use of PAGE_SIZEClosedPublicActions