They are defined in mlx5dv with #define and not enums. I will change to cpdef though. -----Original Message----- From: Jason Gunthorpe <jgg@xxxxxxxxxxxx> Sent: Thursday, November 14, 2019 3:40 PM To: Noa Osherovich <noaos@xxxxxxxxxxxx> Cc: dledford@xxxxxxxxxx; Leon Romanovsky <leonro@xxxxxxxxxxxx>; linux-rdma@xxxxxxxxxxxxxxx; Edward Srouji <edwards@xxxxxxxxxxxx>; Daria Velikovsky <daria@xxxxxxxxxxxx> Subject: Re: [PATCH rdma-core 2/4] pyverbs: Introduce ParentDomain class On Thu, Nov 14, 2019 at 09:37:46AM +0000, Noa Osherovich wrote: > diff --git a/pyverbs/providers/mlx5/mlx5dv_enums.pxd b/pyverbs/providers/mlx5/mlx5dv_enums.pxd > index 038a49111a3b..b02da9bf5001 100644 > +++ b/pyverbs/providers/mlx5/mlx5dv_enums.pxd > @@ -45,3 +45,14 @@ cdef extern from 'infiniband/mlx5dv.h': > MLX5DV_FLOW_ACTION_FLAGS_ESP_AES_GCM_SPI_STEERING = 1 << 2 > MLX5DV_FLOW_ACTION_FLAGS_ESP_AES_GCM_FULL_OFFLOAD = 1 << 3 > MLX5DV_FLOW_ACTION_FLAGS_ESP_AES_GCM_TX_IV_IS_ESN = 1 << 4 > + > + cdef unsigned long long MLX5DV_RES_TYPE_QP > + cdef unsigned long long MLX5DV_RES_TYPE_RWQ > + cdef unsigned long long MLX5DV_RES_TYPE_DBR > + cdef unsigned long long MLX5DV_RES_TYPE_SRQ These are supposed to be enums, and should be cpdef Jason
