linux-kernel - Re: [PATCH v9 0/9] Add simple NVMEM Framework via regmap.

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: <55C87EEF.6020609@linaro.org>
Date:	Mon, 10 Aug 2015 11:37:35 +0100
From:	Srinivas Kandagatla <srinivas.kandagatla@...aro.org>
To:	Andrew Lunn <andrew@...n.ch>
CC:	stefan.wahren@...e.com, devicetree@...r.kernel.org, arnd@...db.de,
	khilman@...nel.org,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	s.hauer@...gutronix.de, sboyd@...eaurora.org,
	linux-kernel@...r.kernel.org, mporter@...sulko.com,
	pantelis.antoniou@...sulko.com, Rob Herring <robh+dt@...nel.org>,
	Mark Brown <broonie@...nel.org>, linux-api@...r.kernel.org,
	linux-arm-msm@...r.kernel.org,
	linux-arm-kernel@...ts.infradead.org, wxt@...k-chips.com
Subject: Re: [PATCH v9 0/9] Add simple NVMEM Framework via regmap.

Hi Andrew,

On 09/08/15 16:19, Andrew Lunn wrote:
> Hi Srinivas
>
> The AT24 eeprom driver contains the comment:
>
>          /*
>           * Export the EEPROM bytes through sysfs, since that's convenient.
>           * By default, only root should see the data (maybe passwords etc)
>           */
>
> and as you would expect from this:
>
> # ls -l ./devices/platform/soc/soc:aips-bus@...00000/40066000.i2c/i2c-0/0-0050/eeprom
> -rw------- 1 root root 512 Aug  9 10:16 ./devices/platform/soc/soc:aips-bus@...00000/40066000.i2c/i2c-0/0-0050/eeprom
>
> The AT25 and the MAX6875 driver are the same.
>
> However nvmem has different defaults:
>
> # ls -l ./devices/platform/soc/soc:aips-bus@...00000/40066000.i2c/i2c-0/0-0050/0-00500/nvmem
> -rw-r--r-- 1 root root 0 Aug  9 10:16 ./devices/platform/soc/soc:aips-bus@...00000/40066000.i2c/i2c-0/0-0050/0-00500/nvmem
>
Yes, nvmem by default sets the read permission for everyone S_IRUGO. 
Which was the most common use case.

> Has this been considered and discussed?
Thanks for bringing this up, No, we did not discuss this explicitly, but 
it should not be an issue to accommodate this. Provider still have 
flexibility to configure such things.

As a first patchset we wanted to keep the framework simple and add 
features as we move on.

Thanks,
srini
>
> Thanks
>      Andrew
>
--
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/