search

lukeiwanski / tensorflow

OpenCL support for TensorFlow via SYCL
Apache License 2.0
65 stars 14 forks source link

FAIL: //tensorflow/python:session_list_devices_test #82

Closed lukeiwanski closed 7 years ago

lukeiwanski commented 7 years ago

System Info

  Name:                      Hawaii
  Vendor:                    Advanced Micro Devices, Inc.
  Device OpenCL C version:           OpenCL C 2.0 
  Driver version:                1912.5 (VM)
  Profile:                   FULL_PROFILE
  Version:                   OpenCL 2.0 AMD-APP (1912.5)
  Extensions:                    cl_khr_fp64 cl_amd_fp64 cl_khr_global_int32_base_atomics cl_khr_global_int32_extended_atomics cl_khr_local_int32_base_atomics cl_khr_local_int32_extended_atomics cl_khr_int64_base_atomics cl_khr_int64_extended_atomics cl_khr_3d_image_writes cl_khr_byte_addressable_store cl_khr_gl_sharing cl_khr_gl_depth_images cl_ext_atomic_counters_32 cl_amd_device_attribute_query cl_amd_vec3 cl_amd_printf cl_amd_media_ops cl_amd_media_ops2 cl_amd_popcnt cl_khr_image2d_from_buffer cl_khr_spir cl_khr_subgroups cl_khr_gl_event cl_khr_depth_images cl_khr_mipmap_image cl_khr_mipmap_image_writes

ComputeCpp 0.2.0

To reproduce

bazel test --config=sycl --local_test_jobs=4 -k --test_lang_filters=cc,py --action_env=LD_PRELOAD=/usr/lib/libOpenCL.so.1 --test_timeout 300,750,1200,3600 //tensorflow/python:session_list_devices_test

Error

FAIL: testListDevices (__main__.SessionListDevicesTest)
...
    self.assertGreaterEqual(1, len(devices), devices)
AssertionError: [_DeviceAttributes(/job:localhost/replica:0/task:0/device:CPU:0, CPU, 50338976), _DeviceAttributes(/job:localhost/replica:0/task:0/device:SYCL:0, SYCL, 54437104)]
jeffmak commented 7 years ago

@lukeiwanski Is list_devices supposed to return both SYCL and CPU devices or just CPU devices when running a default session?

jwlawson commented 7 years ago

This fails on CUDA with the same errors, and is tagged no_gpu in the BUILD file. It looks like we should not be running this test on SYCL and that it only should be run on the CPU.

    self.assertGreaterEqual(1, len(devices), devices)
AssertionError: [_DeviceAttributes(/job:local/replica:0/task:0/device:CPU:0, CPU, 43685504), _DeviceAttributes(/job:local/replica:0/task:0/device:GPU:0, GPU, 43686432)]
lukeiwanski commented 7 years ago

Fair point. Closing