<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Patch Set 1:<br>> > I've confirmed that this does seem to break CDR and CEL on my<br>own<br> > system too. 16 branch is fine, but with this change it is not.<br>> Could this be related to the -fpartial-inlining issue? <br>https://gerrit.asterisk.org/10997 was not reverted in master.</pre></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">That I do not know, but from my quick testing this morning it only<br>occurs with this patch included.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Hi Joshua,</p><p style="white-space: pre-wrap; word-wrap: break-word;">I've checked each of previous commits to the Asterisk-16. Therefore, I found after below commit, all the cdr test was actually failed. I didn't check that why yet, but the thing is, it was wrong since long time ago not only for this time.</p><p style="white-space: pre-wrap; word-wrap: break-word;">c2e6efa63d1fce268455ea0552c9725677f20e0b</p><p style="white-space: pre-wrap; word-wrap: break-word;">https://gerrit.asterisk.org/#/c/asterisk/+/10385/</p><p style="white-space: pre-wrap; word-wrap: break-word;">For example, this test looks ok in jenkins, <br>https://jenkins2.asterisk.org/job/asterisk-unittests/job/16/623/consoleFull</p><p style="white-space: pre-wrap; word-wrap: break-word;">But in the manual test, it returns fail.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Asterisk Ready.<br>*CLI> core show version<br>Asterisk GIT-16-2f8def1453 built by pchero @ debian on a x86_64 running Linux on 2019-02-20 23:28:47 UTC<br>*CLI> test execute category /main/cdr/ name test_cdr_fork_cdr<br>Running all available tests matching category /main/cdr/ and name test_cdr_fork_cdr</p><p style="white-space: pre-wrap; word-wrap: break-word;">START /main/cdr/ - test_cdr_fork_cdr <br>[Feb 21 02:15:45] NOTICE[90910]: cdr.c:4501 cdr_toggle_runtime_options: CDR simple logging enabled.<br>0x7f6d34000f30 - Created CDR for channel CDRTestChannel/Alice<br>0x7f6d34000f30 - Transitioning CDR for CDRTestChannel/Alice from state NONE to Single<br>0x55dfd5ae2f90 - Created CDR for channel CDRTestChannel/Alice<br>0x55dfd5ae2f90 - Transitioning CDR for CDRTestChannel/Alice from state NONE to Single<br>0x55dfd5ae6bb0 - Created CDR for channel CDRTestChannel/Alice<br>0x55dfd5ae6bb0 - Transitioning CDR for CDRTestChannel/Alice from state NONE to Single<br>0x7f6d34000f30 - Transitioning CDR for CDRTestChannel/Alice from state Single to Finalized<br>0x55dfd5ae2f90 - Transitioning CDR for CDRTestChannel/Alice from state Single to Finalized<br>0x55dfd5ae6bb0 - Set answered time to 1550711746.223075<br>Verifying expected record CDRTestChannel/Alice, <none><br>Finished expected record CDRTestChannel/Alice, <none><br>0x55dfd5ae7d10 - Created CDR for channel CDRTestChannel/Alice<br>0x55dfd5ae7d10 - Transitioning CDR for CDRTestChannel/Alice from state NONE to Single<br>0x55dfd5ae7d10 - Set answered time to 1550711747.220814<br>0x55dfd5ae6bb0 - Transitioning CDR for CDRTestChannel/Alice from state Single to Finalized<br>0x55dfd5ae7900 - Created CDR for channel CDRTestChannel/Alice<br>0x55dfd5ae7900 - Transitioning CDR for CDRTestChannel/Alice from state NONE to Single<br>0x55dfd5ae7900 - Set answered time to 1550711747.220864<br>0x55dfd5ae7d10 - Transitioning CDR for CDRTestChannel/Alice from state Single to Finalized<br>0x55dfd5ae7900 - Transitioning CDR for CDRTestChannel/Alice from state Single to Finalized<br>0x7f6d34000f30 - Beginning finalize/dispatch for CDRTestChannel/Alice<br>0x7f6d34000f30 - Dispatching CDR for Party A CDRTestChannel/Alice, Party B <none><br>[test_cdr.c:verify_mock_cdr_record:328]: Unable to find actual CDR record at 1<br>END /main/cdr/ - test_cdr_fork_cdr Time: 6999ms Result: FAIL</p><p style="white-space: pre-wrap; word-wrap: break-word;">1 Test(s) Executed 0 Passed 1 Failed</p><p><a href="https://gerrit.asterisk.org/11013">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/11013">change 11013</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/11013"/><meta itemprop="name" content="View Change"/></div></div>
<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 16 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ie86d125d2966f93de74ee00f47ae6fbc8c081c5f </div>
<div style="display:none"> Gerrit-Change-Number: 11013 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation (1000185) </div>
<div style="display:none"> Gerrit-Reviewer: Joshua C. Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 21 Feb 2019 01:17:03 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>