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

Reply via email to