On Wed, Feb 05, 2020 at 04:00:19PM +0000, Ferruh Yigit wrote:
> On 2/5/2020 3:41 PM, Július Milan wrote:
>> With this patch it is possible to connect 2 DPDK memifs into loopback,
>> i.e. when they have the same id and different roles, as for example:
>>  "--vdev=net_memif0,role=master,id=0"
>>  "--vdev=net_memif1,role=slave,id=0"
>
>Overall looks good idea but this cause a crash in testpmd on exit, can you
>please check?
>
Thank you,
Do you mean this message?
"EAL: Error: Invalid memory"
If not, how can I reproduce it?

>> 
>> Fixes: 09c7e63a71 ("net/memif: introduce memory interface PMD")
>> 
>> Signed-off-by: Július Milan <jmilan....@gmail.com>
>> ---
>>  drivers/net/memif/memif_socket.c | 17 ++++-------------
>>  1 file changed, 4 insertions(+), 13 deletions(-)
>> 
>> diff --git a/drivers/net/memif/memif_socket.c 
>> b/drivers/net/memif/memif_socket.c
>> index ad5e30b96..552d3bec1 100644
>> --- a/drivers/net/memif/memif_socket.c
>> +++ b/drivers/net/memif/memif_socket.c
>> @@ -203,7 +203,7 @@ memif_msg_receive_init(struct memif_control_channel *cc, 
>> memif_msg_t *msg)
>>              dev = elt->dev;
>>              pmd = dev->data->dev_private;
>>              if (((pmd->flags & ETH_MEMIF_FLAG_DISABLED) == 0) &&
>> -                pmd->id == i->id) {
>> +                (pmd->id == i->id) && (pmd->role == MEMIF_ROLE_MASTER)) {
>>                      /* assign control channel to device */
>>                      cc->dev = dev;
>>                      pmd->cc = cc;
>> @@ -965,20 +965,11 @@ memif_socket_init(struct rte_eth_dev *dev, const char 
>> *socket_filename)
>>      }
>>      pmd->socket_filename = socket->filename;
>>  
>> -    if (socket->listener != 0 && pmd->role == MEMIF_ROLE_SLAVE) {
>> -            MIF_LOG(ERR, "Socket is a listener.");
>> -            return -1;
>> -    } else if ((socket->listener == 0) && (pmd->role == MEMIF_ROLE_MASTER)) 
>> {
>> -            MIF_LOG(ERR, "Socket is not a listener.");
>> -            return -1;
>> -    }
>> -
>>      TAILQ_FOREACH(elt, &socket->dev_queue, next) {
>>              tmp_pmd = elt->dev->data->dev_private;
>> -            if (tmp_pmd->id == pmd->id) {
>> -                    MIF_LOG(ERR, "Memif device with id %d already "
>> -                            "exists on socket %s",
>> -                            pmd->id, socket->filename);
>> +            if (tmp_pmd->id == pmd->id && tmp_pmd->role == pmd->role) {
>> +                    MIF_LOG(ERR, "Two interfaces with the same id (%d) can "
>> +                            "not have the same role.", pmd->id);
>>                      return -1;
>>              }
>>      }
>> 
>

Reply via email to