fix: review suggestions

Author: jharlow-intel

benchmarks/README.md

@@ -16,5 +16,7 @@ conda activate dpctl_env
 asv run --environment existing:/full/mamba/path/envs/dpctl_env/bin/python
 ```
 
+For `level_zero` devices, you might see `USM Allocation` errors unless you use the `asv run` command with `--launch-method spawn`
+
 ## Writing new benchmarks
 Read ASV's guidelines for writing benchmarks [here](https://asv.readthedocs.io/en/stable/writing_benchmarks.html)

benchmarks/benchmarks/benchmark_utils.py

@@ -1,14 +1,14 @@
 from asv_runner.benchmarks.mark import SkipNotImplemented
+import dpctl.tensor as dpt
 
-
-def skip_unsupported_datatype(queue, dtype):
+def skip_unsupported_dtype(q, dtype):
     """
     Skip the benchmark if the device does not support the given data type.
     """
     if (
-        (dtype.name == "float64" or dtype.name == "complex128")
-        and not queue.sycl_device.has_aspect_fp64
-    ) or (dtype.name == "float16" and not queue.sycl_device.has_aspect_fp16):
+        (dtype == dpt.float64 or dtype.name == dpt.complex128)
+        and not q.sycl_device.has_aspect_fp64
+    ) or (dtype == dpt.float16 and not q.sycl_device.has_aspect_fp16):
         raise SkipNotImplemented(
             f"Skipping benchmark for {dtype.name} on this device"
             + " as it is not supported."

benchmarks/benchmarks/binary.py

@@ -114,7 +114,7 @@ def generate_benchmark_functions():
             method_name = f"time_{fn_name}_{dtype1.name}_{dtype2.name}"
 
             def benchmark_method(self, fn=fn, dtype1=dtype1, dtype2=dtype2):
-                bench_utils.skip_unsupported_datatype(self.q, dtype1)
+                bench_utils.skip_unsupported_dtype(self.q, dtype1)
                 return self.run_bench(
                     self.q,
                     self.iterations,