<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Apr 18, 2014 at 11:08 AM, Zhenyu Wang <span dir="ltr"><<a href="mailto:zhenyuw@linux.intel.com" target="_blank">zhenyuw@linux.intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 2014.04.17 15:06:10 +0800, Chuanbo Weng wrote:<br>
>    {<br>
> -    const GenMessageTarget sfid = GEN6_SFID_DATAPORT_CONSTANT_CACHE;<br>
> +    // FIXME there is a unknown issue with baytrail-t platform, the DWORD scatter<br>
> +    // message causes a hang at unit test case compiler_global_constant.<br>
> +    // We workaround it to use DATA CACHE instead.<br>
> +    const GenMessageTarget sfid = (p->deviceID == PCI_CHIP_BAYTRAIL_T) ?<br>
> +                                 GEN_SFID_DATAPORT_DATA_CACHE : GEN6_SFID_DATAPORT_CONSTANT_CACHE;<br>
<br>
</div>Use IS_BAYTRAIL_T() will be better as there're several VLV devices<br>
I believe you will add later, you may check kernel or other driver.<br>
Consider our devices have more variants in recent generations, using<br>
kind of macro flag for them is always better.<br></blockquote><div> Good suggestion. But I'm not very sure whether all the baytrial variants have this same issue.</div><div> I will choose to leave it as is currently. If I get chance to test it on the baytrail variants I</div>
<div> will change it to use a macro to include all the affected device IDs in one macro. Any comments?</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<span class="HOEnZb"><font color="#888888"><br>
--<br>
Open Source Technology Center, Intel ltd.<br>
<br>
$gpg --keyserver <a href="http://wwwkeys.pgp.net" target="_blank">wwwkeys.pgp.net</a> --recv-keys 4D781827<br>
</font></span><br>_______________________________________________<br>
Beignet mailing list<br>
<a href="mailto:Beignet@lists.freedesktop.org">Beignet@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/beignet" target="_blank">http://lists.freedesktop.org/mailman/listinfo/beignet</a><br>
<br></blockquote></div><br></div></div>