linux-kernel - Re: [PATCH v9 01/12] coresight: add CoreSight core layer framework

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: <CANLsYkzKB7iBUFxsqmaoHob26mEZvfbVSA1O_ZadFCYHE2viAA@mail.gmail.com>
Date:	Fri, 7 Nov 2014 14:39:31 -0700
From:	Mathieu Poirier <mathieu.poirier@...aro.org>
To:	Greg KH <gregkh@...uxfoundation.org>
Cc:	Russell King - ARM Linux <linux@....linux.org.uk>,
	Pratik Patel <pratikp@...eaurora.org>,
	Vikas Varshney <varshney@...com>, Al Grant <Al.Grant@....com>,
	Jonas Svennebring <jonas.svennebring@...gotech.com>,
	James King <james.king@...aro.org>,
	Kaixu Xia <kaixu.xia@...aro.org>,
	Marcin Jabrzyk <marcin.jabrzyk@...il.com>,
	"r.sengupta@...sung.com" <r.sengupta@...sung.com>,
	trollkarlen Marklund <robbelibobban@...il.com>,
	Patch Tracking <patches@...aro.org>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v9 01/12] coresight: add CoreSight core layer framework

On 7 November 2014 12:30, Greg KH <gregkh@...uxfoundation.org> wrote:
> On Fri, Nov 07, 2014 at 11:30:16AM -0800, Greg KH wrote:
>> On Mon, Nov 03, 2014 at 11:07:35AM -0700, mathieu.poirier@...aro.org wrote:
>> > From: Pratik Patel <pratikp@...eaurora.org>
>> >
>> > CoreSight components are compliant with the ARM CoreSight
>> > architecture specification and can be connected in various
>> > topologies to suit a particular SoC tracing needs. These trace
>> > components can generally be classified as sources, links and
>> > sinks. Trace data produced by one or more sources flows through
>> > the intermediate links connecting the source to the currently
>> > selected sink.
>> >
>> > The CoreSight framework provides an interface for the CoreSight trace
>> > drivers to register themselves with. It's intended to build up a
>> > topological view of the CoreSight components and configure the
>> > correct serie of components on user input via sysfs.
>> >
>> > For eg., when enabling a source, the framework builds up a path
>> > consisting of all the components connecting the source to the
>> > currently selected sink(s) and enables all of them.
>> >
>> > The framework also supports switching between available sinks
>> > and provides status information to user space applications
>> > through the debugfs interface.
>> >
>> > Signed-off-by: Pratik Patel <pratikp@...eaurora.org>
>> > Signed-off-by: Mathieu Poirier <mathieu.poirier@...aro.org>
>>
>> You don't seem to document the sysfs files you create here in
>> Documentation/ABI/ anywhere in the series.  Can you fix that up?

Thanks for the review.   Do you want sysfs entries documented for the
framework only or all the ones created by the drivers as well?

>>
>> Other than that minor nit, this looks good to me.  Feel free to take it
>> through some random ARM tree.
>
> Wait, did you want me to take this through my tree?  If so, I'll be glad
> to do so, I can't remember what we talked about here...

I was hoping to go through yours.

>
> thanks,
>
> greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/