Re: [PATCH v15] This is the ABI for the two halves of a para-virtualized sound driver to communicate with each to other.

[Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>]

On Mon, Jan 30, 2017 at 08:49:50AM +0200, Oleksandr Andrushchenko wrote:
> On 01/27/2017 08:57 PM, Konrad Rzeszutek Wilk wrote:
> > On Fri, Jan 27, 2017 at 08:38:29PM +0200, Oleksandr Andrushchenko wrote:
> > > 
> > > On 01/27/2017 08:13 PM, Konrad Rzeszutek Wilk wrote:
> > > > .snip..
> > > > > I am looking at this from FAE's or integrator's POV, when one would need
> > > > FAE?
> > > > 
> > > Field Applications Engineer
> > > > > to touch different parts of the system. "/0/0/0" makes me feel
> > > > > sad just because either you have to keep all those numbers in mind (like you
> > > > > do)
> > > > > or have documentation available (and know where it is, e.g. sources
> > > > > of Xen or kernel).
> > > > > I have a strong feeling that if you can change configuration without
> > > > > knowing what index stands for it is always better and fail-safer then
> > > > > just having numbers...
> > > > Not sure I follow that.
> > > > 
> > > > How would you change configuration without knowing the index?
> > > > 
> > > > ..snip..
> > > if one looks at
> > > .../pcm-dev-0/stream-1/...
> > > most probably he/she will understand this w/o any documentation,
> > > because it is human readable
> > > 
> > > if one looks at
> > > .../0/1/...
> > > well, I believe you can almost do nothing w/o looking at the documentation
> > I can see the beaty of it.
> > 
> > I can also see the beaty of the old implied mechanism
> > from a maintaince perspective.
> > 
> > I am maintainer so I am leaning towards the second one
> > as having less "special" cases.
> > 
> > Sorry, I feel like I am mutilating your baby with this
> > old boring view of "maintaince" and "conform to the old
> > standard" :-(
> > 
> np, so I will use ".../0/0/0/..." pattern here and in
> displif as well. I will update "Addressing" section
> so it is documented

OK, so I will ignore v16 as of right now it has the old format.
Please at your convience post v16.1 (or v17) with this change.

Thanks!

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel