[dpdk-dev] [PATCH] crypto/qat: fix memzone creation to use a fixed size string

2016-09-19 Thread John Griffin
On 18/09/16 09:16, Yuanhan Liu wrote: > On Wed, Sep 14, 2016 at 04:32:46PM +0100, John Griffin wrote: >> Hi Liu, >> Comments embedded. >> >> Rgds, >> John. >> >> On 05/09/16 04:23, Yuanhan Liu wrote: >>> On Thu, Sep 01, 2016 at 11:21:38AM +0100, John Griffin wrote: Remove the dependency on dev

[dpdk-dev] [PATCH] crypto/qat: fix memzone creation to use a fixed size string

2016-09-18 Thread Yuanhan Liu
On Wed, Sep 14, 2016 at 04:32:46PM +0100, John Griffin wrote: > Hi Liu, > Comments embedded. > > Rgds, > John. > > On 05/09/16 04:23, Yuanhan Liu wrote: > >On Thu, Sep 01, 2016 at 11:21:38AM +0100, John Griffin wrote: > >>Remove the dependency on dev->driver->pci_drv.name when > >>creating the me

[dpdk-dev] [PATCH] crypto/qat: fix memzone creation to use a fixed size string

2016-09-14 Thread John Griffin
Hi Liu, Comments embedded. Rgds, John. On 05/09/16 04:23, Yuanhan Liu wrote: > On Thu, Sep 01, 2016 at 11:21:38AM +0100, John Griffin wrote: >> Remove the dependency on dev->driver->pci_drv.name when >> creating the memzone for the qat hardware queues. >> The pci_drv.name may grow too large for R

[dpdk-dev] [PATCH] crypto/qat: fix memzone creation to use a fixed size string

2016-09-05 Thread Yuanhan Liu
On Thu, Sep 01, 2016 at 11:21:38AM +0100, John Griffin wrote: > Remove the dependency on dev->driver->pci_drv.name when > creating the memzone for the qat hardware queues. > The pci_drv.name may grow too large for RTE_MEMZONE_NAMESIZE. Will the "may grow too large" cause any issues? If so, state i

[dpdk-dev] [PATCH] crypto/qat: fix memzone creation to use a fixed size string

2016-09-01 Thread John Griffin
Remove the dependency on dev->driver->pci_drv.name when creating the memzone for the qat hardware queues. The pci_drv.name may grow too large for RTE_MEMZONE_NAMESIZE. Fixes: 1703e94ac5ce ("qat: add driver for QuickAssist devices") Signed-off-by: John Griffin --- drivers/crypto/qat/qat_qp.c | 2