----- 原始邮件 ----- > 发件人: "Zorro Lang" <[email protected]> > 收件人: [email protected] > 抄送: [email protected], [email protected], "Zorro Lang" <[email protected]> > 发送时间: 星期一, 2015年 8 月 17日 下午 9:49:53 > 主题: [PATCH v3] generic/084: use src/multi_open_unlink to replace tail command > > generic/084 try to run 'tail' command, tail will use inotify. > There're some limit about the number of inotify. For example > fs.inotify.max_user_instances specifies an upper limit on > the number of inotify instances that can be created per real > user ID. > > When I test on a machine with 154 cpu cores, this case run > failed, and hit many warning likes: > > +tail: inotify cannot be used, reverting to polling: Too many open files > > Because the fs.inotify.max_user_instances is 128, so if we > try to tail 154 files, it will be failed. > > So use src/multi_open_unlink to instead of tail will avoid > this problem. > > Signed-off-by: Zorro Lang <[email protected]> > Reviewed-by: Eryu Guan <[email protected]> > Reviewed-by: Dave Chinner <[email protected]>
Hi, Eryu told me I shouldn't add "Reviewed-by" at here. Sorry for my mistake, I just thought I should add everyone who helped me. Please help to review again. Thanks very much. Zorro -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
