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


Groups > linux.kernel > #1621956

Re: please use pci_alloc_irq_vectors in thunderx_edac.

From Borislav Petkov <bp@alien8.de>
Newsgroups linux.kernel
Subject Re: please use pci_alloc_irq_vectors in thunderx_edac.
Date 2017-04-12 11:30 +0200
Message-ID <tvmL8-2Jw-11@gated-at.bofh.it> (permalink)
References <tvjDB-Xu-25@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Wed, Apr 12, 2017 at 08:06:00AM +0200, Christoph Hellwig wrote:
> Hi Sergey,
> 
> you're new thunderx_edac driver uses pci_enable_msix_exact for no
> good reason.  Can you please switch it to use pci_alloc_irq_vectors
> before it gets merged to mainline?

Do we need to mark pci_enable_msix_exact() deprecated or so, or maybe a
checkpatch rule?

It says so in some text file in Documentation:

4.3 Legacy APIs

The following old APIs to enable and disable MSI or MSI-X interrupts should
not be used in new code:

  pci_enable_msi()              /* deprecated */
  pci_disable_msi()             /* deprecated */
  pci_enable_msix_range()       /* deprecated */
  pci_enable_msix_exact()       /* deprecated */
  pci_disable_msix()            /* deprecated */

but maybe we should enforce it?

-- 
Regards/Gruss,
    Boris.

Good mailing practices for 400: avoid top-posting and trim the reply.

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


Thread

please use pci_alloc_irq_vectors in thunderx_edac. Christoph Hellwig <hch@lst.de> - 2017-04-12 08:10 +0200
  Re: please use pci_alloc_irq_vectors in thunderx_edac. Borislav Petkov <bp@alien8.de> - 2017-04-12 11:30 +0200
    Re: please use pci_alloc_irq_vectors in thunderx_edac. Christoph Hellwig <hch@lst.de> - 2017-04-12 19:20 +0200
      Re: please use pci_alloc_irq_vectors in thunderx_edac. Borislav Petkov <bp@alien8.de> - 2017-04-12 19:30 +0200
        Re: please use pci_alloc_irq_vectors in thunderx_edac. Christoph Hellwig <hch@lst.de> - 2017-04-12 19:40 +0200
          Re: please use pci_alloc_irq_vectors in thunderx_edac. Johannes Thumshirn <jthumshirn@suse.de> - 2017-04-13 11:00 +0200

csiph-web