Re: [igt-dev] [PATCH i-g-t] tests/prime_self_import: Swap gtt mapping for cpu

Antonio Argenziano
2019-02-22T16:50:11Z

On 21/02/19 15:37, Chris Wilson wrote:
> Quoting Antonio Argenziano (2019-02-21 22:38:59)
>> The test uses gtt mapping to check two BOs have the same content, since
>> it seems there is no contention on the BOs use cpu mapping to make it
>> compatible with platforms that do not have a mappable aperture.
>>
>> v2:
>> - Need to have an explicit set_domain. (Chris)
>>
>> Signed-off-by: Antonio Argenziano <antonio.argenziano@intel.com>
>
> I don't think has any semantic impact, so no need to verify every write
> to one of the cloned fd is visible in the other, just once is enough.

Keep only one set_domain then?

Any objections to splitting this from the series and merge?

Antonio

>
> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
> -Chris