Re: [dpdk-dev] [PATCH] bus/pci: fix missing case for virtio-crypto

2018-11-05 Thread Thomas Monjalon
01/11/2018 13:10, Fan Zhang: > The recent change to rte_pci_read_config() caused virtio-crypto > device initialization always failed as it has a > RTE_KDRV_UIO_GENERIC driver type. This patch fixes the problem. > > Fixes: 630deed612ca ("bus/pci: compare kernel driver instead of interrupt > handle

Re: [dpdk-dev] [PATCH] bus/pci: fix missing case for virtio-crypto

2018-11-05 Thread Thomas Monjalon
We must also fix rte_pci_write_config() 02/11/2018 11:39, Thomas Monjalon: > +Cc Bruce and Ferruh > > 02/11/2018 11:01, Zhang, Roy Fan: > > Hi Thomas, > > > > You are right, the problem can happen with any PMD falls into > > RTE_KDRV_UIO_GENERIC and RTE_KDRV_NIC_UIO driver type. As riginally >

Re: [dpdk-dev] [PATCH] bus/pci: fix missing case for virtio-crypto

2018-11-02 Thread Thomas Monjalon
+Cc Bruce and Ferruh 02/11/2018 11:01, Zhang, Roy Fan: > Hi Thomas, > > You are right, the problem can happen with any PMD falls into > RTE_KDRV_UIO_GENERIC and RTE_KDRV_NIC_UIO driver type. As riginally > rte_pci_read_config() only covered 2 out of 4 possible driver types' . > > The patch was

Re: [dpdk-dev] [PATCH] bus/pci: fix missing case for virtio-crypto

2018-11-02 Thread Zhang, Roy Fan
Hi Thomas, You are right, the problem can happen with any PMD falls into RTE_KDRV_UIO_GENERIC and RTE_KDRV_NIC_UIO driver type. As riginally rte_pci_read_config() only covered 2 out of 4 possible driver types' . The patch was submitted for fixing the Virtio Crypto Initialization Segmentation F

Re: [dpdk-dev] [PATCH] bus/pci: fix missing case for virtio-crypto

2018-11-01 Thread Thomas Monjalon
01/11/2018 13:10, Fan Zhang: > The recent change to rte_pci_read_config() caused virtio-crypto > device initialization always failed as it has a > RTE_KDRV_UIO_GENERIC driver type. This patch fixes the problem. Hi Fan, this issue is not related at all to virtio-crypto. It can happen with any PMD,

[dpdk-dev] [PATCH] bus/pci: fix missing case for virtio-crypto

2018-11-01 Thread Fan Zhang
The recent change to rte_pci_read_config() caused virtio-crypto device initialization always failed as it has a RTE_KDRV_UIO_GENERIC driver type. This patch fixes the problem. Fixes: 630deed612ca ("bus/pci: compare kernel driver instead of interrupt handler") Signed-off-by: Fan Zhang --- drive