nvme.ko: Don't include nvme_util.c
ClosedPublic
Actions

Authored by jhb on Jun 5 2025, 8:59 PM.

Tags

None

Referenced Files

	F151518923: D50705.id.diff
	Wed, Apr 8, 11:47 PM

	Unknown Object (File)
	Tue, Apr 7, 4:31 AM

	Unknown Object (File)
	Sun, Mar 29, 4:34 PM

	Unknown Object (File)
	Sat, Mar 28, 7:13 PM

	Unknown Object (File)
	Wed, Mar 25, 7:09 PM

	Unknown Object (File)
	Wed, Mar 25, 12:25 PM

	Unknown Object (File)
	Mar 9 2026, 3:51 PM

	Unknown Object (File)
	Mar 9 2026, 8:01 AM

View All 61 Files

Subscribers

dab

Details

Reviewers

imp

Commits

rG45165aab2d22: nvme.ko: Don't include nvme_util.c

Summary

This file is already included in cam.ko which is nvme.ko depends on
unconditionally.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 64664
Build 61548: arc lint + arc unit

Event Timeline

jhb created this revision.Jun 5 2025, 8:59 PM

Herald added a subscriber: dab. · View Herald TranscriptJun 5 2025, 8:59 PM

jhb requested review of this revision.Jun 5 2025, 8:59 PM

Harbormaster completed remote builds in B64664: Diff 156590.Jun 5 2025, 8:59 PM

I've tested this with numerous kernels combinations:

kernel without CAM but with nvme + nvd
kernel without nvme (MINIMAL was too big) but with CAM, nvme.ko can be kldloaded and works
kernel without CAM or nvme, nvme.ko can be kldloaded and pulls in cam.ko as an automatic dependency and works

imp accepted this revision.Jun 5 2025, 9:34 PM

This revision is now accepted and ready to land.Jun 5 2025, 9:34 PM

Closed by commit rG45165aab2d22: nvme.ko: Don't include nvme_util.c (authored by jhb). · Explain WhyJun 16 2025, 5:57 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG45165aab2d22: nvme.ko: Don't include nvme_util.c.

Revision Contents
Changeset List

Path

Size

sys/

modules/

nvme/

Makefile

1 line

Diff 156590

View Options

nvme.ko: Don't include nvme_util.cClosedPublicActions