perf: net_dropmonitor: Do not assume ordering of dictionaries
authorBen Hutchings <ben@decadent.org.uk>
Mon, 20 May 2013 14:45:34 +0000 (14:45 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 22 May 2013 22:10:10 +0000 (15:10 -0700)
The sort order of dictionaries in Python is undocumented.  Use
tuples instead, which are documented to be lexically ordered.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Signed-off-by: David S. Miller <davem@davemloft.net>
tools/perf/scripts/python/net_dropmonitor.py

index 4c1160560917b1d2ddf36c2ca614f0c011519d36..6acdc82ef47f7dec8f973f3245d08c222ae8823d 100755 (executable)
@@ -32,7 +32,7 @@ def get_kallsyms_table():
                j = j +1
                if ((j % 100) == 0):
                        print "\r" + str(j) + "/" + str(linecount),
-               kallsyms.append({ 'loc': loc, 'name' : name})
+               kallsyms.append((loc, name))
 
        print "\r" + str(j) + "/" + str(linecount)
        kallsyms.sort()
@@ -40,9 +40,9 @@ def get_kallsyms_table():
 
 def get_sym(sloc):
        loc = int(sloc)
-       for i in kallsyms[::-1]:
-               if loc >= i['loc']:
-                       return (i['name'], loc - i['loc'])
+       for symloc, name in kallsyms[::-1]:
+               if loc >= symloc:
+                       return (name, loc - symloc)
        return (None, 0)
 
 def print_drop_table():