Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Dmitry Rokosov <DDRokosov@sberdevices.ru>
To:     "broonie@kernel.org" <broonie@kernel.org>,
        "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
        "rafael@kernel.org" <rafael@kernel.org>,
        "jic23@kernel.org" <jic23@kernel.org>,
        "linux@rasmusvillemoes.dk>" <linux@rasmusvillemoes.dk>,
        "tglx@linutronix.de" <tglx@linutronix.de>,
        "maz@kernel.org>" <maz@kernel.org>,
        "andy.shevchenko@gmail.com" <andy.shevchenko@gmail.com>
CC:     kernel <kernel@sberdevices.ru>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v1] regmap: introduce value tracing for regmap bulk
 operations
Thread-Topic: [PATCH v1] regmap: introduce value tracing for regmap bulk
 operations
Thread-Index: AQHYsZwRTOzZdLxKvkqpBNcAgWIdzq20YtWA
Date:   Thu, 18 Aug 2022 12:15:00 +0000
Message-ID: <20220818121515.6azkxyqetnunwsc6@CAB-WSD-L081021.sigma.sbrf.ru>
References: <20220816181451.5628-1-ddrokosov@sberdevices.ru>
In-Reply-To: <20220816181451.5628-1-ddrokosov@sberdevices.ru>
Accept-Language: ru-RU, en-US
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-ID: <AC9EAFFC92E14A4EA157ADBF6AF36FBC@sberdevices.ru>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Precedence: bulk

+Thomas, Rasmus and Marc

On Tue, Aug 16, 2022 at 06:14:48PM +0000, Dmitry Rokosov wrote:
> Currently, only one-register io operations support tracepoints with
> value logging. For the regmap bulk operations developer can view
> hw_start/hw_done tracepoints with starting reg number and registers
> count to be reading or writing. This patch injects tracepoints with
> dumping registers values in the hex format to regmap bulk reading
> and writing.
>=20
> Signed-off-by: Dmitry Rokosov <ddrokosov@sberdevices.ru>
> ---
>  drivers/base/regmap/regmap.c |  7 ++++++
>  drivers/base/regmap/trace.h  | 43 ++++++++++++++++++++++++++++++++++++
>  2 files changed, 50 insertions(+)
>=20
> diff --git a/drivers/base/regmap/regmap.c b/drivers/base/regmap/regmap.c
> index c3517ccc3159..673ad37df11f 100644
> --- a/drivers/base/regmap/regmap.c
> +++ b/drivers/base/regmap/regmap.c
> @@ -2323,6 +2323,10 @@ int regmap_bulk_write(struct regmap *map, unsigned=
 int reg, const void *val,
> =20
>  		kfree(wval);
>  	}
> +
> +	if (!ret)
> +		trace_regmap_bulk_write(map, reg, val, val_bytes * val_count);
> +
>  	return ret;
>  }
>  EXPORT_SYMBOL_GPL(regmap_bulk_write);
> @@ -3068,6 +3072,9 @@ int regmap_bulk_read(struct regmap *map, unsigned i=
nt reg, void *val,
>  		map->unlock(map->lock_arg);
>  	}
> =20
> +	if (!ret)
> +		trace_regmap_bulk_read(map, reg, val, val_bytes * val_count);
> +
>  	return ret;
>  }
>  EXPORT_SYMBOL_GPL(regmap_bulk_read);
> diff --git a/drivers/base/regmap/trace.h b/drivers/base/regmap/trace.h
> index 9abee14df9ee..04329ba68ec5 100644
> --- a/drivers/base/regmap/trace.h
> +++ b/drivers/base/regmap/trace.h
> @@ -64,6 +64,49 @@ DEFINE_EVENT(regmap_reg, regmap_reg_read_cache,
> =20
>  );
> =20
> +DECLARE_EVENT_CLASS(regmap_bulk,
> +
> +	TP_PROTO(struct regmap *map, unsigned int reg,
> +		 const void *val, int val_len),
> +
> +	TP_ARGS(map, reg, val, val_len),
> +
> +	TP_STRUCT__entry(
> +		__string(name, regmap_name(map))
> +		__field(unsigned int, reg)
> +		__dynamic_array(char, buf, val_len)
> +		__field(int, val_len)
> +	),
> +
> +	TP_fast_assign(
> +		__assign_str(name, regmap_name(map));
> +		__entry->reg =3D reg;
> +		__entry->val_len =3D val_len;
> +		if (val)
> +			memcpy(__get_dynamic_array(buf), val, val_len);
> +	),
> +
> +	TP_printk("%s reg=3D%x val=3D%s", __get_str(name),
> +		  (unsigned int)__entry->reg,
> +		  __print_hex(__get_dynamic_array(buf), __entry->val_len))
> +);
> +
> +DEFINE_EVENT(regmap_bulk, regmap_bulk_write,
> +
> +	TP_PROTO(struct regmap *map, unsigned int reg,
> +		 const void *val, int val_len),
> +
> +	TP_ARGS(map, reg, val, val_len)
> +);
> +
> +DEFINE_EVENT(regmap_bulk, regmap_bulk_read,
> +
> +	TP_PROTO(struct regmap *map, unsigned int reg,
> +		 const void *val, int val_len),
> +
> +	TP_ARGS(map, reg, val, val_len)
> +);
> +
>  DECLARE_EVENT_CLASS(regmap_block,
> =20
>  	TP_PROTO(struct regmap *map, unsigned int reg, int count),
> --=20
> 2.36.0

--=20
Thank you,
Dmitry=