From mboxrd@z Thu Jan  1 00:00:00 1970
Content-Type: multipart/mixed; boundary="===============2245260336158615726=="
MIME-Version: 1.0
From: Dave Jiang <dave.jiang at intel.com>
Subject: [Accel-config] Re: [PATCH 1/2] accel-config: Add
 accfg_wq_get_occupancy API to library
Date: Wed, 11 Aug 2021 08:30:17 -0700
Message-ID: <88b72c9b-23c0-34c5-1153-e8312c7fb50c@intel.com>
In-Reply-To: 20210811025735.1182805-2-ramesh.thomas@intel.com
To: accel-config@lists.01.org
List-ID: <accel-config@lists.01.org>

--===============2245260336158615726==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable


On 8/10/2021 7:57 PM, ramesh.thomas(a)intel.com wrote:
> From: Ramesh Thomas <ramesh.thomas(a)intel.com>
>
> Add library implementation of API to get wq occupancy
>
> Signed-off-by: Ramesh Thomas <ramesh.thomas(a)intel.com>
> ---
>   accfg/lib/libaccel-config.sym | 5 +++++
>   accfg/lib/libaccfg.c          | 2 ++
>   accfg/lib/private.h           | 1 +
>   accfg/libaccel_config.h       | 1 +
>   4 files changed, 9 insertions(+)
>
> diff --git a/accfg/lib/libaccel-config.sym b/accfg/lib/libaccel-config.sym
> index f44a3ca..7d6b396 100644
> --- a/accfg/lib/libaccel-config.sym
> +++ b/accfg/lib/libaccel-config.sym
> @@ -161,3 +161,8 @@ global:
>   	accfg_ctx_get_last_error_group;
>   	accfg_ctx_get_last_error_engine;
>   } LIBACCFG_9;
> +
> +LIBACCFG_11 {
> +global:
> +	accfg_wq_get_occupancy;
> +} LIBACCFG_10;
> diff --git a/accfg/lib/libaccfg.c b/accfg/lib/libaccfg.c
> index 5b63b62..d41a6b3 100644
> --- a/accfg/lib/libaccfg.c
> +++ b/accfg/lib/libaccfg.c
> @@ -801,6 +801,7 @@ static void *add_wq(void *parent, int id, const char =
*wq_base,
>   	wq->max_batch_size =3D  accfg_get_param_long(ctx, dfd, "max_batch_size=
");
>   	wq->max_transfer_size =3D  accfg_get_param_long(ctx, dfd, "max_transfe=
r_size");
>   	wq->ats_disable =3D accfg_get_param_long(ctx, dfd, "ats_disable");
> +	wq->occupancy =3D accfg_get_param_long(ctx, dfd, "occupancy");
We need to read this value real time. When the user calls it, we need to =

read from sysfs directly. We don't need to store it though.
>   =

>   	wq_parse_type(wq, wq_type);
>   	free(wq_type);
> @@ -2452,6 +2453,7 @@ accfg_wq_get_field(wq, threshold)
>   accfg_wq_get_field(wq, group_id)
>   accfg_wq_get_field(wq, block_on_fault)
>   accfg_wq_get_field(wq, ats_disable)
> +accfg_wq_get_field(wq, occupancy)
>   =

>   ACCFG_EXPORT int accfg_wq_set_mode(struct accfg_wq *wq,
>   		enum accfg_wq_mode wq_mode)
> diff --git a/accfg/lib/private.h b/accfg/lib/private.h
> index 57aa1ed..579fa6b 100644
> --- a/accfg/lib/private.h
> +++ b/accfg/lib/private.h
> @@ -127,6 +127,7 @@ struct accfg_wq {
>   	unsigned int max_batch_size;
>   	uint64_t max_transfer_size;
>   	int ats_disable;
> +	int occupancy;
>   };
>   =

>   #define ACCFG_EXPORT __attribute__ ((visibility("default")))
> diff --git a/accfg/libaccel_config.h b/accfg/libaccel_config.h
> index f8b9bf9..92b1da5 100644
> --- a/accfg/libaccel_config.h
> +++ b/accfg/libaccel_config.h
> @@ -277,6 +277,7 @@ uint64_t accfg_wq_get_max_transfer_size(struct accfg_=
wq *wq);
>   int accfg_wq_get_threshold(struct accfg_wq *wq);
>   int accfg_wq_get_clients(struct accfg_wq *wq);
>   int accfg_wq_get_ats_disable(struct accfg_wq *wq);
> +int accfg_wq_get_occupancy(struct accfg_wq *wq);
>   int accfg_wq_is_enabled(struct accfg_wq *wq);
>   int accfg_wq_set_size(struct accfg_wq *wq, int val);
>   int accfg_wq_set_priority(struct accfg_wq *wq, int val);
--===============2245260336158615726==--