linux-kernel - Re: [PATCH v3 0/2] remoteproc: Add sysfs interface

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <bbd51198-5fd2-e007-a788-3cb0b94e525b@imgtec.com>
Date:   Tue, 1 Nov 2016 09:25:09 +0000
From:   Matt Redfearn <matt.redfearn@...tec.com>
To:     Bjorn Andersson <bjorn.andersson@...aro.org>
CC:     Ohad Ben-Cohen <ohad@...ery.com>,
        <linux-remoteproc@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v3 0/2] remoteproc: Add sysfs interface



On 01/11/16 00:31, Bjorn Andersson wrote:
> On Thu 27 Oct 07:10 PDT 2016, Matt Redfearn wrote:
>
>> On 19/10/16 13:05, Matt Redfearn wrote:
>>> It is often desireable to boot a remote processor with different
>>> firmware files, depending on the needs of the system at a particular
>>> time. This series adds a sysfs interface to the remoteproc core,
>>> exposing interfaces to manipulate the remote processor. One interface is
>>> the "state" file which performs the same function as the one in debugfs
>>> (which is removed later in the series). The other is a "firmware" file
>>> which allows retrieval of the name of the running firmware, and allows a
>>> new firmware to be loaded when written, as long as the remote processor
>>> is currently stopped.
>>>
>>> Some groundwork must be laid first, changing the storage mechanism of
>>> the firmware name such that it can be rewritten easily, then that is
>>> wired up to the new sysfs interface.
>>>
>>> This series is based on v4.9-rc1
>>>
>>>
>>> Changes in v3:
>>> Drop call to rproc_add_virtio_devices from sysfs firmware_store
>>> Use strcspn to find firmware name length
>>> Explicit indexes for state strings
>>>
>>> Changes in v2:
>>> Have firmware_store perform the necessary steps inline.
>>> Use sysfs_streq when dealing with writes to sysfs files
>>>
>>> Matt Redfearn (2):
>>>    remoteproc: Add a sysfs interface for firmware and state
>>>    remoteproc: debugfs: Remove state entry which is duplicated is sysfs
>>>
>>>   Documentation/ABI/testing/sysfs-class-remoteproc |  50 ++++++++
>>>   drivers/remoteproc/Makefile                      |   1 +
>>>   drivers/remoteproc/remoteproc_core.c             |   3 +
>>>   drivers/remoteproc/remoteproc_debugfs.c          |  71 -----------
>>>   drivers/remoteproc/remoteproc_internal.h         |   5 +
>>>   drivers/remoteproc/remoteproc_sysfs.c            | 151 +++++++++++++++++++++++
>>>   6 files changed, 210 insertions(+), 71 deletions(-)
>>>   create mode 100644 Documentation/ABI/testing/sysfs-class-remoteproc
>>>   create mode 100644 drivers/remoteproc/remoteproc_sysfs.c
>>>
>> Hi Bjorn,
>>
>> Is this version of the patchset ok? Any feedback welcome :-)
>>
> Hi Matt,
>
> I was hoping for some comments or verification from others, so I
> conveniently did let it hang out in my TODO list.
>
> I think the new interface looks good and have merged the two patches!

Great, thanks! :-)

Matt

>
> Regards,
> Bjorn