Fix mpr(4) panics caused by bad drive mapping tables
ClosedPublic
Actions

Authored by asomers on Nov 1 2017, 10:00 PM.

Tags

None

Referenced Files

	F81584295: D12901.id34643.diff
	Thu, Apr 18, 12:23 PM

	F81559682: D12901.id34643.diff
	Thu, Apr 18, 1:54 AM

	F81555876: D12901.id34643.diff
	Thu, Apr 18, 12:27 AM

	F81555471: D12901.id34647.diff
	Thu, Apr 18, 12:16 AM

	Unknown Object (File)
	Wed, Apr 17, 3:46 PM

	Unknown Object (File)
	Feb 7 2024, 6:39 AM

	Unknown Object (File)
	Dec 20 2023, 12:44 AM

	Unknown Object (File)
	Dec 16 2023, 4:44 PM

Subscribers

Details

Reviewers

slm

Commits

rS326341: MFC r325363:
rS326323: MFC r325363:
rS325363: Fix mpr(4) panics caused by bad drive mapping tables

Summary

Fix mpr(4) panics caused by bad drive mapping tables

sys/dev/mpr/mpr_mapping.c
If _mapping_process_dpm_pg0 detects inconsistencies in the drive
mapping table (stored in the HBA's NVRAM), abort reading it and
continue to boot as if the mapping table were blank.

Test Plan

Try booting with an mpr card that contains a known-bad mapping table

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

asomers created this revision.Nov 1 2017, 10:00 PM

Herald added a subscriber: imp. · View Herald TranscriptNov 1 2017, 10:00 PM

slm requested changes to this revision.Nov 1 2017, 10:26 PM

slm added inline comments.

sys/dev/mpr/mpr_mapping.c
2382 ↗	(On Diff #34643)	Doesn't look like you need 'dpm_entry' here.

This revision now requires changes to proceed.Nov 1 2017, 10:26 PM

Fix copypaste error found by slm

Harbormaster completed remote builds in B12387: Diff 34647.Nov 1 2017, 11:05 PM

slm accepted this revision.Nov 2 2017, 2:55 PM

This revision is now accepted and ready to land.Nov 2 2017, 2:55 PM

Closed by commit rS325363: Fix mpr(4) panics caused by bad drive mapping tables (authored by asomers). · Explain WhyNov 3 2017, 3:07 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

dev/

mpr/

mpr_mapping.c

26 lines

Diff 34720

View Options

head/sys/dev/mpr/mpr_mapping.c

Fix mpr(4) panics caused by bad drive mapping tablesClosedPublicActions