From: Crystal Kolipe <kolipe.c@exoticsilicon.com>
Subject: Re: sndio: show the real device name in server.device control
To: Alexandre Ratchov <alex@caoua.org>
Cc: tech@openbsd.org, jmc@kerhand.co.uk
Date: Thu, 7 Mar 2024 13:50:47 -0300

On Thu, Mar 07, 2024 at 01:17:22PM +0100, Alexandre Ratchov wrote:
> On Thu, Mar 07, 2024 at 08:26:46AM -0300, Crystal Kolipe wrote:
> > With this first version of the code, these strings are unique.  Do we
> > intend to guarantee this when the names are expanded to include further
> > chipset information?
> > 
> 
> You're right, the display strings can't be unique,
> unfortunately. Furthermore, currently they are not stable with time.
> 
> > If not, should we mention this somewhere to avoid userland code relying
> > on behaviour that will change later on?
> > 
> 
> Sure. Something like this?

Looks good, but I think we should s/no/nor

Cc'ed Jason for a second opinion on this.

> 
> --- a/libsndio/sioctl_open.3
> +++ b/libsndio/sioctl_open.3
> @@ -244,6 +244,7 @@ The
>  .Fa display
>  attribute contains an optional free-format string providing additional
>  hints about the control, like the hardware model, or the units.
> +It is neither unique at a given time, no stable with time.

 +It is neither unique at a given time, nor stable with time.

( Maybe s/with/over/ as well? )