Differential D42807

mpi3mr: Replace can't happen DataLength == 0 with an assert
ClosedPublic
Actions

Authored by imp on Nov 28 2023, 9:06 PM.

Tags

None

Referenced Files

	F144429112: D42807.id130692.diff
	Sun, Feb 8, 9:44 AM

	F144311508: D42807.id130732.diff
	Sat, Feb 7, 7:11 PM

	Unknown Object (File)
	Thu, Feb 5, 11:10 AM

	Unknown Object (File)
	Sat, Jan 31, 4:12 AM

	Unknown Object (File)
	Jan 8 2026, 11:19 AM

	Unknown Object (File)
	Dec 12 2025, 3:08 AM

	Unknown Object (File)
	Dec 8 2025, 10:49 PM

	Unknown Object (File)
	Nov 1 2025, 7:58 AM

View All 64 Files

Subscribers

None

Details

Reviewers

jhb
chs
mav
sumit.saxena_broadcom.com
chandrakanth.patil_broadcom.com

Commits

rGac7d9eaf066b: mpi3mr: Replace can't happen DataLength == 0 with an assert
rG2361a0056fc1: mpi3mr: Replace can't happen DataLength == 0 with an assert

Summary

Replace the test for DataLength == 0 with an assert. It can't happen,
but an assert doesn't hurt. Emacs removed some trailing white space too.

Sponsored by: Netflix

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

imp requested review of this revision.Nov 28 2023, 9:06 PM

imp created this revision.

Harbormaster completed remote builds in B54663: Diff 130692.Nov 28 2023, 9:06 PM

imp added a child revision: D42808: mpi3mr: Assume dma_hiaddr is BUS_SPACE_MAXADDR.Nov 28 2023, 9:06 PM

imp added reviewers: jhb, chs, mav, sumit.saxena_broadcom.com, chandrakanth.patil_broadcom.com.Nov 28 2023, 9:06 PM

imp added a parent revision: D42607: mpi3mr: Use template for main busdma tag..

This revision was not accepted when it landed; it landed in state Needs Review.Nov 29 2023, 1:56 AM

Closed by commit rG2361a0056fc1: mpi3mr: Replace can't happen DataLength == 0 with an assert (authored by imp). · Explain Why

This revision was automatically updated to reflect the committed changes.

imp added a commit: rG2361a0056fc1: mpi3mr: Replace can't happen DataLength == 0 with an assert.

mav added a commit: rGac7d9eaf066b: mpi3mr: Replace can't happen DataLength == 0 with an assert.Jan 19 2024, 5:18 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

mpi3mr/

18 lines

Diff 130732

sys/dev/mpi3mr/mpi3mr_cam.c

Loading...