<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Apr 20, 2015 at 2:40 PM, Lennart Poettering <span dir="ltr"><<a href="mailto:lennart@poettering.net" target="_blank">lennart@poettering.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Mon, 20.04.15 20:39, Lennart Poettering (<a href="mailto:lennart@poettering.net">lennart@poettering.net</a>) wrote:<br>
<br>
> On Mon, 20.04.15 20:00, Lennart Poettering (<a href="mailto:lennart@poettering.net">lennart@poettering.net</a>) wrote:<br>
><br>
> > On Fri, 10.04.15 22:27, Adam Goode (<a href="mailto:agoode@google.com">agoode@google.com</a>) wrote:<br>
> ><br>
> > > The ALSA id sysattr is generated by the sound subsystem and is not<br>
> > > a stable identifier. It is generated though some string manipulation<br>
> > > then made unique if there is a conflict. This means that it is<br>
> > > enumeration-dependent and shouldn't be used for ID_ID.<br>
> > ><br>
> > > If ID_ID is supposed to be system-unique, it is not already since<br>
> > > for firewire it is generated from the guid and there are broken<br>
> > > firewire devices that have duplicate guids across devices.<br>
> ><br>
> > Hmm, this patch pretty much reverts<br>
> > ed1b2d9fc7d5c5bfe2a67b0b8ff9e5ea8694268e. Now I am not sure if that<br>
> > commit from 6 years ago was a good idea, but we should have some<br>
> > clarity about this.<br>
> ><br>
> > What is ID_ID actually supposed to be? Should it really be system<br>
> > unique?<br>
> ><br>
> > I do have the suspicion this is something that better should be fixed in<br>
> > PA rather then in these udev rules, so I figure your patch might be a<br>
> > good idea?<br>
> ><br>
> > Opinions?<br>
> ><br>
> > If we apply the patch somebody should at least post a bug report<br>
> > against PA to be aware of this change.<br>
><br>
> OK, after talking to some folks I think we should merge your<br>
> patch. Any chance you can post a bug to PA though, reference this<br>
> discussion, and then include the bug report in the commit message and<br>
> resend the patch?<br>
<br>
</div></div>Also, please mention in the commit msg that this is basically a revert<br>
of ed1b2d9fc7d5c5bfe2a67b0b8ff9e5ea8694268e.<br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div><br></div><div>Thanks, happy to do this. Look for it soon.</div><div><br></div><div><br></div><div>Adam</div><div><br></div></div></div></div>