From: Greg KH <greg@kroah.com>
To: linux-hotplug@vger.kernel.org
Subject: Re: udev and sysfs permissions
Date: Fri, 20 May 2005 21:40:20 +0000 [thread overview]
Message-ID: <20050520214020.GA19677@kroah.com> (raw)
In-Reply-To: <9e47339105051915025188e535@mail.gmail.com>
On Fri, May 20, 2005 at 05:26:09PM -0400, Jon Smirl wrote:
> On 5/20/05, Greg KH <greg@kroah.com> wrote:
> > On Fri, May 20, 2005 at 05:11:01PM -0400, Jon Smirl wrote:
> > > On 5/20/05, Greg KH <greg@kroah.com> wrote:
> > > > On Fri, May 20, 2005 at 10:06:24AM -0400, Jon Smirl wrote:
> > > > > On 5/20/05, Greg KH <greg@kroah.com> wrote:
> > > > > > Nope, the kernel is. You must have provided enough memory pressure to
> > > > > > push the file out of the dcache, and then when you went to look at it
> > > > > > again, it was created on the fly from scratch again, with the proper
> > > > > > permissions (as the kernel thinks the files have.) Nice to see it's all
> > > > > > working properly :)
> > > > > >
> > > > > > > Can udev control sysfs permissions (I though it only controlled the
> > > > > > > device permissions).
> > > > > >
> > > > > > No, only the kernel can control sysfs permissions.
> > > > >
> > > > > We were planning on having PAM assign ownership of the video device
> > > > > and sysfs attributes to the logged in user.
> > > >
> > > > video device, fine. sysfs attributes, no.
> > > >
> > > > > I need read/write access to the sysfs attributes but it need to be
> > > > > restricted to whoever owns the device.
> > > >
> > > > Ick. what kind of attributes do you want the logged in user to be able
> > > > to change?
> > >
> > > After everyone complained that IOCTLs were so evil and that sysfs
> > > attributes were the way to go, I added a bunch of attributes for
> > > controlling the framebuffer device. Load a fbdev driver and look in
> > > /sys/class/graphics/fb0.
> > >
> > > [jonsmirl@jonsmirl fb0]$ ls
> > > bits_per_pixel color_map cursor device modes virtual_size
> > > blank console dev mode pan
> > > [jonsmirl@jonsmirl fb0]$
> > >
> > > You can change the mode, cursor position, screen size, pan, etc by
> > > writing to sysfs attributes. These attributes need to only be writable
> > > only by the person who owns the device.
> >
> > Very nice.
> >
> > > If I can't control permissions on these attributes I'll just get rid
> > > of them all and go back to IOCTLs.
> >
> > How about two more files, "user" and "group" that are writable only by
> > root that your pam module writes to to set the user and group of the
> > files? That way the kernel does the modification, and will always keep
> > the proper permissions.
>
> How about marking sysfs attributes at creation time to say that their
> permissions mimic the permissions assigned to the /dev device?
How can the kernel know what userspace uses to create a /dev node with?
> Then PAM can switch the ownership of the device and all of the marked
> attributes will automatically follow.
Again, how would the kernel learn of this switch?
> That would make things simpler for driver writers if more devices are
> going to follow this model.
I think you are going to be pretty unique here, as no one else has come
up with a situation yet that requires this.
thanks,
greg k-h
-------------------------------------------------------
This SF.Net email is sponsored by Oracle Space Sweepstakes
Want to be the first software developer in space?
Enter now for the Oracle Space Sweepstakes!
http://ads.osdn.com/?ad_idt12&alloc_id\x16344&op=click
_______________________________________________
Linux-hotplug-devel mailing list http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel
next prev parent reply other threads:[~2005-05-20 21:40 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-05-19 22:02 udev and sysfs permissions Jon Smirl
2005-05-19 22:10 ` Kay Sievers
2005-05-20 4:33 ` Greg KH
2005-05-20 14:06 ` Jon Smirl
2005-05-20 18:33 ` Greg KH
2005-05-20 21:11 ` Jon Smirl
2005-05-20 21:26 ` Jon Smirl
2005-05-20 21:27 ` Greg KH
2005-05-20 21:40 ` Jon Smirl
2005-05-20 21:40 ` Greg KH [this message]
2005-05-20 21:41 ` Jon Smirl
2005-05-20 21:53 ` Jon Smirl
2005-05-20 21:54 ` Greg KH
2005-05-20 21:56 ` Greg KH
2005-05-20 22:07 ` Kay Sievers
2005-05-20 22:09 ` Greg KH
2005-05-26 23:09 ` Greg KH
2005-05-27 12:44 ` Maneesh Soni
2005-05-27 16:39 ` Jon Smirl
2005-05-27 21:51 ` Greg KH
2005-05-28 5:06 ` maneesh
2005-05-28 5:08 ` maneesh
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20050520214020.GA19677@kroah.com \
--to=greg@kroah.com \
--cc=linux-hotplug@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).