Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1448142

Re: pci: automatic interrupt affinity for MSI/MSI-X capable devices V2

From Bjorn Helgaas <helgaas@kernel.org>
Newsgroups linux.kernel
Subject Re: pci: automatic interrupt affinity for MSI/MSI-X capable devices V2
Date 2016-07-21 23:10 +0200
Message-ID <rXtoe-4vY-13@gated-at.bofh.it> (permalink)
References <rU2aR-2Zl-9@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Tue, Jul 12, 2016 at 06:20:13PM +0900, Christoph Hellwig wrote:
> This series adds a new set of functions that transparently use the right
> type of interrupt (MSI-X, MSI, legacy interrupt line) for a PCI device,
> and if multiple vectors are supported automatically spreads the irq
> routing to different CPUs.  This will allow the block layer (and hopefully
> other consumers in the future) to use this information for mapping
> queues to fit the interrupt affinity.
> 
> For the last patche to work you need to merge the irq/for-block branch of
> 
>    https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
> 
> first.
> 
> There also is a git tree available at:
> 
>    git://git.infradead.org/users/hch/block.git pci-irq-spreading
> 
> Gitweb:
> 
>    http://git.infradead.org/users/hch/block.git/shortlog/refs/heads/pci-irq-spreading
> 
> Changes since V1
>  - change return values for the minmum vector check
>  - improve the documentation
> 
> Changes since automatic interrupt affinity for MSI/MSI-X capable devices V3:
>  - add PCI_IRQ_NOLEGACY flag
>  - various error code fixes
>  - reuse the pci_enable_msi(x)_range code instead of duplicating it
>  - don't allocate msix_entry structures for the MSI-X case

I merged the irq/for-block branch, applied this series, and added
Alexander's Reviewed-by to the patches that didn't have it already.

As far as the if/return/else construct:

  if (x)
    return y;
  else if (z)
    ...

I removed the "else" in the other occurrences so the style is
consistently:

  if (x)
    return y;
  if (z)
    ...

This is all on my pci/msi-affinity branch for v4.8.

Thanks for all the work; I think this is a very nice change, and it's
great that all the documentation is updated as well.

Bjorn

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

pci: automatic interrupt affinity for MSI/MSI-X capable devices V2 Christoph Hellwig <hch@lst.de> - 2016-07-12 11:30 +0200
  [PATCH 4/5] pci: Provide sensible irq vector alloc/free routines Christoph Hellwig <hch@lst.de> - 2016-07-12 11:30 +0200
    Re: [PATCH 4/5] pci: Provide sensible irq vector alloc/free routines Alexander Gordeev <agordeev@redhat.com> - 2016-07-12 15:00 +0200
  [PATCH 2/5] pci: switch msix_program_entries to use pci_msix_desc_addr Christoph Hellwig <hch@lst.de> - 2016-07-12 11:30 +0200
    Re: [PATCH 2/5] pci: switch msix_program_entries to use  pci_msix_desc_addr Alexander Gordeev <agordeev@redhat.com> - 2016-07-12 15:00 +0200
  Re: pci: automatic interrupt affinity for MSI/MSI-X capable devices  V2 Bjorn Helgaas <helgaas@kernel.org> - 2016-07-21 23:10 +0200
    Re: pci: automatic interrupt affinity for MSI/MSI-X capable  devices V2 Christoph Hellwig <hch@lst.de> - 2016-07-21 23:20 +0200

csiph-web