linux-kernel - Re: [PATCH v9 4/7] staging: fpga manager: add sysfs interface document

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]

Date:	Fri, 24 Jul 2015 07:39:15 -0500
From:	atull <atull@...nsource.altera.com>
To:	Pavel Machek <pavel@...x.de>
CC:	<gregkh@...uxfoundation.org>, <jgunthorpe@...idianresearch.com>,
	<hpa@...or.com>, <monstr@...str.eu>, <michal.simek@...inx.com>,
	<rdunlap@...radead.org>, <linux-kernel@...r.kernel.org>,
	<devicetree@...r.kernel.org>, <pantelis.antoniou@...sulko.com>,
	<robh+dt@...nel.org>, <grant.likely@...aro.org>,
	<iws@...o.caltech.edu>, <linux-doc@...r.kernel.org>,
	<broonie@...nel.org>, <philip@...ister.org>, <rubini@...dd.com>,
	<s.trumtrar@...gutronix.de>, <jason@...edaemon.net>,
	<kyle.teske@...com>, <nico@...aro.org>, <balbi@...com>,
	<m.chehab@...sung.com>, <davidb@...eaurora.org>, <rob@...dley.net>,
	<davem@...emloft.net>, <cesarb@...arb.net>,
	<sameo@...ux.intel.com>, <akpm@...ux-foundation.org>,
	<linus.walleij@...aro.org>, <pawel.moll@....com>,
	<mark.rutland@....com>, <ijc+devicetree@...lion.org.uk>,
	<galak@...eaurora.org>, <devel@...verdev.osuosl.org>,
	Petr Cvek <petr.cvek@....cz>, <delicious.quinoa@...il.com>,
	<dinguyen@...nsource.altera.com>
Subject: Re: [PATCH v9 4/7] staging: fpga manager: add sysfs interface
 document

On Fri, 24 Jul 2015, Pavel Machek wrote:

Hi Pavel,

Thanks for your your feedback in cleaning up these docs.

> Hi!
> 
> > +What:		/sys/class/fpga_manager/<fpga>/state
> > +Date:		July 2015
> > +KernelVersion:	4.2
> > +Contact:	Alan Tull <atull@...nsource.altera.com>
> > +Description:	Read fpga manager state as a string.
> 
> fpga->FPGA.

Yep

> 
> > +		Valid states may vary by manufacturer; superset is:
> > +		* unknown		= can't determine state
> > +		* power off		= FPGA power is off
> > +		* power up		= FPGA reports power is up
> > +		* reset			= FPGA held in reset state
> > +		* firmware request	= firmware class request in progress
> > +		* firmware request error = firmware request failed
> > +		* write init		= FPGA being prepared for programming
> > +		* write init error	= Error while preparing FPGA for
> > +					  programming
> > +		* write			= FPGA ready to receive image data
> > +		* write error		= Error while programming
> > +		* write complete	= Doing post programming steps
> > +		* write complete error	= Error while doing post programming
> > +		* operating		= FPGA is programmed and operating
> 

If I can make my intent clear, maybe we can figure out what will be most
useful here.  

The intent is to provide enough detail that if something goes wrong with
the FPGA programming (something that the driver can't take care of) then 
userspace can know that.  Such as if the firmware request fails, that 
could be due to not being able to find the firmware file.

> This will need some more details. "firmware request" is hardly a
> hardware state, does it belong here? 

This is a superset of FPGA states and fpga manager driver states as the
fpga manager driver is walking through the steps to get the FPGA into
a known operating state.  So it's a sequence, though some steps may get
skipped. If there is an error, then userspace can know what step failed.

Maybe this should be separated into fpga_state for hardware state and
fpga_mgr_status (to report what step of progress the fpga manager driver
is at during programming).  I want this to be useful and still not be 
device (FPGA) specific.

> Is power off or on while firmware
> is being requested? 

On.  It's a sequence.

> How does the fpga get into power up phase?
> Normally, you'd only power it on to do something more with it...?
> 
> 									Pavel



> 
> -- 
> (english) http://www.livejournal.com/~pavelmachek
> (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
> 
--
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/