[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Xen-devel] [PATCH v15] This is the ABI for the two halves of a para-virtualized sound driver to communicate with each to other.
On 02/07/2017 09:16 PM, Konrad Rzeszutek Wilk wrote:
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 needFAE?Field Applications Engineerto 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 documentationI 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 documentedOK, 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.
Thank you _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx https://lists.xen.org/xen-devel
Lists.xenproject.org is hosted with RackSpace, monitoring our