On Wed, 24 Feb 2021 10:49:41 +0100, Jaroslav Kysela wrote:
Dne 24. 02. 21 v 10:38 Takashi Iwai napsal(a):
It seems that you misunderstood the number of issues which my code is trying to resolve:
- set LED based on state from multiple cards (so you cannot trigger LED
inside single driver / single control element); we need one arbiter; this is the main argument 2) unifies the audio LED interface 3) reduce the hardware driver code
Those purposes are all fine. But they don't need to be exposed for user controls that can be abused. That's the very concern.
So, how to handle this feature for AMD ACP without PA / PipeWire modifications?
And if we add an user space channel to the audio LED arbiter code, how it differs from my proposed control API extension?
As the early patch does, creating a kernel control (but not a generic "Capture" switch but specific to LED) and control it in UCM profile. What's the practical problem there? That's what I might have missed as the starting point of the discussion.
We have already locking mechanism for the user control element to one task, so it's possible to create safe user space implementation (depending on the standard task priviledges) on demand even with my proposal.
Could you elaborate the abuse you mean? Three bits?
You can create up to 1028 user controls per card and each of them can fire the led trigger... That's an interesting experiment :)
So far, a user control is merely storing the value, let read/write via the control API. That's all, and nothing wrong can happen just by that. Now if it interacts with other subsystem...
A more serious concern is rather the fragility of the setup; for enabling the mute LED control, you'd have to create a new user-space control, the function of the control has to be ignored by some application and some not, etc. This has to be done on each machine when the system got updated. And, not everyone is using alsa-lib. Does tiny ALSA and other existing backend support the user control element management? Some uncertainty here.
thanks,
Takashi