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

Cyril Hrubis chrubis@suse.cz
Tue May 26 16:02:19 CEST 2026


Hi!
> ...
> > +/* 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.

I would avoid that for now, since we are likely going to change the API
after the release. Is that okay?

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list