From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 3758041C2D; Tue, 7 Feb 2023 10:28:12 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 22A7D42D32; Tue, 7 Feb 2023 10:28:12 +0100 (CET) Received: from mail-vs1-f50.google.com (mail-vs1-f50.google.com [209.85.217.50]) by mails.dpdk.org (Postfix) with ESMTP id F36D540042 for ; Tue, 7 Feb 2023 10:28:10 +0100 (CET) Received: by mail-vs1-f50.google.com with SMTP id g3so4240573vsr.10 for ; Tue, 07 Feb 2023 01:28:10 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=K5lrtPPMDgikZtKC7nkorg0OF4wpgGHouLb0zz4r9eo=; b=qOkgdS/NCP4vksMpfzMUTD+n9qUxHvLS7BXvtUIunSoTJm5FJ+tNMl2fsD1uCbbZLT bx+6I+o/BD000/JOIVJ4p2Kw8mW6VlH1hCoRtjHdA94aCSWwoZ+TRheYxl2C+LnCK650 8rgFrGxISwURAkXR6PR4R0Nb7WQYmL8is6zIVfnixzmw1sayqm8zNJFIDcoUM1L8lzuy 3Jp9vZC6/DdVEn3s606TZrWnpBMO3Z25dKdj8RgOKsunnAMMQVZ/5qm4J7apRK7P/oi+ dDTRLn8k9bghAnBVA7/FFb6LAGASqE81pP+nFk5cNuUnVejGMKOMwmGMcy4deVF+hmmW ftyw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=K5lrtPPMDgikZtKC7nkorg0OF4wpgGHouLb0zz4r9eo=; b=J6H/IUsVQjo4kJFLYmN3aTa5iUX5IrE5vxuu3EbraosldQyE5WUd/RF0xu3nuVCjLh tXDoIO+Jho6B0edwL6WPdJuVUxSy6wM9KwM5vUO0E8X8cxWQ7nd4JNiNLlTzTP7RnUY9 O6SpdkIRmrQA+LfjMOe/kTSONOhsSNbUwUiX6dGdN81fZr/L7208lVA2jcAof8elWQU1 njqE/u7LXXFSTFNAlWQuFoaCIb3dPS1W92JauvCWiZfpV16cP+oQ261ZADmEhH114zrk q7INqgr5iP6kbW1YDOkhb7txaY5e6RwdcmCejkZ8T1DeCmYRpZ+3DmC8Lk7+V1atyvTS 8kOg== X-Gm-Message-State: AO0yUKUOZYRi+qv+MICgnZRLnQ7qDdg21j1y32HBmBwsXWyMVQ3yVUPq rvRpms7hCubkiTovjSEn3OMCUzmADT1eLSPchg8= X-Google-Smtp-Source: AK7set9k9PQGGmJK9PkxZZNB4x29V6K8QymWqBvRe51FKgs20+PsRvTILm8lNnEx283TqzjAi1y6zQrrBI3ENwY5jIg= X-Received: by 2002:a67:de9a:0:b0:40f:b3a4:ccbc with SMTP id r26-20020a67de9a000000b0040fb3a4ccbcmr554869vsk.45.1675762090311; Tue, 07 Feb 2023 01:28:10 -0800 (PST) MIME-Version: 1.0 References: <20230205063347.6469-1-pbhagavatula@marvell.com> In-Reply-To: <20230205063347.6469-1-pbhagavatula@marvell.com> From: Jerin Jacob Date: Tue, 7 Feb 2023 14:57:44 +0530 Message-ID: Subject: Re: [PATCH] event/cnxk: fix timer ops init in secondary To: pbhagavatula@marvell.com Cc: jerinj@marvell.com, Shijith Thotton , Anatoly Burakov , dev@dpdk.org Content-Type: text/plain; charset="UTF-8" X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org On Sun, Feb 5, 2023 at 12:04 PM wrote: > > From: Pavan Nikhilesh > > Fix event timer fast-path ops not being initialized in secondary > process. > > Fixes: dd519f83dd96 ("event/cnxk: add timer adapter capabilities") > > Signed-off-by: Pavan Nikhilesh Applied to dpdk-next-net-eventdev/for-main. Thanks > --- > drivers/event/cnxk/cnxk_tim_evdev.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/drivers/event/cnxk/cnxk_tim_evdev.c b/drivers/event/cnxk/cnxk_tim_evdev.c > index 5dd79cbd47..c155764b77 100644 > --- a/drivers/event/cnxk/cnxk_tim_evdev.c > +++ b/drivers/event/cnxk/cnxk_tim_evdev.c > @@ -381,6 +381,7 @@ cnxk_tim_caps_get(const struct rte_eventdev *evdev, uint64_t flags, > cnxk_sso_set_priv_mem_t priv_mem_fn) > { > struct cnxk_tim_evdev *dev = cnxk_tim_priv_get(); > + struct cnxk_tim_ring *tim_ring; > > RTE_SET_USED(flags); > > @@ -403,6 +404,12 @@ cnxk_tim_caps_get(const struct rte_eventdev *evdev, uint64_t flags, > dev->event_dev = (struct rte_eventdev *)(uintptr_t)evdev; > *caps = RTE_EVENT_TIMER_ADAPTER_CAP_INTERNAL_PORT | > RTE_EVENT_TIMER_ADAPTER_CAP_PERIODIC; > + > + tim_ring = ((struct rte_event_timer_adapter_data > + *)((char *)caps - offsetof(struct rte_event_timer_adapter_data, caps))) > + ->adapter_priv; > + if (tim_ring != NULL && rte_eal_process_type() == RTE_PROC_SECONDARY) > + cnxk_tim_set_fp_ops(tim_ring); > *ops = &cnxk_tim_ops; > > return 0; > -- > 2.25.1 >