Bug 229267 - [patch] mps(4) irq allocation regression from r321714: INTx used instead of MSI if MSI-X unavail
Summary: [patch] mps(4) irq allocation regression from r321714: INTx used instead of M...
Status: Closed FIXED
Alias: None
Product: Base System
Classification: Unclassified
Component: kern (show other bugs)
Version: 11.2-STABLE
Hardware: Any Any
: --- Affects Some People
Assignee: freebsd-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-06-24 08:32 UTC by Harald Schmalzbauer
Modified: 2018-11-13 19:18 UTC (History)
0 users

See Also:


Attachments
Fix MSI allocation check for mps(4) and mpr(4) after r321714 (594 bytes, patch)
2018-06-24 08:32 UTC, Harald Schmalzbauer
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Harald Schmalzbauer 2018-06-24 08:32:30 UTC
Created attachment 194551 [details]
Fix MSI allocation check for mps(4) and mpr(4) after r321714

Hello,

already reported on freebsd-scsi@ (¹), I'd like to make sure this doesn't get lost.

In r321714(²), which is part of MFC r322658(³), there's an unintended change that causes MSI allocation to be skipped in case MSI-X is not available.

Please find attached a patch which I'm using for mps(4) on 11.2 and the same for mpr(4), which I haven't tested!

(¹) https://lists.freebsd.org/pipermail/freebsd-scsi/2018-June/007743.html
(²) https://svnweb.freebsd.org/base?view=revision&revision=321714
(³) https://svnweb.freebsd.org/base?view=revision&revision=322658
Comment 1 Harald Schmalzbauer 2018-11-13 19:17:25 UTC
Author: scottl
Date: Tue Nov 13 18:49:43 2018
New Revision: 340403
URL: https://svnweb.freebsd.org/changeset/base/340403

Log:
  Fix a regression from prior to 11.2 that caused MSI (not MSI-X) interrupt
  allocation to fail.  While here, refactor the code so that it's more clear
  and less likely to break in the future.  This is not an MFC due to the code
  in 12/head being very different, but it follows the latter's structure
  more closely than before.