linux-kernel - Re: [PATCH V2] firmware: tegra: add BPMP debugfs support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20171017104456.GE30589@ulmo>
Date:   Tue, 17 Oct 2017 12:44:56 +0200
From:   Thierry Reding <thierry.reding@...il.com>
To:     Timo Alho <talho@...dia.com>
Cc:     jonathanh@...dia.com, linux-tegra@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH V2] firmware: tegra: add BPMP debugfs support

On Tue, Oct 03, 2017 at 09:12:13AM +0300, Timo Alho wrote:
> Tegra power management firmware running on co-processor (BPMP)
> implements a simple pseudo file system akin to debugfs. The file
> system can be used for debugging purposes to examine and change the
> status of selected resources controlled by the firmware (such as
> clocks, resets, voltages, powergates, ...).
> 
> Add support to "mirror" the firmware's file system to debugfs. At
> boot, query firmware for a list of all possible files and create
> corresponding debugfs entries. Read/write of individual files is
> implemented by sending a Message ReQuest (MRQ) that passes the full
> file path name and data to firmware via DRAM.
> 
> Signed-off-by: Timo Alho <talho@...dia.com>
> ---
> Changes in v2:
>  - Address Jonathan's review feedback
>   * restructure error printing and what error codes passed to higher
>     layers
>   * don't use IS_ERR_OR_NULL()
>   * avoid overwriting last-character of filename in one corner case
>     (name length = 255)
>   
>  drivers/firmware/tegra/Makefile       |   4 +-
>  drivers/firmware/tegra/bpmp.c         |   4 +
>  drivers/firmware/tegra/bpmp_debugfs.c | 444 ++++++++++++++++++++++++++++++++++
>  include/soc/tegra/bpmp.h              |  14 ++
>  4 files changed, 465 insertions(+), 1 deletion(-)
>  create mode 100644 drivers/firmware/tegra/bpmp_debugfs.c

I've applied this, though I turned the _ to - in the filename for
consistency with other files in the above directory.

Thanks,
Thierry

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)