: Richardson, Bruce ; tho...@monjalon.net
Subject: Re: [dpdk-dev] [PATCH v3 2/3] eal: add synchronous
multi-process communication
On 25-Jan-18 1:05 PM, Burakov, Anatoly wrote:
On 25-Jan-18 1:00 PM, Ananyev, Konstantin wrote:
-Original Message-
From: Burakov, Anatoly
Sent: Thursday
Subject: Re: [dpdk-dev] [PATCH v3 2/3] eal: add synchronous
multi-process communication
On 25-Jan-18 1:05 PM, Burakov, Anatoly wrote:
On 25-Jan-18 1:00 PM, Ananyev, Konstantin wrote:
-Original Message-
From: Burakov, Anatoly
Sent: Thursday, January 25, 2018 12:26 PM
To: Ananyev
On 25-Jan-18 3:03 PM, Ananyev, Konstantin wrote:
-Original Message-
From: Burakov, Anatoly
Sent: Thursday, January 25, 2018 1:10 PM
To: Ananyev, Konstantin ; Tan, Jianfeng
; dev@dpdk.org
Cc: Richardson, Bruce ; tho...@monjalon.net
Subject: Re: [dpdk-dev] [PATCH v3 2/3] eal: add
> -Original Message-
> From: Burakov, Anatoly
> Sent: Thursday, January 25, 2018 1:10 PM
> To: Ananyev, Konstantin ; Tan, Jianfeng
> ; dev@dpdk.org
> Cc: Richardson, Bruce ; tho...@monjalon.net
> Subject: Re: [dpdk-dev] [PATCH v3 2/3] eal: add synchronous multi-pro
On 25-Jan-18 1:05 PM, Burakov, Anatoly wrote:
On 25-Jan-18 1:00 PM, Ananyev, Konstantin wrote:
-Original Message-
From: Burakov, Anatoly
Sent: Thursday, January 25, 2018 12:26 PM
To: Ananyev, Konstantin ; Tan, Jianfeng
; dev@dpdk.org
Cc: Richardson, Bruce ; tho...@monjalon.net
Subje
On 25-Jan-18 1:00 PM, Ananyev, Konstantin wrote:
-Original Message-
From: Burakov, Anatoly
Sent: Thursday, January 25, 2018 12:26 PM
To: Ananyev, Konstantin ; Tan, Jianfeng
; dev@dpdk.org
Cc: Richardson, Bruce ; tho...@monjalon.net
Subject: Re: [PATCH v3 2/3] eal: add synchronous mult
> -Original Message-
> From: Burakov, Anatoly
> Sent: Thursday, January 25, 2018 12:26 PM
> To: Ananyev, Konstantin ; Tan, Jianfeng
> ; dev@dpdk.org
> Cc: Richardson, Bruce ; tho...@monjalon.net
> Subject: Re: [PATCH v3 2/3] eal: add synchronous multi-process communication
>
> On 25-Jan
On 25-Jan-18 12:19 PM, Ananyev, Konstantin wrote:
-Original Message-
From: Burakov, Anatoly
Sent: Thursday, January 25, 2018 12:00 PM
To: Tan, Jianfeng ; dev@dpdk.org
Cc: Richardson, Bruce ; Ananyev, Konstantin
; tho...@monjalon.net
Subject: Re: [PATCH v3 2/3] eal: add synchronous mul
> We need the synchronous way for multi-process communication,
> i.e., blockingly waiting for reply message when we send a request
> to the peer process.
>
> We add two APIs rte_eal_mp_request() and rte_eal_mp_reply() for
> such use case. By invoking rte_eal_mp_request(), a request message
> is s
> -Original Message-
> From: Burakov, Anatoly
> Sent: Thursday, January 25, 2018 12:00 PM
> To: Tan, Jianfeng ; dev@dpdk.org
> Cc: Richardson, Bruce ; Ananyev, Konstantin
> ; tho...@monjalon.net
> Subject: Re: [PATCH v3 2/3] eal: add synchronous multi-process communication
>
> On the ov
See above: it would be much more useful to return number of replies
received, rather than number of messages sent, as that's the number we
are most interested in. Otherwise, if we e.g. sent 5 messages but
received 1 reply, you're essentially not telling the user how far can he
index the reply
On the overall patch,
Reviewed-by: Anatoly Burakov
For request(), returning number of replies received actually makes
sense, because now we get use the value to read our replies, if we were
a primary process sending messages to secondary processes.
Few comments below.
On 25-Jan-18 4:16 AM,
12 matches
Mail list logo
