Gonglei (Arei)
2022-May-06 09:34 UTC
[PATCH v5 5/5] virtio-crypto: enable retry for virtio-crypto-dev
> -----Original Message----- > From: zhenwei pi [mailto:pizhenwei at bytedance.com] > Sent: Thursday, May 5, 2022 5:24 PM > To: Gonglei (Arei) <arei.gonglei at huawei.com>; mst at redhat.com > Cc: jasowang at redhat.com; herbert at gondor.apana.org.au; > linux-kernel at vger.kernel.org; virtualization at lists.linux-foundation.org; > linux-crypto at vger.kernel.org; helei.sig11 at bytedance.com; > pizhenwei at bytedance.com; davem at davemloft.net > Subject: [PATCH v5 5/5] virtio-crypto: enable retry for virtio-crypto-dev > > From: lei he <helei.sig11 at bytedance.com> > > Enable retry for virtio-crypto-dev, so that crypto-engine can process > cipher-requests parallelly. > > Cc: Michael S. Tsirkin <mst at redhat.com> > Cc: Jason Wang <jasowang at redhat.com> > Cc: Gonglei <arei.gonglei at huawei.com> > Signed-off-by: lei he <helei.sig11 at bytedance.com> > Signed-off-by: zhenwei pi <pizhenwei at bytedance.com> > --- > drivers/crypto/virtio/virtio_crypto_core.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/crypto/virtio/virtio_crypto_core.c > b/drivers/crypto/virtio/virtio_crypto_core.c > index 60490ffa3df1..f67e0d4c1b0c 100644 > --- a/drivers/crypto/virtio/virtio_crypto_core.c > +++ b/drivers/crypto/virtio/virtio_crypto_core.c > @@ -144,7 +144,8 @@ static int virtcrypto_find_vqs(struct virtio_crypto *vi) > spin_lock_init(&vi->data_vq[i].lock); > vi->data_vq[i].vq = vqs[i]; > /* Initialize crypto engine */ > - vi->data_vq[i].engine = crypto_engine_alloc_init(dev, 1); > + vi->data_vq[i].engine = crypto_engine_alloc_init_and_set(dev, true, > NULL, 1, > + virtqueue_get_vring_size(vqs[i]));Here the '1' can be 'true' too. Sure, you can add Reviewed-by: Gonglei <arei.gonglei at huawei.com> Regards, -Gonglei> if (!vi->data_vq[i].engine) { > ret = -ENOMEM; > goto err_engine; > -- > 2.20.1
zhenwei pi
2022-May-06 09:55 UTC
[PATCH v5 5/5] virtio-crypto: enable retry for virtio-crypto-dev
On 5/6/22 17:34, Gonglei (Arei) wrote:> > >> -----Original Message----- >> From: zhenwei pi [mailto:pizhenwei at bytedance.com] >> Sent: Thursday, May 5, 2022 5:24 PM >> To: Gonglei (Arei) <arei.gonglei at huawei.com>; mst at redhat.com >> Cc: jasowang at redhat.com; herbert at gondor.apana.org.au; >> linux-kernel at vger.kernel.org; virtualization at lists.linux-foundation.org; >> linux-crypto at vger.kernel.org; helei.sig11 at bytedance.com; >> pizhenwei at bytedance.com; davem at davemloft.net >> Subject: [PATCH v5 5/5] virtio-crypto: enable retry for virtio-crypto-dev >> >> From: lei he <helei.sig11 at bytedance.com> >> >> Enable retry for virtio-crypto-dev, so that crypto-engine can process >> cipher-requests parallelly. >> >> Cc: Michael S. Tsirkin <mst at redhat.com> >> Cc: Jason Wang <jasowang at redhat.com> >> Cc: Gonglei <arei.gonglei at huawei.com> >> Signed-off-by: lei he <helei.sig11 at bytedance.com> >> Signed-off-by: zhenwei pi <pizhenwei at bytedance.com> >> --- >> drivers/crypto/virtio/virtio_crypto_core.c | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/crypto/virtio/virtio_crypto_core.c >> b/drivers/crypto/virtio/virtio_crypto_core.c >> index 60490ffa3df1..f67e0d4c1b0c 100644 >> --- a/drivers/crypto/virtio/virtio_crypto_core.c >> +++ b/drivers/crypto/virtio/virtio_crypto_core.c >> @@ -144,7 +144,8 @@ static int virtcrypto_find_vqs(struct virtio_crypto *vi) >> spin_lock_init(&vi->data_vq[i].lock); >> vi->data_vq[i].vq = vqs[i]; >> /* Initialize crypto engine */ >> - vi->data_vq[i].engine = crypto_engine_alloc_init(dev, 1); >> + vi->data_vq[i].engine = crypto_engine_alloc_init_and_set(dev, true, >> NULL, 1, >> + virtqueue_get_vring_size(vqs[i])); > > Here the '1' can be 'true' too. > > Sure, you can add > > Reviewed-by: Gonglei <arei.gonglei at huawei.com> > > Regards, > -Gonglei > >> if (!vi->data_vq[i].engine) { >> ret = -ENOMEM; >> goto err_engine; >> -- >> 2.20.1 >Thanks to Lei! Hi, Michael I would appreciate it if you could apply this minor change, or I send the v6 series, which one do you prefer? -- zhenwei pi