[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] ACPICA: ACPI 6.0: Add support for IORT table.



Appreciate the feedback. I want to use these defines for parsing out SMMUv3 
components defined in ACPI. Since, I am picking these defines directly from 
Linux, I did not want to make this a part of a newly developed patch set.
 
I will update the cc list.

Thanks,
Sameer 


On 2/16/2017 2:57 AM, Jan Beulich wrote:
>>>> On 15.02.17 at 23:34, <sgoel@xxxxxxxxxxxxxx> wrote:
>> From: Lv Zheng <lv.zheng@xxxxxxxxx>
>>
>> ACPICA commit 5de82757aef5d6163e37064033aacbce193abbca
>>
>> This patch adds support for IORT (IO Remapping Table) in iasl.
>>
>> Note that some field names are modified to shrink their length or the
>> decompiled IORT ASL will contain fields with ugly ":" alignment.
>>
>> The IORT contains field definitions around "Memory Access Properties". This
>> patch also adds support to encode/decode it using inline table.
>>
>> This patch doesn't add inline table support for the SMMU interrupt fields
>> due to a limitation in current ACPICA data table support. Lv Zheng.
>>
>> Link: https://github.com/acpica/acpica/commit/5de82757 
>> Signed-off-by: Lv Zheng <lv.zheng@xxxxxxxxx>
>> Signed-off-by: Bob Moore <robert.moore@xxxxxxxxx>
>> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
>> [Linux commit 874f6a723e56d0da9e481629b17482bcd3801ecf]
>> [only port the IORT changes]
>> Signed-off-by: Sameer Goel <sgoel@xxxxxxxxxxxxxx>
> 
> While you've Cc-ed quite a few people, you didn't Cc the Xen
> side maintainers of the code you change. Plus with this not
> being part of a series, it would also help if you clarified what
> the change is going to be needed for, as on its own it'll be an
> addition of dead code.
> 
> Jan
> 
> 

-- 
Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project.

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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.