From: Ben Elliston <bje@au.ibm.com>
Date: Thu, 15 Feb 2001 21:14:40 +0000 (+0000)
Subject: * profiling bug fixes.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=c43ad8eb1e91da3406b76c24f2711729320e03d8;p=binutils-gdb.git

* profiling bug fixes.

2001-02-09  Ben Elliston  <bje@redhat.com>

	* (profile_print_pc): Write header out in target byte order.

2001-02-09  Ben Elliston  <bje@redhat.com>

	* sim-profile.c (profile_pc_init): Correct bug in loop logic when
	adjusting the pc shift value.
---

diff --git a/sim/common/ChangeLog b/sim/common/ChangeLog
index 7f1fadfb81a..35830a9e5a8 100644
--- a/sim/common/ChangeLog
+++ b/sim/common/ChangeLog
@@ -1,3 +1,12 @@
+2001-02-09  Ben Elliston  <bje@redhat.com>
+
+	* (profile_print_pc): Write header out in target byte order.
+
+2001-02-09  Ben Elliston  <bje@redhat.com>
+
+	* sim-profile.c (profile_pc_init): Correct bug in loop logic when
+	adjusting the pc shift value.
+
 2001-01-12  Chris Demetriou  <cgd@sibyte.com>
 
 	* aclocal.m4 (SIM_AC_OPTION_SCACHE): Properly handle the case
diff --git a/sim/common/sim-profile.c b/sim/common/sim-profile.c
index 1ac148d1a9b..20d7a541f31 100644
--- a/sim/common/sim-profile.c
+++ b/sim/common/sim-profile.c
@@ -1,5 +1,5 @@
 /* Default profiling support.
-   Copyright (C) 1996, 1997, 1998, 2000 Free Software Foundation, Inc.
+   Copyright (C) 1996, 1997, 1998, 2000, 2001 Free Software Foundation, Inc.
    Contributed by Cygnus Support.
 
 This file is part of GDB, the GNU debugger.
@@ -564,7 +564,7 @@ profile_pc_init (SIM_DESC sd)
 				+ PROFILE_PC_NR_BUCKETS (data) - 1)
 			       / PROFILE_PC_NR_BUCKETS (data));
 	      PROFILE_PC_SHIFT (data) = 0;
-	      while (bucket_size < PROFILE_PC_BUCKET_SIZE (data))
+	      while (bucket_size > PROFILE_PC_BUCKET_SIZE (data))
 		{
 		  PROFILE_PC_SHIFT (data) += 1;
 		}
@@ -684,6 +684,12 @@ profile_print_pc (sim_cpu *cpu, int verbose)
 	  }
 	/* size of sample buffer (+ header) */
 	header[2] = PROFILE_PC_NR_BUCKETS (profile) * 2 + sizeof (header);
+
+	/* Header must be written out in target byte order.  */
+	H2T (header[0]);
+	H2T (header[1]);
+	H2T (header[2]);
+
 	ok = fwrite (&header, sizeof (header), 1, pf);
 	for (loop = 0;
 	     ok && (loop < PROFILE_PC_NR_BUCKETS (profile));