<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Thu, Nov 8, 2018 at 8:11 AM Andy Whitcroft <<a href="mailto:apw@canonical.com">apw@canonical.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
As the new functionality is off by default this should be a low risk.<br>
The only obvious remaining risk is that we scan all of the pci busses<br>
where we did not necessarily do so before.  And as the patch itself<br>
mentions this might have been done to avoid bugs in Nvidia chipsets.<br>
Likely some nvidia based and specific testing is warrented before this<br>
cycle releases.<br></blockquote><div><br></div><div>I'll rework the new pci bus scan to be gated by the cmdline option as well,</div><div>as requested by Cascardo in the other review/reply, which should address</div><div>the remaining concern here.  Thanks for reviewing!<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Acked-by: Andy Whitcroft <<a href="mailto:apw@canonical.com" target="_blank">apw@canonical.com</a>><br>
<br>
-apw<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><span><div><div dir="ltr"><div><div dir="ltr"><div>Mauricio Faria de Oliveira</div></div></div></div></div></span></div></div></div>