Re: [dpdk-dev] [PATCH 0/3] generic channel for multi-process communication

["Burakov, Anatoly" <anatoly.burakov@intel.com>]

On 30-Nov-17 6:44 PM, Jianfeng Tan wrote:
> This patchset adds a generic channel for multi-process (primary/secondary)
> communication.
> 
> Patch 1: addess the purpose and howto;
> Patch 2: add a syncrhonous way for those messages which need a response immediately.
> Patch 3: Rework vfio to use this generic communication channel.
> 

Hi Jianfeng,

Just a general comment: I am assuming this has the limitation of 
"everything happens through primary process's involvement". This will 
work for VFIO, as secondary always needs to ask the primary before doing 
anything, but it doesn't address other issues that could have been 
addressed with IPC.

For example, if a primary process would've hotplugged a device, it can't 
notify all secondary processes about this; rather, it has to wait until 
secondary processes ask for this info. Neither can it do anything if 
secondary requests a primary to do something, and notify other secondary 
processes about it (i.e. if secondary wants to hotplug a device, but 
there are other secondaries also running). It would be great to have a 
standard way of doing things like this in future revisions of our IPC.

-- 
Thanks,
Anatoly