[asterisk-dev] [Code Review] 3235: config: Add file size and nanosecond resolution fields to the cached modified config file information.

rmudgett reviewboard at asterisk.org
Tue Feb 18 10:55:34 CST 2014


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

(Updated Feb. 18, 2014, 10:55 a.m.)


Review request for Asterisk Developers.


Bugs: AST-1303
    https://issues.asterisk.org/jira/browse/AST-1303


Repository: Asterisk


Description (updated)
-------

Repeatedly modifying config files and reloading too fast sometimes fails to reload the configuration because the cached modification timestamp has one second resolution.

* Added file size and nanosecond resolution fields to the cached config file modification timestamp information.  Now if the file size changes or the file system supports nanosecond resolution the modified file has a better chance of being detected for reload.

* Added a missing unlock in an off-nominal code path.


Diffs
-----

  /branches/1.8/main/config.c 408290 

Diff: https://reviewboard.asterisk.org/r/3235/diff/


Testing
-------

Added a debug statement to the cfmstat_cmp() and checked that the saved stat() information changed when expected.
Did several reloads of app_queue which indicates if it is not reloading queuerules.conf if the file did not change.


Thanks,

rmudgett

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140218/596c2dfe/attachment.html>


More information about the asterisk-dev mailing list