On Thu, Sep 25, 2014 at 03:00:53PM +0200, Jacek Danecki wrote:
> On 09/25/14 14:26, Chris Wilson wrote:
> > If we apply the fix in the kernel we break the
> > one user of it in beignet but fix all the existing users of mesa.
>
> Are you talking about fix in kernel which will provide 36 bits GPU ti
On 09/25/14 14:26, Chris Wilson wrote:
> The problem is that beignet already works around the broken hw read
> whereas mesa does not.
There is workaround in mesa already:
static uint64_t
ilo_get_timestamp(struct pipe_screen *screen)
{
struct ilo_screen *is = ilo_screen(screen);
union {
On Mon, Sep 22, 2014 at 06:22:53PM +0200, Jacek Danecki wrote:
> Current implementation of reading GPU timestamp is broken.
> It returns lower 32 bits shifted by 32 bits ( instead of
> ).
> Below change is adding possibility to read hi part of that register
> separ
On 09/23/14 10:37, Daniel Vetter wrote:
> On Mon, Sep 22, 2014 at 06:22:53PM +0200, Jacek Danecki wrote:
>> Current implementation of reading GPU timestamp is broken.
>> It returns lower 32 bits shifted by 32 bits ( instead of
>> ).
>> Below change is adding possibi
On Mon, Sep 22, 2014 at 06:22:53PM +0200, Jacek Danecki wrote:
> Current implementation of reading GPU timestamp is broken.
> It returns lower 32 bits shifted by 32 bits ( instead of
> ).
> Below change is adding possibility to read hi part of that register
> separ
Current implementation of reading GPU timestamp is broken.
It returns lower 32 bits shifted by 32 bits ( instead of
).
Below change is adding possibility to read hi part of that register separately.
Signed-off-by: Jacek Danecki jacek.dane...@intel.com
---
diff --g
