public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH 0/3] Arm*Pkg: convert LFs to CRLF, expand hard TABs
@ 2020-02-27 21:39 Laszlo Ersek
  2020-02-27 21:39 ` [PATCH 1/3] ArmPkg: " Laszlo Ersek
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ messages in thread
From: Laszlo Ersek @ 2020-02-27 21:39 UTC (permalink / raw)
  To: edk2-devel-groups-io
  Cc: Ard Biesheuvel, Leif Lindholm, Philippe Mathieu-Daudé

Bugzilla: https://bugzilla.tianocore.org/show_bug.cgi?id=1659
Repo:     https://github.com/lersek/edk2.git
Branch:   crlf_bz_1659

Future patches for some Arm*Pkg source files would either introduce CRLF
lines into LF files, or add more LF lines. The first would increase
inconsistency, the second would make "PatchCheck.py" yell. Convert these
files to CRLF uniformly (and while at it, expand hard TABS, because
"PatchCheck.py" catches those too).

Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Philippe Mathieu-Daudé <philmd@redhat.com>

Thanks,
Laszlo

Laszlo Ersek (3):
  ArmPkg: convert LFs to CRLF, expand hard TABs
  ArmVirtPkg: convert LFs to CRLF
  ArmPlatformPkg: convert LFs to CRLF, expand hard TABs

 ArmPkg/Library/ArmSmcLibNull/ArmSmcLibNull.c     |  32 +-
 ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c | 566 ++++++++---------
 ArmPkg/Library/CompilerIntrinsicsLib/memset.c    | 110 ++--
 ArmPkg/Library/GccLto/liblto-aarch64.s           |  42 +-
 ArmPkg/Library/GccLto/liblto-arm.s               | 110 ++--
 ArmPlatformPkg/Scripts/Ds5/profile.py            | 656 ++++++++++----------
 ArmVirtPkg/Include/Platform/Hidden.h             |  44 +-
 7 files changed, 780 insertions(+), 780 deletions(-)

-- 
2.19.1.3.g30247aa5d201


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 1/3] ArmPkg: convert LFs to CRLF, expand hard TABs
  2020-02-27 21:39 [PATCH 0/3] Arm*Pkg: convert LFs to CRLF, expand hard TABs Laszlo Ersek
@ 2020-02-27 21:39 ` Laszlo Ersek
  2020-02-27 21:39 ` [PATCH 2/3] ArmVirtPkg: convert LFs to CRLF Laszlo Ersek
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Laszlo Ersek @ 2020-02-27 21:39 UTC (permalink / raw)
  To: edk2-devel-groups-io
  Cc: Ard Biesheuvel, Leif Lindholm, Philippe Mathieu-Daudé

We're going to switch the internal line terminators globally to LF at some
point, but until then, let's use CRLF consistently. Convert source files
with LFs in them to CRLF, using "unix2dos".

"git show -b" prints no code changes for this patch.

(I collected all the file name suffixes in this package, with:

$ git ls-files -- $PACKAGE | rev | cut -f 1 -d . | sort -u | rev

I eliminated those suffixes that didn't stand for text files, then
blanket-converted the rest with unix2dos. Finally, picked up the actual
changes with git-add.)

At the same time, the following three files had to undergo TAB expansion:

  ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c
  ArmPkg/Library/GccLto/liblto-aarch64.s
  ArmPkg/Library/GccLto/liblto-arm.s

I used "expand -t 2", in order to stay close to the edk2 coding style
(which uses two spaces for indentation.)

Both the CRLF conversion and the TAB expansion are motivated by
"PatchCheck.py". "PatchCheck.py" is also the reason why CRLF conversion
and TAB expansion have to happen in the same patch.

Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1659
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
---
 ArmPkg/Library/ArmSmcLibNull/ArmSmcLibNull.c     |  32 +-
 ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c | 566 ++++++++++----------
 ArmPkg/Library/CompilerIntrinsicsLib/memset.c    | 110 ++--
 ArmPkg/Library/GccLto/liblto-aarch64.s           |  42 +-
 ArmPkg/Library/GccLto/liblto-arm.s               | 110 ++--
 5 files changed, 430 insertions(+), 430 deletions(-)

diff --git a/ArmPkg/Library/ArmSmcLibNull/ArmSmcLibNull.c b/ArmPkg/Library/ArmSmcLibNull/ArmSmcLibNull.c
index 2cd292aabf88..2d79aadaf1fa 100644
--- a/ArmPkg/Library/ArmSmcLibNull/ArmSmcLibNull.c
+++ b/ArmPkg/Library/ArmSmcLibNull/ArmSmcLibNull.c
@@ -1,16 +1,16 @@
-//
-//  Copyright (c) 2016, Linaro Limited. All rights reserved.
-//
-//  SPDX-License-Identifier: BSD-2-Clause-Patent
-//
-//
-
-#include <Base.h>
-#include <Library/ArmSmcLib.h>
-
-VOID
-ArmCallSmc (
-  IN OUT ARM_SMC_ARGS *Args
-  )
-{
-}
+//
+//  Copyright (c) 2016, Linaro Limited. All rights reserved.
+//
+//  SPDX-License-Identifier: BSD-2-Clause-Patent
+//
+//
+
+#include <Base.h>
+#include <Library/ArmSmcLib.h>
+
+VOID
+ArmCallSmc (
+  IN OUT ARM_SMC_ARGS *Args
+  )
+{
+}
diff --git a/ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c b/ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c
index 42bed7700c99..77e2473678a2 100644
--- a/ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c
+++ b/ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c
@@ -1,283 +1,283 @@
-/*
- * Copyright (c) 2015 - 2019, Linaro Limited
- *
- * SPDX-License-Identifier: BSD-2-Clause-Patent
- */
-
-#include "platform.h"
-#include <softfloat.h>
-
-/*
- * On ARM32 EABI defines both a soft-float ABI and a hard-float ABI,
- * hard-float is basically a super set of soft-float. Hard-float requires
- * all the support routines provided for soft-float, but the compiler may
- * choose to optimize to not use some of them.
- *
- * The AEABI functions uses soft-float calling convention even if the
- * functions are compiled for hard-float. So where float and double would
- * have been expected we use aeabi_float_t and aeabi_double_t respectively
- * instead.
- */
-typedef uint32_t aeabi_float_t;
-typedef uint64_t aeabi_double_t;
-
-/*
- * Helpers to convert between float32 and aeabi_float_t, and float64 and
- * aeabi_double_t used by the AEABI functions below.
- */
-static aeabi_float_t f32_to_f(float32_t val)
-{
-	return val.v;
-}
-
-static float32_t f32_from_f(aeabi_float_t val)
-{
-	float32_t res;
-
-	res.v = val;
-
-	return res;
-}
-
-static aeabi_double_t f64_to_d(float64_t val)
-{
-	return val.v;
-}
-
-static float64_t f64_from_d(aeabi_double_t val)
-{
-	float64_t res;
-
-	res.v = val;
-
-	return res;
-}
-
-/*
- * From ARM Run-time ABI for ARM Architecture
- * ARM IHI 0043D, current through ABI release 2.09
- *
- * 4.1.2 The floating-point helper functions
- */
-
-/*
- * Table 2, Standard aeabi_double_t precision floating-point arithmetic helper
- * functions
- */
-
-aeabi_double_t __aeabi_dadd(aeabi_double_t a, aeabi_double_t b)
-{
-	return f64_to_d(f64_add(f64_from_d(a), f64_from_d(b)));
-}
-
-aeabi_double_t __aeabi_ddiv(aeabi_double_t a, aeabi_double_t b)
-{
-	return f64_to_d(f64_div(f64_from_d(a), f64_from_d(b)));
-}
-
-aeabi_double_t __aeabi_dmul(aeabi_double_t a, aeabi_double_t b)
-{
-	return f64_to_d(f64_mul(f64_from_d(a), f64_from_d(b)));
-}
-
-
-aeabi_double_t __aeabi_drsub(aeabi_double_t a, aeabi_double_t b)
-{
-	return f64_to_d(f64_sub(f64_from_d(b), f64_from_d(a)));
-}
-
-aeabi_double_t __aeabi_dsub(aeabi_double_t a, aeabi_double_t b)
-{
-	return f64_to_d(f64_sub(f64_from_d(a), f64_from_d(b)));
-}
-
-/*
- * Table 3, double precision floating-point comparison helper functions
- */
-
-int __aeabi_dcmpeq(aeabi_double_t a, aeabi_double_t b)
-{
-	return f64_eq(f64_from_d(a), f64_from_d(b));
-}
-
-int __aeabi_dcmplt(aeabi_double_t a, aeabi_double_t b)
-{
-	return f64_lt(f64_from_d(a), f64_from_d(b));
-}
-
-int __aeabi_dcmple(aeabi_double_t a, aeabi_double_t b)
-{
-	return f64_le(f64_from_d(a), f64_from_d(b));
-}
-
-int __aeabi_dcmpge(aeabi_double_t a, aeabi_double_t b)
-{
-	return f64_le(f64_from_d(b), f64_from_d(a));
-}
-
-int __aeabi_dcmpgt(aeabi_double_t a, aeabi_double_t b)
-{
-	return f64_lt(f64_from_d(b), f64_from_d(a));
-}
-
-/*
- * Table 4, Standard single precision floating-point arithmetic helper
- * functions
- */
-
-aeabi_float_t __aeabi_fadd(aeabi_float_t a, aeabi_float_t b)
-{
-	return f32_to_f(f32_add(f32_from_f(a), f32_from_f(b)));
-}
-
-aeabi_float_t __aeabi_fdiv(aeabi_float_t a, aeabi_float_t b)
-{
-	return f32_to_f(f32_div(f32_from_f(a), f32_from_f(b)));
-}
-
-aeabi_float_t __aeabi_fmul(aeabi_float_t a, aeabi_float_t b)
-{
-	return f32_to_f(f32_mul(f32_from_f(a), f32_from_f(b)));
-}
-
-aeabi_float_t __aeabi_frsub(aeabi_float_t a, aeabi_float_t b)
-{
-	return f32_to_f(f32_sub(f32_from_f(b), f32_from_f(a)));
-}
-
-aeabi_float_t __aeabi_fsub(aeabi_float_t a, aeabi_float_t b)
-{
-	return f32_to_f(f32_sub(f32_from_f(a), f32_from_f(b)));
-}
-
-/*
- * Table 5, Standard single precision floating-point comparison helper
- * functions
- */
-
-int __aeabi_fcmpeq(aeabi_float_t a, aeabi_float_t b)
-{
-	return f32_eq(f32_from_f(a), f32_from_f(b));
-}
-
-int __aeabi_fcmplt(aeabi_float_t a, aeabi_float_t b)
-{
-	return f32_lt(f32_from_f(a), f32_from_f(b));
-}
-
-int __aeabi_fcmple(aeabi_float_t a, aeabi_float_t b)
-{
-	return f32_le(f32_from_f(a), f32_from_f(b));
-}
-
-int __aeabi_fcmpge(aeabi_float_t a, aeabi_float_t b)
-{
-	return f32_le(f32_from_f(b), f32_from_f(a));
-}
-
-int __aeabi_fcmpgt(aeabi_float_t a, aeabi_float_t b)
-{
-	return f32_lt(f32_from_f(b), f32_from_f(a));
-}
-
-/*
- * Table 6, Standard floating-point to integer conversions
- */
-
-int __aeabi_d2iz(aeabi_double_t a)
-{
-	return f64_to_i32_r_minMag(f64_from_d(a), false);
-}
-
-unsigned __aeabi_d2uiz(aeabi_double_t a)
-{
-	return f64_to_ui32_r_minMag(f64_from_d(a), false);
-}
-
-long long __aeabi_d2lz(aeabi_double_t a)
-{
-	return f64_to_i64_r_minMag(f64_from_d(a), false);
-}
-
-unsigned long long __aeabi_d2ulz(aeabi_double_t a)
-{
-	return f64_to_ui64_r_minMag(f64_from_d(a), false);
-}
-
-int __aeabi_f2iz(aeabi_float_t a)
-{
-	return f32_to_i32_r_minMag(f32_from_f(a), false);
-}
-
-unsigned __aeabi_f2uiz(aeabi_float_t a)
-{
-	return f32_to_ui32_r_minMag(f32_from_f(a), false);
-}
-
-long long __aeabi_f2lz(aeabi_float_t a)
-{
-	return f32_to_i64_r_minMag(f32_from_f(a), false);
-}
-
-unsigned long long __aeabi_f2ulz(aeabi_float_t a)
-{
-	return f32_to_ui64_r_minMag(f32_from_f(a), false);
-}
-
-/*
- * Table 7, Standard conversions between floating types
- */
-
-aeabi_float_t __aeabi_d2f(aeabi_double_t a)
-{
-	return f32_to_f(f64_to_f32(f64_from_d(a)));
-}
-
-aeabi_double_t __aeabi_f2d(aeabi_float_t a)
-{
-	return f64_to_d(f32_to_f64(f32_from_f(a)));
-}
-
-/*
- * Table 8, Standard integer to floating-point conversions
- */
-
-aeabi_double_t __aeabi_i2d(int a)
-{
-	return f64_to_d(i32_to_f64(a));
-}
-
-aeabi_double_t __aeabi_ui2d(unsigned a)
-{
-	return f64_to_d(ui32_to_f64(a));
-}
-
-aeabi_double_t __aeabi_l2d(long long a)
-{
-	return f64_to_d(i64_to_f64(a));
-}
-
-aeabi_double_t __aeabi_ul2d(unsigned long long a)
-{
-	return f64_to_d(ui64_to_f64(a));
-}
-
-aeabi_float_t __aeabi_i2f(int a)
-{
-	return f32_to_f(i32_to_f32(a));
-}
-
-aeabi_float_t __aeabi_ui2f(unsigned a)
-{
-	return f32_to_f(ui32_to_f32(a));
-}
-
-aeabi_float_t __aeabi_l2f(long long a)
-{
-	return f32_to_f(i64_to_f32(a));
-}
-
-aeabi_float_t __aeabi_ul2f(unsigned long long a)
-{
-	return f32_to_f(ui64_to_f32(a));
-}
+/*
+ * Copyright (c) 2015 - 2019, Linaro Limited
+ *
+ * SPDX-License-Identifier: BSD-2-Clause-Patent
+ */
+
+#include "platform.h"
+#include <softfloat.h>
+
+/*
+ * On ARM32 EABI defines both a soft-float ABI and a hard-float ABI,
+ * hard-float is basically a super set of soft-float. Hard-float requires
+ * all the support routines provided for soft-float, but the compiler may
+ * choose to optimize to not use some of them.
+ *
+ * The AEABI functions uses soft-float calling convention even if the
+ * functions are compiled for hard-float. So where float and double would
+ * have been expected we use aeabi_float_t and aeabi_double_t respectively
+ * instead.
+ */
+typedef uint32_t aeabi_float_t;
+typedef uint64_t aeabi_double_t;
+
+/*
+ * Helpers to convert between float32 and aeabi_float_t, and float64 and
+ * aeabi_double_t used by the AEABI functions below.
+ */
+static aeabi_float_t f32_to_f(float32_t val)
+{
+  return val.v;
+}
+
+static float32_t f32_from_f(aeabi_float_t val)
+{
+  float32_t res;
+
+  res.v = val;
+
+  return res;
+}
+
+static aeabi_double_t f64_to_d(float64_t val)
+{
+  return val.v;
+}
+
+static float64_t f64_from_d(aeabi_double_t val)
+{
+  float64_t res;
+
+  res.v = val;
+
+  return res;
+}
+
+/*
+ * From ARM Run-time ABI for ARM Architecture
+ * ARM IHI 0043D, current through ABI release 2.09
+ *
+ * 4.1.2 The floating-point helper functions
+ */
+
+/*
+ * Table 2, Standard aeabi_double_t precision floating-point arithmetic helper
+ * functions
+ */
+
+aeabi_double_t __aeabi_dadd(aeabi_double_t a, aeabi_double_t b)
+{
+  return f64_to_d(f64_add(f64_from_d(a), f64_from_d(b)));
+}
+
+aeabi_double_t __aeabi_ddiv(aeabi_double_t a, aeabi_double_t b)
+{
+  return f64_to_d(f64_div(f64_from_d(a), f64_from_d(b)));
+}
+
+aeabi_double_t __aeabi_dmul(aeabi_double_t a, aeabi_double_t b)
+{
+  return f64_to_d(f64_mul(f64_from_d(a), f64_from_d(b)));
+}
+
+
+aeabi_double_t __aeabi_drsub(aeabi_double_t a, aeabi_double_t b)
+{
+  return f64_to_d(f64_sub(f64_from_d(b), f64_from_d(a)));
+}
+
+aeabi_double_t __aeabi_dsub(aeabi_double_t a, aeabi_double_t b)
+{
+  return f64_to_d(f64_sub(f64_from_d(a), f64_from_d(b)));
+}
+
+/*
+ * Table 3, double precision floating-point comparison helper functions
+ */
+
+int __aeabi_dcmpeq(aeabi_double_t a, aeabi_double_t b)
+{
+  return f64_eq(f64_from_d(a), f64_from_d(b));
+}
+
+int __aeabi_dcmplt(aeabi_double_t a, aeabi_double_t b)
+{
+  return f64_lt(f64_from_d(a), f64_from_d(b));
+}
+
+int __aeabi_dcmple(aeabi_double_t a, aeabi_double_t b)
+{
+  return f64_le(f64_from_d(a), f64_from_d(b));
+}
+
+int __aeabi_dcmpge(aeabi_double_t a, aeabi_double_t b)
+{
+  return f64_le(f64_from_d(b), f64_from_d(a));
+}
+
+int __aeabi_dcmpgt(aeabi_double_t a, aeabi_double_t b)
+{
+  return f64_lt(f64_from_d(b), f64_from_d(a));
+}
+
+/*
+ * Table 4, Standard single precision floating-point arithmetic helper
+ * functions
+ */
+
+aeabi_float_t __aeabi_fadd(aeabi_float_t a, aeabi_float_t b)
+{
+  return f32_to_f(f32_add(f32_from_f(a), f32_from_f(b)));
+}
+
+aeabi_float_t __aeabi_fdiv(aeabi_float_t a, aeabi_float_t b)
+{
+  return f32_to_f(f32_div(f32_from_f(a), f32_from_f(b)));
+}
+
+aeabi_float_t __aeabi_fmul(aeabi_float_t a, aeabi_float_t b)
+{
+  return f32_to_f(f32_mul(f32_from_f(a), f32_from_f(b)));
+}
+
+aeabi_float_t __aeabi_frsub(aeabi_float_t a, aeabi_float_t b)
+{
+  return f32_to_f(f32_sub(f32_from_f(b), f32_from_f(a)));
+}
+
+aeabi_float_t __aeabi_fsub(aeabi_float_t a, aeabi_float_t b)
+{
+  return f32_to_f(f32_sub(f32_from_f(a), f32_from_f(b)));
+}
+
+/*
+ * Table 5, Standard single precision floating-point comparison helper
+ * functions
+ */
+
+int __aeabi_fcmpeq(aeabi_float_t a, aeabi_float_t b)
+{
+  return f32_eq(f32_from_f(a), f32_from_f(b));
+}
+
+int __aeabi_fcmplt(aeabi_float_t a, aeabi_float_t b)
+{
+  return f32_lt(f32_from_f(a), f32_from_f(b));
+}
+
+int __aeabi_fcmple(aeabi_float_t a, aeabi_float_t b)
+{
+  return f32_le(f32_from_f(a), f32_from_f(b));
+}
+
+int __aeabi_fcmpge(aeabi_float_t a, aeabi_float_t b)
+{
+  return f32_le(f32_from_f(b), f32_from_f(a));
+}
+
+int __aeabi_fcmpgt(aeabi_float_t a, aeabi_float_t b)
+{
+  return f32_lt(f32_from_f(b), f32_from_f(a));
+}
+
+/*
+ * Table 6, Standard floating-point to integer conversions
+ */
+
+int __aeabi_d2iz(aeabi_double_t a)
+{
+  return f64_to_i32_r_minMag(f64_from_d(a), false);
+}
+
+unsigned __aeabi_d2uiz(aeabi_double_t a)
+{
+  return f64_to_ui32_r_minMag(f64_from_d(a), false);
+}
+
+long long __aeabi_d2lz(aeabi_double_t a)
+{
+  return f64_to_i64_r_minMag(f64_from_d(a), false);
+}
+
+unsigned long long __aeabi_d2ulz(aeabi_double_t a)
+{
+  return f64_to_ui64_r_minMag(f64_from_d(a), false);
+}
+
+int __aeabi_f2iz(aeabi_float_t a)
+{
+  return f32_to_i32_r_minMag(f32_from_f(a), false);
+}
+
+unsigned __aeabi_f2uiz(aeabi_float_t a)
+{
+  return f32_to_ui32_r_minMag(f32_from_f(a), false);
+}
+
+long long __aeabi_f2lz(aeabi_float_t a)
+{
+  return f32_to_i64_r_minMag(f32_from_f(a), false);
+}
+
+unsigned long long __aeabi_f2ulz(aeabi_float_t a)
+{
+  return f32_to_ui64_r_minMag(f32_from_f(a), false);
+}
+
+/*
+ * Table 7, Standard conversions between floating types
+ */
+
+aeabi_float_t __aeabi_d2f(aeabi_double_t a)
+{
+  return f32_to_f(f64_to_f32(f64_from_d(a)));
+}
+
+aeabi_double_t __aeabi_f2d(aeabi_float_t a)
+{
+  return f64_to_d(f32_to_f64(f32_from_f(a)));
+}
+
+/*
+ * Table 8, Standard integer to floating-point conversions
+ */
+
+aeabi_double_t __aeabi_i2d(int a)
+{
+  return f64_to_d(i32_to_f64(a));
+}
+
+aeabi_double_t __aeabi_ui2d(unsigned a)
+{
+  return f64_to_d(ui32_to_f64(a));
+}
+
+aeabi_double_t __aeabi_l2d(long long a)
+{
+  return f64_to_d(i64_to_f64(a));
+}
+
+aeabi_double_t __aeabi_ul2d(unsigned long long a)
+{
+  return f64_to_d(ui64_to_f64(a));
+}
+
+aeabi_float_t __aeabi_i2f(int a)
+{
+  return f32_to_f(i32_to_f32(a));
+}
+
+aeabi_float_t __aeabi_ui2f(unsigned a)
+{
+  return f32_to_f(ui32_to_f32(a));
+}
+
+aeabi_float_t __aeabi_l2f(long long a)
+{
+  return f32_to_f(i64_to_f32(a));
+}
+
+aeabi_float_t __aeabi_ul2f(unsigned long long a)
+{
+  return f32_to_f(ui64_to_f32(a));
+}
diff --git a/ArmPkg/Library/CompilerIntrinsicsLib/memset.c b/ArmPkg/Library/CompilerIntrinsicsLib/memset.c
index c5ae32e5ee71..24398d591f79 100644
--- a/ArmPkg/Library/CompilerIntrinsicsLib/memset.c
+++ b/ArmPkg/Library/CompilerIntrinsicsLib/memset.c
@@ -1,55 +1,55 @@
-//------------------------------------------------------------------------------
-//
-// Copyright (c) 2016, Linaro Ltd. All rights reserved.<BR>
-//
-// SPDX-License-Identifier: BSD-2-Clause-Patent
-//
-//------------------------------------------------------------------------------
-
-typedef __SIZE_TYPE__ size_t;
-
-static __attribute__((__used__))
-void *__memset(void *s, int c, size_t n)
-{
-  unsigned char *d = s;
-
-  while (n--)
-    *d++ = c;
-
-  return s;
-}
-
-//
-// Other modules (such as CryptoPkg/IntrinsicLib) may provide another
-// implementation of memset(), which may conflict with this one if this
-// object was pulled into the link due to the definitions below. So make
-// our memset() 'weak' to let the other implementation take precedence.
-//
-__attribute__((__weak__, __alias__("__memset")))
-void *memset(void *dest, int c, size_t n);
-
-#ifdef __arm__
-
-void __aeabi_memset(void *dest, size_t n, int c)
-{
-  __memset(dest, c, n);
-}
-
-__attribute__((__alias__("__aeabi_memset")))
-void __aeabi_memset4(void *dest, size_t n, int c);
-
-__attribute__((__alias__("__aeabi_memset")))
-void __aeabi_memset8(void *dest, size_t n, int c);
-
-void __aeabi_memclr(void *dest, size_t n)
-{
-  __memset(dest, 0, n);
-}
-
-__attribute__((__alias__("__aeabi_memclr")))
-void __aeabi_memclr4(void *dest, size_t n);
-
-__attribute__((__alias__("__aeabi_memclr")))
-void __aeabi_memclr8(void *dest, size_t n);
-
-#endif
+//------------------------------------------------------------------------------
+//
+// Copyright (c) 2016, Linaro Ltd. All rights reserved.<BR>
+//
+// SPDX-License-Identifier: BSD-2-Clause-Patent
+//
+//------------------------------------------------------------------------------
+
+typedef __SIZE_TYPE__ size_t;
+
+static __attribute__((__used__))
+void *__memset(void *s, int c, size_t n)
+{
+  unsigned char *d = s;
+
+  while (n--)
+    *d++ = c;
+
+  return s;
+}
+
+//
+// Other modules (such as CryptoPkg/IntrinsicLib) may provide another
+// implementation of memset(), which may conflict with this one if this
+// object was pulled into the link due to the definitions below. So make
+// our memset() 'weak' to let the other implementation take precedence.
+//
+__attribute__((__weak__, __alias__("__memset")))
+void *memset(void *dest, int c, size_t n);
+
+#ifdef __arm__
+
+void __aeabi_memset(void *dest, size_t n, int c)
+{
+  __memset(dest, c, n);
+}
+
+__attribute__((__alias__("__aeabi_memset")))
+void __aeabi_memset4(void *dest, size_t n, int c);
+
+__attribute__((__alias__("__aeabi_memset")))
+void __aeabi_memset8(void *dest, size_t n, int c);
+
+void __aeabi_memclr(void *dest, size_t n)
+{
+  __memset(dest, 0, n);
+}
+
+__attribute__((__alias__("__aeabi_memclr")))
+void __aeabi_memclr4(void *dest, size_t n);
+
+__attribute__((__alias__("__aeabi_memclr")))
+void __aeabi_memclr8(void *dest, size_t n);
+
+#endif
diff --git a/ArmPkg/Library/GccLto/liblto-aarch64.s b/ArmPkg/Library/GccLto/liblto-aarch64.s
index 66819534a045..02a55ef4456c 100644
--- a/ArmPkg/Library/GccLto/liblto-aarch64.s
+++ b/ArmPkg/Library/GccLto/liblto-aarch64.s
@@ -1,21 +1,21 @@
-//
-// Copyright (c) 2016, Linaro Ltd. All rights reserved.<BR>
-//
-// SPDX-License-Identifier: BSD-2-Clause-Patent
-//
-
-//
-// GCC in LTO mode interoperates poorly with non-standard libraries that
-// provide implementations of compiler intrinsics such as memcpy/memset
-// or the stack protector entry points.
-//
-// By referencing these functions from a non-LTO object that can be passed
-// to the linker via the -plugin-opt=-pass-through=-lxxx options, the
-// intrinsics are included in the link in a way that allows them to be
-// pruned again if no other references to them exist.
-//
-
-	.long	memcpy - .
-	.long	memset - .
-	.long	__stack_chk_fail - .
-	.long	__stack_chk_guard - .
+//
+// Copyright (c) 2016, Linaro Ltd. All rights reserved.<BR>
+//
+// SPDX-License-Identifier: BSD-2-Clause-Patent
+//
+
+//
+// GCC in LTO mode interoperates poorly with non-standard libraries that
+// provide implementations of compiler intrinsics such as memcpy/memset
+// or the stack protector entry points.
+//
+// By referencing these functions from a non-LTO object that can be passed
+// to the linker via the -plugin-opt=-pass-through=-lxxx options, the
+// intrinsics are included in the link in a way that allows them to be
+// pruned again if no other references to them exist.
+//
+
+  .long memcpy - .
+  .long memset - .
+  .long __stack_chk_fail - .
+  .long __stack_chk_guard - .
diff --git a/ArmPkg/Library/GccLto/liblto-arm.s b/ArmPkg/Library/GccLto/liblto-arm.s
index 4b26d4320d52..f19fb455519d 100644
--- a/ArmPkg/Library/GccLto/liblto-arm.s
+++ b/ArmPkg/Library/GccLto/liblto-arm.s
@@ -1,55 +1,55 @@
-//
-// Copyright (c) 2016, Linaro Ltd. All rights reserved.<BR>
-//
-// SPDX-License-Identifier: BSD-2-Clause-Patent
-//
-
-//
-// GCC in LTO mode interoperates poorly with non-standard libraries that
-// provide implementations of compiler intrinsics such as memcpy/memset
-// or the stack protector entry points.
-//
-// By referencing these functions from a non-LTO object that can be passed
-// to the linker via the -plugin-opt=-pass-through=-lxxx options, the
-// intrinsics are included in the link in a way that allows them to be
-// pruned again if no other references to them exist.
-//
-
-	.long	memcpy - .
-	.long	memset - .
-	.long	__stack_chk_fail - .
-	.long	__stack_chk_guard - .
-	.long __ashrdi3 - .
-	.long __ashldi3 - .
-	.long __aeabi_idiv - .
-	.long __aeabi_idivmod - .
-	.long __aeabi_uidiv - .
-	.long __aeabi_uidivmod - .
-	.long __divdi3 - .
-	.long __divsi3 - .
-	.long __lshrdi3 - .
-	.long __aeabi_memcpy - .
-	.long __aeabi_memset - .
-	.long memmove - .
-	.long __modsi3 - .
-	.long __moddi3 - .
-	.long __muldi3 - .
-	.long __aeabi_lmul - .
-	.long __ARM_ll_mullu - .
-	.long __udivsi3 - .
-	.long __umodsi3 - .
-	.long __udivdi3 - .
-	.long __umoddi3 - .
-	.long __udivmoddi4 - .
-	.long __clzsi2 - .
-	.long __ctzsi2 - .
-	.long __ucmpdi2 - .
-	.long __switch8 - .
-	.long __switchu8 - .
-	.long __switch16 - .
-	.long __switch32 - .
-	.long __aeabi_ulcmp - .
-	.long __aeabi_uldivmod - .
-	.long __aeabi_ldivmod - .
-	.long __aeabi_llsr - .
-	.long __aeabi_llsl - .
+//
+// Copyright (c) 2016, Linaro Ltd. All rights reserved.<BR>
+//
+// SPDX-License-Identifier: BSD-2-Clause-Patent
+//
+
+//
+// GCC in LTO mode interoperates poorly with non-standard libraries that
+// provide implementations of compiler intrinsics such as memcpy/memset
+// or the stack protector entry points.
+//
+// By referencing these functions from a non-LTO object that can be passed
+// to the linker via the -plugin-opt=-pass-through=-lxxx options, the
+// intrinsics are included in the link in a way that allows them to be
+// pruned again if no other references to them exist.
+//
+
+  .long memcpy - .
+  .long memset - .
+  .long __stack_chk_fail - .
+  .long __stack_chk_guard - .
+  .long __ashrdi3 - .
+  .long __ashldi3 - .
+  .long __aeabi_idiv - .
+  .long __aeabi_idivmod - .
+  .long __aeabi_uidiv - .
+  .long __aeabi_uidivmod - .
+  .long __divdi3 - .
+  .long __divsi3 - .
+  .long __lshrdi3 - .
+  .long __aeabi_memcpy - .
+  .long __aeabi_memset - .
+  .long memmove - .
+  .long __modsi3 - .
+  .long __moddi3 - .
+  .long __muldi3 - .
+  .long __aeabi_lmul - .
+  .long __ARM_ll_mullu - .
+  .long __udivsi3 - .
+  .long __umodsi3 - .
+  .long __udivdi3 - .
+  .long __umoddi3 - .
+  .long __udivmoddi4 - .
+  .long __clzsi2 - .
+  .long __ctzsi2 - .
+  .long __ucmpdi2 - .
+  .long __switch8 - .
+  .long __switchu8 - .
+  .long __switch16 - .
+  .long __switch32 - .
+  .long __aeabi_ulcmp - .
+  .long __aeabi_uldivmod - .
+  .long __aeabi_ldivmod - .
+  .long __aeabi_llsr - .
+  .long __aeabi_llsl - .
-- 
2.19.1.3.g30247aa5d201



^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 2/3] ArmVirtPkg: convert LFs to CRLF
  2020-02-27 21:39 [PATCH 0/3] Arm*Pkg: convert LFs to CRLF, expand hard TABs Laszlo Ersek
  2020-02-27 21:39 ` [PATCH 1/3] ArmPkg: " Laszlo Ersek
@ 2020-02-27 21:39 ` Laszlo Ersek
  2020-02-27 21:39 ` [PATCH 3/3] ArmPlatformPkg: convert LFs to CRLF, expand hard TABs Laszlo Ersek
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Laszlo Ersek @ 2020-02-27 21:39 UTC (permalink / raw)
  To: edk2-devel-groups-io
  Cc: Ard Biesheuvel, Leif Lindholm, Philippe Mathieu-Daudé

We're going to switch the internal line terminators globally to LF at some
point, but until then, let's use CRLF consistently. Convert source files
with LFs in them to CRLF, using "unix2dos".

"git show -b" prints no code changes for this patch.

(I collected all the file name suffixes in this package, with:

$ git ls-files -- $PACKAGE | rev | cut -f 1 -d . | sort -u | rev

I eliminated those suffixes that didn't stand for text files, then
blanket-converted the rest with unix2dos. Finally, picked up the actual
changes with git-add.)

The CRLF conversion is motivated by "PatchCheck.py".

Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1659
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
---
 ArmVirtPkg/Include/Platform/Hidden.h | 44 ++++++++++----------
 1 file changed, 22 insertions(+), 22 deletions(-)

diff --git a/ArmVirtPkg/Include/Platform/Hidden.h b/ArmVirtPkg/Include/Platform/Hidden.h
index c69cc662fa62..7a7bdb42b8bd 100644
--- a/ArmVirtPkg/Include/Platform/Hidden.h
+++ b/ArmVirtPkg/Include/Platform/Hidden.h
@@ -1,22 +1,22 @@
-/** @file
-
-  Copyright (c) 2018, Linaro Limited. All rights reserved.
-
-  SPDX-License-Identifier: BSD-2-Clause-Patent
-
-**/
-
-#ifndef __PLATFORM_HIDDEN_H
-#define __PLATFORM_HIDDEN_H
-
-//
-// Setting the GCC -fvisibility=hidden command line option is not quite the same
-// as setting the pragma below: the former only affects definitions, whereas the
-// pragma affects extern declarations as well. So if we want to ensure that no
-// GOT indirected symbol references are emitted, we need to use the pragma, or
-// GOT based cross object references could be emitted, e.g., in libraries, and
-// these cannot be relaxed to ordinary symbol references at link time.
-//
-#pragma GCC visibility push (hidden)
-
-#endif
+/** @file
+
+  Copyright (c) 2018, Linaro Limited. All rights reserved.
+
+  SPDX-License-Identifier: BSD-2-Clause-Patent
+
+**/
+
+#ifndef __PLATFORM_HIDDEN_H
+#define __PLATFORM_HIDDEN_H
+
+//
+// Setting the GCC -fvisibility=hidden command line option is not quite the same
+// as setting the pragma below: the former only affects definitions, whereas the
+// pragma affects extern declarations as well. So if we want to ensure that no
+// GOT indirected symbol references are emitted, we need to use the pragma, or
+// GOT based cross object references could be emitted, e.g., in libraries, and
+// these cannot be relaxed to ordinary symbol references at link time.
+//
+#pragma GCC visibility push (hidden)
+
+#endif
-- 
2.19.1.3.g30247aa5d201



^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 3/3] ArmPlatformPkg: convert LFs to CRLF, expand hard TABs
  2020-02-27 21:39 [PATCH 0/3] Arm*Pkg: convert LFs to CRLF, expand hard TABs Laszlo Ersek
  2020-02-27 21:39 ` [PATCH 1/3] ArmPkg: " Laszlo Ersek
  2020-02-27 21:39 ` [PATCH 2/3] ArmVirtPkg: convert LFs to CRLF Laszlo Ersek
@ 2020-02-27 21:39 ` Laszlo Ersek
  2020-02-27 22:06 ` [PATCH 0/3] Arm*Pkg: " Ard Biesheuvel
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Laszlo Ersek @ 2020-02-27 21:39 UTC (permalink / raw)
  To: edk2-devel-groups-io
  Cc: Ard Biesheuvel, Leif Lindholm, Philippe Mathieu-Daudé

We're going to switch the internal line terminators globally to LF at some
point, but until then, let's use CRLF consistently. Convert source files
with LFs in them to CRLF, using "unix2dos".

"git show -b" prints no code changes for this patch.

(I collected all the file name suffixes in this package, with:

$ git ls-files -- $PACKAGE | rev | cut -f 1 -d . | sort -u | rev

I eliminated those suffixes that didn't stand for text files, then
blanket-converted the rest with unix2dos. Finally, picked up the actual
changes with git-add.)

At the same time, the following file had to undergo TAB expansion:

  ArmPlatformPkg/Scripts/Ds5/profile.py

I used "expand -t 4", conforming to the Indentation section of PEP-8
<https://www.python.org/dev/peps/pep-0008/#indentation>.

Both the CRLF conversion and the TAB expansion are motivated by
"PatchCheck.py". "PatchCheck.py" is also the reason why CRLF conversion
and TAB expansion have to happen in the same patch.

Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1659
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
---
 ArmPlatformPkg/Scripts/Ds5/profile.py | 656 ++++++++++----------
 1 file changed, 328 insertions(+), 328 deletions(-)

diff --git a/ArmPlatformPkg/Scripts/Ds5/profile.py b/ArmPlatformPkg/Scripts/Ds5/profile.py
index f87dee24695c..979c6ea2bdc2 100644
--- a/ArmPlatformPkg/Scripts/Ds5/profile.py
+++ b/ArmPlatformPkg/Scripts/Ds5/profile.py
@@ -1,328 +1,328 @@
-#!/usr/bin/python
-
-#
-#  Copyright (c) 2014, ARM Limited. All rights reserved.
-#
-#  SPDX-License-Identifier: BSD-2-Clause-Patent
-#
-
-import getopt
-import operator
-import os
-import pickle
-import sys
-from sys import argv
-from cStringIO import StringIO
-
-modules = {}
-functions = {}
-functions_addr = {}
-
-def usage():
-	print "-t,--trace: Location of the Trace file"
-	print "-s,--symbols: Location of the symbols and modules"
-
-def get_address_from_string(address):
-	return int(address.strip("S:").strip("N:").strip("EL2:").strip("EL1:"), 16)
-
-def get_module_from_addr(modules, addr):
-	for key,value in modules.items():
-		if (value['start'] <= addr) and (addr <= value['end']):
-			return key
-	return None
-
-def add_cycles_to_function(functions, func_name, addr, cycles):
-	if func_name != "<Unknown>":
-		# Check if we are still in the previous function
-		if add_cycles_to_function.prev_func_name == func_name:
-			add_cycles_to_function.prev_entry['cycles'] += cycles
-			return (add_cycles_to_function.prev_func_name, add_cycles_to_function.prev_module_name)
-
-		if func_name in functions.keys():
-			for module_name, module_value in functions[func_name].iteritems():
-				if (module_value['start'] <= addr) and (addr < module_value['end']):
-					module_value['cycles'] += cycles
-
-					add_cycles_to_function.prev_func_name   = func_name
-					add_cycles_to_function.prev_module_name = module_name
-					add_cycles_to_function.prev_entry       = module_value
-					return (func_name, module_name)
-				elif (module_value['end'] == 0):
-					module_value['cycles'] += cycles
-
-					add_cycles_to_function.prev_func_name   = func_name
-					add_cycles_to_function.prev_module_name = module_name
-					add_cycles_to_function.prev_entry       = module_value
-					return (func_name, module_name)
-
-		# Workaround to fix the 'info func' limitation that does not expose the 'static' function
-		module_name = get_module_from_addr(modules, addr)
-		functions[func_name] = {}
-		functions[func_name][module_name] = {}
-		functions[func_name][module_name]['start']  = 0
-		functions[func_name][module_name]['end']    = 0
-		functions[func_name][module_name]['cycles'] = cycles
-		functions[func_name][module_name]['count']  = 0
-
-		add_cycles_to_function.prev_func_name   = func_name
-		add_cycles_to_function.prev_module_name = module_name
-		add_cycles_to_function.prev_entry       = functions[func_name][module_name]
-		return (func_name, module_name)
-	else:
-		# Check if we are still in the previous function
-		if (add_cycles_to_function.prev_entry is not None) and (add_cycles_to_function.prev_entry['start'] <= addr) and (addr < add_cycles_to_function.prev_entry['end']):
-			add_cycles_to_function.prev_entry['cycles'] += cycles
-			return (add_cycles_to_function.prev_func_name, add_cycles_to_function.prev_module_name)
-
-		# Generate the key for the given address
-		key = addr & ~0x0FFF
-
-		if key not in functions_addr.keys():
-			if 'Unknown' not in functions.keys():
-				functions['Unknown'] = {}
-			if 'Unknown' not in functions['Unknown'].keys():
-				functions['Unknown']['Unknown'] = {}
-				functions['Unknown']['Unknown']['cycles'] = 0
-				functions['Unknown']['Unknown']['count'] = 0
-			functions['Unknown']['Unknown']['cycles'] += cycles
-
-			add_cycles_to_function.prev_func_name = None
-			return None
-
-		for func_key, module in functions_addr[key].iteritems():
-			for module_key, module_value in module.iteritems():
-				if (module_value['start'] <= addr) and (addr < module_value['end']):
-					module_value['cycles'] += cycles
-
-					# In case o <Unknown> we prefer to fallback on the direct search
-					add_cycles_to_function.prev_func_name   = func_key
-					add_cycles_to_function.prev_module_name = module_key
-					add_cycles_to_function.prev_entry       = module_value
-					return (func_key, module_key)
-
-	print "Warning: Function %s @ 0x%x not found" % (func_name, addr)
-
-	add_cycles_to_function.prev_func_name = None
-	return None
-
-# Static variables for the previous function
-add_cycles_to_function.prev_func_name = None
-add_cycles_to_function.prev_entry     = None
-
-def trace_read():
-	global trace_process
-	line = trace.readline()
-	trace_process += len(line)
-	return line
-
-#
-# Parse arguments
-#
-trace_name = None
-symbols_file = None
-
-opts,args = getopt.getopt(sys.argv[1:], "ht:vs:v", ["help","trace=","symbols="])
-if (opts is None) or (not opts):
-	usage()
-	sys.exit()
-
-for o,a in opts:
-    if o in ("-h","--help"):
-        usage()
-        sys.exit()
-    elif o in ("-t","--trace"):
-        trace_name = a
-    elif o in ("-s","--symbols"):
-        symbols_file = a
-    else:
-        assert False, "Unhandled option (%s)" % o
-
-#
-# We try first to see if we run the script from DS-5
-#
-try:
-	from arm_ds.debugger_v1 import Debugger
-	from arm_ds.debugger_v1 import DebugException
-
-	# Debugger object for accessing the debugger
-	debugger = Debugger()
-
-	# Initialisation commands
-	ec = debugger.getExecutionContext(0)
-	ec.getExecutionService().stop()
-	ec.getExecutionService().waitForStop()
-	# in case the execution context reference is out of date
-	ec = debugger.getExecutionContext(0)
-
-	#
-	# Get the module name and their memory range
-	#
-	info_file = ec.executeDSCommand("info file")
-	info_file_str = StringIO(info_file)
-
-	line = info_file_str.readline().strip('\n')
-	while line != '':
-		if ("Symbols from" in line):
-			# Get the module name from the line 'Symbols from "/home/...."'
-			module_name = line.split("\"")[1].split("/")[-1]
-			modules[module_name] = {}
-
-			# Look for the text section
-			line = info_file_str.readline().strip('\n')
-			while (line != '') and ("Symbols from" not in line):
-				if ("ER_RO" in line):
-					modules[module_name]['start'] = get_address_from_string(line.split()[0])
-					modules[module_name]['end']   = get_address_from_string(line.split()[2])
-					line = info_file_str.readline().strip('\n')
-					break;
-				if (".text" in line):
-					modules[module_name]['start'] = get_address_from_string(line.split()[0])
-					modules[module_name]['end']   = get_address_from_string(line.split()[2])
-					line = info_file_str.readline().strip('\n')
-					break;
-				line = info_file_str.readline().strip('\n')
-		line = info_file_str.readline().strip('\n')
-
-	#
-	# Get the function name and their memory range
-	#
-	info_func = ec.executeDSCommand("info func")
-	info_func_str = StringIO(info_func)
-
-	# Skip the first line 'Low-level symbols ...'
-	line = info_func_str.readline().strip('\n')
-	func_prev = None
-	while line != '':
-		# We ignore all the functions after 'Functions in'
-		if ("Functions in " in line):
-			line = info_func_str.readline().strip('\n')
-			while line != '':
-				line = info_func_str.readline().strip('\n')
-			line = info_func_str.readline().strip('\n')
-			continue
-
-		if ("Low-level symbols" in line):
-			# We need to fixup the last function of the module
-			if func_prev is not None:
-				func_prev['end'] = modules[module_name]['end']
-				func_prev = None
-
-			line = info_func_str.readline().strip('\n')
-			continue
-
-		func_name = line.split()[1]
-		func_start = get_address_from_string(line.split()[0])
-		module_name = get_module_from_addr(modules, func_start)
-
-		if func_name not in functions.keys():
-			functions[func_name] = {}
-		functions[func_name][module_name] = {}
-		functions[func_name][module_name]['start'] = func_start
-		functions[func_name][module_name]['cycles'] = 0
-		functions[func_name][module_name]['count'] = 0
-
-		# Set the end address of the previous function
-		if func_prev is not None:
-			func_prev['end'] = func_start
-		func_prev = functions[func_name][module_name]
-
-		line = info_func_str.readline().strip('\n')
-
-	# Fixup the last function
-	func_prev['end'] = modules[module_name]['end']
-
-	if symbols_file is not None:
-		pickle.dump((modules, functions), open(symbols_file, "w"))
-except:
-	if symbols_file is None:
-		print "Error: Symbols file is required when run out of ARM DS-5"
-		sys.exit()
-
-	(modules, functions) = pickle.load(open(symbols_file, "r"))
-
-#
-# Build optimized table for the <Unknown> functions
-#
-functions_addr = {}
-for func_key, module in functions.iteritems():
-	for module_key, module_value in module.iteritems():
-		key = module_value['start'] & ~0x0FFF
-		if key not in functions_addr.keys():
-			functions_addr[key] = {}
-		if func_key not in functions_addr[key].keys():
-			functions_addr[key][func_key] = {}
-		functions_addr[key][func_key][module_key] = module_value
-
-#
-# Process the trace file
-#
-if trace_name is None:
-	sys.exit()
-
-trace = open(trace_name, "r")
-trace_size = os.path.getsize(trace_name)
-trace_process = 0
-
-# Get the column names from the first line
-columns = trace_read().split()
-column_addr     = columns.index('Address')
-column_cycles   = columns.index('Cycles')
-column_function = columns.index('Function')
-
-line = trace_read()
-i = 0
-prev_callee = None
-while line:
-	try:
-		func_name = line.split('\t')[column_function].strip()
-		address   = get_address_from_string(line.split('\t')[column_addr])
-		cycles    = int(line.split('\t')[column_cycles])
-		callee = add_cycles_to_function(functions, func_name, address, cycles)
-		if (prev_callee != None) and (prev_callee != callee):
-			functions[prev_callee[0]][prev_callee[1]]['count'] += 1
-		prev_callee = callee
-	except ValueError:
-		pass
-	line = trace_read()
-	if ((i % 1000000) == 0) and (i != 0):
-		percent = (trace_process * 100.00) / trace_size
-		print "Processing file ... (%.2f %%)" % (percent)
-	i = i + 1
-
-# Fixup the last callee
-functions[prev_callee[0]][prev_callee[1]]['count'] += 1
-
-#
-# Process results
-#
-functions_cycles     = {}
-all_functions_cycles = {}
-total_cycles         = 0
-
-for func_key, module in functions.iteritems():
-	for module_key, module_value in module.iteritems():
-		key = "%s/%s" % (module_key, func_key)
-		functions_cycles[key] = (module_value['cycles'], module_value['count'])
-		total_cycles += module_value['cycles']
-
-		if func_key not in all_functions_cycles.keys():
-			all_functions_cycles[func_key] = (module_value['cycles'], module_value['count'])
-		else:
-			all_functions_cycles[func_key] = tuple(map(sum, zip(all_functions_cycles[func_key], (module_value['cycles'], module_value['count']))))
-
-sorted_functions_cycles     = sorted(functions_cycles.iteritems(), key=operator.itemgetter(1), reverse = True)
-sorted_all_functions_cycles = sorted(all_functions_cycles.items(), key=operator.itemgetter(1), reverse = True)
-
-print
-print "----"
-for (key,value) in sorted_functions_cycles[:20]:
-	if value[0] != 0:
-		print "%s (cycles: %d - %d%%, count: %d)" % (key, value[0], (value[0] * 100) / total_cycles, value[1])
-	else:
-		break;
-print "----"
-for (key,value) in sorted_all_functions_cycles[:20]:
-	if value[0] != 0:
-		print "%s (cycles: %d - %d%%, count: %d)" % (key, value[0], (value[0] * 100) / total_cycles, value[1])
-	else:
-		break;
+#!/usr/bin/python
+
+#
+#  Copyright (c) 2014, ARM Limited. All rights reserved.
+#
+#  SPDX-License-Identifier: BSD-2-Clause-Patent
+#
+
+import getopt
+import operator
+import os
+import pickle
+import sys
+from sys import argv
+from cStringIO import StringIO
+
+modules = {}
+functions = {}
+functions_addr = {}
+
+def usage():
+    print "-t,--trace: Location of the Trace file"
+    print "-s,--symbols: Location of the symbols and modules"
+
+def get_address_from_string(address):
+    return int(address.strip("S:").strip("N:").strip("EL2:").strip("EL1:"), 16)
+
+def get_module_from_addr(modules, addr):
+    for key,value in modules.items():
+        if (value['start'] <= addr) and (addr <= value['end']):
+            return key
+    return None
+
+def add_cycles_to_function(functions, func_name, addr, cycles):
+    if func_name != "<Unknown>":
+        # Check if we are still in the previous function
+        if add_cycles_to_function.prev_func_name == func_name:
+            add_cycles_to_function.prev_entry['cycles'] += cycles
+            return (add_cycles_to_function.prev_func_name, add_cycles_to_function.prev_module_name)
+
+        if func_name in functions.keys():
+            for module_name, module_value in functions[func_name].iteritems():
+                if (module_value['start'] <= addr) and (addr < module_value['end']):
+                    module_value['cycles'] += cycles
+
+                    add_cycles_to_function.prev_func_name   = func_name
+                    add_cycles_to_function.prev_module_name = module_name
+                    add_cycles_to_function.prev_entry       = module_value
+                    return (func_name, module_name)
+                elif (module_value['end'] == 0):
+                    module_value['cycles'] += cycles
+
+                    add_cycles_to_function.prev_func_name   = func_name
+                    add_cycles_to_function.prev_module_name = module_name
+                    add_cycles_to_function.prev_entry       = module_value
+                    return (func_name, module_name)
+
+        # Workaround to fix the 'info func' limitation that does not expose the 'static' function
+        module_name = get_module_from_addr(modules, addr)
+        functions[func_name] = {}
+        functions[func_name][module_name] = {}
+        functions[func_name][module_name]['start']  = 0
+        functions[func_name][module_name]['end']    = 0
+        functions[func_name][module_name]['cycles'] = cycles
+        functions[func_name][module_name]['count']  = 0
+
+        add_cycles_to_function.prev_func_name   = func_name
+        add_cycles_to_function.prev_module_name = module_name
+        add_cycles_to_function.prev_entry       = functions[func_name][module_name]
+        return (func_name, module_name)
+    else:
+        # Check if we are still in the previous function
+        if (add_cycles_to_function.prev_entry is not None) and (add_cycles_to_function.prev_entry['start'] <= addr) and (addr < add_cycles_to_function.prev_entry['end']):
+            add_cycles_to_function.prev_entry['cycles'] += cycles
+            return (add_cycles_to_function.prev_func_name, add_cycles_to_function.prev_module_name)
+
+        # Generate the key for the given address
+        key = addr & ~0x0FFF
+
+        if key not in functions_addr.keys():
+            if 'Unknown' not in functions.keys():
+                functions['Unknown'] = {}
+            if 'Unknown' not in functions['Unknown'].keys():
+                functions['Unknown']['Unknown'] = {}
+                functions['Unknown']['Unknown']['cycles'] = 0
+                functions['Unknown']['Unknown']['count'] = 0
+            functions['Unknown']['Unknown']['cycles'] += cycles
+
+            add_cycles_to_function.prev_func_name = None
+            return None
+
+        for func_key, module in functions_addr[key].iteritems():
+            for module_key, module_value in module.iteritems():
+                if (module_value['start'] <= addr) and (addr < module_value['end']):
+                    module_value['cycles'] += cycles
+
+                    # In case o <Unknown> we prefer to fallback on the direct search
+                    add_cycles_to_function.prev_func_name   = func_key
+                    add_cycles_to_function.prev_module_name = module_key
+                    add_cycles_to_function.prev_entry       = module_value
+                    return (func_key, module_key)
+
+    print "Warning: Function %s @ 0x%x not found" % (func_name, addr)
+
+    add_cycles_to_function.prev_func_name = None
+    return None
+
+# Static variables for the previous function
+add_cycles_to_function.prev_func_name = None
+add_cycles_to_function.prev_entry     = None
+
+def trace_read():
+    global trace_process
+    line = trace.readline()
+    trace_process += len(line)
+    return line
+
+#
+# Parse arguments
+#
+trace_name = None
+symbols_file = None
+
+opts,args = getopt.getopt(sys.argv[1:], "ht:vs:v", ["help","trace=","symbols="])
+if (opts is None) or (not opts):
+    usage()
+    sys.exit()
+
+for o,a in opts:
+    if o in ("-h","--help"):
+        usage()
+        sys.exit()
+    elif o in ("-t","--trace"):
+        trace_name = a
+    elif o in ("-s","--symbols"):
+        symbols_file = a
+    else:
+        assert False, "Unhandled option (%s)" % o
+
+#
+# We try first to see if we run the script from DS-5
+#
+try:
+    from arm_ds.debugger_v1 import Debugger
+    from arm_ds.debugger_v1 import DebugException
+
+    # Debugger object for accessing the debugger
+    debugger = Debugger()
+
+    # Initialisation commands
+    ec = debugger.getExecutionContext(0)
+    ec.getExecutionService().stop()
+    ec.getExecutionService().waitForStop()
+    # in case the execution context reference is out of date
+    ec = debugger.getExecutionContext(0)
+
+    #
+    # Get the module name and their memory range
+    #
+    info_file = ec.executeDSCommand("info file")
+    info_file_str = StringIO(info_file)
+
+    line = info_file_str.readline().strip('\n')
+    while line != '':
+        if ("Symbols from" in line):
+            # Get the module name from the line 'Symbols from "/home/...."'
+            module_name = line.split("\"")[1].split("/")[-1]
+            modules[module_name] = {}
+
+            # Look for the text section
+            line = info_file_str.readline().strip('\n')
+            while (line != '') and ("Symbols from" not in line):
+                if ("ER_RO" in line):
+                    modules[module_name]['start'] = get_address_from_string(line.split()[0])
+                    modules[module_name]['end']   = get_address_from_string(line.split()[2])
+                    line = info_file_str.readline().strip('\n')
+                    break;
+                if (".text" in line):
+                    modules[module_name]['start'] = get_address_from_string(line.split()[0])
+                    modules[module_name]['end']   = get_address_from_string(line.split()[2])
+                    line = info_file_str.readline().strip('\n')
+                    break;
+                line = info_file_str.readline().strip('\n')
+        line = info_file_str.readline().strip('\n')
+
+    #
+    # Get the function name and their memory range
+    #
+    info_func = ec.executeDSCommand("info func")
+    info_func_str = StringIO(info_func)
+
+    # Skip the first line 'Low-level symbols ...'
+    line = info_func_str.readline().strip('\n')
+    func_prev = None
+    while line != '':
+        # We ignore all the functions after 'Functions in'
+        if ("Functions in " in line):
+            line = info_func_str.readline().strip('\n')
+            while line != '':
+                line = info_func_str.readline().strip('\n')
+            line = info_func_str.readline().strip('\n')
+            continue
+
+        if ("Low-level symbols" in line):
+            # We need to fixup the last function of the module
+            if func_prev is not None:
+                func_prev['end'] = modules[module_name]['end']
+                func_prev = None
+
+            line = info_func_str.readline().strip('\n')
+            continue
+
+        func_name = line.split()[1]
+        func_start = get_address_from_string(line.split()[0])
+        module_name = get_module_from_addr(modules, func_start)
+
+        if func_name not in functions.keys():
+            functions[func_name] = {}
+        functions[func_name][module_name] = {}
+        functions[func_name][module_name]['start'] = func_start
+        functions[func_name][module_name]['cycles'] = 0
+        functions[func_name][module_name]['count'] = 0
+
+        # Set the end address of the previous function
+        if func_prev is not None:
+            func_prev['end'] = func_start
+        func_prev = functions[func_name][module_name]
+
+        line = info_func_str.readline().strip('\n')
+
+    # Fixup the last function
+    func_prev['end'] = modules[module_name]['end']
+
+    if symbols_file is not None:
+        pickle.dump((modules, functions), open(symbols_file, "w"))
+except:
+    if symbols_file is None:
+        print "Error: Symbols file is required when run out of ARM DS-5"
+        sys.exit()
+
+    (modules, functions) = pickle.load(open(symbols_file, "r"))
+
+#
+# Build optimized table for the <Unknown> functions
+#
+functions_addr = {}
+for func_key, module in functions.iteritems():
+    for module_key, module_value in module.iteritems():
+        key = module_value['start'] & ~0x0FFF
+        if key not in functions_addr.keys():
+            functions_addr[key] = {}
+        if func_key not in functions_addr[key].keys():
+            functions_addr[key][func_key] = {}
+        functions_addr[key][func_key][module_key] = module_value
+
+#
+# Process the trace file
+#
+if trace_name is None:
+    sys.exit()
+
+trace = open(trace_name, "r")
+trace_size = os.path.getsize(trace_name)
+trace_process = 0
+
+# Get the column names from the first line
+columns = trace_read().split()
+column_addr     = columns.index('Address')
+column_cycles   = columns.index('Cycles')
+column_function = columns.index('Function')
+
+line = trace_read()
+i = 0
+prev_callee = None
+while line:
+    try:
+        func_name = line.split('\t')[column_function].strip()
+        address   = get_address_from_string(line.split('\t')[column_addr])
+        cycles    = int(line.split('\t')[column_cycles])
+        callee = add_cycles_to_function(functions, func_name, address, cycles)
+        if (prev_callee != None) and (prev_callee != callee):
+            functions[prev_callee[0]][prev_callee[1]]['count'] += 1
+        prev_callee = callee
+    except ValueError:
+        pass
+    line = trace_read()
+    if ((i % 1000000) == 0) and (i != 0):
+        percent = (trace_process * 100.00) / trace_size
+        print "Processing file ... (%.2f %%)" % (percent)
+    i = i + 1
+
+# Fixup the last callee
+functions[prev_callee[0]][prev_callee[1]]['count'] += 1
+
+#
+# Process results
+#
+functions_cycles     = {}
+all_functions_cycles = {}
+total_cycles         = 0
+
+for func_key, module in functions.iteritems():
+    for module_key, module_value in module.iteritems():
+        key = "%s/%s" % (module_key, func_key)
+        functions_cycles[key] = (module_value['cycles'], module_value['count'])
+        total_cycles += module_value['cycles']
+
+        if func_key not in all_functions_cycles.keys():
+            all_functions_cycles[func_key] = (module_value['cycles'], module_value['count'])
+        else:
+            all_functions_cycles[func_key] = tuple(map(sum, zip(all_functions_cycles[func_key], (module_value['cycles'], module_value['count']))))
+
+sorted_functions_cycles     = sorted(functions_cycles.iteritems(), key=operator.itemgetter(1), reverse = True)
+sorted_all_functions_cycles = sorted(all_functions_cycles.items(), key=operator.itemgetter(1), reverse = True)
+
+print
+print "----"
+for (key,value) in sorted_functions_cycles[:20]:
+    if value[0] != 0:
+        print "%s (cycles: %d - %d%%, count: %d)" % (key, value[0], (value[0] * 100) / total_cycles, value[1])
+    else:
+        break;
+print "----"
+for (key,value) in sorted_all_functions_cycles[:20]:
+    if value[0] != 0:
+        print "%s (cycles: %d - %d%%, count: %d)" % (key, value[0], (value[0] * 100) / total_cycles, value[1])
+    else:
+        break;
-- 
2.19.1.3.g30247aa5d201


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH 0/3] Arm*Pkg: convert LFs to CRLF, expand hard TABs
  2020-02-27 21:39 [PATCH 0/3] Arm*Pkg: convert LFs to CRLF, expand hard TABs Laszlo Ersek
                   ` (2 preceding siblings ...)
  2020-02-27 21:39 ` [PATCH 3/3] ArmPlatformPkg: convert LFs to CRLF, expand hard TABs Laszlo Ersek
@ 2020-02-27 22:06 ` Ard Biesheuvel
  2020-02-28  9:26 ` Philippe Mathieu-Daudé
  2020-03-04 12:45 ` [edk2-devel] " Laszlo Ersek
  5 siblings, 0 replies; 7+ messages in thread
From: Ard Biesheuvel @ 2020-02-27 22:06 UTC (permalink / raw)
  To: Laszlo Ersek
  Cc: edk2-devel-groups-io, Leif Lindholm, Philippe Mathieu-Daudé

On Thu, 27 Feb 2020 at 22:39, Laszlo Ersek <lersek@redhat.com> wrote:
>
> Bugzilla: https://bugzilla.tianocore.org/show_bug.cgi?id=1659
> Repo:     https://github.com/lersek/edk2.git
> Branch:   crlf_bz_1659
>
> Future patches for some Arm*Pkg source files would either introduce CRLF
> lines into LF files, or add more LF lines. The first would increase
> inconsistency, the second would make "PatchCheck.py" yell. Convert these
> files to CRLF uniformly (and while at it, expand hard TABS, because
> "PatchCheck.py" catches those too).
>
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
>
> Thanks,
> Laszlo
>
> Laszlo Ersek (3):
>   ArmPkg: convert LFs to CRLF, expand hard TABs
>   ArmVirtPkg: convert LFs to CRLF
>   ArmPlatformPkg: convert LFs to CRLF, expand hard TABs
>

For the series,

Acked-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>

>  ArmPkg/Library/ArmSmcLibNull/ArmSmcLibNull.c     |  32 +-
>  ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c | 566 ++++++++---------
>  ArmPkg/Library/CompilerIntrinsicsLib/memset.c    | 110 ++--
>  ArmPkg/Library/GccLto/liblto-aarch64.s           |  42 +-
>  ArmPkg/Library/GccLto/liblto-arm.s               | 110 ++--
>  ArmPlatformPkg/Scripts/Ds5/profile.py            | 656 ++++++++++----------
>  ArmVirtPkg/Include/Platform/Hidden.h             |  44 +-
>  7 files changed, 780 insertions(+), 780 deletions(-)
>
> --
> 2.19.1.3.g30247aa5d201
>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 0/3] Arm*Pkg: convert LFs to CRLF, expand hard TABs
  2020-02-27 21:39 [PATCH 0/3] Arm*Pkg: convert LFs to CRLF, expand hard TABs Laszlo Ersek
                   ` (3 preceding siblings ...)
  2020-02-27 22:06 ` [PATCH 0/3] Arm*Pkg: " Ard Biesheuvel
@ 2020-02-28  9:26 ` Philippe Mathieu-Daudé
  2020-03-04 12:45 ` [edk2-devel] " Laszlo Ersek
  5 siblings, 0 replies; 7+ messages in thread
From: Philippe Mathieu-Daudé @ 2020-02-28  9:26 UTC (permalink / raw)
  To: Laszlo Ersek, edk2-devel-groups-io; +Cc: Ard Biesheuvel, Leif Lindholm

On 2/27/20 10:39 PM, Laszlo Ersek wrote:
> Bugzilla: https://bugzilla.tianocore.org/show_bug.cgi?id=1659
> Repo:     https://github.com/lersek/edk2.git
> Branch:   crlf_bz_1659
> 
> Future patches for some Arm*Pkg source files would either introduce CRLF
> lines into LF files, or add more LF lines. The first would increase
> inconsistency, the second would make "PatchCheck.py" yell. Convert these
> files to CRLF uniformly (and while at it, expand hard TABS, because
> "PatchCheck.py" catches those too).
> 
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
> 
> Thanks,
> Laszlo
> 
> Laszlo Ersek (3):
>    ArmPkg: convert LFs to CRLF, expand hard TABs
>    ArmVirtPkg: convert LFs to CRLF
>    ArmPlatformPkg: convert LFs to CRLF, expand hard TABs
> 
>   ArmPkg/Library/ArmSmcLibNull/ArmSmcLibNull.c     |  32 +-
>   ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c | 566 ++++++++---------
>   ArmPkg/Library/CompilerIntrinsicsLib/memset.c    | 110 ++--
>   ArmPkg/Library/GccLto/liblto-aarch64.s           |  42 +-
>   ArmPkg/Library/GccLto/liblto-arm.s               | 110 ++--
>   ArmPlatformPkg/Scripts/Ds5/profile.py            | 656 ++++++++++----------
>   ArmVirtPkg/Include/Platform/Hidden.h             |  44 +-
>   7 files changed, 780 insertions(+), 780 deletions(-)
> 

Series:
Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [edk2-devel] [PATCH 0/3] Arm*Pkg: convert LFs to CRLF, expand hard TABs
  2020-02-27 21:39 [PATCH 0/3] Arm*Pkg: convert LFs to CRLF, expand hard TABs Laszlo Ersek
                   ` (4 preceding siblings ...)
  2020-02-28  9:26 ` Philippe Mathieu-Daudé
@ 2020-03-04 12:45 ` Laszlo Ersek
  5 siblings, 0 replies; 7+ messages in thread
From: Laszlo Ersek @ 2020-03-04 12:45 UTC (permalink / raw)
  To: edk2-devel-groups-io
  Cc: Ard Biesheuvel, Leif Lindholm, Philippe Mathieu-Daudé

On 02/27/20 22:39, Laszlo Ersek wrote:
> Bugzilla: https://bugzilla.tianocore.org/show_bug.cgi?id=1659
> Repo:     https://github.com/lersek/edk2.git
> Branch:   crlf_bz_1659
> 
> Future patches for some Arm*Pkg source files would either introduce CRLF
> lines into LF files, or add more LF lines. The first would increase
> inconsistency, the second would make "PatchCheck.py" yell. Convert these
> files to CRLF uniformly (and while at it, expand hard TABS, because
> "PatchCheck.py" catches those too).
> 
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
> 
> Thanks,
> Laszlo
> 
> Laszlo Ersek (3):
>   ArmPkg: convert LFs to CRLF, expand hard TABs
>   ArmVirtPkg: convert LFs to CRLF
>   ArmPlatformPkg: convert LFs to CRLF, expand hard TABs
> 
>  ArmPkg/Library/ArmSmcLibNull/ArmSmcLibNull.c     |  32 +-
>  ArmPkg/Library/ArmSoftFloatLib/ArmSoftFloatLib.c | 566 ++++++++---------
>  ArmPkg/Library/CompilerIntrinsicsLib/memset.c    | 110 ++--
>  ArmPkg/Library/GccLto/liblto-aarch64.s           |  42 +-
>  ArmPkg/Library/GccLto/liblto-arm.s               | 110 ++--
>  ArmPlatformPkg/Scripts/Ds5/profile.py            | 656 ++++++++++----------
>  ArmVirtPkg/Include/Platform/Hidden.h             |  44 +-
>  7 files changed, 780 insertions(+), 780 deletions(-)
> 

Merged as commit range 1158fc8e2c7b..e63d54db9526, via
<https://github.com/tianocore/edk2/pull/416/>.

Thanks
Laszlo


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2020-03-04 12:45 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-02-27 21:39 [PATCH 0/3] Arm*Pkg: convert LFs to CRLF, expand hard TABs Laszlo Ersek
2020-02-27 21:39 ` [PATCH 1/3] ArmPkg: " Laszlo Ersek
2020-02-27 21:39 ` [PATCH 2/3] ArmVirtPkg: convert LFs to CRLF Laszlo Ersek
2020-02-27 21:39 ` [PATCH 3/3] ArmPlatformPkg: convert LFs to CRLF, expand hard TABs Laszlo Ersek
2020-02-27 22:06 ` [PATCH 0/3] Arm*Pkg: " Ard Biesheuvel
2020-02-28  9:26 ` Philippe Mathieu-Daudé
2020-03-04 12:45 ` [edk2-devel] " Laszlo Ersek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox