<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from text --><style><!-- .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left: #800000 2px solid; } --></style>
</head>
<body>
<meta content="text/html; charset=UTF-8">
<style type="text/css" style="">
<!--
p
        {margin-top:0;
        margin-bottom:0}
-->
</style>
<div dir="ltr">
<div id="x_divtagdefaultwrapper" dir="ltr" style="font-size:12pt; color:#000000; font-family:Calibri,Helvetica,sans-serif">
<p>Remember when I have been working in Ericsson we had been using a gerrit review and you simply</p>
<p>would not be able to merge anything until it has +2(r-b) and no tests failing. Or you have to exclude</p>
<p>unstable tests at least until those are fixed.</p>
<p><br>
</p>
<p>It was simply automated and no exceptions, no matter how good person you are <span>
😊</span></p>
<p></p>
<p><br>
</p>
<div id="x_Signature">
<div style="font-family:Tahoma; font-size:13px"><font size="2"><span style="font-size:10pt">Best Regards,<br>
<br>
Lisovskiy Stanislav <br>
<br>
Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo<br>
</span></font></div>
</div>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> Hiler, Arkadiusz<br>
<b>Sent:</b> Wednesday, March 18, 2020 10:55:19 AM<br>
<b>To:</b> Lisovskiy, Stanislav<br>
<b>Cc:</b> intel-gfx@lists.freedesktop.org; Peres, Martin; Vudum, Lakshminarayana<br>
<b>Subject:</b> Re: âœ— Fi.CI.DOCS: warning for Consider DBuf bandwidth when calculating CDCLK (rev2)</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">On Wed, Mar 18, 2020 at 10:43:29AM +0200, Lisovskiy, Stanislav wrote:<br>
> Wonder, how we end up merging _stuff_ with failed IGT and loads of warnings..<br>
> <br>
> <a href="https://patchwork.freedesktop.org/series/74738/">https://patchwork.freedesktop.org/series/74738/</a><br>
> <br>
> ... while I get beaten for each and every single warning in my patches ðŸ˜Š<br>
<br>
True. This shouldn't get merged like this. Ask the authors and the<br>
commiters why this got in without anyone looking at the docs issue and<br>
without any explanation of the failures in Fi.CI.IGT.<br>
<br>
-- <br>
Cheers,<br>
Arek<br>
</div>
</span></font>
</body>
</html>