On 5/6/20 10:26 PM, jan wrote:
In the past the file was named 'built-in.o'.
Nowadays it is 'built-in.a'.

The script is modified to work with both.  I expect
it will not happen that there are built-in.a and built-in.o
files in the same kernel.

Signed-off-by: Jan Vermaete <[email protected]>
---
  scripts/tiny/ksize.py | 8 ++++----
  1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/scripts/tiny/ksize.py b/scripts/tiny/ksize.py
index 8316b85cb1..63c5495c91 100755
--- a/scripts/tiny/ksize.py
+++ b/scripts/tiny/ksize.py
@@ -4,7 +4,7 @@
  #
  # SPDX-License-Identifier: GPL-2.0-or-later
  #
-# Display details of the kernel build size, broken up by built-in.o. Sort
+# Display details of the kernel build size, broken up by built-in.[o,a]. Sort
  # the objects by size. Run from the top level kernel build directory.
  #
  # Author: Darren Hart <[email protected]>
@@ -48,7 +48,7 @@ class Report:
          r = Report(filename, title)
          path = os.path.dirname(filename)
- p = Popen("ls " + str(path) + "/*.o | grep -v built-in.o",
+        p = Popen("ls " + str(path) + "/*.o | grep -v built-in.o | grep -v 
built-in.a",

Looks to me like you don't need to add the last grep; you're only listing *.o to begin with, no ?

                    shell=True, stdout=PIPE, stderr=PIPE, 
universal_newlines=True)
          glob = ' '.join(p.communicate()[0].splitlines())
          oreport = Report(glob, str(path) + "/*.o")
@@ -59,7 +59,7 @@ class Report:
              p = Popen("ls " + subglob, shell=True, stdout=PIPE, stderr=PIPE, 
universal_newlines=True)
              for f in p.communicate()[0].splitlines():
                  path = os.path.dirname(f)
-                r.parts.append(Report.create(f, path, str(path) + 
"/*/built-in.o"))
+                r.parts.append(Report.create(f, path, str(path) + 
"/*/built-in.[o,a]"))
              r.parts.sort(reverse=True)
for b in r.parts:
@@ -139,7 +139,7 @@ def main():
          else:
              assert False, "unhandled option"
- glob = "arch/*/built-in.o */built-in.o"
+    glob = "arch/*/built-in.[o,a] */built-in.[o,a]"
      vmlinux = Report.create("vmlinux",  "Linux Kernel", glob)
vmlinux.show()




-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#138001): 
https://lists.openembedded.org/g/openembedded-core/message/138001
Mute This Topic: https://lists.openembedded.org/mt/74037483/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to