<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Mar 20, 2014 at 8:23 PM, Rick Jones <span dir="ltr"><<a href="mailto:rick.jones2@hp.com" target="_blank">rick.jones2@hp.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 03/20/2014 09:07 AM, Yuriy Taraday wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">
On Thu, Mar 20, 2014 at 7:28 PM, Rick Jones <<a href="mailto:rick.jones2@hp.com" target="_blank">rick.jones2@hp.com</a><br></div><div class="">
<mailto:<a href="mailto:rick.jones2@hp.com" target="_blank">rick.jones2@hp.com</a>>> wrote:<br>
Interesting result. Which versions of sudo and ip and with how many<br>
interfaces on the system?<br>
<br>
<br>
Here are the numbers:<br>
<br>
% sudo -V<br>
Sudo version 1.8.6p7<br>
Sudoers policy plugin version 1.8.6p7<br>
Sudoers file grammar version 42<br>
Sudoers I/O plugin version 1.8.6p7<br>
% ip -V<br>
ip utility, iproute2-ss130221<br>
% ip a | grep '^[^ ]' | wc -l<br>
5<br>
<br>
For consistency's sake (however foolish it may be) and purposes of<br>
others being able to reproduce results and all that, stating the<br>
number of interfaces on the system and versions and such would be a<br>
Good Thing.<br>
<br>
<br>
Ok, I'll add them to benchmark output.<br>
</div></blockquote>
<br>
Since there are only five interfaces on the system, it likely doesn't make much of a difference in your specific benchmark but the top-of-trunk version of sudo has the fix/enhancement to allow one to tell it via sudo.conf to not grab the list of interfaces on the system.<br>
<br>
Might be worthwhile though to take the interface count out to 2000 or more in the name of doing things at scale. Namespace count as well.</blockquote><div><br></div><div>Given that this benchmark is created to show that my changes are worth doing and they already show that my approach is almost 2x faster than sudo, slowing down sudo will only enhance this difference. I don't think we should add this to the benchmark itself.</div>
<div> </div></div>-- <br><br><div>Kind regards, Yuriy.</div>
</div></div>