From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Jan Beulich" <JBeulich@suse.com>
Subject: Re: Design doc of adding ACPI support for arm64 on Xen
 - version 5
Date: Mon, 31 Aug 2015 05:46:53 -0600
Message-ID: <55E45ACD020000780009E5B1@prv-mh.provo.novell.com>
References: <55E02DC5.4090202@huawei.com>
	<55E09528020000780009DE26@prv-mh.provo.novell.com>
	<55E10AEF.6020806@linaro.org>
	<55E420ED020000780009E2D4@prv-mh.provo.novell.com>
	<55E41598.70701@huawei.com>
	<55E43D3A020000780009E3B2@prv-mh.provo.novell.com>
	<55E43B06.3080406@huawei.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: <55E43B06.3080406@huawei.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: Shannon Zhao <zhaoshenglong@huawei.com>, Shannon Zhao <shannon.zhao@linaro.org>
Cc: Hangaohuai <hangaohuai@huawei.com>, Ian Campbell <ian.campbell@citrix.com>, Stefano Stabellini <stefano.stabellini@eu.citrix.com>, andrew@fubar.geek.nz, "Huangpeng (Peter)" <peter.huangpeng@huawei.com>, Julien Grall <julien.grall@citrix.com>, Stefano Stabellini <stefano.stabellini@citrix.com>, David Vrabel <david.vrabel@citrix.com>, BorisOstrovsky <boris.ostrovsky@oracle.com>, xen-devel <xen-devel@lists.xen.org>, Parth Dixit <parth.dixit@linaro.org>, Christoffer Dall <christoffer.dall@linaro.org>, RogerPauMonne <roger.pau@citrix.com>
List-Id: xen-devel@lists.xenproject.org

>>> On 31.08.15 at 13:31, <zhaoshenglong@huawei.com> wrote:
> On 2015/8/31 17:40, Jan Beulich wrote:
>>>>> On 31.08.15 at 10:51, <zhaoshenglong@huawei.com> wrote:
>>> (I wonder why you didn't get this if you have a glance at the booting
>>> process. uefi_init(arch/arm64/kernel/efi.c of Linux) -->
>>> efi_config_parse_tables --> match_config_table. It will save the
>>> VenderTable to efi.acpi20 and when Linux call acpi_os_get_root_pointer
>>> to get ACPI root address, it will return efi.acpi20)
>> 
>> See above - if I hadn't realized you, in every single place you use it,
>> really mean "the VendorTable field of the Configuration Table entry
>> using ACPI_20_TABLE_GUID", I would have complained here again.
>> (Of course you don't need to spell it this way every time, but you
>> should imo spell it this or a similar way at least once in each section.)
> 
> So you can't get the meaning of "the value of VendorTable in EFI
> Configuration Table"?

Without proper context, VendorTable to me refers to a table, not
the field of a particular table's slot.

Jan