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

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

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Oct 13, 1:13 AM

	Unknown Object (File)
	Mon, Oct 13, 1:13 AM

	Unknown Object (File)
	Mon, Oct 13, 1:13 AM

	Unknown Object (File)
	Sun, Oct 12, 1:33 PM

	Unknown Object (File)
	Thu, Oct 9, 4:37 AM

	Unknown Object (File)
	Mon, Oct 6, 9:37 AM

	Unknown Object (File)
	Mon, Sep 22, 2:00 PM

	Unknown Object (File)
	Sun, Sep 21, 11:20 PM

View All 32 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 Not Applicable

Unit

Tests Not Applicable

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 157121

View Options

nvme.ko: Don't include nvme_util.cClosedPublicActions