Re: [Beignet] [PATCH V2] GBE: optimize phi elimination.

2015-07-14 Thread Yang, Rong R
LGTM, will push it later, thanks. -Original Message- From: Beignet [mailto:beignet-boun...@lists.freedesktop.org] On Behalf Of Ruiling Song Sent: Monday, July 6, 2015 14:40 To: beignet@lists.freedesktop.org Cc: Song, Ruiling Subject: [Beignet] [PATCH V2] GBE: optimize phi

[Beignet] [PATCH] Reorder GBE_BIN_GENERATER arguments.

2015-07-14 Thread Koop Mast
Basically, it boils down to a difference in getopt(3). The getopt(3) on (Free)BSD will exit parsing arguments at the first unknown argument. Signed-off-by: Koop Mast k...@rainbow-runner.nl --- src/CMakeLists.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [Beignet] [PATCH] libocl: Add macro define for fp16.

2015-07-14 Thread Yang, Rong R
LGTM, will push it later, thanks. -Original Message- From: Beignet [mailto:beignet-boun...@lists.freedesktop.org] On Behalf Of junyan...@inbox.com Sent: Monday, July 13, 2015 16:55 To: beignet@lists.freedesktop.org Cc: Junyan He Subject: [Beignet] [PATCH] libocl: Add macro define

Re: [Beignet] [PATCH] fix utest bug.

2015-07-14 Thread Yang, Rong R
LGTM, will push it later, thanks. -Original Message- From: Beignet [mailto:beignet-boun...@lists.freedesktop.org] On Behalf Of xionghu@intel.com Sent: Tuesday, July 14, 2015 02:52 To: beignet@lists.freedesktop.org Cc: Luo, Xionghu Subject: [Beignet] [PATCH] fix utest bug.

Re: [Beignet] [PATCH] Fix comparison in if() check.

2015-07-14 Thread Yang, Rong R
LGTM, will push it later, thanks. -Original Message- From: Beignet [mailto:beignet-boun...@lists.freedesktop.org] On Behalf Of Koop Mast Sent: Monday, July 13, 2015 03:50 To: beignet@lists.freedesktop.org Cc: Koop Mast Subject: [Beignet] [PATCH] Fix comparison in if() check.

Re: [Beignet] [PATCH] Runtime: Add default extension for platforms before BDW.

2015-07-14 Thread Yang, Rong R
Memcpy is not safe in cl_intel_platform_get_default_extension, but the device extension array size is same as platform's extension array size, will not overflow (has '\0'). Maybe need a MACRO to define these arrays' size. Will push it later. -Original Message- From: Beignet

Re: [Beignet] [PATCH] runtime: Need to separate atomic in L3 test and SLM test in self_test().

2015-07-14 Thread Luo, Xionghu
LGTM, Thanks. Luo Xionghu Best Regards -Original Message- From: Beignet [mailto:beignet-boun...@lists.freedesktop.org] On Behalf Of Zhigang Gong Sent: Wednesday, July 15, 2015 9:49 AM To: beignet@lists.freedesktop.org Cc: Gong, Zhigang Subject: [Beignet] [PATCH] runtime: Need to

Re: [Beignet] NUC5i7RYX Intel Iris Graphics 6100 support

2015-07-14 Thread Yang, Rong R
Yes, you are right, the device 162b is BDW and missing. The patch is ok. In fact, this series device ids all are all missing, include 0x160B(GT1), 0x161B(GT2), 0x162B(GT3), could you send to a new version patch to include them? The Failed test, test_load_program_from_spir has been fixed. The

Re: [Beignet] [PATCH 1/2] Fixed a thread safe bug.

2015-07-14 Thread Zhigang Gong
Please ignore this version as I forgot to remove the debug message. Just sent out the version 2. Thanks, Zhigang Gong. On Wed, Jul 15, 2015 at 08:54:33AM +0800, Zhigang Gong wrote: From: Zhigang Gong zhigang.g...@linux.intel.com last_event and current_event should be thread private data.

[Beignet] [Patch v2 1/2] Fixed a thread safe bug.

2015-07-14 Thread Zhigang Gong
From: Zhigang Gong zhigang.g...@linux.intel.com last_event and current_event should be thread private data. Signed-off-by: Zhigang Gong zhigang.g...@linux.intel.com --- src/cl_api.c | 2 +- src/cl_command_queue.c | 17 +++-- src/cl_command_queue.h | 2 -- src/cl_event.c