Re: [dpdk-dev] [PATCH] net/memif: add multiple memif data transmission support

[Jakub Grajciar -X (jgrajcia - PANTHEON TECHNOLOGIES at Cisco)]

> -----Original Message-----
> From: Anand Sunkad <anand.sun...@benisontech.com>
> Sent: Wednesday, October 16, 2019 3:22 PM
> To: Jakub Grajciar -X (jgrajcia - PANTHEON TECHNOLOGIES at Cisco)
> <jgraj...@cisco.com>
> Cc: dev@dpdk.org; Vivek Gupta <viv...@benisontech.com>; Thomas
> Mulamangalath <thomas.mulamangal...@benisontech.com>; Anand Sunkad
> <anand.sun...@benisontech.com>
> Subject: [PATCH] net/memif: add multiple memif data transmission support
> 
> When Multiple slave/master Memif interfaces are created in single process
> data transmission over second connection is not successful.
> 
> Issue is because of "mq->in_port" is not initialized with "dev->data->port_id"
> in memif_msg_enq_add_ring() and
> memif_msg_receive_add_ring() functions, and while transmitting packets
> over second connection in eth_memif_tx function it refer "mq->in_port"
> which is always zero, and leads to data transmission always in 0th port.
> 
> To mitigate the issue,"mq->in_port" is initialized with "dev->data->port_id" 
> in
> memif_msg_enq_add_ring() and
> memif_msg_receive_add_ring() functions.
> 
> Signed-off-by: Anand Sunkad <anand.sun...@benisontech.com>

Good catch, however you want to initialize mq->in_port in 
memif_tx_queue_setup().

Thanks,
Jakub