From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id A514A93FA for ; Fri, 12 Feb 2016 17:02:48 +0100 (CET) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga103.fm.intel.com with ESMTP; 12 Feb 2016 08:02:37 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,436,1449561600"; d="scan'208";a="901591071" Received: from gklab-246-018.igk.intel.com (HELO stargo) ([10.217.246.18]) by fmsmga001.fm.intel.com with SMTP; 12 Feb 2016 08:02:35 -0800 Received: by stargo (sSMTP sendmail emulation); Fri, 12 Feb 2016 17:05:38 +0100 From: Marcin Kerlin To: dev@dpdk.org Date: Fri, 12 Feb 2016 17:04:41 +0100 Message-Id: <1455293081-13094-1-git-send-email-marcinx.kerlin@intel.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1454081497-13015-1-git-send-email-marcinx.kerlin@intel.com> References: <1454081497-13015-1-git-send-email-marcinx.kerlin@intel.com> Subject: [dpdk-dev] [PATCH v3 1/1] jobstats: added function abort for job X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 12 Feb 2016 16:02:49 -0000 This patch adds new function rte_jobstats_abort. It marks *job* as finished and time of this work will be add to management time instead of execution time. This function should be used instead of rte_jobstats_finish if condition occurs, condition is defined by the application for example when receiving n>0 packets. Example of usage is added to the example l2fwd-jobstats. At maximum load do-while loop inside Idle job will be execute once because one or more jobs waiting to be executed, so this time should not be include as the execution time by calling rte_jobstats_abort(). v2: * removed redundant field v3: * added an example of using Signed-off-by: Marcin Kerlin --- examples/l2fwd-jobstats/main.c | 9 ++++++++- lib/librte_jobstats/rte_jobstats.c | 20 ++++++++++++++++++++ lib/librte_jobstats/rte_jobstats.h | 14 ++++++++++++++ lib/librte_jobstats/rte_jobstats_version.map | 7 +++++++ 4 files changed, 49 insertions(+), 1 deletion(-) diff --git a/examples/l2fwd-jobstats/main.c b/examples/l2fwd-jobstats/main.c index 7b59f4e..bd64e74 100644 --- a/examples/l2fwd-jobstats/main.c +++ b/examples/l2fwd-jobstats/main.c @@ -577,10 +577,13 @@ l2fwd_main_loop(void) */ rte_jobstats_start(&qconf->jobs_context, &qconf->idle_job); + uint64_t repeats = 0; + do { uint8_t i; uint64_t now = rte_get_timer_cycles(); + repeats++; need_manage = qconf->flush_timer.expire < now; /* Check if we was esked to give a stats. */ stats_read_pending = @@ -591,7 +594,11 @@ l2fwd_main_loop(void) need_manage = qconf->rx_timers[i].expire < now; } while (!need_manage); - rte_jobstats_finish(&qconf->idle_job, qconf->idle_job.target); + + if (likely(repeats != 1)) + rte_jobstats_finish(&qconf->idle_job, qconf->idle_job.target); + else + rte_jobstats_abort(&qconf->idle_job); rte_timer_manage(); rte_jobstats_context_finish(&qconf->jobs_context); diff --git a/lib/librte_jobstats/rte_jobstats.c b/lib/librte_jobstats/rte_jobstats.c index 2eaac0c..2b42050 100644 --- a/lib/librte_jobstats/rte_jobstats.c +++ b/lib/librte_jobstats/rte_jobstats.c @@ -170,6 +170,26 @@ rte_jobstats_start(struct rte_jobstats_context *ctx, struct rte_jobstats *job) } int +rte_jobstats_abort(struct rte_jobstats *job) +{ + struct rte_jobstats_context *ctx; + uint64_t now, exec_time; + + /* Some sanity check. */ + if (unlikely(job == NULL || job->context == NULL)) + return -EINVAL; + + ctx = job->context; + now = get_time(); + exec_time = now - ctx->state_time; + ADD_TIME_MIN_MAX(ctx, management, exec_time); + ctx->state_time = now; + job->context = NULL; + + return 0; +} + +int rte_jobstats_finish(struct rte_jobstats *job, int64_t job_value) { struct rte_jobstats_context *ctx; diff --git a/lib/librte_jobstats/rte_jobstats.h b/lib/librte_jobstats/rte_jobstats.h index de6a89a..c2b285f 100644 --- a/lib/librte_jobstats/rte_jobstats.h +++ b/lib/librte_jobstats/rte_jobstats.h @@ -237,6 +237,20 @@ int rte_jobstats_start(struct rte_jobstats_context *ctx, struct rte_jobstats *job); /** + * Mark that *job* finished its execution, but time of this work will be skipped + * and added to management time. + * + * @param job + * Job object. + * + * @return + * 0 on success + * -EINVAL if job is NULL or job was not started (it have no context). + */ +int +rte_jobstats_abort(struct rte_jobstats *job); + +/** * Mark that *job* finished its execution. Context in which it was executing * will receive stat update. After this function call *job* object is ready to * be executed in other context. diff --git a/lib/librte_jobstats/rte_jobstats_version.map b/lib/librte_jobstats/rte_jobstats_version.map index cb01bfd..e3b21ca 100644 --- a/lib/librte_jobstats/rte_jobstats_version.map +++ b/lib/librte_jobstats/rte_jobstats_version.map @@ -17,3 +17,10 @@ DPDK_2.0 { local: *; }; + +DPDK_2.3 { + global: + + rte_jobstats_abort; + +} DPDK_2.0; -- 1.9.1