From: Jerin Jacob <jerinjacobk@gmail.com>
To: Pavan Nikhilesh <pbhagavatula@marvell.com>
Cc: Jerin Jacob <jerinj@marvell.com>,
Nithin Dabilpuram <ndabilpuram@marvell.com>,
Kiran Kumar K <kirankumark@marvell.com>,
Sunil Kumar Kori <skori@marvell.com>,
Satha Rao <skoteshwar@marvell.com>, Ray Kinsella <mdr@ashroe.eu>,
Shijith Thotton <sthotton@marvell.com>, dpdk-dev <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH v4 1/2] event/cnxk: update min interval calculation
Date: Thu, 14 Oct 2021 17:21:28 +0530 [thread overview]
Message-ID: <CALBAE1Pdj-pYfgwiKNhi__f9MDner6MqicRWGcmAW-y+h1HnLA@mail.gmail.com> (raw)
In-Reply-To: <20211009080426.18482-1-pbhagavatula@marvell.com>
On Sat, Oct 9, 2021 at 1:34 PM <pbhagavatula@marvell.com> wrote:
>
> From: Pavan Nikhilesh <pbhagavatula@marvell.com>
>
> Minimum supported interval should now be retrieved from
> mailbox based on the clock source and clock frequency.
>
> Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
> ---
> v4:
> - Rebase on master, fix NULL checks.
> v3:
> - Add new mbox interface.
> v2:
> - Fixed devargs parsing and rebased.
Please rebase
[for-main]dell[dpdk-next-eventdev] $ git pw series apply 19482
Applying: event/cnxk: update min interval calculation
Applying: event/cnxk: add external clock support for timer
error: sha1 information is lacking or useless
(drivers/event/cnxk/cnxk_tim_evdev.c).
error: could not build fake ancestor
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0002 event/cnxk: add external clock support for timer
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
>
> drivers/common/cnxk/roc_mbox.h | 17 +++++++
> drivers/common/cnxk/roc_tim.c | 32 +++++++++++-
> drivers/common/cnxk/roc_tim.h | 9 +++-
> drivers/common/cnxk/version.map | 1 +
> drivers/event/cnxk/cnxk_tim_evdev.c | 69 +++++++++++++++++--------
> drivers/event/cnxk/cnxk_tim_evdev.h | 79 ++++++++++++++++++-----------
> 6 files changed, 155 insertions(+), 52 deletions(-)
>
> diff --git a/drivers/common/cnxk/roc_mbox.h b/drivers/common/cnxk/roc_mbox.h
> index 75d1ff1ef3..c3688de065 100644
> --- a/drivers/common/cnxk/roc_mbox.h
> +++ b/drivers/common/cnxk/roc_mbox.h
> @@ -131,6 +131,8 @@ struct mbox_msghdr {
> M(TIM_ENABLE_RING, 0x803, tim_enable_ring, tim_ring_req, \
> tim_enable_rsp) \
> M(TIM_DISABLE_RING, 0x804, tim_disable_ring, tim_ring_req, msg_rsp) \
> + M(TIM_GET_MIN_INTVL, 0x805, tim_get_min_intvl, tim_intvl_req, \
> + tim_intvl_rsp) \
> /* CPT mbox IDs (range 0xA00 - 0xBFF) */ \
> M(CPT_LF_ALLOC, 0xA00, cpt_lf_alloc, cpt_lf_alloc_req_msg, msg_rsp) \
> M(CPT_LF_FREE, 0xA01, cpt_lf_free, msg_req, msg_rsp) \
> @@ -1756,6 +1758,9 @@ struct tim_config_req {
> uint32_t __io chunksize;
> uint32_t __io interval;
> uint8_t __io gpioedge;
> + uint8_t __io rsvd[7];
> + uint64_t __io intervalns;
> + uint64_t __io clockfreq;
> };
>
> struct tim_lf_alloc_rsp {
> @@ -1769,6 +1774,18 @@ struct tim_enable_rsp {
> uint32_t __io currentbucket;
> };
>
> +struct tim_intvl_req {
> + struct mbox_msghdr hdr;
> + uint8_t __io clocksource;
> + uint64_t __io clockfreq;
> +};
> +
> +struct tim_intvl_rsp {
> + struct mbox_msghdr hdr;
> + uint64_t __io intvl_cyc;
> + uint64_t __io intvl_ns;
> +};
> +
> struct sdp_node_info {
> /* Node to which this PF belons to */
> uint8_t __io node_id;
> diff --git a/drivers/common/cnxk/roc_tim.c b/drivers/common/cnxk/roc_tim.c
> index 387164bb1d..eefa8253bc 100644
> --- a/drivers/common/cnxk/roc_tim.c
> +++ b/drivers/common/cnxk/roc_tim.c
> @@ -145,7 +145,7 @@ int
> roc_tim_lf_config(struct roc_tim *roc_tim, uint8_t ring_id,
> enum roc_tim_clk_src clk_src, uint8_t ena_periodic,
> uint8_t ena_dfb, uint32_t bucket_sz, uint32_t chunk_sz,
> - uint32_t interval)
> + uint32_t interval, uint64_t intervalns, uint64_t clockfreq)
> {
> struct dev *dev = &roc_sso_to_sso_priv(roc_tim->roc_sso)->dev;
> struct tim_config_req *req;
> @@ -162,6 +162,8 @@ roc_tim_lf_config(struct roc_tim *roc_tim, uint8_t ring_id,
> req->enableperiodic = ena_periodic;
> req->enabledontfreebuffer = ena_dfb;
> req->interval = interval;
> + req->intervalns = intervalns;
> + req->clockfreq = clockfreq;
> req->gpioedge = TIM_GPIO_LTOH_TRANS;
>
> rc = mbox_process(dev->mbox);
> @@ -173,6 +175,34 @@ roc_tim_lf_config(struct roc_tim *roc_tim, uint8_t ring_id,
> return 0;
> }
>
> +int
> +roc_tim_lf_interval(struct roc_tim *roc_tim, enum roc_tim_clk_src clk_src,
> + uint64_t clockfreq, uint64_t *intervalns,
> + uint64_t *interval)
> +{
> + struct dev *dev = &roc_sso_to_sso_priv(roc_tim->roc_sso)->dev;
> + struct tim_intvl_req *req;
> + struct tim_intvl_rsp *rsp;
> + int rc = -ENOSPC;
> +
> + req = mbox_alloc_msg_tim_get_min_intvl(dev->mbox);
> + if (req == NULL)
> + return rc;
> +
> + req->clockfreq = clockfreq;
> + req->clocksource = clk_src;
> + rc = mbox_process_msg(dev->mbox, (void **)&rsp);
> + if (rc < 0) {
> + tim_err_desc(rc);
> + return rc;
> + }
> +
> + *intervalns = rsp->intvl_ns;
> + *interval = rsp->intvl_cyc;
> +
> + return 0;
> +}
> +
> int
> roc_tim_lf_alloc(struct roc_tim *roc_tim, uint8_t ring_id, uint64_t *clk)
> {
> diff --git a/drivers/common/cnxk/roc_tim.h b/drivers/common/cnxk/roc_tim.h
> index 159b021a31..392732eae2 100644
> --- a/drivers/common/cnxk/roc_tim.h
> +++ b/drivers/common/cnxk/roc_tim.h
> @@ -10,6 +10,8 @@ enum roc_tim_clk_src {
> ROC_TIM_CLK_SRC_GPIO,
> ROC_TIM_CLK_SRC_GTI,
> ROC_TIM_CLK_SRC_PTP,
> + ROC_TIM_CLK_SRC_SYNCE,
> + ROC_TIM_CLK_SRC_BTS,
> ROC_TIM_CLK_SRC_INVALID,
> };
>
> @@ -33,7 +35,12 @@ int __roc_api roc_tim_lf_config(struct roc_tim *roc_tim, uint8_t ring_id,
> enum roc_tim_clk_src clk_src,
> uint8_t ena_periodic, uint8_t ena_dfb,
> uint32_t bucket_sz, uint32_t chunk_sz,
> - uint32_t interval);
> + uint32_t interval, uint64_t intervalns,
> + uint64_t clockfreq);
> +int __roc_api roc_tim_lf_interval(struct roc_tim *roc_tim,
> + enum roc_tim_clk_src clk_src,
> + uint64_t clockfreq, uint64_t *intervalns,
> + uint64_t *interval);
> int __roc_api roc_tim_lf_alloc(struct roc_tim *roc_tim, uint8_t ring_id,
> uint64_t *clk);
> int __roc_api roc_tim_lf_free(struct roc_tim *roc_tim, uint8_t ring_id);
> diff --git a/drivers/common/cnxk/version.map b/drivers/common/cnxk/version.map
> index fff7902b25..5791629ce9 100644
> --- a/drivers/common/cnxk/version.map
> +++ b/drivers/common/cnxk/version.map
> @@ -279,6 +279,7 @@ INTERNAL {
> roc_tim_lf_disable;
> roc_tim_lf_enable;
> roc_tim_lf_free;
> + roc_tim_lf_interval;
> roc_se_ctx_swap;
>
> local: *;
> diff --git a/drivers/event/cnxk/cnxk_tim_evdev.c b/drivers/event/cnxk/cnxk_tim_evdev.c
> index 9d40e336d7..369f198444 100644
> --- a/drivers/event/cnxk/cnxk_tim_evdev.c
> +++ b/drivers/event/cnxk/cnxk_tim_evdev.c
> @@ -2,6 +2,8 @@
> * Copyright(C) 2021 Marvell.
> */
>
> +#include <math.h>
> +
> #include "cnxk_eventdev.h"
> #include "cnxk_tim_evdev.h"
>
> @@ -120,7 +122,10 @@ cnxk_tim_ring_create(struct rte_event_timer_adapter *adptr)
> {
> struct rte_event_timer_adapter_conf *rcfg = &adptr->data->conf;
> struct cnxk_tim_evdev *dev = cnxk_tim_priv_get();
> + uint64_t min_intvl_ns, min_intvl_cyc;
> struct cnxk_tim_ring *tim_ring;
> + enum roc_tim_clk_src clk_src;
> + uint64_t clk_freq = 0;
> int i, rc;
>
> if (dev == NULL)
> @@ -139,25 +144,52 @@ cnxk_tim_ring_create(struct rte_event_timer_adapter *adptr)
> goto tim_ring_free;
> }
>
> - if (NSEC2TICK(RTE_ALIGN_MUL_CEIL(
> - rcfg->timer_tick_ns,
> - cnxk_tim_min_resolution_ns(cnxk_tim_cntfrq())),
> - cnxk_tim_cntfrq()) <
> - cnxk_tim_min_tmo_ticks(cnxk_tim_cntfrq())) {
> - if (rcfg->flags & RTE_EVENT_TIMER_ADAPTER_F_ADJUST_RES)
> - rcfg->timer_tick_ns = TICK2NSEC(
> - cnxk_tim_min_tmo_ticks(cnxk_tim_cntfrq()),
> - cnxk_tim_cntfrq());
> - else {
> + clk_src = cnxk_tim_convert_clk_src(rcfg->clk_src);
> + if (clk_src == ROC_TIM_CLK_SRC_INVALID) {
> + plt_err("Invalid clock source");
> + goto tim_hw_free;
> + }
> +
> + rc = cnxk_tim_get_clk_freq(dev, clk_src, &clk_freq);
> + if (rc < 0) {
> + plt_err("Failed to get clock frequency");
> + goto tim_hw_free;
> + }
> +
> + rc = roc_tim_lf_interval(&dev->tim, clk_src, clk_freq, &min_intvl_ns,
> + &min_intvl_cyc);
> + if (rc < 0) {
> + plt_err("Failed to get min interval details");
> + goto tim_hw_free;
> + }
> +
> + if (rcfg->timer_tick_ns < min_intvl_ns) {
> + if (rcfg->flags & RTE_EVENT_TIMER_ADAPTER_F_ADJUST_RES) {
> + rcfg->timer_tick_ns = min_intvl_ns;
> + } else {
> rc = -ERANGE;
> goto tim_hw_free;
> }
> }
> +
> + if (rcfg->timer_tick_ns > rcfg->max_tmo_ns) {
> + plt_err("Max timeout to too high");
> + rc = -ERANGE;
> + goto tim_hw_free;
> + }
> +
> + /* Round */
> + tim_ring->tck_nsec =
> + round(RTE_ALIGN_MUL_NEAR((long double)rcfg->timer_tick_ns,
> + cnxk_tim_ns_per_tck(clk_freq)));
> +
> + tim_ring->tck_int = round((long double)tim_ring->tck_nsec /
> + cnxk_tim_ns_per_tck(clk_freq));
> + tim_ring->tck_nsec =
> + ceil(tim_ring->tck_int * cnxk_tim_ns_per_tck(clk_freq));
> +
> tim_ring->ring_id = adptr->data->id;
> - tim_ring->clk_src = (int)rcfg->clk_src;
> - tim_ring->tck_nsec = RTE_ALIGN_MUL_CEIL(
> - rcfg->timer_tick_ns,
> - cnxk_tim_min_resolution_ns(cnxk_tim_cntfrq()));
> + tim_ring->clk_src = clk_src;
> tim_ring->max_tout = rcfg->max_tmo_ns;
> tim_ring->nb_bkts = (tim_ring->max_tout / tim_ring->tck_nsec);
> tim_ring->nb_timers = rcfg->nb_timers;
> @@ -201,11 +233,9 @@ cnxk_tim_ring_create(struct rte_event_timer_adapter *adptr)
> if (rc < 0)
> goto tim_bkt_free;
>
> - rc = roc_tim_lf_config(
> - &dev->tim, tim_ring->ring_id,
> - cnxk_tim_convert_clk_src(tim_ring->clk_src), 0, 0,
> - tim_ring->nb_bkts, tim_ring->chunk_sz,
> - NSEC2TICK(tim_ring->tck_nsec, cnxk_tim_cntfrq()));
> + rc = roc_tim_lf_config(&dev->tim, tim_ring->ring_id, clk_src, 0, 0,
> + tim_ring->nb_bkts, tim_ring->chunk_sz,
> + tim_ring->tck_int, tim_ring->tck_nsec, clk_freq);
> if (rc < 0) {
> plt_err("Failed to configure timer ring");
> goto tim_chnk_free;
> @@ -300,7 +330,6 @@ cnxk_tim_ring_start(const struct rte_event_timer_adapter *adptr)
> if (rc < 0)
> return rc;
>
> - tim_ring->tck_int = NSEC2TICK(tim_ring->tck_nsec, cnxk_tim_cntfrq());
> tim_ring->tot_int = tim_ring->tck_int * tim_ring->nb_bkts;
> tim_ring->fast_div = rte_reciprocal_value_u64(tim_ring->tck_int);
> tim_ring->fast_bkt = rte_reciprocal_value_u64(tim_ring->nb_bkts);
> diff --git a/drivers/event/cnxk/cnxk_tim_evdev.h b/drivers/event/cnxk/cnxk_tim_evdev.h
> index c369f6f472..4afbcb8fb2 100644
> --- a/drivers/event/cnxk/cnxk_tim_evdev.h
> +++ b/drivers/event/cnxk/cnxk_tim_evdev.h
> @@ -97,13 +97,6 @@ struct cnxk_tim_evdev {
> struct cnxk_tim_ctl *ring_ctl_data;
> };
>
> -enum cnxk_tim_clk_src {
> - CNXK_TIM_CLK_SRC_10NS = RTE_EVENT_TIMER_ADAPTER_CPU_CLK,
> - CNXK_TIM_CLK_SRC_GPIO = RTE_EVENT_TIMER_ADAPTER_EXT_CLK0,
> - CNXK_TIM_CLK_SRC_GTI = RTE_EVENT_TIMER_ADAPTER_EXT_CLK1,
> - CNXK_TIM_CLK_SRC_PTP = RTE_EVENT_TIMER_ADAPTER_EXT_CLK2,
> -};
> -
> struct cnxk_tim_bkt {
> uint64_t first_chunk;
> union {
> @@ -146,7 +139,7 @@ struct cnxk_tim_ring {
> uint64_t max_tout;
> uint64_t nb_chunks;
> uint64_t chunk_sz;
> - enum cnxk_tim_clk_src clk_src;
> + enum roc_tim_clk_src clk_src;
> } __rte_cache_aligned;
>
> struct cnxk_tim_ent {
> @@ -166,32 +159,13 @@ cnxk_tim_priv_get(void)
> return mz->addr;
> }
>
> -static inline uint64_t
> -cnxk_tim_min_tmo_ticks(uint64_t freq)
> +static inline long double
> +cnxk_tim_ns_per_tck(uint64_t freq)
> {
> - if (roc_model_runtime_is_cn9k())
> - return CN9K_TIM_MIN_TMO_TKS;
> - else /* CN10K min tick is of 1us */
> - return freq / USECPERSEC;
> + return (long double)NSECPERSEC / freq;
> }
>
> -static inline uint64_t
> -cnxk_tim_min_resolution_ns(uint64_t freq)
> -{
> - return NSECPERSEC / freq;
> -}
>
> -static inline enum roc_tim_clk_src
> -cnxk_tim_convert_clk_src(enum cnxk_tim_clk_src clk_src)
> -{
> - switch (clk_src) {
> - case RTE_EVENT_TIMER_ADAPTER_CPU_CLK:
> - return roc_model_runtime_is_cn9k() ? ROC_TIM_CLK_SRC_10NS :
> - ROC_TIM_CLK_SRC_GTI;
> - default:
> - return ROC_TIM_CLK_SRC_INVALID;
> - }
> -}
>
> #ifdef RTE_ARCH_ARM64
> static inline uint64_t
> @@ -225,6 +199,51 @@ cnxk_tim_cntfrq(void)
> }
> #endif
>
> +static inline enum roc_tim_clk_src
> +cnxk_tim_convert_clk_src(enum rte_event_timer_adapter_clk_src clk_src)
> +{
> + switch (clk_src) {
> + case RTE_EVENT_TIMER_ADAPTER_CPU_CLK:
> + return ROC_TIM_CLK_SRC_GTI;
> + case RTE_EVENT_TIMER_ADAPTER_EXT_CLK0:
> + return ROC_TIM_CLK_SRC_10NS;
> + case RTE_EVENT_TIMER_ADAPTER_EXT_CLK1:
> + return ROC_TIM_CLK_SRC_GPIO;
> + case RTE_EVENT_TIMER_ADAPTER_EXT_CLK2:
> + return ROC_TIM_CLK_SRC_PTP;
> + case RTE_EVENT_TIMER_ADAPTER_EXT_CLK3:
> + return roc_model_constant_is_cn9k() ? ROC_TIM_CLK_SRC_INVALID :
> + ROC_TIM_CLK_SRC_SYNCE;
> + default:
> + return ROC_TIM_CLK_SRC_INVALID;
> + }
> +}
> +
> +static inline int
> +cnxk_tim_get_clk_freq(struct cnxk_tim_evdev *dev, enum roc_tim_clk_src clk_src,
> + uint64_t *freq)
> +{
> + if (freq == NULL)
> + return -EINVAL;
> +
> + PLT_SET_USED(dev);
> + switch (clk_src) {
> + case ROC_TIM_CLK_SRC_GTI:
> + *freq = cnxk_tim_cntfrq();
> + break;
> + case ROC_TIM_CLK_SRC_10NS:
> + *freq = 1E8;
> + break;
> + case ROC_TIM_CLK_SRC_GPIO:
> + case ROC_TIM_CLK_SRC_PTP:
> + case ROC_TIM_CLK_SRC_SYNCE:
> + default:
> + return -EINVAL;
> + }
> +
> + return 0;
> +}
> +
> #define TIM_ARM_FASTPATH_MODES \
> FP(sp, 0, 0, 0, CNXK_TIM_ENA_DFB | CNXK_TIM_SP) \
> FP(mp, 0, 0, 1, CNXK_TIM_ENA_DFB | CNXK_TIM_MP) \
> --
> 2.17.1
>
next prev parent reply other threads:[~2021-10-14 11:51 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-28 15:24 [dpdk-dev] [PATCH] drivers: add external clock support for cnxk timer Shijith Thotton
2021-09-15 16:03 ` [dpdk-dev] [PATCH v2] " Shijith Thotton
2021-09-23 21:30 ` [dpdk-dev] [PATCH v3 1/2] event/cnxk: update min interval calculation pbhagavatula
2021-09-23 21:30 ` [dpdk-dev] [PATCH v3 2/2] event/cnxk: add external clock support for timer pbhagavatula
2021-10-09 8:04 ` [dpdk-dev] [PATCH v4 1/2] event/cnxk: update min interval calculation pbhagavatula
2021-10-09 8:04 ` [dpdk-dev] [PATCH v4 2/2] event/cnxk: add external clock support for timer pbhagavatula
2021-10-14 11:51 ` Jerin Jacob [this message]
2021-12-13 11:13 ` [PATCH v5 1/2] event/cnxk: update min interval calculation pbhagavatula
2021-12-13 11:13 ` [PATCH v5 2/2] event/cnxk: add external clock support for timer pbhagavatula
2022-01-21 10:18 ` [PATCH v5 1/2] event/cnxk: update min interval calculation Jerin Jacob
2022-01-25 9:54 ` Ray Kinsella
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CALBAE1Pdj-pYfgwiKNhi__f9MDner6MqicRWGcmAW-y+h1HnLA@mail.gmail.com \
--to=jerinjacobk@gmail.com \
--cc=dev@dpdk.org \
--cc=jerinj@marvell.com \
--cc=kirankumark@marvell.com \
--cc=mdr@ashroe.eu \
--cc=ndabilpuram@marvell.com \
--cc=pbhagavatula@marvell.com \
--cc=skori@marvell.com \
--cc=skoteshwar@marvell.com \
--cc=sthotton@marvell.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).