[asterisk-dev] [Code Review] inotify(7) support for pbx_spool

Sean Bright sean.bright at gmail.com
Thu Mar 25 13:50:11 CDT 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/137/#review1767
-----------------------------------------------------------

Ship it!


Looks good

- Sean


On 2010-03-18 16:53:46, Tilghman Lesher wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/137/
> -----------------------------------------------------------
> 
> (Updated 2010-03-18 16:53:46)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> Instead of scanning the directory and trying to keep tabs on the mtime of the directory, an approach which has already been shown to have an issue with directory entry races, use the kernel builtin method of inotify(7) to identify every file as it gets created in the spool directory.
> 
> 
> Diffs
> -----
> 
>   /trunk/pbx/pbx_spool.c 253361 
> 
> Diff: https://reviewboard.asterisk.org/r/137/diff
> 
> 
> Testing
> -------
> 
> Compile-time testing only.
> 
> 
> Thanks,
> 
> Tilghman
> 
>




More information about the asterisk-dev mailing list