[LTP] [PATCH 2/2] inotify: Add SAFE_MYINOTIFY_ADD_WATCH() helper
Petr Vorel
pvorel@suse.cz
Wed Jun 5 17:02:48 CEST 2019
Hi Li,
> You are right. And seems the problem is only exist in original code, it put
> reap_wd in wrong place and mislead my sight.
> if ((wd = myinotify_add_watch(fd_notify, DIR_PATH, IN_ALL_EVENTS)) <
> 0) {
> tst_brk(TBROK | TERRNO,
> "inotify_add_watch (%d, " DIR_PATH ", IN_ALL_EVENTS)
> failed",
> fd_notify);
> reap_wd = 1;
> };
Thanks for pointing it out. I was surprised why this is there.
I'll note it in git commit.
...
> You patch set looks good.
> Sorry for the error in judgment, that remind me it's time to go to bed
> now:).
Really, thanks a lot for a review! (I'll add your ack).
Kind regards,
Petr
More information about the ltp
mailing list