[LTP] [PATCH v2 1/2] safe_macros: Add SAFE_READ_ANY_EAGAIN

Petr Vorel pvorel@suse.cz
Tue May 26 15:39:22 CEST 2026


Hi Cyril,

...
> +/* supported values for safe_read() len_strict parameter */
Could we have kerneldoc (ok to fix before merge).

/**
 * enum safe_read_opts - supported values for safe_read() len_strict parameter */
 *
 * @SAFE_READ_ANY: no length strictness, short reads are ok.
 * @SAFE_READ_ALL: strict length, short reads trigger :c:enum:`TBROK <tst_res_flags>`.
 * @SAFE_READ_ANY_EAGAIN: converts EAGAIN to read that returns 0.
 */

Otherwise LGTM, thanks for spotting the error.

Reviewed-by: Petr Vorel <pvorel@suse.cz>

Kind regards,
Petr

> +enum safe_read_opts {
> +	/* no length strictness, short reads are ok */
> +	SAFE_READ_ANY = 0,
> +
> +	/* strict length, short reads trigger TBROK */
> +	SAFE_READ_ALL = 1,
> +
> +	/* converts EAGAIN to read that returns 0 */
> +	SAFE_READ_ANY_EAGAIN = 2,
> +};
...


More information about the ltp mailing list