Page MenuHomeFreeBSD
Differential D26857

IOMMU quirks added
ClosedPublic
Actions

Authored by br on Oct 19 2020, 1:35 PM.
Tags
None
Referenced Files
F86228565: D26857.id78472.diff
Mon, Jun 17, 8:01 AM
Unknown Object (File)
May 12 2024, 7:58 AM
Unknown Object (File)
May 11 2024, 10:20 PM
Unknown Object (File)
May 11 2024, 9:50 AM
Unknown Object (File)
May 11 2024, 5:32 AM
Unknown Object (File)
May 9 2024, 11:33 PM
Unknown Object (File)
Apr 27 2024, 9:59 PM
Unknown Object (File)
Apr 24 2024, 3:00 AM
View All 20 Files
Subscribers
imp

Details

Reviewers
kib
mav
Commits
rS366863: Add IOMMU_BUSWIDE ahci quirk.
Summary

Add the Marvel SATA quirk.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

br requested review of this revision.Oct 19 2020, 1:35 PM
br created this revision.
kib added a comment.Oct 19 2020, 1:56 PM

No, this should be in SATA driver.

br updated this revision to Diff 78433.Oct 19 2020, 3:27 PM

Move the quirk to ahci driver

kib added a reviewer: mav.Oct 19 2020, 3:44 PM
kib added inline comments.
sys/dev/ahci/ahci.h
628 ↗(On Diff #78433)

This define should be updated as well.

br updated this revision to Diff 78439.Oct 19 2020, 3:59 PM

Update AHCI quirk bitstring

kib accepted this revision.Oct 19 2020, 4:54 PM
This revision is now accepted and ready to land.Oct 19 2020, 4:54 PM
mav added a comment.Oct 19 2020, 5:36 PM

I have no objections, but the link specified lead me to Linux' drivers/pci/quirks.c file, where I've found the same done for 12 different Marvell HBAs. Why not do the same?

br updated this revision to Diff 78456.Oct 19 2020, 7:03 PM

Add IOMMU quirks for all the devices listed in linux

This revision now requires review to proceed.Oct 19 2020, 7:03 PM
mav accepted this revision.Oct 19 2020, 8:22 PM

Thanks!

This revision is now accepted and ready to land.Oct 19 2020, 8:22 PM
Closed by commit rS366863: Add IOMMU_BUSWIDE ahci quirk. (authored by br). · Explain WhyOct 19 2020, 9:27 PM
This revision was automatically updated to reflect the committed changes.
br added a commit: rS366863: Add IOMMU_BUSWIDE ahci quirk..
Herald added a subscriber: imp. · View Herald TranscriptOct 19 2020, 9:27 PM

Revision Contents
Changeset List

PathSize
head/
sys/
dev/
ahci/
4 lines
52 lines

Diff 78472

View Options

head/sys/dev/ahci/ahci.h

Loading...
View Options

head/sys/dev/ahci/ahci_pci.c

Loading...