nvme_ctrlr_linux_passthru_cmd: correct size of upages_small array
ClosedPublic
Actions

Authored by rlibby on Apr 12 2026, 8:14 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Jul 15, 7:16 AM

	Unknown Object (File)
	Wed, Jul 15, 7:16 AM

	Unknown Object (File)
	Thu, Jul 2, 2:58 PM

	Unknown Object (File)
	Wed, Jul 1, 10:05 PM

	Unknown Object (File)
	Jun 14 2026, 12:52 AM

	Unknown Object (File)
	May 27 2026, 6:35 PM

	Unknown Object (File)
	May 27 2026, 8:28 AM

	Unknown Object (File)
	May 27 2026, 5:06 AM

View All 41 Files

Subscribers

dab

imp

Details

Reviewers

imp

Commits

rG8085c5a5c1e5: nvme_ctrlr_linux_passthru_cmd: correct size of upages_small array

Summary

The size broke when upages was converted from array to double pointer.

Reported-by: gcc -Wsizeof-pointer-div
Fixes: 82ff1c334b97 ("nvme: Allow larger user request sizes")

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

rlibby created this revision.Apr 12 2026, 8:14 PM

Herald added subscribers: dab, imp. · View Herald TranscriptApr 12 2026, 8:14 PM

rlibby requested review of this revision.Apr 12 2026, 8:14 PM

Harbormaster completed remote builds in B72158: Diff 175378.Apr 12 2026, 8:15 PM

rlibby added a reviewer: imp.Apr 12 2026, 8:28 PM

<doh!>
Thanks!

This revision is now accepted and ready to land.Apr 12 2026, 8:46 PM

Closed by commit rG8085c5a5c1e5: nvme_ctrlr_linux_passthru_cmd: correct size of upages_small array (authored by rlibby). · Explain WhyApr 13 2026, 12:21 AM

This revision was automatically updated to reflect the committed changes.

rlibby added a commit: rG8085c5a5c1e5: nvme_ctrlr_linux_passthru_cmd: correct size of upages_small array.

Revision Contents
Changeset List

Path

Size

sys/

dev/

nvme/

nvme_ctrlr.c

4 lines

Diff 175390

View Options

nvme_ctrlr_linux_passthru_cmd: correct size of upages_small arrayClosedPublicActions