On 10/5/2018 9:57 AM, Burakov, Anatoly wrote:
> On 03-Oct-18 1:39 PM, Darek Stojaczyk wrote:
>> So far each process in MP used to have a separate container
>> and relied on the primary process to register all memsegs.
>>
>> Mapping external memory via rte_vfio_container_dma_map()
>> in secondary processes was broken, because the default
>> (process-local) container had no groups bound. There was
>> even no way to bind any groups to it, because the container
>> fd was deeply encapsulated within EAL.
>>
>> This patch introduces a new SOCKET_REQ_DEFAULT_CONTAINER
>> message type for MP synchronization, makes all processes
>> within a MP party use a single default container, and hence
>> fixes rte_vfio_container_dma_map() for secondary processes.
>>
>> >From what I checked this behavior was always the same, but
>> started to be invalid/insufficient once mapping external
>> memory was allowed.
>>
>> Fixes: 73a639085938 ("vfio: allow to map other memory regions")
>> Cc: anatoly.bura...@intel.com
>> Cc: pawelx.wodkow...@intel.com
>> Cc: gowrishanka...@linux.vnet.ibm.com
>> Cc: sta...@dpdk.org
>>
>> While here, fix up the comment on rte_vfio_get_container_fd().
>> This function always opens a new container, never reuses
>> an old one.
>>
>> Signed-off-by: Darek Stojaczyk <dariusz.stojac...@intel.com>
>> ---
>
> <snip>
>
>> + if (internal_config.process_type == RTE_PROC_PRIMARY) {
>> + /* if we were secondary process we would try requesting
>> + * container fd from the primary, but we're the primary
>> + * process so just exit here
>> + */
>> + return -1;
>> + }
>> +
>> + p->req = SOCKET_REQ_DEFAULT_CONTAINER;
>> + strcpy(mp_req.name, EAL_VFIO_MP);
>
> strlcpy?
even better, rte_strscpy?
>
> Otherwise,
> Reviewed-by: Anatoly Burakov <anatoly.bura...@intel.com>
>
