[PATCH v2 1/1] drm/amdkfd: Add sysfs bitfields and enums to uAPI

Kim, Jonathan Jonathan.Kim at amd.com
Thu Nov 4 16:19:01 UTC 2021


[AMD Official Use Only]

> -----Original Message-----
> From: amd-gfx <amd-gfx-bounces at lists.freedesktop.org> On Behalf Of Felix
> Kuehling
> Sent: September 13, 2021 5:23 PM
> To: amd-gfx at lists.freedesktop.org
> Subject: [PATCH v2 1/1] drm/amdkfd: Add sysfs bitfields and enums to
> uAPI
>
> [CAUTION: External Email]
>
> These bits are de-facto part of the uAPI, so declare them in a uAPI header.
>
> The corresponding bit-fields and enums in user mode are defined in
> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit
> hub.com%2FRadeonOpenCompute%2FROCT-Thunk-
> Interface%2Fblob%2Fmaster%2Finclude%2Fhsakmttypes.h&data=04%
> 7C01%7Cjonathan.kim%40amd.com%7C60c91f7b30794bf670c808d976fcc
> 000%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637671650
> 194006492%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQI
> joiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=WQU
> JKcixWV0gxBYSxUzZx05nAtyYjoRNz7oJO%2BnjsLA%3D&reserved=0
>
> HSA_CAP_...           -> HSA_CAPABILITY
> HSA_MEM_HEAP_TYPE_... -> HSA_HEAPTYPE
> HSA_MEM_FLAGS_...     -> HSA_MEMORYPROPERTY
> HSA_CACHE_TYPE_...    -> HsaCacheType
> HSA_IOLINK_TYPE_...   -> HSA_IOLINKTYPE
> HSA_IOLINK_FLAGS_...  -> HSA_LINKPROPERTY
>
> Signed-off-by: Felix Kuehling <Felix.Kuehling at amd.com>

Reviewed-by: Jonathan Kim <jonathan.kim at amd.com>

> ---
>  MAINTAINERS                               |   1 +
>  drivers/gpu/drm/amd/amdkfd/kfd_topology.h |  46 +--------
>  include/uapi/linux/kfd_sysfs.h            | 108 ++++++++++++++++++++++
>  3 files changed, 110 insertions(+), 45 deletions(-)  create mode 100644
> include/uapi/linux/kfd_sysfs.h
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 84cd16694640..7554ec928ee2 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -930,6 +930,7 @@ F:
> drivers/gpu/drm/amd/include/kgd_kfd_interface.h
>  F:     drivers/gpu/drm/amd/include/v9_structs.h
>  F:     drivers/gpu/drm/amd/include/vi_structs.h
>  F:     include/uapi/linux/kfd_ioctl.h
> +F:     include/uapi/linux/kfd_sysfs.h
>
>  AMD SPI DRIVER
>  M:     Sanjay R Mehta <sanju.mehta at amd.com>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_topology.h
> b/drivers/gpu/drm/amd/amdkfd/kfd_topology.h
> index a8db017c9b8e..f0cc59d2fd5d 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_topology.h
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_topology.h
> @@ -25,38 +25,11 @@
>
>  #include <linux/types.h>
>  #include <linux/list.h>
> +#include <linux/kfd_sysfs.h>
>  #include "kfd_crat.h"
>
>  #define KFD_TOPOLOGY_PUBLIC_NAME_SIZE 32
>
> -#define HSA_CAP_HOT_PLUGGABLE                  0x00000001
> -#define HSA_CAP_ATS_PRESENT                    0x00000002
> -#define HSA_CAP_SHARED_WITH_GRAPHICS           0x00000004
> -#define HSA_CAP_QUEUE_SIZE_POW2                        0x00000008
> -#define HSA_CAP_QUEUE_SIZE_32BIT               0x00000010
> -#define HSA_CAP_QUEUE_IDLE_EVENT               0x00000020
> -#define HSA_CAP_VA_LIMIT                       0x00000040
> -#define HSA_CAP_WATCH_POINTS_SUPPORTED         0x00000080
> -#define HSA_CAP_WATCH_POINTS_TOTALBITS_MASK    0x00000f00
> -#define HSA_CAP_WATCH_POINTS_TOTALBITS_SHIFT   8
> -#define HSA_CAP_DOORBELL_TYPE_TOTALBITS_MASK   0x00003000
> -#define HSA_CAP_DOORBELL_TYPE_TOTALBITS_SHIFT  12
> -
> -#define HSA_CAP_DOORBELL_TYPE_PRE_1_0          0x0
> -#define HSA_CAP_DOORBELL_TYPE_1_0              0x1
> -#define HSA_CAP_DOORBELL_TYPE_2_0              0x2
> -#define HSA_CAP_AQL_QUEUE_DOUBLE_MAP           0x00004000
> -
> -#define HSA_CAP_RESERVED_WAS_SRAM_EDCSUPPORTED 0x00080000
> /* Old buggy user mode depends on this being 0 */
> -#define HSA_CAP_MEM_EDCSUPPORTED               0x00100000
> -#define HSA_CAP_RASEVENTNOTIFY                 0x00200000
> -#define HSA_CAP_ASIC_REVISION_MASK             0x03c00000
> -#define HSA_CAP_ASIC_REVISION_SHIFT            22
> -#define HSA_CAP_SRAM_EDCSUPPORTED              0x04000000
> -#define HSA_CAP_SVMAPI_SUPPORTED               0x08000000
> -#define HSA_CAP_FLAGS_COHERENTHOSTACCESS       0x10000000
> -#define HSA_CAP_RESERVED                       0xe00f8000
> -
>  struct kfd_node_properties {
>         uint64_t hive_id;
>         uint32_t cpu_cores_count;
> @@ -93,17 +66,6 @@ struct kfd_node_properties {
>         char name[KFD_TOPOLOGY_PUBLIC_NAME_SIZE];
>  };
>
> -#define HSA_MEM_HEAP_TYPE_SYSTEM       0
> -#define HSA_MEM_HEAP_TYPE_FB_PUBLIC    1
> -#define HSA_MEM_HEAP_TYPE_FB_PRIVATE   2
> -#define HSA_MEM_HEAP_TYPE_GPU_GDS      3
> -#define HSA_MEM_HEAP_TYPE_GPU_LDS      4
> -#define HSA_MEM_HEAP_TYPE_GPU_SCRATCH  5
> -
> -#define HSA_MEM_FLAGS_HOT_PLUGGABLE            0x00000001
> -#define HSA_MEM_FLAGS_NON_VOLATILE             0x00000002
> -#define HSA_MEM_FLAGS_RESERVED                 0xfffffffc
> -
>  struct kfd_mem_properties {
>         struct list_head        list;
>         uint32_t                heap_type;
> @@ -116,12 +78,6 @@ struct kfd_mem_properties {
>         struct attribute        attr;
>  };
>
> -#define HSA_CACHE_TYPE_DATA            0x00000001
> -#define HSA_CACHE_TYPE_INSTRUCTION     0x00000002
> -#define HSA_CACHE_TYPE_CPU             0x00000004
> -#define HSA_CACHE_TYPE_HSACU           0x00000008
> -#define HSA_CACHE_TYPE_RESERVED                0xfffffff0
> -
>  struct kfd_cache_properties {
>         struct list_head        list;
>         uint32_t                processor_id_low;
> diff --git a/include/uapi/linux/kfd_sysfs.h b/include/uapi/linux/kfd_sysfs.h
> new file mode 100644 index 000000000000..e1fb78b4bf09
> --- /dev/null
> +++ b/include/uapi/linux/kfd_sysfs.h
> @@ -0,0 +1,108 @@
> +/* SPDX-License-Identifier: GPL-2.0 OR MIT WITH Linux-syscall-note */
> +/*
> + * Copyright 2021 Advanced Micro Devices, Inc.
> + *
> + * Permission is hereby granted, free of charge, to any person
> +obtaining a
> + * copy of this software and associated documentation files (the
> +"Software"),
> + * to deal in the Software without restriction, including without
> +limitation
> + * the rights to use, copy, modify, merge, publish, distribute,
> +sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom
> +the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice shall be
> +included in
> + * all copies or substantial portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY
> KIND,
> +EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> +MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO
> EVENT
> +SHALL
> + * THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM,
> +DAMAGES OR
> + * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR
> +OTHERWISE,
> + * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR
> THE USE
> +OR
> + * OTHER DEALINGS IN THE SOFTWARE.
> + */
> +
> +#ifndef KFD_SYSFS_H_INCLUDED
> +#define KFD_SYSFS_H_INCLUDED
> +
> +/* Capability bits in node properties */
> +#define HSA_CAP_HOT_PLUGGABLE                  0x00000001
> +#define HSA_CAP_ATS_PRESENT                    0x00000002
> +#define HSA_CAP_SHARED_WITH_GRAPHICS           0x00000004
> +#define HSA_CAP_QUEUE_SIZE_POW2                        0x00000008
> +#define HSA_CAP_QUEUE_SIZE_32BIT               0x00000010
> +#define HSA_CAP_QUEUE_IDLE_EVENT               0x00000020
> +#define HSA_CAP_VA_LIMIT                       0x00000040
> +#define HSA_CAP_WATCH_POINTS_SUPPORTED         0x00000080
> +#define HSA_CAP_WATCH_POINTS_TOTALBITS_MASK    0x00000f00
> +#define HSA_CAP_WATCH_POINTS_TOTALBITS_SHIFT   8
> +#define HSA_CAP_DOORBELL_TYPE_TOTALBITS_MASK   0x00003000
> +#define HSA_CAP_DOORBELL_TYPE_TOTALBITS_SHIFT  12
> +
> +#define HSA_CAP_DOORBELL_TYPE_PRE_1_0          0x0
> +#define HSA_CAP_DOORBELL_TYPE_1_0              0x1
> +#define HSA_CAP_DOORBELL_TYPE_2_0              0x2
> +#define HSA_CAP_AQL_QUEUE_DOUBLE_MAP           0x00004000
> +
> +/* Old buggy user mode depends on this being 0 */ #define
> +HSA_CAP_RESERVED_WAS_SRAM_EDCSUPPORTED 0x00080000
> +
> +#define HSA_CAP_MEM_EDCSUPPORTED               0x00100000
> +#define HSA_CAP_RASEVENTNOTIFY                 0x00200000
> +#define HSA_CAP_ASIC_REVISION_MASK             0x03c00000
> +#define HSA_CAP_ASIC_REVISION_SHIFT            22
> +#define HSA_CAP_SRAM_EDCSUPPORTED              0x04000000
> +#define HSA_CAP_SVMAPI_SUPPORTED               0x08000000
> +#define HSA_CAP_FLAGS_COHERENTHOSTACCESS       0x10000000
> +#define HSA_CAP_RESERVED                       0xe00f8000
> +
> +/* Heap types in memory properties */
> +#define HSA_MEM_HEAP_TYPE_SYSTEM       0
> +#define HSA_MEM_HEAP_TYPE_FB_PUBLIC    1
> +#define HSA_MEM_HEAP_TYPE_FB_PRIVATE   2
> +#define HSA_MEM_HEAP_TYPE_GPU_GDS      3
> +#define HSA_MEM_HEAP_TYPE_GPU_LDS      4
> +#define HSA_MEM_HEAP_TYPE_GPU_SCRATCH  5
> +
> +/* Flag bits in memory properties */
> +#define HSA_MEM_FLAGS_HOT_PLUGGABLE            0x00000001
> +#define HSA_MEM_FLAGS_NON_VOLATILE             0x00000002
> +#define HSA_MEM_FLAGS_RESERVED                 0xfffffffc
> +
> +/* Cache types in cache properties */
> +#define HSA_CACHE_TYPE_DATA            0x00000001
> +#define HSA_CACHE_TYPE_INSTRUCTION     0x00000002
> +#define HSA_CACHE_TYPE_CPU             0x00000004
> +#define HSA_CACHE_TYPE_HSACU           0x00000008
> +#define HSA_CACHE_TYPE_RESERVED                0xfffffff0
> +
> +/* Link types in IO link properties (matches CRAT link types) */
> +#define HSA_IOLINK_TYPE_UNDEFINED      0
> +#define HSA_IOLINK_TYPE_HYPERTRANSPORT 1
> +#define HSA_IOLINK_TYPE_PCIEXPRESS     2
> +#define HSA_IOLINK_TYPE_AMBA           3
> +#define HSA_IOLINK_TYPE_MIPI           4
> +#define HSA_IOLINK_TYPE_QPI_1_1        5
> +#define HSA_IOLINK_TYPE_RESERVED1      6
> +#define HSA_IOLINK_TYPE_RESERVED2      7
> +#define HSA_IOLINK_TYPE_RAPID_IO       8
> +#define HSA_IOLINK_TYPE_INFINIBAND     9
> +#define HSA_IOLINK_TYPE_RESERVED3      10
> +#define HSA_IOLINK_TYPE_XGMI           11
> +#define HSA_IOLINK_TYPE_XGOP           12
> +#define HSA_IOLINK_TYPE_GZ             13
> +#define HSA_IOLINK_TYPE_ETHERNET_RDMA  14
> +#define HSA_IOLINK_TYPE_RDMA_OTHER     15
> +#define HSA_IOLINK_TYPE_OTHER          16
> +
> +/* Flag bits in IO link properties (matches CRAT flags, excluding the
> + * bi-directional flag, which is not offially part of the CRAT spec,
> +and
> + * only used internally in KFD)
> + */
> +#define HSA_IOLINK_FLAGS_ENABLED               (1 << 0)
> +#define HSA_IOLINK_FLAGS_NON_COHERENT          (1 << 1)
> +#define HSA_IOLINK_FLAGS_NO_ATOMICS_32_BIT     (1 << 2)
> +#define HSA_IOLINK_FLAGS_NO_ATOMICS_64_BIT     (1 << 3)
> +#define HSA_IOLINK_FLAGS_NO_PEER_TO_PEER_DMA   (1 << 4)
> +#define HSA_IOLINK_FLAGS_RESERVED              0xffffffe0
> +
> +#endif
> --
> 2.32.0



More information about the amd-gfx mailing list