event/cnxk: initialize work slot read cache

Message ID 20220620112616.3710246-1-vfialko@marvell.com (mailing list archive)
State Accepted, archived
Delegated to: Jerin Jacob
Headers
Series event/cnxk: initialize work slot read cache |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-aarch64-unit-testing success Testing PASS
ci/iol-aarch64-compile-testing success Testing PASS
ci/iol-x86_64-unit-testing success Testing PASS
ci/github-robot: build success github build: passed
ci/iol-x86_64-compile-testing success Testing PASS
ci/iol-abi-testing success Testing PASS

Commit Message

Volodymyr Fialko June 20, 2022, 11:26 a.m. UTC
  Initialize gw_rdata with tag type EMPTY. Leaving tag type as
zero(ATOMIC) may cause some unnecessary head wait, if cache will be used
before the first update in dequeue/get_work functions.

Signed-off-by: Volodymyr Fialko <vfialko@marvell.com>
---
 drivers/event/cnxk/cn10k_eventdev.c | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Jerin Jacob June 20, 2022, 7:02 p.m. UTC | #1
On Mon, Jun 20, 2022 at 4:56 PM Volodymyr Fialko <vfialko@marvell.com> wrote:
>
> Initialize gw_rdata with tag type EMPTY. Leaving tag type as
> zero(ATOMIC) may cause some unnecessary head wait, if cache will be used
> before the first update in dequeue/get_work functions.
>
> Signed-off-by: Volodymyr Fialko <vfialko@marvell.com>

Applied to dpdk-next-net-eventdev/for-main. Thanks


> ---
>  drivers/event/cnxk/cn10k_eventdev.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/event/cnxk/cn10k_eventdev.c b/drivers/event/cnxk/cn10k_eventdev.c
> index 2f15ff8692..d28eeb3f2f 100644
> --- a/drivers/event/cnxk/cn10k_eventdev.c
> +++ b/drivers/event/cnxk/cn10k_eventdev.c
> @@ -56,6 +56,7 @@ cn10k_sso_init_hws_mem(void *arg, uint8_t port_id)
>         ws->hws_id = port_id;
>         ws->swtag_req = 0;
>         ws->gw_wdata = cn10k_sso_gw_mode_wdata(dev);
> +       ws->gw_rdata = SSO_TT_EMPTY << 32;
>         ws->lmt_base = dev->sso.lmt_base;
>
>         return ws;
> --
> 2.25.1
>
  

Patch

diff --git a/drivers/event/cnxk/cn10k_eventdev.c b/drivers/event/cnxk/cn10k_eventdev.c
index 2f15ff8692..d28eeb3f2f 100644
--- a/drivers/event/cnxk/cn10k_eventdev.c
+++ b/drivers/event/cnxk/cn10k_eventdev.c
@@ -56,6 +56,7 @@  cn10k_sso_init_hws_mem(void *arg, uint8_t port_id)
 	ws->hws_id = port_id;
 	ws->swtag_req = 0;
 	ws->gw_wdata = cn10k_sso_gw_mode_wdata(dev);
+	ws->gw_rdata = SSO_TT_EMPTY << 32;
 	ws->lmt_base = dev->sso.lmt_base;
 
 	return ws;