DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH] linux/igb_uio: fix build with kernel 5.18+
@ 2022-10-06  9:37 Georg Müller
  2022-10-06 18:51 ` [PATCH v2] " Georg Müller
  0 siblings, 1 reply; 3+ messages in thread
From: Georg Müller @ 2022-10-06  9:37 UTC (permalink / raw)
  To: dev; +Cc: Georg Müller

pci_set_dma_mask() and pci_set_consistent_dma_mask() were removed with
kernel 5.18. They both were just wrappers for dma_set_mask() and
dma_set_coherent_mask().

Instead, use dma_set_mask_and_coherent(), which is a combination of
dma_set_mask() and dma_set_coherent_mask().

dma_set_mask_and_coherent() exists since kernel 3.13.
---
 linux/igb_uio/igb_uio.c | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/linux/igb_uio/igb_uio.c b/linux/igb_uio/igb_uio.c
index 33e0e02..aea67da 100644
--- a/linux/igb_uio/igb_uio.c
+++ b/linux/igb_uio/igb_uio.c
@@ -512,18 +512,12 @@ igbuio_pci_probe(struct pci_dev *dev, const struct pci_device_id *id)
 		goto fail_release_iomem;

 	/* set 64-bit DMA mask */
-	err = pci_set_dma_mask(dev,  DMA_BIT_MASK(64));
+	err = dma_set_mask_and_coherent(&dev->dev,  DMA_BIT_MASK(64));
 	if (err != 0) {
 		dev_err(&dev->dev, "Cannot set DMA mask\n");
 		goto fail_release_iomem;
 	}

-	err = pci_set_consistent_dma_mask(dev, DMA_BIT_MASK(64));
-	if (err != 0) {
-		dev_err(&dev->dev, "Cannot set consistent DMA mask\n");
-		goto fail_release_iomem;
-	}
-
 	/* fill uio infos */
 	udev->info.name = "igb_uio";
 	udev->info.version = "0.1";
--
2.37.3


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH v2] linux/igb_uio: fix build with kernel 5.18+
  2022-10-06  9:37 [PATCH] linux/igb_uio: fix build with kernel 5.18+ Georg Müller
@ 2022-10-06 18:51 ` Georg Müller
  2023-02-05 17:59   ` Thomas Monjalon
  0 siblings, 1 reply; 3+ messages in thread
From: Georg Müller @ 2022-10-06 18:51 UTC (permalink / raw)
  To: dev; +Cc: Georg Müller

pci_set_dma_mask() and pci_set_consistent_dma_mask() were removed with
kernel 5.18. They both were just wrappers for dma_set_mask() and
dma_set_coherent_mask().

Instead, use dma_set_mask_and_coherent(), which is a combination of
dma_set_mask() and dma_set_coherent_mask().

dma_set_mask_and_coherent() exists since kernel 3.13.

Signed-off-by: Georg Müller <georgmueller@gmx.net>
---
v2: - add Signed-off-by
---
 linux/igb_uio/igb_uio.c | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/linux/igb_uio/igb_uio.c b/linux/igb_uio/igb_uio.c
index 33e0e02..aea67da 100644
--- a/linux/igb_uio/igb_uio.c
+++ b/linux/igb_uio/igb_uio.c
@@ -512,18 +512,12 @@ igbuio_pci_probe(struct pci_dev *dev, const struct pci_device_id *id)
 		goto fail_release_iomem;

 	/* set 64-bit DMA mask */
-	err = pci_set_dma_mask(dev,  DMA_BIT_MASK(64));
+	err = dma_set_mask_and_coherent(&dev->dev,  DMA_BIT_MASK(64));
 	if (err != 0) {
 		dev_err(&dev->dev, "Cannot set DMA mask\n");
 		goto fail_release_iomem;
 	}

-	err = pci_set_consistent_dma_mask(dev, DMA_BIT_MASK(64));
-	if (err != 0) {
-		dev_err(&dev->dev, "Cannot set consistent DMA mask\n");
-		goto fail_release_iomem;
-	}
-
 	/* fill uio infos */
 	udev->info.name = "igb_uio";
 	udev->info.version = "0.1";
--
2.37.3


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] linux/igb_uio: fix build with kernel 5.18+
  2022-10-06 18:51 ` [PATCH v2] " Georg Müller
@ 2023-02-05 17:59   ` Thomas Monjalon
  0 siblings, 0 replies; 3+ messages in thread
From: Thomas Monjalon @ 2023-02-05 17:59 UTC (permalink / raw)
  To: Georg Müller; +Cc: dev, ferruh.yigit

06/10/2022 20:51, Georg Müller:
> pci_set_dma_mask() and pci_set_consistent_dma_mask() were removed with
> kernel 5.18. They both were just wrappers for dma_set_mask() and
> dma_set_coherent_mask().
> 
> Instead, use dma_set_mask_and_coherent(), which is a combination of
> dma_set_mask() and dma_set_coherent_mask().
> 
> dma_set_mask_and_coherent() exists since kernel 3.13.
> 
> Signed-off-by: Georg Müller <georgmueller@gmx.net>

Applied, thanks.




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-02-05 17:59 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-06  9:37 [PATCH] linux/igb_uio: fix build with kernel 5.18+ Georg Müller
2022-10-06 18:51 ` [PATCH v2] " Georg Müller
2023-02-05 17:59   ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).