Thread (45 messages) 45 messages, 7 authors, 2020-07-08

Re: [dpdk-dev] [PATCH 1/3] eventdev: fix race condition on timer list counter

From: Phil Yang <hidden>
Date: 2020-07-02 03:28:42

-----Original Message-----
From: Jerin Jacob <redacted>
Sent: Wednesday, July 1, 2020 7:22 PM
To: Phil Yang <redacted>
Cc: Honnappa Nagarahalli <redacted>; Carrillo, Erik G
[off-list ref]; dev@dpdk.org; drc@linux.vnet.ibm.com;
Ruifeng Wang [off-list ref]; Dharmik Thakkar
[off-list ref]; nd [off-list ref]; stable@dpdk.org
Subject: Re: [dpdk-dev] [PATCH 1/3] eventdev: fix race condition on timer list
counter

On Mon, Jun 22, 2020 at 3:18 PM Phil Yang [off-list ref] wrote:
quoted
quoted
-----Original Message-----
From: Honnappa Nagarahalli <redacted>
Sent: Friday, June 19, 2020 2:26 AM
To: Carrillo, Erik G <redacted>; Phil Yang
[off-list ref]; dev@dpdk.org
Cc: drc@linux.vnet.ibm.com; Ruifeng Wang <redacted>;
Dharmik Thakkar [off-list ref]; nd [off-list ref];
stable@dpdk.org; Honnappa Nagarahalli
[off-list ref];
quoted
quoted
nd [off-list ref]
Subject: RE: [PATCH 1/3] eventdev: fix race condition on timer list counter
quoted
quoted
Since this commit will be back ported, we should prefer to use rte_atomic
APIs for this commit. Otherwise, we will have a mix of rte_atomic and C11
APIs.
My suggestion is to fix this bug using rte_atomic so that backported code
will
quoted
quoted
have only rte_atomic APIs. Add another commit (if required) in this series
to
quoted
quoted
make the bug fix use C11 APIs (this commit will not be backported).
Agree.
I will change this patch to the rte_atomic version in the next version.
Hi Phil,

Could you send the next version? I would like to take this series for
RC1(next-eventdev tree)
Thanks, Jerin.
I will upstream the new patch series soon.

quoted
Thanks,
Phil
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help