[EXT] [dpdk-dev v1] lib/cryptodev: multi-process IPC request handler

[Zhang, Roy Fan]

Hi Akhil,

> -----Original Message-----
> From: Akhil Goyal <gakhil at marvell.com>
> Sent: Wednesday, July 27, 2022 5:26 AM
> To: Ji, Kai <kai.ji at intel.com>; dev at dpdk.org
> Cc: Anoob Joseph <anoobj at marvell.com>; hemant.agrawal at nxp.com;
> Zhang, Roy Fan <roy.fan.zhang at intel.com>; Chandubabu Namburu
> <chandu at amd.com>; Ruifeng Wang <ruifeng.wang at arm.com>;
> ajit.khaparde at broadcom.com; Michael Shamis <michaelsh at marvell.com>;
> Nagadheeraj Rottela <rnagadheeraj at marvell.com>; matan at nvidia.com; Jay
> Zhou <jianjay.zhou at huawei.com>
> Subject: RE: [EXT] [dpdk-dev v1] lib/cryptodev: multi-process IPC request
> handler
> 
> This is a library change you should cc all PMD owners while sending patch.
Kai is in holiday at the moment and will be back in a week. I will sync with him then.
> 
> > This patch add in multi-process IPC request handler function in rte
> > cryptodev. This function intend to support a queue-pair configuration
> > request to allow the secondary process to reconfigure the queue-pair
> > setup'ed by the primary process.
> 
> Who will release the queue pair already setup by primary in the first place?

Fan: If the queue pair already setup by primary the secondary shall not recreate it
but use it instead.

> Currently, all queues are setup by primary and secondary uses them.
> So if a queue is re-initialized by secondary, and if it is being used in primary
> process,
> Wont that drop packets abruptly if the queue is re-initialized?

You are right. What about creating a variable in the queue pair with either PID
or thread id who own the queue pair?

> 
> Also, I see register API but not deregister.

There will be V2 for that.

Regards,
Fan