linux-kernel - Re: char interface to sdio, chdio.c

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:	Wed, 16 Jul 2014 16:08:11 -0700
From:	Stephen Boyd <sboyd@...eaurora.org>
To:	Greg KH <greg@...ah.com>
CC:	linux-kernel@...r.kernel.org, linux-mmc@...r.kernel.org,
	"linux-arm-msm@...r.kernel.org" <linux-arm-msm@...r.kernel.org>,
	Alexander Kolesnikov <akolesni@...eaurora.org>,
	Konstantin Dorfman <kdorfman@...eaurora.org>
Subject: Re: char interface to sdio, chdio.c

On 07/16/14 12:38, Stephen Boyd wrote:
> On 07/16/14 11:44, Greg KH wrote:
>> My specific question is why 2 different interfaces?
> Ok. It definitely looks odd to support set/get of the VDD with sysfs and
> ioctl interfaces. I'm just guessing, but I suspect it's a permissions
> thing given that the patch that introduces the VDD stuff also updates
> the permissions of the device node. Of course, the commit text only
> mentions an ioctl, nothing about sysfs. The only other sysfs stuff seems
> to be module parameters to change the vendor ids.
>
> I'll have to go repo diving to figure out if the sysfs interface was
> ever used. Maybe Alex or Konstantin can respond before I have to do that.
>
>

I couldn't find any usage of the sysfs interface. The only thing I see
used is the host_name field when the module is inserted, if that even
happens at all (I'm not sure because it's behind some ifdef that may or
may not be defined). Sorry this doesn't explain why there are two
different interfaces for the VDD stuff. Hopefully Alex or Konstantin can
help.

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by The Linux Foundation

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