[LTP] [PATCH] inotify: cleanup - limit masks, use SAFE_ wrappers
Andrea Cervesato
andrea.cervesato@suse.com
Tue Feb 17 14:29:23 CET 2026
On Mon Feb 16, 2026 at 5:50 PM CET, Jinseok Kim wrote:
> Hi Vorel,
>
> Thanks for the review!
>
> - inotify12.c needs read() because EAGAIN (“no second event”) is a
> valid case that must be explicitly detected and treated as IN_IGNORED.
>
> - inotify07.c does not require this distinction: SAFE_READ() is enough,
> since EAGAIN just means end-of-events, while real errors are handled as
> TBROK, matching the original test intent.
>
> Best regards,
> Jinseok.
LGTM
Reviewed-by: Andrea Cervesato <andrea.cervesato@suse.com>
--
Andrea Cervesato
SUSE QE Automation Engineer Linux
andrea.cervesato@suse.com
More information about the ltp
mailing list