This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


[Xen-changelog] [xen-4.0-testing] xenpm: remove wrong and pointless "cur

To: xen-changelog@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-changelog] [xen-4.0-testing] xenpm: remove wrong and pointless "current" indicator
From: "Xen patchbot-4.0-testing" <patchbot-4.0-testing@xxxxxxxxxxxxxxxxxxx>
Date: Fri, 14 May 2010 00:41:39 -0700
Delivery-date: Fri, 14 May 2010 00:47:32 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-changelog-request@lists.xensource.com?subject=help>
List-id: BK change log <xen-changelog.lists.xensource.com>
List-post: <mailto:xen-changelog@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-changelog>, <mailto:xen-changelog-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-changelog>, <mailto:xen-changelog-request@lists.xensource.com?subject=unsubscribe>
Reply-to: xen-devel@xxxxxxxxxxxxxxxxxxx
Sender: xen-changelog-bounces@xxxxxxxxxxxxxxxxxxx
# HG changeset patch
# User Keir Fraser <keir.fraser@xxxxxxxxxx>
# Date 1273481705 -3600
# Node ID 7404865abab684586bee827032df5c222d8eb184
# Parent  783cfba8867a01d8288234be6f94d740d981f136
xenpm: remove wrong and pointless "current" indicator

Using the CPU number to compare with an index into an array containing
only a subset of CPUs isn't valid. And indicator isn't necessary here
at all since the CPU number being dealt with gets printed right before
this line.

Signed-off-by: Jan Beulich <jbeulich@xxxxxxxxxx>
xen-unstable changeset:   21324:3b7228548a64
xen-unstable date:        Fri May 07 09:50:17 2010 +0100
 tools/misc/xenpm.c |    5 +----
 1 files changed, 1 insertion(+), 4 deletions(-)

diff -r 783cfba8867a -r 7404865abab6 tools/misc/xenpm.c
--- a/tools/misc/xenpm.c        Mon May 10 09:54:18 2010 +0100
+++ b/tools/misc/xenpm.c        Mon May 10 09:55:05 2010 +0100
@@ -495,10 +495,7 @@ static void print_cpufreq_para(int cpuid
     printf("affected_cpus        :");
     for ( i = 0; i < p_cpufreq->cpu_num; i++ )
-        if ( i == cpuid )
-            printf(" *%d", p_cpufreq->affected_cpus[i]);
-        else
-            printf(" %d", p_cpufreq->affected_cpus[i]);
+        printf(" %d", p_cpufreq->affected_cpus[i]);
     printf("cpuinfo frequency    : max [%u] min [%u] cur [%u]\n",

Xen-changelog mailing list

<Prev in Thread] Current Thread [Next in Thread>
  • [Xen-changelog] [xen-4.0-testing] xenpm: remove wrong and pointless "current" indicator, Xen patchbot-4.0-testing <=