The patch LGTM
-Original Message-
From: Beignet [mailto:beignet-boun...@lists.freedesktop.org] On Behalf Of
Zhigang Gong
Sent: Wednesday, August 27, 2014 12:13 PM
To: beignet@lists.freedesktop.org
Cc: Gong, Zhigang
Subject: [Beignet] [PATCH 2/2] GBE: optimize unaligned char and short data
That OK
On 二, 2014-09-02 at 10:36 +0800, Zhigang Gong wrote:
> Similar as the bug found by junyan, some events are
> accessed before assigned.
>
> Signed-off-by: Zhigang Gong
> ---
> utests/runtime_event.cpp | 2 +-
> utests/runtime_marker_list.cpp | 2 +-
> 2 files changed, 2 insertions(
The patch LGTM
-Original Message-
From: Beignet [mailto:beignet-boun...@lists.freedesktop.org] On Behalf Of
Zhigang Gong
Sent: Tuesday, September 02, 2014 12:37 PM
To: Gong, Zhigang
Cc: beignet@lists.freedesktop.org
Subject: Re: [Beignet] [PATCH] GBE: fix error in the rootn fastpath funct
Ping for review.
On Fri, Aug 29, 2014 at 10:13:57AM +0800, Zhigang Gong wrote:
> The fastpath is to lose some accuracy but get fast speed. It is not
> to generate error result. The rootn has many special input and need
> to be taken care before we call the native pow directly.
> This patch fix all
On Wed, Aug 27, 2014 at 12:12:44PM +0800, Zhigang Gong wrote:
> The gather the contiguous short/char loads into a single load instruction
> could give us a good pportunity to use untyped load to optimize them.
>
> This patch enable the short/char load gathering at the load store optimize
> pass. T
Ping for review including this 2 patches and the previous 2 patches.
Thanks.
On Thu, Aug 28, 2014 at 10:46:03AM +0800, Zhigang Gong wrote:
> Previous restrication is that the vector size must be multiple
> of DWORD. This restrication prevent the vload2/3 of char or
> vload3 of ushort to be optimiz
Similar as the bug found by junyan, some events are
accessed before assigned.
Signed-off-by: Zhigang Gong
---
utests/runtime_event.cpp | 2 +-
utests/runtime_marker_list.cpp | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/utests/runtime_event.cpp b/utests/runtime_eve
LGTM, pushed, thanks.
On Mon, Sep 01, 2014 at 04:18:45PM +0800, junyan...@inbox.com wrote:
> From: Junyan He
>
> When there are multi printf statements in multi kernel
> fucntions within the same translate unit, if they have
> the same sting parameter, the Clang will just generate
> one global s
LGTM, pushed, thanks.
On Tue, Sep 02, 2014 at 10:37:02AM +0800, junyan...@inbox.com wrote:
> From: Junyan He
>
> Signed-off-by: Junyan He
> ---
> utests/runtime_barrier_list.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/utests/runtime_barrier_list.cpp b/utests/r
LGTM, pushed, thanks.
On Mon, Sep 01, 2014 at 01:05:05PM +0800, Yang Rong wrote:
> 1. Some systems don't define ulong type, use unsigned long instead of..
> 2. Use sA, sB... instead of sa, sb... to access vector 16, because sometimes
> sa, sb will cause clang error.
>
> Signed-off-by: Yang Rong
Hi, David
It may be caused by drm's command parser. Can you disable the command parser
simple and try again?
The command parser disable by following command with root:
echo 0 > /sys/module/i915/parameters/enable_cmd_parser
Thanks,
Yang Rong
> -Original Message-
> From: Zhigang Gong [m
From: Junyan He
Signed-off-by: Junyan He
---
utests/runtime_barrier_list.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/utests/runtime_barrier_list.cpp b/utests/runtime_barrier_list.cpp
index 6987d5e..135996f 100644
--- a/utests/runtime_barrier_list.cpp
+++ b/utests/run
That's better than memset,
It's OK
On 一, 2014-09-01 at 09:27 +0800, Zhigang Gong wrote:
> I just checked the test case. This may not the best fix. The issue should be
> the first time to access all the events which also inclues the uninitialized
> event 3,4,5, And the following patch should be b
It may be another kernel driver issue.
CC to Rong, could you help to check whether this is a kernel related issue.
The kernel version is 3.16.1-301.fc21.x86_64.
Thanks,
Zhigang Gong.
On Mon, Sep 1, 2014 at 8:59 PM, David Liebman wrote:
> I now have the patched version installed on my laptop. I r
I'll check it tonight.
--
-Igor Gnatenko
On Sep 1, 2014 4:59 PM, "David Liebman" wrote:
> I now have the patched version installed on my laptop. I ran the 'sum'
> program and it didn't work. I will include the output of the 'sum' program
> below, but it is the same as in previous emails. One thin
I now have the patched version installed on my laptop. I ran the 'sum'
program and it didn't work. I will include the output of the 'sum'
program below, but it is the same as in previous emails. One thing to
note about the 'sum' program is that it does ask for the user to input
which platform y
From: Junyan He
The bit code linker will load the beignet.bc as a
lib module and link the module of the kernel together.
Then we will filter out all the dead bit code by create
an InternalizePass for the module.
After this stage, the ir will include the bitcode just
used by the cl kernel.
Signed
From: Junyan He
When there are multi printf statements in multi kernel
fucntions within the same translate unit, if they have
the same sting parameter, the Clang will just generate
one global string named .strXXX to represent that string.
So when translating the kernel to gen, we can not unref
th
From: Junyan He
We disable the valid check for the PCH to avoid path
and modified time check, which brings us some trouble.
Signed-off-by: Junyan He
---
backend/src/backend/program.cpp | 22 --
1 file changed, 20 insertions(+), 2 deletions(-)
diff --git a/backend/src/bac
From: Junyan He
The llvm ir print out is modified.
>From the OCL_OUTPUT_LLVM_BEFORE_EXTRA_PASS and
OCL_OUTPUT_LLVM, we change to
OCL_OUTPUT_LLVM_BEFORE_LINK
OCL_OUTPUT_LLVM_AFTER_LINK
OCL_OUTPUT_LLVM_AFTER_GEN
The first one print out the IR before link the bitcode lib.
The second one print out th
20 matches
Mail list logo