<html dir="ltr">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
<style>
<!--
@font-face
        {font-family:SimSun}
@font-face
        {font-family:"Cambria Math"}
@font-face
        {font-family:Calibri}
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif}
a:link, span.MsoHyperlink
        {color:#0563C1;
        text-decoration:underline}
a:visited, span.MsoHyperlinkFollowed
        {color:#954F72;
        text-decoration:underline}
span.EmailStyle17
        {font-family:"Calibri",sans-serif;
        color:windowtext}
.MsoChpDefault
        {font-family:"Calibri",sans-serif}
@page WordSection1
        {margin:1.0in 1.0in 1.0in 1.0in}
-->
</style><style type="text/css" id="owaParaStyle">P {margin-top:0;margin-bottom:0;}</style>
</head>
<body fpstyle="1" ocsi="0" vlink="#954F72" link="#0563C1" lang="EN-US">
<div style="direction: ltr;font-family: Tahoma;color: #000000;font-size: 10pt;">
<div>Hi Liang,</div>
<div><br>
</div>
<div>No objections here, mark this patch as abandoned.</div>
<div><br>
</div>
<div>Regards,</div>
<div>Daniel</div>
<div><br>
</div>
<div style="font-family: Times New Roman; color: #000000; font-size: 16px">
<hr tabindex="-1">
<div id="divRpF971867" style="direction: ltr;"><font size="2" face="Tahoma" color="#000000"><b>From:</b> Fang, Liang A [liang.a.fang@intel.com]<br>
<b>Sent:</b> Tuesday, November 06, 2018 10:55<br>
<b>To:</b> Badea, Daniel; Poncea, Ovidiu<br>
<b>Cc:</b> starlingx-discuss@lists.starlingx.io<br>
<b>Subject:</b> [Starlingx-discuss] Analysis of patch 294a63c for StarlingX Upstreaming<br>
</font><br>
</div>
<div></div>
<div>
<div class="WordSection1">
<p class="MsoNormal">Hi Daniel and Ovidiu</p>
<p class="MsoNormal"> </p>
<p class="MsoNormal">Repo: cinder</p>
<p class="MsoNormal">Commit: 294a63c821233e3307f142965f366e8eb3d0c1de</p>
<p class="MsoNormal">Title: Pike Rebase: Fix Cinder memory leak</p>
<p class="MsoNormal"> </p>
<p class="MsoNormal">This patch is to call novaclient’s API “set_timings_max_len”, to limit the max timing logs, so that the log list will not be to huge. But novaclient(both the upstream version and stx- version) don’t contain the API set_timings_max_len,
 I also cannot find in repo history. It seems this patch may co-worked with some private version of novaclient previously, but not the public version. In fact current novaclient contains an API “reset_timings” which can be called to purge the timing logs. I
 want to mark this patch as abandoned, do you have any comments?</p>
<p class="MsoNormal"> </p>
<p class="MsoNormal">Regards</p>
<p class="MsoNormal">Liang</p>
<p class="MsoNormal"> </p>
</div>
</div>
</div>
</div>
</body>
</html>