[v1,7/8] common/mlx5: exclude OS dependency in devx commands
Checks
Commit Message
Shared function mlx5_devx_cmd_mkey_create() reads the OS pagesize by
calling a Linux API: 'sysconf(_SC_PAGESIZE)'. Wrap this call with a
shared API 'mlx5_os_get_page_size()' which contains the specific OS
implementation.
Signed-off-by: Ophir Munk <ophirmu@mellanox.com>
---
drivers/common/mlx5/linux/mlx5_common_os.c | 12 ++++++++++++
drivers/common/mlx5/mlx5_common.h | 1 +
drivers/common/mlx5/mlx5_devx_cmds.c | 2 +-
3 files changed, 14 insertions(+), 1 deletion(-)
Comments
10/06/2020 11:32, Ophir Munk:
> Shared function mlx5_devx_cmd_mkey_create() reads the OS pagesize by
> calling a Linux API: 'sysconf(_SC_PAGESIZE)'. Wrap this call with a
> shared API 'mlx5_os_get_page_size()' which contains the specific OS
> implementation.
>
> Signed-off-by: Ophir Munk <ophirmu@mellanox.com>
Sorry, I drop this patch while pulling next-net.
> +/**
> + * Get OS page size
> + *
> + * @return
> + * OS pagesize
> + */
> +size_t
> +mlx5_os_get_page_size(void)
> +{
> + return sysconf(_SC_PAGESIZE);
> +}
The same purpose is achieved with rte_mem_page_size(),
which was added in EAL recently for Windows memory management.
In general, such basic need should not be implemented in a PMD.
+Cc Ferruh for info
19/06/2020 00:47, Thomas Monjalon:
> 10/06/2020 11:32, Ophir Munk:
> > Shared function mlx5_devx_cmd_mkey_create() reads the OS pagesize by
> > calling a Linux API: 'sysconf(_SC_PAGESIZE)'. Wrap this call with a
> > shared API 'mlx5_os_get_page_size()' which contains the specific OS
> > implementation.
> >
> > Signed-off-by: Ophir Munk <ophirmu@mellanox.com>
>
> Sorry, I drop this patch while pulling next-net.
>
> > +/**
> > + * Get OS page size
> > + *
> > + * @return
> > + * OS pagesize
> > + */
> > +size_t
> > +mlx5_os_get_page_size(void)
> > +{
> > + return sysconf(_SC_PAGESIZE);
> > +}
>
> The same purpose is achieved with rte_mem_page_size(),
> which was added in EAL recently for Windows memory management.
>
> In general, such basic need should not be implemented in a PMD.
@@ -125,6 +125,18 @@ mlx5_translate_port_name(const char *port_name_in,
port_info_out->name_type = MLX5_PHYS_PORT_NAME_TYPE_UNKNOWN;
}
+/**
+ * Get OS page size
+ *
+ * @return
+ * OS pagesize
+ */
+size_t
+mlx5_os_get_page_size(void)
+{
+ return sysconf(_SC_PAGESIZE);
+}
+
#ifdef MLX5_GLUE
/**
@@ -213,6 +213,7 @@ __rte_internal
void mlx5_translate_port_name(const char *port_name_in,
struct mlx5_switch_info *port_info_out);
void mlx5_glue_constructor(void);
+size_t mlx5_os_get_page_size(void);
extern uint8_t haswell_broadwell_cpu;
@@ -158,7 +158,7 @@ mlx5_devx_cmd_mkey_create(void *ctx,
return NULL;
}
memset(in, 0, in_size_dw * 4);
- pgsize = sysconf(_SC_PAGESIZE);
+ pgsize = mlx5_os_get_page_size();
MLX5_SET(create_mkey_in, in, opcode, MLX5_CMD_OP_CREATE_MKEY);
mkc = MLX5_ADDR_OF(create_mkey_in, in, memory_key_mkey_entry);
if (klm_num > 0) {