linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: Mathieu Poirier <mathieu.poirier@linaro.org>
To: Mike Leach <mike.leach@linaro.org>
Cc: linux-arm-kernel@lists.infradead.org, coresight@lists.linaro.org,
	suzuki.poulose@arm.com, leo.yan@linaro.org
Subject: Re: [PATCH v2 0/6] coresight: syscfg: Extend configfs for config load
Date: Mon, 10 Jan 2022 11:58:50 -0700	[thread overview]
Message-ID: <20220110185850.GA771798@p14s> (raw)
In-Reply-To: <20211130220100.25888-1-mike.leach@linaro.org>

Good morning Mike,

On Tue, Nov 30, 2021 at 10:00:54PM +0000, Mike Leach wrote:
> This set extends the configfs support to allow loading and unloading of
> configurations as binary files via configfs.
> 
> Additional attributes - load, unload and last_load_status are provided to
> implement the load functionality.
> 
> Routines to generate binary configuration files are supplied in
> ./samples/coresight.
> 
> Example generator and reader applications are provided.
> 
> Additional Makefile.host supplied to build the generator and reader
> applications on the host system separate from a cross compiled kernel.
> 
> Documentation is updated to describe feature usage.
> 
> Applies and tested on latest coresight/next that includes the
> previous coresight configuration dynamic load patchset.
> 

I have started to review this set.

Thanks,
Mathieu

> Changes since v1:
> 1) Rebased to coresight/next - 5.16-rc1 with previous coresight config
> set applied.
> 2) Makefile.host fixed to default to all target.
> 
> Mike Leach (6):
>   coresight: configfs: Add in functionality for load via configfs
>   coresight: configfs: Add in binary attributes to load files
>   coresight: configfs: Modify config files to allow userspace use
>   coresight: samples: Add an example config writer for configfs load
>   coresight: samples: Add coresight file reader sample program
>   Documentation: coresight: docs for config load via configfs
> 
>  .../trace/coresight/coresight-config.rst      | 151 +++++-
>  drivers/hwtracing/coresight/Makefile          |   2 +-
>  .../coresight/coresight-config-file.c         | 472 ++++++++++++++++++
>  .../coresight/coresight-config-file.h         | 158 ++++++
>  .../hwtracing/coresight/coresight-config.h    |  38 ++
>  .../coresight/coresight-syscfg-configfs.c     | 148 +++++-
>  .../coresight/coresight-syscfg-configfs.h     |   8 +
>  .../hwtracing/coresight/coresight-syscfg.c    |  36 ++
>  .../hwtracing/coresight/coresight-syscfg.h    |   2 +
>  samples/coresight/Makefile                    |  23 +
>  samples/coresight/Makefile.host               |  47 ++
>  samples/coresight/coresight-cfg-bufw.c        | 302 +++++++++++
>  samples/coresight/coresight-cfg-bufw.h        |  24 +
>  samples/coresight/coresight-cfg-file-read.c   | 191 +++++++
>  samples/coresight/coresight-cfg-filegen.c     |  89 ++++
>  15 files changed, 1677 insertions(+), 14 deletions(-)
>  create mode 100644 drivers/hwtracing/coresight/coresight-config-file.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-config-file.h
>  create mode 100644 samples/coresight/Makefile.host
>  create mode 100644 samples/coresight/coresight-cfg-bufw.c
>  create mode 100644 samples/coresight/coresight-cfg-bufw.h
>  create mode 100644 samples/coresight/coresight-cfg-file-read.c
>  create mode 100644 samples/coresight/coresight-cfg-filegen.c
> 
> -- 
> 2.17.1
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

      parent reply	other threads:[~2022-01-10 19:00 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-11-30 22:00 [PATCH v2 0/6] coresight: syscfg: Extend configfs for config load Mike Leach
2021-11-30 22:00 ` [PATCH v2 1/6] coresight: configfs: Add in functionality for load via configfs Mike Leach
2022-01-28 18:08   ` Mathieu Poirier
2022-02-02 20:40     ` Mike Leach
2021-11-30 22:00 ` [PATCH v2 2/6] coresight: configfs: Add in binary attributes to load files Mike Leach
2022-01-28 18:17   ` Mathieu Poirier
2022-02-02 20:33     ` Mike Leach
2022-02-02 22:33       ` Mathieu Poirier
2021-11-30 22:00 ` [PATCH v2 3/6] coresight: configfs: Modify config files to allow userspace use Mike Leach
2022-01-28 18:33   ` Mathieu Poirier
2022-02-02 20:48     ` Mike Leach
2021-11-30 22:00 ` [PATCH v2 4/6] coresight: samples: Add an example config writer for configfs load Mike Leach
2022-01-13 17:56   ` Mathieu Poirier
2022-01-18 16:38     ` Mike Leach
2022-01-28 18:43   ` Mathieu Poirier
2022-02-02 20:54     ` Mike Leach
2021-11-30 22:00 ` [PATCH v2 5/6] coresight: samples: Add coresight file reader sample program Mike Leach
2021-11-30 22:01 ` [PATCH v2 6/6] Documentation: coresight: docs for config load via configfs Mike Leach
2022-01-12 17:22   ` Mathieu Poirier
2022-01-12 18:43     ` Mathieu Poirier
2022-01-13 18:15   ` Mathieu Poirier
2022-01-18 16:41     ` Mike Leach
2022-01-10 18:58 ` Mathieu Poirier [this message]

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=20220110185850.GA771798@p14s \
    --to=mathieu.poirier@linaro.org \
    --cc=coresight@lists.linaro.org \
    --cc=leo.yan@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=mike.leach@linaro.org \
    --cc=suzuki.poulose@arm.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).