patches for DPDK stable branches
 help / color / mirror / Atom feed
* [dpdk-stable] [PATCH] common/qat: add missing kmod dependency
@ 2020-10-23 15:44 Didier Pallard
  2020-10-23 16:32 ` David Marchand
  2020-10-28 11:49 ` [dpdk-stable] [dpdk-dev] " Akhil Goyal
  0 siblings, 2 replies; 4+ messages in thread
From: Didier Pallard @ 2020-10-23 15:44 UTC (permalink / raw)
  To: dev
  Cc: stable, John Griffin, Fiona Trahe, Deepak Kumar Jain,
	Olivier Matz, Adrien Mazarguil

Dependency on kmod needed to manage crypto devices is missing
in qat crypto pmd.

Cc: stable@dpdk.org
Fixes: 0880c40113ef ("drivers: advertise kmod dependencies in pmdinfo")
Signed-off-by: Didier Pallard <didier.pallard@6wind.com>
---
 drivers/common/qat/qat_device.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/common/qat/qat_device.c b/drivers/common/qat/qat_device.c
index b050ce20e4fc..9fa142b5e5bb 100644
--- a/drivers/common/qat/qat_device.c
+++ b/drivers/common/qat/qat_device.c
@@ -423,3 +423,4 @@ qat_comp_dev_destroy(struct qat_pci_device *qat_pci_dev __rte_unused)
 
 RTE_PMD_REGISTER_PCI(QAT_PCI_NAME, rte_qat_pmd);
 RTE_PMD_REGISTER_PCI_TABLE(QAT_PCI_NAME, pci_id_qat_map);
+RTE_PMD_REGISTER_KMOD_DEP(QAT_PCI_NAME, "* igb_uio | uio_pci_generic | vfio-pci");
-- 
2.20.1


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

* Re: [dpdk-stable] [PATCH] common/qat: add missing kmod dependency
  2020-10-23 15:44 [dpdk-stable] [PATCH] common/qat: add missing kmod dependency Didier Pallard
@ 2020-10-23 16:32 ` David Marchand
  2020-10-23 17:20   ` Trahe, Fiona
  2020-10-28 11:49 ` [dpdk-stable] [dpdk-dev] " Akhil Goyal
  1 sibling, 1 reply; 4+ messages in thread
From: David Marchand @ 2020-10-23 16:32 UTC (permalink / raw)
  To: Didier Pallard
  Cc: dev, dpdk stable, John Griffin, Fiona Trahe, Deepak Kumar Jain,
	Olivier Matz, Adrien Mazarguil

Salut Didier,

On Fri, Oct 23, 2020 at 5:45 PM Didier Pallard <didier.pallard@6wind.com> wrote:
>
> Dependency on kmod needed to manage crypto devices is missing
> in qat crypto pmd.
>
> Cc: stable@dpdk.org
> Fixes: 0880c40113ef ("drivers: advertise kmod dependencies in pmdinfo")

The Fixes: tag goes first, then Cc:, then empty line.


> Signed-off-by: Didier Pallard <didier.pallard@6wind.com>
> ---
>  drivers/common/qat/qat_device.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/common/qat/qat_device.c b/drivers/common/qat/qat_device.c
> index b050ce20e4fc..9fa142b5e5bb 100644
> --- a/drivers/common/qat/qat_device.c
> +++ b/drivers/common/qat/qat_device.c
> @@ -423,3 +423,4 @@ qat_comp_dev_destroy(struct qat_pci_device *qat_pci_dev __rte_unused)
>
>  RTE_PMD_REGISTER_PCI(QAT_PCI_NAME, rte_qat_pmd);
>  RTE_PMD_REGISTER_PCI_TABLE(QAT_PCI_NAME, pci_id_qat_map);
> +RTE_PMD_REGISTER_KMOD_DEP(QAT_PCI_NAME, "* igb_uio | uio_pci_generic | vfio-pci");
> --
> 2.20.1
>

Reviewed-by: David Marchand <david.marchand@redhat.com>


There are some other drivers that have the issue.
If there is a volunteer, we have other drivers with the same issue
(but probably worth double checking):
$ git grep -l 'drv_flags = RTE_PCI_DRV_NEED_MAPPING' drivers/ |xargs
dirname |sort -u|while read dir; do dir=${dir%%/base}; git grep -q
RTE_PMD_REGISTER_KMOD_DEP $dir || echo $dir; done
drivers/baseband/acc100
drivers/baseband/fpga_5gnr_fec
drivers/baseband/fpga_lte_fec
drivers/common/qat # done
drivers/event/octeontx
drivers/event/skeleton
drivers/mempool/octeontx
drivers/net/avp
drivers/net/hinic
drivers/net/octeontx
drivers/regex/octeontx2


-- 
David Marchand


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

* Re: [dpdk-stable] [PATCH] common/qat: add missing kmod dependency
  2020-10-23 16:32 ` David Marchand
@ 2020-10-23 17:20   ` Trahe, Fiona
  0 siblings, 0 replies; 4+ messages in thread
From: Trahe, Fiona @ 2020-10-23 17:20 UTC (permalink / raw)
  To: David Marchand, Didier Pallard
  Cc: dev, dpdk stable, Griffin, John, Jain, Deepak K, Olivier Matz,
	Adrien Mazarguil



> -----Original Message-----
> From: David Marchand <david.marchand@redhat.com>
> Sent: Friday, October 23, 2020 5:32 PM
> To: Didier Pallard <didier.pallard@6wind.com>
> Cc: dev <dev@dpdk.org>; dpdk stable <stable@dpdk.org>; Griffin, John <john.griffin@intel.com>; Trahe,
> Fiona <fiona.trahe@intel.com>; Jain, Deepak K <deepak.k.jain@intel.com>; Olivier Matz
> <olivier.matz@6wind.com>; Adrien Mazarguil <adrien.mazarguil@6wind.com>
> Subject: Re: [dpdk-stable] [PATCH] common/qat: add missing kmod dependency
> 
> Salut Didier,
> 
> On Fri, Oct 23, 2020 at 5:45 PM Didier Pallard <didier.pallard@6wind.com> wrote:
> >
> > Dependency on kmod needed to manage crypto devices is missing
> > in qat crypto pmd.
> >
> > Cc: stable@dpdk.org
> > Fixes: 0880c40113ef ("drivers: advertise kmod dependencies in pmdinfo")
> 
> The Fixes: tag goes first, then Cc:, then empty line.
> 
> 
> > Signed-off-by: Didier Pallard <didier.pallard@6wind.com>

Thanks!
Acked-by: Fiona Trahe <fiona.trahe@intel.com>

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

* Re: [dpdk-stable] [dpdk-dev] [PATCH] common/qat: add missing kmod dependency
  2020-10-23 15:44 [dpdk-stable] [PATCH] common/qat: add missing kmod dependency Didier Pallard
  2020-10-23 16:32 ` David Marchand
@ 2020-10-28 11:49 ` Akhil Goyal
  1 sibling, 0 replies; 4+ messages in thread
From: Akhil Goyal @ 2020-10-28 11:49 UTC (permalink / raw)
  To: Didier Pallard, dev
  Cc: stable, John Griffin, Fiona Trahe, Deepak Kumar Jain,
	Olivier Matz, Adrien Mazarguil

> Subject: [dpdk-dev] [PATCH] common/qat: add missing kmod dependency
> 
> Dependency on kmod needed to manage crypto devices is missing
> in qat crypto pmd.
> 
> Cc: stable@dpdk.org
> Fixes: 0880c40113ef ("drivers: advertise kmod dependencies in pmdinfo")
> Signed-off-by: Didier Pallard <didier.pallard@6wind.com>
Applied to dpdk-next-crypto

Thanks/


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

end of thread, other threads:[~2020-10-28 11:49 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-23 15:44 [dpdk-stable] [PATCH] common/qat: add missing kmod dependency Didier Pallard
2020-10-23 16:32 ` David Marchand
2020-10-23 17:20   ` Trahe, Fiona
2020-10-28 11:49 ` [dpdk-stable] [dpdk-dev] " Akhil Goyal

patches for DPDK stable branches

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/stable/0 stable/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 stable stable/ https://inbox.dpdk.org/stable \
		stable@dpdk.org
	public-inbox-index stable

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.stable


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git