On 13/06/17 12:02, Manish Jaggi wrote:
Will the below code be ok?
If you noticed, I didn't say this code is wrong. Instead I asked why
you use the same ID. Meaning, is there anything in the DSDT requiring
this value?
+ int tras_id = 0;
unsigned.
+ list_for_each_entry(its_data, &host_its_list, entry)
+ {
+ gic_its->translation_id = ++trans_id;
You start the translation ID at 1. Why?
as per the ACPI spec the value should be unique to each GIC ITS unit.
Does starting with 1 break anything? Or should I start with a magic number?
Rather than arguing on the start value here, you should have first
answer to the question regarding the usage of translation_id.
I understand that nobody is using it today. However, when I asked around
me nobody ruled out to any future usage of GIC ITS ID and request this
to be kept as it is.
This means that you can simply copy over the ACPI tables. Rather
regenerating them.
Cheers,
--
Julien Grall
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
