Re: [PATCH v3] generic/084: use src/multi_open_unlink to replace tail command

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]




----- 原始邮件 -----
> 发件人: "Zorro Lang" <zlang@xxxxxxxxxx>
> 收件人: fstests@xxxxxxxxxxxxxxx
> 抄送: eguan@xxxxxxxxxx, david@xxxxxxxxxxxxx, "Zorro Lang" <zlang@xxxxxxxxxx>
> 发送时间: 星期一, 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 <zlang@xxxxxxxxxx>
> Reviewed-by: Eryu Guan <eguan@xxxxxxxxxx>
> Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>

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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux