nvme: Use size_t for payload size
ClosedPublic
Actions

Authored by des on Feb 13 2026, 5:58 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Jun 18, 2:48 PM

	Unknown Object (File)
	Thu, Jun 18, 1:42 PM

	Unknown Object (File)
	Sun, May 24, 9:08 AM

	Unknown Object (File)
	May 18 2026, 10:22 AM

	Unknown Object (File)
	May 18 2026, 10:18 AM

	Unknown Object (File)
	May 12 2026, 2:47 PM

	Unknown Object (File)
	May 12 2026, 7:18 AM

	Unknown Object (File)
	May 11 2026, 5:31 PM

View All 46 Files

Subscribers

dab

Details

Reviewers

imp

Group Reviewers

Klara

Commits

rG3562c80f4144: nvme: Use size_t for payload size
rGbe7e4dc878ea: nvme: Use size_t for payload size

Summary

Using uint32_t here can cause spurious compiler warnings. I assume it
was done to constrain the range; use a KASSERT instead.

Sponsored by: Klara, Inc.
Sponsored by: NetApp, Inc.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

des created this revision.Feb 13 2026, 5:58 PM

Herald added a subscriber: dab. · View Herald TranscriptFeb 13 2026, 5:58 PM

des requested review of this revision.Feb 13 2026, 5:58 PM

Harbormaster completed remote builds in B70665: Diff 171876.Feb 13 2026, 5:58 PM

imp accepted this revision.Feb 13 2026, 6:05 PM

This revision is now accepted and ready to land.Feb 13 2026, 6:05 PM

Closed by commit rGbe7e4dc878ea: nvme: Use size_t for payload size (authored by des). · Explain WhyFeb 13 2026, 6:38 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rGbe7e4dc878ea: nvme: Use size_t for payload size.

des added a commit: rG3562c80f4144: nvme: Use size_t for payload size.Feb 18 2026, 12:26 AM

Revision Contents
Changeset List

Path

Size

sys/

dev/

nvme/

nvme_private.h

4 lines

Diff 171879

View Options

sys/dev/nvme/nvme_private.h

nvme: Use size_t for payload sizeClosedPublicActions