March 2012 - U-Boot

[U-Boot] cam_enc_4xx build failure
by Marek Vasut 07 Mar '12

07 Mar '12

Dear Heiko Schocher, cam_enc_4xx does not build with ELDK4.2 . Can you please come up with a patch? Thanks in advance! Copyright (C) 2007 Free Software Foundation, Inc. This is free software; see the source for copying conditions. There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. Configuring for cam_enc_4xx board... cam_enc_4xx.c: In function 'menu_handle': cam_enc_4xx.c:609: warning: dereferencing type-punned pointer will break strict- aliasing rules arm-linux-ld: u-boot-spl: Not enough room for program headers, try linking with -N arm-linux-ld: final link failed: Bad value make[1]: *** [/opt/work/jenkins/jobs/U- Boot/workspace/workspace/compiler/eldk4.2/device/arm___cam_enc_4xx/spl/u-boot- spl] Error 1 make: *** [spl/u-boot-spl.bin] Error 2 make: *** Waiting for unfinished jobs.... cmd_pxe.c: In function 'do_pxe_get': cmd_pxe.c:334: warning: dereferencing type-punned pointer will break strict- aliasing rules cmd_pxe.c: In function 'get_relfile_envaddr': cmd_pxe.c:374: warning: dereferencing type-punned pointer will break strict- aliasing rules arm-linux-size: './u-boot': No such file --------------------- SUMMARY ---------------------------- Boards compiled: 1 Boards with warnings or errors: 1 ( cam_enc_4xx ) ---------------------------------------------------------- Best regards, Marek Vasut

4 5

Re: [U-Boot] [PATCH v3 6/7] arm: add a common .lds link script
by Simon Glass 07 Mar '12

07 Mar '12

Hi Albert, On Tue, Feb 28, 2012 at 1:25 PM, Albert ARIBAUD <albert.u.boot(a)aribaud.net> wrote: > Hi Simon, > > Le 21/02/2012 21:14, Simon Glass a écrit : >> >> Hi Albert, >> >> >> On Tue, Feb 21, 2012 at 11:52 AM, Albert ARIBAUD >> <albert.u.boot(a)aribaud.net> wrote: >>> >>> Hi Simon, >>> >>> Le 21/02/2012 18:02, Simon Glass a écrit : >>> >>> >>>>>> + . = ALIGN(4); >>>>> >>>>> >>>>> >>>>> >>>>> This "ALIGN(4)" is what upsets my edminiv2 build and makes it not >>>>> binary-identical. It is not in the current arm926ejs liner script. I >>>>> will >>>>> check if adding it to the current arm926ejs script produces a working >>>>> and >>>>> binary-identical u-boot. >>>> >>>> >>>> >>>> OK, but realistically we copy a word at a time (at least) and the >>>> .rel.dyn region is going to be word-aligned I think. All we are doing >>>> here is making sure that the image ends on a word boundary, which >>>> seems reasonable to me. >>>> >>>> In other words, I would like the __image_copy_start and >>>> __image_copy_end symbols to be word-aligned. >>> >>> >>> >>> I understand the requirement and agree to it -- only, it seems to me it >>> is >>> already fulfilled in the current lds files, so I still fail to see what >>> breaks 'near binary identity'. >> >> >> If we add any symbols or change the value of any of them, then the >> binary file may change. You can see the differences by doing something >> like: > > > This statement is true in a very general fashion, but becomes false for many > cases, including the one where the symbol is added at link stage and not > referenced in the build, which is the case for edminiv2. Then the ELF file's > symbol sections may change, but the .text and .data sections have no reason > to. I agree with that. But do you see this behaviour? I did initially, until I realized that it was just that '-dirty' was be appended to the version string, which was increasing the .rodata size and just changing all the offsets in the text section. Once I committed the change and rebuilt, the problem went away. Please can you confirm what differences you are seeing with things like: armv7a-cros-linux-gnueabi-objdump -h u-boot armv7a-cros-linux-gnueabi-objdump -t u-boot armv7a-cros-linux-gnueabi-objdump -d u-boot ... > > Plus, you did not address my note that the current .lds already aligns the > start and end of copy section to a multiple of 4. I think I missed that comment. I saw that you said that the ALIGN before __image_copy_end was not in the arm926-ejs link script, but when I looked I did in fact see it, e.g. in e37ae40: arch/arm/cpu/arm926ejs/u-boot.lds: ... . = .; __u_boot_cmd_start = .; .u_boot_cmd : { *(.u_boot_cmd) } __u_boot_cmd_end = .; . = ALIGN(4); ^^^^^^^^^^^^^ .rel.dyn : { ... In the link script in this patch I have: . = ALIGN(4); . = .; __u_boot_cmd_start = .; .u_boot_cmd : { *(.u_boot_cmd) } __u_boot_cmd_end = .; . = ALIGN(4); __image_copy_end = .; .rel.dyn : { __rel_dyn_start = .; Granted, the last ALIGN should not be needed since the size of commands should be word-aligned anyway, and the first command is clearly word-aligned, but it does make things clear I think. Can you please explain more what you mean by ' the current .lds already aligns the start and end of copy section to a multiple of 4'. And also, how you would like me to change this patch. Just remove the last ALIGN? > > >> armv7a-cros-linux-gnueabi-objdump -b binary -m arm -D u-boot.bin >> >> on each file (but I guess you know that). > > > Yes, plus 'od -t x1z -A x u-boot.bin > u-boot.lst' :) and it does not > explain why the .text and .data sections should undergo any change when the > .lds introduces a symbol that neither section references. See my note above about 'dirty', but if you are saying it is due to the ALIGN, I am not sure what to say. I don't see that, so perhaps it is best to email the commands you are using so I can replicate. Regards, Smion > > >> Regards, >> Simon > > > Amicalement, > -- > Albert.

1 0

[U-Boot] [PATCH 1/4] doc: Add documentation for mpc85xx debugger support
by Prabhakar Kushwaha 07 Mar '12

07 Mar '12

This describes requirement of e500 and e500v2 processor to support any debugger. it also provide an insight of switch used and defined. Signed-off-by: Radu Lazarescu <radu.lazarescu(a)freescale.com> Signed-off-by: Prabhakar Kushwaha <prabhakar(a)freescale.com> --- Applies on http://git.denx.de/u-boot.git branch master doc/README.mpc85xx_debugger | 44 +++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 44 insertions(+), 0 deletions(-) create mode 100644 doc/README.mpc85xx_debugger diff --git a/doc/README.mpc85xx_debugger b/doc/README.mpc85xx_debugger new file mode 100644 index 0000000..b35d710 --- /dev/null +++ b/doc/README.mpc85xx_debugger @@ -0,0 +1,44 @@ +Overview +-------- +Debugger's ability to debug an application is constrained by the +architecture's debug IP / run-control solution that may impose certain +requirements for the application itself. + +Similarly, when referring to the e500 and e500v2 architecture, there are two +basic rules any application has to respect in order to allow full debugging +support: + 1. Keep MSR[DE] bit set + 2. Have a valid opcode that can be fetched from the debug exception + vector [IVPR|IVOR15]. +Where: + MSR : Machine State register + IVPR : Interrupt Vector Prefix Register + IVOR : Interrupt Vector Offset Register + +Depending upon above 2 points there various place in powerpc/mpc85xx/ code +which break the rules: + - MSR[DE] is not se + - Changing the context with a rfi instruction, but omitting to preserve + the [DE] bit in SRR1 + - Changing IVPR/IVOR15 to a new location, before that location is + un-accessible + - Changing IVPR/IVOR15 to a location that does not have a valid opcode + at [IVOR|IVOR15] + - While executing in translated space (AS=1), whenever a debug + exception is generated, the MSR[DS/IS] gets cleared and the processor + tries to fetch an instruction from the debug exception vector + (IVPR|IVOR15); since now we are in AS=0, the application needs to + ensure the proper configuration to have IVOR|IVOR15 accessible from + AS=0 also + +Config Switches: +---------------- +CONFIG_E500_V1_V2 Enables code taking care of above mentioned rule. +CONFIG_DEBUGGER_TEMP_TLB Define temporary TLB number. + It will be used to create temporary TLB for AS0 + during execution in AS1. The TLB entry will be + created for debug exception vector. + Becuase on debug exception MSR[DS/IS] get + cleared i.e. execution space is shifted back + to AS0 and a TLB is required to have debug + exception vecor accessible. -- 1.7.5.4

2 2

[U-Boot] [PATCH v7 09/20] tegra: fdt: Add additional USB binding
by Simon Glass 07 Mar '12

07 Mar '12

This adds a property to indicate a port which can switch between host and device mode. Signed-off-by: Simon Glass <sjg(a)chromium.org> --- Changes in v5: - Add dr_mode property to control host/device/otg mode - Add nvidia,has-legacy-mode property per review comments Changes in v7: - Fix space indent nit doc/device-tree-bindings/usb/tegra-usb.txt | 12 ++++++++++++ 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/doc/device-tree-bindings/usb/tegra-usb.txt b/doc/device-tree-bindings/usb/tegra-usb.txt index 035d63d..5282d44 100644 --- a/doc/device-tree-bindings/usb/tegra-usb.txt +++ b/doc/device-tree-bindings/usb/tegra-usb.txt @@ -11,3 +11,15 @@ Required properties : - phy_type : Should be one of "ulpi" or "utmi". - nvidia,vbus-gpio : If present, specifies a gpio that needs to be activated for the bus to be powered. + +Optional properties: + - dr_mode : dual role mode. Indicates the working mode for + nvidia,tegra20-ehci compatible controllers. Can be "host", "peripheral", + or "otg". Default to "host" if not defined for backward compatibility. + host means this is a host controller + peripheral means it is device controller + otg means it can operate as either ("on the go") + - nvidia,has-legacy-mode : boolean indicates whether this controller can + operate in legacy mode (as APX 2500 / 2600). In legacy mode some + registers are accessed through the APB_MISC base address instead of + the USB controller. -- 1.7.7.3

3 10

[U-Boot] [PATCH v6 0/20] tegra: Add fdt definitions and USB driver
by Simon Glass 07 Mar '12

07 Mar '12

This series brings in the Linux kernel fdt file and provides a working USB driver for Tegra2 Seaboard. (I have done this in one series since otherwise most of the fdt additions will just look like dead code.) The driver requires CONFIG_OF_CONTROL and a device tree to operate. Some enhancements to fdtdec are required to make this easier, and these are included in the series also. I have had to bring in basic GPIO support due to the request to put the USB VBUS into the fdt. Since the kernel recently got a very minimal USB binding, I have started with that and extended it where appropriate. Tegra likes to have cache-aligned buffers. I have dropped the patch which implements this since we will solve this problem by making callers align their buffers (as we did with MMC). Changes in v2: - Add setting of pinmux for USB VBUS GPIO - Decode USB VBUS GPIO from the fdt - Decode phy type differently (to match new kernel fdt) - Improve debug() printouts in case of failure to init USB - Remove 0x from fdt aliases - Remove non-fdt operation of USB, since it is not needed - Remove unneeded CONFIG_TEGRA_USBx defines - Rename params to timing - Rename tegra20-usb to tegra20-ehcui (to match new kernel fdt) - Store entire fdt config in port list, not just register pointer - Use "okay" instead of "ok" for fdt node status Changes in v3: - Disable USB2 which is not used on Seaboard - Drop Tegra USB alignment patch as we will deal with this another way - Fix device tree indenting with tabs instead of spaces - Remove "okay" from nodes since this is the default anyway - Remove usbparams properties from fdt and moved them to C code Changes in v4: - Add clock bindings for Tegra2x - Add fdtdec function to return peripheral ID - Add staging area for device tree bindings used in U-Boot - Use peripheral clock node to obtain peripheral ID - Use updated fdtdec alias functiona to get USB aliases Changes in v5: - Add CONFIG_EHCI_DCACHE which is needed for dcache operation - Add additional debugging to report active USB ports - Add dr_mode property to control host/device/otg mode - Add nvidia,has-legacy-mode property per review comments - Allow any port to operate in otg mode - Change device tree comment style from // to /* */ - Correct PTS_MASK value to be unsigned - Drop unused CONFIG_USB_EHCI_DATA_ALIGN option - Fixed endian bug in fdtdec_decode_gpios() - Implement new device tree properties - Make sure GPIO name is NULL if incorrectly decoded - Put pinmux setting into a board-specific function - Remove checking of peripheral ID and try to use only device tree - Remove support-host-mode property - Report error if phy clock does not start up - Update README to indicate that we will commit fdt documentation to U-Boot Changes in v6: - Add clock bindings from Stephen Warren's latest patch - Add new patch to bring in clock bindings to seaboard - Drop fdt alignment patch as we will handle this after generic reloc - Move peripheral decode function into Tegra's clock.c - Remove dr_mode properties from SOC .dtsi file and move to boards - Use updated clock_decode_periph_id() function Simon Glass (20): fdt: Tidy up a few fdtdec problems fdt: Add functions to access phandles, arrays and bools fdt: Add basic support for decoding GPIO definitions arm: fdt: Add skeleton device tree file from kernel tegra: fdt: Add Tegra2x device tree file from kernel tegra: fdt: Add device tree file for Tegra2 Seaboard from kernel fdt: Add staging area for device tree binding documentation fdt: Add tegra-usb bindings file from linux tegra: fdt: Add additional USB binding tegra: fdt: Add clock bindings tegra: fdt: Add clock bindings for Tegra2 Seaboard tegra: usb: fdt: Add additional device tree definitions for USB ports tegra: usb: fdt: Add USB definitions for Tegra2 Seaboard usb: Add support for txfifo threshold tegra: fdt: Add function to return peripheral/clock ID tegra: usb: Add support for Tegra USB peripheral tegra: usb: Add USB support to nvidia boards tegra: usb: Add common USB defines for tegra2 boards tegra: usb: Enable USB on Seaboard tegra: fdt: Enable FDT support for Seaboard README | 3 + arch/arm/cpu/armv7/tegra2/Makefile | 4 +- arch/arm/cpu/armv7/tegra2/clock.c | 19 + arch/arm/cpu/armv7/tegra2/config.mk | 2 + arch/arm/cpu/armv7/tegra2/usb.c | 460 ++++++++++++++++++++ arch/arm/dts/skeleton.dtsi | 13 + arch/arm/dts/tegra20.dtsi | 188 ++++++++ arch/arm/include/asm/arch-tegra2/clock.h | 13 + arch/arm/include/asm/arch-tegra2/tegra2.h | 2 + arch/arm/include/asm/arch-tegra2/usb.h | 252 +++++++++++ board/nvidia/common/board.c | 12 + board/nvidia/common/board.h | 6 + board/nvidia/dts/tegra2-seaboard.dts | 57 +++ board/nvidia/seaboard/seaboard.c | 6 + doc/device-tree-bindings/README | 17 + .../clock/nvidia,tegra20-car.txt | 207 +++++++++ doc/device-tree-bindings/usb/tegra-usb.txt | 25 + drivers/usb/host/Makefile | 1 + drivers/usb/host/ehci-hcd.c | 7 + drivers/usb/host/ehci-tegra.c | 62 +++ drivers/usb/host/ehci.h | 6 +- include/configs/seaboard.h | 12 + include/configs/tegra2-common.h | 10 + include/fdtdec.h | 108 +++++- lib/fdtdec.c | 169 +++++++- 25 files changed, 1649 insertions(+), 12 deletions(-) create mode 100644 arch/arm/cpu/armv7/tegra2/usb.c create mode 100644 arch/arm/dts/skeleton.dtsi create mode 100644 arch/arm/dts/tegra20.dtsi create mode 100644 arch/arm/include/asm/arch-tegra2/usb.h create mode 100644 board/nvidia/dts/tegra2-seaboard.dts create mode 100644 doc/device-tree-bindings/README create mode 100644 doc/device-tree-bindings/clock/nvidia,tegra20-car.txt create mode 100644 doc/device-tree-bindings/usb/tegra-usb.txt create mode 100644 drivers/usb/host/ehci-tegra.c -- 1.7.7.3

4 43

[U-Boot] [RFC PATCH 0/5] add bootable partition support
by Rob Herring 07 Mar '12

07 Mar '12

From: Rob Herring <rob.herring(a)calxeda.com> The primary goal of this series is to enable using the 1st bootable partition as the default partition for disk boot related commands. Every command that takes a "<dev>[:<part>]" option duplicates the same parsing code, so this series consolidates the parsing code to a single function. There are obviously other block device commands that need to be converted still. I will do these once there is some basic agreement on the initial support. Rob Rob Herring (5): disk/part: check bootable flag for DOS partitions disk/part: introduce get_device_and_partition cmd_ext2: use common get_device_and_partition function cmd_fat: use common get_device_and_partition function cmd_scsi: use common get_device_and_partition function common/cmd_ext2.c | 71 +++++++++++---------------------------------------- common/cmd_fat.c | 72 +++++++++++++++++++--------------------------------- common/cmd_scsi.c | 58 ++++++++---------------------------------- disk/part.c | 60 ++++++++++++++++++++++++++++++++++++++++++++ disk/part_dos.c | 11 ++++++- include/part.h | 5 +++ 6 files changed, 127 insertions(+), 150 deletions(-) -- 1.7.5.4

1 5

[U-Boot] [PATCH V2 0/1] net: fec_mxc: allow use with cache enabled
by Eric Nelson 06 Mar '12

06 Mar '12

V2 addresses some concerns from the ML: - Use readl()/writel() instead of mapped data structure accesses - Wrong comment style - &rbd_base[0] == rbd_base Also removed 'volatile' from fec_send(). Eric Nelson (1): net: fec_mxc: allow use with cache enabled drivers/net/fec_mxc.c | 243 +++++++++++++++++++++++++++++++++++++------------ drivers/net/fec_mxc.h | 19 +---- 2 files changed, 184 insertions(+), 78 deletions(-) -- 1.7.9

3 13

[U-Boot] [PATCH v2 1/3] image: Make image_get_fdt work like image_get_{ram_disk, kernel}
by Stephen Warren 06 Mar '12

06 Mar '12

image_get_ram_disk() and image_get_kernel() perform operations in a consistent order. Modify image_get_fdt() to do things the same way. This allows a later change to insert some image header manipulations into these three functions in a consistent fashion. v2: New patch Signed-off-by: Stephen Warren <swarren(a)nvidia.com> --- common/image.c | 9 +++++++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/common/image.c b/common/image.c index 555d9d9..b773505 100644 --- a/common/image.c +++ b/common/image.c @@ -1131,14 +1131,19 @@ static const image_header_t *image_get_fdt(ulong fdt_addr) { const image_header_t *fdt_hdr = (const image_header_t *)fdt_addr; - image_print_contents(fdt_hdr); + if (!image_check_magic(fdt_hdr)) { + fdt_error("fdt header bad magic number\n"); + return NULL; + } - puts(" Verifying Checksum ... "); if (!image_check_hcrc(fdt_hdr)) { fdt_error("fdt header checksum invalid"); return NULL; } + image_print_contents(fdt_hdr); + + puts(" Verifying Checksum ... "); if (!image_check_dcrc(fdt_hdr)) { fdt_error("fdt checksum invalid"); return NULL; -- 1.7.0.4

5 35

[U-Boot] Conflicting definitions of flush_dcache
by Stefan Kristiansson 06 Mar '12

06 Mar '12

Hi, I noticed the following dilemma when I tried to enable both CONFIG_ETHOC and CONFIG_CMD_CACHE: The ethoc ethernet driver expects: void flush_dcache(unsigned long start, unsigned long size) while cmd_cache.c expect it to be: void flush_dcache(void) Grepping around the sources, I found that apart from drivers/net/ethoc.c also drivers/net/altera_tse.c uses flush_dcache with the two ulong parameters. No architecture apart from nios2 seems to provide flush_dcache like this and flush_dcache is not declared in common.h, so my gut feeling is that nios2 and the 2 ethernet drivers should be changed to use for example flush_dcache_range(?) Perhaps cmd_cache.c should also be fixed to use flush_dcache_all()? flush_icache() ofcourse suffer from the same problem. Stefan

4 5

[U-Boot] [PATCH] tools: checkstack.pl from Linux added to tools
by Tom Rini 06 Mar '12

06 Mar '12

Added from Linux - commit 62aa2b537c6f5957afd98e29f96897419ed5ebab Signed-off-by: Tom Rini <trini(a)ti.com> --- Makefile | 5 ++ tools/checkstack.pl | 172 +++++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 177 insertions(+), 0 deletions(-) create mode 100755 tools/checkstack.pl diff --git a/Makefile b/Makefile index 36246b6..7931615 100644 --- a/Makefile +++ b/Makefile @@ -529,6 +529,11 @@ TAG_SUBDIRS += include FIND := find FINDFLAGS := -L +checkstack: + $(CROSS_COMPILE)objdump -d $(obj)u-boot \ + `$(FIND) $(obj) -name u-boot-spl -print` | \ + perl $(src)tools/checkstack.pl $(ARCH) + tags ctags: ctags -w -o $(obj)ctags `$(FIND) $(FINDFLAGS) $(TAG_SUBDIRS) \ -name '*.[chS]' -print` diff --git a/tools/checkstack.pl b/tools/checkstack.pl new file mode 100755 index 0000000..17e3843 --- /dev/null +++ b/tools/checkstack.pl @@ -0,0 +1,172 @@ +#!/usr/bin/perl + +# Check the stack usage of functions +# +# Copyright Joern Engel <joern(a)lazybastard.org> +# Inspired by Linus Torvalds +# Original idea maybe from Keith Owens +# s390 port and big speedup by Arnd Bergmann <arnd(a)bergmann-dalldorf.de> +# Mips port by Juan Quintela <quintela(a)mandrakesoft.com> +# IA64 port via Andreas Dilger +# Arm port by Holger Schurig +# sh64 port by Paul Mundt +# Random bits by Matt Mackall <mpm(a)selenic.com> +# M68k port by Geert Uytterhoeven and Andreas Schwab +# AVR32 port by Haavard Skinnemoen (Atmel) +# PARISC port by Kyle McMartin <kyle(a)parisc-linux.org> +# sparc port by Martin Habets <errandir_news(a)mph.eclipse.co.uk> +# +# Usage: +# objdump -d vmlinux | scripts/checkstack.pl [arch] +# +# TODO : Port to all architectures (one regex per arch) + +use strict; + +# check for arch +# +# $re is used for two matches: +# $& (whole re) matches the complete objdump line with the stack growth +# $1 (first bracket) matches the size of the stack growth +# +# $dre is similar, but for dynamic stack redutions: +# $& (whole re) matches the complete objdump line with the stack growth +# $1 (first bracket) matches the dynamic amount of the stack growth +# +# use anything else and feel the pain ;) +my (@stack, $re, $dre, $x, $xs); +{ + my $arch = shift; + if ($arch eq "") { + $arch = `uname -m`; + chomp($arch); + } + + $x = "[0-9a-f]"; # hex character + $xs = "[0-9a-f ]"; # hex character or space + if ($arch eq 'arm') { + #c0008ffc: e24dd064 sub sp, sp, #100 ; 0x64 + $re = qr/.*sub.*sp, sp, #(([0-9]{2}|[3-9])[0-9]{2})/o; + } elsif ($arch eq 'avr32') { + #8000008a: 20 1d sub sp,4 + #80000ca8: fa cd 05 b0 sub sp,sp,1456 + $re = qr/^.*sub.*sp.*,([0-9]{1,8})/o; + } elsif ($arch =~ /^i[3456]86$/) { + #c0105234: 81 ec ac 05 00 00 sub $0x5ac,%esp + $re = qr/^.*[as][du][db] \$(0x$x{1,8}),\%esp$/o; + $dre = qr/^.*[as][du][db] (%.*),\%esp$/o; + } elsif ($arch eq 'x86_64') { + # 2f60: 48 81 ec e8 05 00 00 sub $0x5e8,%rsp + $re = qr/^.*[as][du][db] \$(0x$x{1,8}),\%rsp$/o; + $dre = qr/^.*[as][du][db] (\%.*),\%rsp$/o; + } elsif ($arch eq 'ia64') { + #e0000000044011fc: 01 0f fc 8c adds r12=-384,r12 + $re = qr/.*adds.*r12=-(([0-9]{2}|[3-9])[0-9]{2}),r12/o; + } elsif ($arch eq 'm68k') { + # 2b6c: 4e56 fb70 linkw %fp,#-1168 + # 1df770: defc ffe4 addaw #-28,%sp + $re = qr/.*(?:linkw %fp,|addaw )#-([0-9]{1,4})(?:,%sp)?$/o; + } elsif ($arch eq 'mips64') { + #8800402c: 67bdfff0 daddiu sp,sp,-16 + $re = qr/.*daddiu.*sp,sp,-(([0-9]{2}|[3-9])[0-9]{2})/o; + } elsif ($arch eq 'mips') { + #88003254: 27bdffe0 addiu sp,sp,-32 + $re = qr/.*addiu.*sp,sp,-(([0-9]{2}|[3-9])[0-9]{2})/o; + } elsif ($arch eq 'parisc' || $arch eq 'parisc64') { + $re = qr/.*ldo ($x{1,8})$sp$,sp/o; + } elsif ($arch eq 'ppc') { + #c00029f4: 94 21 ff 30 stwu r1,-208(r1) + $re = qr/.*stwu.*r1,-($x{1,8})$r1$/o; + } elsif ($arch eq 'ppc64') { + #XXX + $re = qr/.*stdu.*r1,-($x{1,8})$r1$/o; + } elsif ($arch eq 'powerpc') { + $re = qr/.*st[dw]u.*r1,-($x{1,8})$r1$/o; + } elsif ($arch =~ /^s390x?$/) { + # 11160: a7 fb ff 60 aghi %r15,-160 + # or + # 100092: e3 f0 ff c8 ff 71 lay %r15,-56(%r15) + $re = qr/.*(?:lay|ag?hi).*\%r15,-(([0-9]{2}|[3-9])[0-9]{2}) + (?:$\%r15$)?$/ox; + } elsif ($arch =~ /^sh64$/) { + #XXX: we only check for the immediate case presently, + # though we will want to check for the movi/sub + # pair for larger users. -- PFM. + #a00048e0: d4fc40f0 addi.l r15,-240,r15 + $re = qr/.*addi\.l.*r15,-(([0-9]{2}|[3-9])[0-9]{2}),r15/o; + } elsif ($arch =~ /^blackfin$/) { + # 0: 00 e8 38 01 LINK 0x4e0; + $re = qr/.*[[:space:]]LINK[[:space:]]*(0x$x{1,8})/o; + } elsif ($arch eq 'sparc' || $arch eq 'sparc64') { + # f0019d10: 9d e3 bf 90 save %sp, -112, %sp + $re = qr/.*save.*%sp, -(([0-9]{2}|[3-9])[0-9]{2}), %sp/o; + } else { + print("wrong or unknown architecture \"$arch\"\n"); + exit + } +} + +# +# main() +# +my $funcre = qr/^$x* <(.*)>:$/; +my ($func, $file, $lastslash); + +while (my $line = <STDIN>) { + if ($line =~ m/$funcre/) { + $func = $1; + } + elsif ($line =~ m/(.*):\s*file format/) { + $file = $1; + $file =~ s/\.ko//; + $lastslash = rindex($file, "/"); + if ($lastslash != -1) { + $file = substr($file, $lastslash + 1); + } + } + elsif ($line =~ m/$re/) { + my $size = $1; + $size = hex($size) if ($size =~ /^0x/); + + if ($size > 0xf0000000) { + $size = - $size; + $size += 0x80000000; + $size += 0x80000000; + } + next if ($size > 0x10000000); + + next if $line !~ m/^($xs*)/; + my $addr = $1; + $addr =~ s/ /0/g; + $addr = "0x$addr"; + + my $intro = "$addr $func [$file]:"; + my $padlen = 56 - length($intro); + while ($padlen > 0) { + $intro .= ' '; + $padlen -= 8; + } + next if ($size < 100); + push @stack, "$intro$size\n"; + } + elsif (defined $dre && $line =~ m/$dre/) { + my $size = "Dynamic ($1)"; + + next if $line !~ m/^($xs*)/; + my $addr = $1; + $addr =~ s/ /0/g; + $addr = "0x$addr"; + + my $intro = "$addr $func [$file]:"; + my $padlen = 56 - length($intro); + while ($padlen > 0) { + $intro .= ' '; + $padlen -= 8; + } + push @stack, "$intro$size\n"; + } +} + +# Sort output by size (last field) +print sort { ($b =~ /:\t*(\d+)$/)[0] <=> ($a =~ /:\t*(\d+)$/)[0] } @stack; + -- 1.7.0.4

2 1