From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Jan Beulich" <JBeulich@suse.com>
Subject: Re: [PATCH v2 26/41] arm : acpi add xen environment
 table
Date: Thu, 21 May 2015 11:46:02 +0100
Message-ID: <555DD38A020000780007CAF5@mail.emea.novell.com>
References: <1431893048-5214-1-git-send-email-parth.dixit@linaro.org>
	<1431893048-5214-27-git-send-email-parth.dixit@linaro.org>
	<555CD101020000780007C65C@mail.emea.novell.com>
	<555CBDCB.6010600@citrix.com>
	<555D95AA020000780007C87E@mail.emea.novell.com>
	<555DB4BD.9070308@citrix.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <555DB4BD.9070308@citrix.com>
Content-Disposition: inline
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Julien Grall <julien.grall@citrix.com>, Parth Dixit <parth.dixit@linaro.org>
Cc: keir@xen.org, ian.campbell@citrix.com, andrew.cooper3@citrix.com, tim@xen.org, xen-devel@lists.xen.org, stefano.stabellini@citrix.com, christoffer.dall@linaro.org
List-Id: xen-devel@lists.xenproject.org

>>> On 21.05.15 at 12:34, <julien.grall@citrix.com> wrote:
> On 21/05/15 07:22, Jan Beulich wrote:
>> The linked to document (on our wiki) is versioned 0.<something>,
>> which doesn't look like a final stable version. The same applies to
>> the other (STAO?) one.
> 
> That's a mistake in the version number. Those tables has been reviewed
> by Citrix and Linaro people and we agreed about the final tables.

And Citriy+Linaro are the standardizing body here? With no-one
else involved?

>>> For the device tree, we
>>> include a new node. For ACPI, this table allow us to know the we are
>>> running on Xen.
>> 
>> Which seems superseded by 6.0's hypervisor vendor identification
>> in FADT. And the OEM IDs in various table headers could have
>> served such identification purposes too, as could have "OEMx"
>> tables.
> 
> ACPI 6.0 has been released few months after Parth and Naresh began to
> implement ACPI for Xen. We could take advantage of this new field.

If at all possible - yes please, in favor of any custom tables.

Jan