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: <20110325222716.GT3130@pulham.picochip.com>
Date:	Fri, 25 Mar 2011 22:27:16 +0000
From:	Jamie Iles <jamie@...ieiles.com>
To:	Mike Frysinger <vapier@...too.org>
Cc:	Jamie Iles <jamie@...ieiles.com>, linux-kernel@...r.kernel.org,
	gregkh@...e.de
Subject: Re: [RFC PATCHv2 1/4] drivers/otp: add initial support for OTP memory

On Fri, Mar 25, 2011 at 05:32:54PM -0400, Mike Frysinger wrote:
> On Fri, Mar 25, 2011 at 06:08, Jamie Iles wrote:
> > I guess in this framework we could make the OTP look like a single
> > region of the 0x80 data pages concatenated or provide separate regions
> > for each group of 0x80 pages but the control and ECC pages aren't
> > directly accessible by the user.
> 
> well, in the current driver, i let people dump the ECC.  for
> debugging, i think that's valuable.

Hmm, that does sound potentially useful.  In v3 I've created a single 
partition that excludes the control bits and the ecc.  How about I 
change that for the next spin so there are two partitions where the 
first contains the raw data and the second is all of the ECC lumped 
together.  It may also be useful to have a name attribute for each 
region to make this clear.

It's either that or add another ioctl() for getting the ECC but that 
doesn't feel very nice.

> > The other question that brings up is whether any compatibility with the
> > old driver is required.
> 
> it's already going to be changing names ;).  but what other
> compatibility are you thinking of ?  the requirements are you have to
> be able to:
>  - read it
>  - write in full/aligned 64bit chunks
>  - do an OTPLOCK ioctl (the drivers uses this to manage the first few
> pages which flags all other pages as "locked")

Well the current version will do all of that so I think if we add the 
ECC dumping ability then everything is there.

Jamie
--
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/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ