From: Paul Fox <pgf@laptop.org>
To: linux-hotplug@vger.kernel.org
Subject: Re: keymap rule selection for non-DMI platforms
Date: Tue, 16 Aug 2011 20:49:16 +0000 [thread overview]
Message-ID: <3031.1313527756@foxharp.boston.ma.us> (raw)
In-Reply-To: <3483.1313087020@foxharp.boston.ma.us>
kay wrote:
> On Tue, Aug 16, 2011 at 22:09, Daniel Drake <dsd@laptop.org> wrote:
> > On Tue, Aug 16, 2011 at 8:34 PM, Kay Sievers <kay.sievers@vrfy.org> wrote:
> >> Reading such things from /proc is kinda taboo from code we maintain in
> >> udev. All things not related to processes really do not belong into
> >> /proc and udev code should never get into the way of possibly
> >> deprecating these things in the long run, even when they might never
> >> happen. I know, there is sometimes no other simple option, but we
> >> generally prefer the inconvenience it causes, over adding hacks to
> >> upstream code, to make a move to a generally useful solution (which
> >> isn't /proc/*) more attractive.
> >
> > I agree that the use of /proc is strange, but just to make sure you
> > understand: /proc/device-tree is a standard upstream kernel thing and
> > has been for a long time. It is not some OLPC-specific oddity to
> > access our manufacturing data. It is *the* way to access device tree
> > info on ARM, PPC, SPARC (and x86). And device tree is specifically
> > built as a way of identifying hardware info which the silicon can't
> > tell you. udev implementing support for device tree will solve OLPC's
> > keyboard identification issue, but you'll also solve a whole class of
> > problems for the wide range of platforms that use device tree (and
> > those that will soon use it).
>
> That might all be true, I don't complain, it's just that udev upstream
> does not read things like that from /proc, no matter how common the
> use is.
>
> Stuff belongs into /sys/firmware/ /sys/wherever not /proc, and udev
> can not read things like that from /proc because that would prevent
> anybody from ever fixing it with the argument that one of the main
> components relies on it.
so just to be sure i understand your objection: if /proc/device-tree
were to move to /sys/device-tree (or similar path, and perhaps be
doubly mounted there during a transition period), it would be
acceptable to propose that udev start using it?
paul
>
> >> I guess one sensible option is to register the /sys/class/dmi
> >> interface to ARM too, even when it's not called that way for ARM
> >> hardware. 'Desktop Management Interface' makes not much sense anyway,
> >> not even for x86, but hey it's only 3 characters, whatever they mean.
> >> :)
> >
> > I think you're too late to suggest a new solution to this problem.
> > This is exactly what device tree is for, and Linux has been steadily
> > going in this direction for a while.
>
> Sure, if there is something we all can use, we will switch over to it.
> Until that happens, hacks have to be maintained by the people relying
> on them, not by udev upstream.
>
> > However, the location inside of /proc is certainly something that can
> > be criticised.
>
> It's just nothing udev will use. Archs can do what they think it's
> right, and they might be right, I'll not argue here.
>
> But userspace should step back working around such things, and people
> should start working on the 'proper' solution.
>
> >> The alternative is to replace /sys/class/dmi with some completely arch
> >> and platform independent interface and export there what dmi currently
> >> supports and plug-in the other platforms.
> >
> > Device tree is already arch and platform independent, but I'm not sure
> > how you would make DMI info look like a device tree. Despite both
> > being able to provide identification info, they are quite different
> > beasts.
>
> Well, then upstream udev will wait for the common solution. :)
>
> Kay
=---------------------
paul fox, pgf@laptop.org
next prev parent reply other threads:[~2011-08-16 20:49 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-08-11 18:23 keymap rule selection for non-DMI platforms Paul Fox
2011-08-16 18:54 ` Paul Fox
2011-08-16 19:34 ` Kay Sievers
2011-08-16 20:09 ` Daniel Drake
2011-08-16 20:30 ` Kay Sievers
2011-08-16 20:39 ` Daniel Drake
2011-08-16 20:49 ` Paul Fox [this message]
2011-08-16 21:47 ` Greg KH
2011-08-16 22:27 ` Kay Sievers
2011-08-16 22:34 ` Kay Sievers
2011-08-16 22:37 ` Kay Sievers
2011-08-16 22:54 ` Daniel Drake
2011-08-16 23:35 ` Prarit Bhargava
2011-08-17 2:09 ` Karl O. Pinc
2011-08-17 3:32 ` Paul Fox
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=3031.1313527756@foxharp.boston.ma.us \
--to=pgf@laptop.org \
--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).