Re: [PATCH v5 03/10] lib/igt_drm_netlink: Introduce DRM RAS Netlink interface library

["Purkait, Soham" <soham.purkait@intel.com>]

Hi Ravi,

On 14-10-2026 00:36, Ravi Kishore Koppuravuri wrote:
> Introduce an IGT library for communicating with DRM RAS Generic Netlink
> interface. Add helpers for Generic Netlink socket initialization and
> cleanup.
>
> Cc: Soham Purkait <soham.purkait@intel.com>
> Cc: Riana Tauro <riana.tauro@intel.com>
> Cc: Anshuman Gupta <anshuman.gupta@intel.com>
> Cc: Raag Jadav <raag.jadav@intel.com>
> Cc: Badal Nilawar <badal.nilawar@intel.com>
> Cc: Mallesh Koujalagi <mallesh.koujalagi@intel.com>
> Cc: Harish Chegondi <harish.chegondi@intel.com>
> Signed-off-by: Ravi Kishore Koppuravuri <ravi.kishore.koppuravuri@intel.com>
> ---
> v5:Renamed library function names prefixed with igt_drm_ras_nl_*
>     (Riana and Soham)
> v4:Renamed library function names prefixed with igt_drm_netlink_*
>     (Nikula Jani and Kamil)
>     Moved the headers to other commits which are not necessary in this
>     commit (Nikula Jani)
> ---
>   lib/igt_drm_ras_nl.c | 50 ++++++++++++++++++++++++++++++++++++++++++++
>   lib/igt_drm_ras_nl.h | 20 ++++++++++++++++++
>   lib/meson.build      |  1 +
>   3 files changed, 71 insertions(+)
>   create mode 100644 lib/igt_drm_ras_nl.c
>   create mode 100644 lib/igt_drm_ras_nl.h
>
> diff --git a/lib/igt_drm_ras_nl.c b/lib/igt_drm_ras_nl.c
> new file mode 100644
> index 000000000..2822361d3
> --- /dev/null
> +++ b/lib/igt_drm_ras_nl.c
> @@ -0,0 +1,50 @@
> +// SPDX-License-Identifier: MIT
> +/*
> + * Copyright © 2026 Intel Corporation
> + */
> +
> +#include <stdint.h>
> +
> +#include <netlink/genl/ctrl.h>
> +#include <netlink/genl/genl.h>
> +
> +#include "igt_core.h"
> +#include "igt_drm_ras_nl.h"
> +
> +void igt_drm_ras_nl_cleanup(struct igt_drm_ras_nl_context *ctx)
> +{
> +	if (!ctx || !ctx->sock)
> +		return;
> +
> +	nl_close(ctx->sock);
> +	nl_socket_free(ctx->sock);
> +	ctx->sock = NULL;
> +	ctx->family_id = -1;
> +
> +	igt_debug("Cleaned up netlink socket.\n");
> +}
> +
> +int igt_drm_ras_nl_init(struct igt_drm_ras_nl_context *ctx)
> +{
> +	ctx->sock = nl_socket_alloc();
> +	if (!ctx->sock)
> +		return -1;
> +
> +	if (genl_connect(ctx->sock) < 0) {
> +		igt_drm_ras_nl_cleanup(ctx);
> +		return -1;
> +	}
> +
> +	ctx->family_id = genl_ctrl_resolve(ctx->sock, DRM_RAS_FAMILY_NAME);
> +	if (ctx->family_id < 0) {
> +		fprintf(stderr,
> +			"Failed to resolve Netlink family '%s': %s\n",
> +			DRM_RAS_FAMILY_NAME, nl_geterror(ctx->family_id));
> +		igt_drm_ras_nl_cleanup(ctx);
> +		return -1;
> +	}
> +
> +	igt_debug("Resolved Generic Netlink family '%s' with id %d.\n",
> +		  DRM_RAS_FAMILY_NAME, ctx->family_id);
> +	return 0;
> +}
> diff --git a/lib/igt_drm_ras_nl.h b/lib/igt_drm_ras_nl.h
> new file mode 100644
> index 000000000..011f3bc05
> --- /dev/null
> +++ b/lib/igt_drm_ras_nl.h
> @@ -0,0 +1,20 @@
> +/* SPDX-License-Identifier: MIT */
> +/*
> + * Copyright © 2026 Intel Corporation
> + */
> +
> +#ifndef IGT_DRM_RAS_NL_H
> +#define IGT_DRM_RAS_NL_H
> +
> +#include <drm-uapi/drm_ras.h>
> +
> +struct igt_drm_ras_nl_context {
> +	struct nl_sock *sock;
> +	int family_id;
> +};
> +
> +void igt_drm_ras_nl_cleanup(struct igt_drm_ras_nl_context *ctx);
> +int igt_drm_ras_nl_init(struct igt_drm_ras_nl_context *ctx);
> +
> +#endif /* IGT_DRM_RAS_NL_H */
> +
> diff --git a/lib/meson.build b/lib/meson.build
> index df3046f46..c4e03c831 100644
> --- a/lib/meson.build
> +++ b/lib/meson.build
> @@ -27,6 +27,7 @@ lib_sources = [
>   	'igt_device_scan.c',
>   	'igt_drm_clients.h',
>   	'igt_drm_fdinfo.c',
> +	'igt_drm_ras_nl.c',
>           'igt_fs.c',
>   	'igt_aux.c',
>   	'igt_dp.c',
LGTM
Reviewed-by: Soham Purkait <soham.purkait@intel.com>

Regards,
Soham