From: Lennart Poettering <mznyfn@0pointer.de>
To: systemd-devel@lists.freedesktop.org, alsa-devel@alsa-project.org,
ffado-devel@lists.sourceforge.net,
linux1394-devel@lists.sourceforge.net, clemens@ladisch.de
Subject: Re: udev: Access permission of fw character device in which fw node includes unit handled by ALSA
Date: Mon, 17 Jun 2019 15:26:42 +0200 [thread overview]
Message-ID: <20190617132642.GA25074@gardel-login> (raw)
In-Reply-To: <20190520122724.GA14864@workstation>
On Mo, 20.05.19 21:27, Takashi Sakamoto (o-takashi@sakamocchi.jp) wrote:
> Hi all,
>
> I'm an ALSA developer, mainly committing to drivers for audio and
> music units on IEEE 1394 bus (yes, it's legacy FireWire). I have a
> concern about access permission for fw character device.
I am not sure I understand firewire well enough to say something too
smart. But if I understand correctly, then we (in systemd upstream)
currently only have these four firewire specific rules in our tree:
SUBSYSTEM=="firewire", ATTR{units}=="*0x00a02d:0x00010*", GROUP="video"
SUBSYSTEM=="firewire", ATTR{units}=="*0x00b09d:0x00010*", GROUP="video"
SUBSYSTEM=="firewire", ATTR{units}=="*0x00a02d:0x010001*", GROUP="video"
SUBSYSTEM=="firewire", ATTR{units}=="*0x00a02d:0x014001*", GROUP="video"
And all of these are handled by some firewire core driver? The devices
you care about, how would the rules look like for this?
In general, when it comes to auto-loading of drivers we usually expect
the kernel to add modalias metadata to .ko files.
In the hwdb usually mostly only "auxiliary" data is stored, i.e. stuff
that is great to have but is not essential for a device to work
basically.
Rules files should generally be kept short. i.e. if you add 3 more
rules for the devices you care about, that'd be fine — if you add 100,
then that's not so great however, and another approach should be
found.
Anyway, given that I am not a firewire pro, I figure you have to
decide yourself what the best approach is, and then prep a PR and we
can have a look. Without looking at an (maybe just RFC) patch I am not
sure I grok enugh to properly comment on it.
Lennart
--
Lennart Poettering, Berlin
_______________________________________________
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/systemd-devel
next prev parent reply other threads:[~2019-06-17 13:26 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-20 12:27 udev: Access permission of fw character device in which fw node includes unit handled by ALSA Takashi Sakamoto
2019-06-17 13:26 ` Lennart Poettering [this message]
2019-06-18 15:05 ` Takashi Sakamoto
2019-06-18 0:05 ` [FFADO-devel] " Jonathan Woithe
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=20190617132642.GA25074@gardel-login \
--to=mznyfn@0pointer.de \
--cc=alsa-devel@alsa-project.org \
--cc=clemens@ladisch.de \
--cc=ffado-devel@lists.sourceforge.net \
--cc=linux1394-devel@lists.sourceforge.net \
--cc=systemd-devel@lists.freedesktop.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