All of lore.kernel.org
 help / color / mirror / Atom feed
From: Don Slutz <Don@CloudSwitch.com>
To: <qemu-devel@nongnu.org>, <mtosatti@redhat.com>,
	<ehabkost@redhat.com>, <imammedo@redhat.com>, <avi@redhat.com>,
	<afaerber@suse.de>, <peter.maydell@linaro.org>,
	<kvm@vger.kernel.org>, <anthony@codemonkey.ws>
Cc: Don Slutz <Don@CloudSwitch.com>,
	Fred Oliveira <foliveira@cloudswitch.com>
Subject: [PATCH v3 01/17] target-i386: Allow tsc-frequency to be larger then 2.147G
Date: Mon, 17 Sep 2012 09:39:27 -0400	[thread overview]
Message-ID: <1347889189-9199-4-git-send-email-Don@CloudSwitch.com> (raw)
In-Reply-To: <1346354435-21685-1-git-send-email-Don@CloudSwitch.com>

The check using INT_MAX (2147483647) is wrong in this case.

Signed-off-by: Fred Oliveira <foliveira@cloudswitch.com>
Signed-off-by: Don Slutz <Don@CloudSwitch.com>
---
 target-i386/cpu.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/target-i386/cpu.c b/target-i386/cpu.c
index af50a8f..0313cf5 100644
--- a/target-i386/cpu.c
+++ b/target-i386/cpu.c
@@ -1146,7 +1146,7 @@ static void x86_cpuid_set_tsc_freq(Object *obj, Visitor *v, void *opaque,
 {
     X86CPU *cpu = X86_CPU(obj);
     const int64_t min = 0;
-    const int64_t max = INT_MAX;
+    const int64_t max = INT64_MAX;
     int64_t value;
 
     visit_type_freq(v, &value, name, errp);
-- 
1.7.1


WARNING: multiple messages have this Message-ID (diff)
From: Don Slutz <Don@CloudSwitch.com>
To: qemu-devel@nongnu.org, mtosatti@redhat.com, ehabkost@redhat.com,
	imammedo@redhat.com, avi@redhat.com, afaerber@suse.de,
	peter.maydell@linaro.org, kvm@vger.kernel.org,
	anthony@codemonkey.ws
Cc: Don Slutz <Don@CloudSwitch.com>,
	Fred Oliveira <foliveira@cloudswitch.com>
Subject: [Qemu-devel] [PATCH v3 01/17] target-i386: Allow tsc-frequency to be larger then 2.147G
Date: Mon, 17 Sep 2012 09:39:27 -0400	[thread overview]
Message-ID: <1347889189-9199-4-git-send-email-Don@CloudSwitch.com> (raw)
In-Reply-To: <1346354435-21685-1-git-send-email-Don@CloudSwitch.com>

The check using INT_MAX (2147483647) is wrong in this case.

Signed-off-by: Fred Oliveira <foliveira@cloudswitch.com>
Signed-off-by: Don Slutz <Don@CloudSwitch.com>
---
 target-i386/cpu.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/target-i386/cpu.c b/target-i386/cpu.c
index af50a8f..0313cf5 100644
--- a/target-i386/cpu.c
+++ b/target-i386/cpu.c
@@ -1146,7 +1146,7 @@ static void x86_cpuid_set_tsc_freq(Object *obj, Visitor *v, void *opaque,
 {
     X86CPU *cpu = X86_CPU(obj);
     const int64_t min = 0;
-    const int64_t max = INT_MAX;
+    const int64_t max = INT64_MAX;
     int64_t value;
 
     visit_type_freq(v, &value, name, errp);
-- 
1.7.1

  parent reply	other threads:[~2012-09-17 13:40 UTC|newest]

Thread overview: 130+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-30 19:20 [PATCH] target-i386: Allow changing of Hypervisor CPUIDs Don Slutz
2012-08-30 19:20 ` [Qemu-devel] " Don Slutz
2012-09-05 16:48 ` Marcelo Tosatti
2012-09-05 16:48   ` [Qemu-devel] " Marcelo Tosatti
2012-09-11 13:57   ` Don Slutz
2012-09-11 13:57     ` [Qemu-devel] " Don Slutz
2012-09-06 10:28 ` Andreas Färber
2012-09-06 10:28   ` Andreas Färber
2012-09-06 18:40   ` Eduardo Habkost
2012-09-06 18:40     ` Eduardo Habkost
2012-09-06 19:00     ` Don Slutz
2012-09-06 19:00       ` Don Slutz
2012-09-06 19:36       ` Eduardo Habkost
2012-09-06 19:36         ` Eduardo Habkost
2012-09-11 14:07 ` [PATCH v2 0/4] " Don Slutz
2012-09-11 14:07   ` [Qemu-devel] " Don Slutz
2012-09-11 14:07 ` [PATCH v2 1/4] target-i386: Allow tsc-frequency to be larger then 2.147G Don Slutz
2012-09-11 14:07   ` [Qemu-devel] " Don Slutz
2012-09-11 14:07 ` [PATCH v2 2/4] target-i386: Add missing kvm bits Don Slutz
2012-09-11 14:07   ` [Qemu-devel] " Don Slutz
2012-09-11 14:07 ` [PATCH v2 3/4] target-i386: Allow changing of Hypervisor CPUIDs Don Slutz
2012-09-11 14:07   ` [Qemu-devel] " Don Slutz
2012-09-12 17:55   ` Marcelo Tosatti
2012-09-12 17:55     ` [Qemu-devel] " Marcelo Tosatti
2012-09-13 18:43     ` Don Slutz
2012-09-13 18:43       ` [Qemu-devel] " Don Slutz
2012-09-14 10:01       ` Marcelo Tosatti
2012-09-14 10:01         ` [Qemu-devel] " Marcelo Tosatti
2012-09-11 14:07 ` [PATCH v2 4/4] target-i386: Add Hypervisor CPUIDs in accel=tcg mode Don Slutz
2012-09-11 14:07   ` [Qemu-devel] " Don Slutz
2012-09-17 13:39 ` [PATCH v3 00/17] Allow changing of Hypervisor CPUIDs Don Slutz
2012-09-17 13:39   ` [Qemu-devel] " Don Slutz
2012-09-17 13:39 ` [PATCH v3 00/17] *** SUBJECT HERE *** Don Slutz
2012-09-17 13:39   ` [Qemu-devel] " Don Slutz
2012-09-17 13:49   ` Don Slutz
2012-09-17 13:49     ` [Qemu-devel] " Don Slutz
2012-09-17 13:51     ` Don Slutz
2012-09-17 13:51       ` [Qemu-devel] " Don Slutz
2012-09-17 13:39 ` [PATCH v3 00/17] Allow changing of Hypervisor CPUIDs Don Slutz
2012-09-17 13:39   ` [Qemu-devel] " Don Slutz
2012-09-17 13:39 ` Don Slutz [this message]
2012-09-17 13:39   ` [Qemu-devel] [PATCH v3 01/17] target-i386: Allow tsc-frequency to be larger then 2.147G Don Slutz
2012-09-17 13:39 ` [PATCH v3 02/17] target-i386: Add missing kvm bits Don Slutz
2012-09-17 13:39   ` [Qemu-devel] " Don Slutz
2012-09-17 13:39 ` [PATCH v3 03/17] target-i386: Add Hypervisor level Don Slutz
2012-09-17 13:39   ` [Qemu-devel] " Don Slutz
2012-09-17 13:39 ` Don Slutz
2012-09-17 13:39   ` [Qemu-devel] " Don Slutz
2012-09-17 13:39 ` [PATCH v3 04/17] target-i386: Add cpu object access routines for " Don Slutz
2012-09-17 13:39   ` [Qemu-devel] " Don Slutz
2012-09-17 13:39 ` [PATCH v3 05/17] target-i386: Add x86_set_hyperv Don Slutz
2012-09-17 13:39   ` [Qemu-devel] " Don Slutz
2012-09-17 13:39 ` [PATCH v3 06/17] target-i386: Use Hypervisor level in -machine pc,accel=kvm Don Slutz
2012-09-17 13:39   ` [Qemu-devel] [PATCH v3 06/17] target-i386: Use Hypervisor level in -machine pc, accel=kvm Don Slutz
2012-09-17 13:39 ` [PATCH v3 06/17] target-i386: Use Hypervisor level in -machine pc,accel=kvm Don Slutz
2012-09-17 13:39   ` [Qemu-devel] [PATCH v3 06/17] target-i386: Use Hypervisor level in -machine pc, accel=kvm Don Slutz
2012-09-17 13:39 ` [PATCH v3 07/17] target-i386: Use Hypervisor level in -machine pc,accel=tcg Don Slutz
2012-09-17 13:39   ` [Qemu-devel] [PATCH v3 07/17] target-i386: Use Hypervisor level in -machine pc, accel=tcg Don Slutz
2012-09-17 13:39 ` [PATCH v3 07/17] target-i386: Use Hypervisor level in -machine pc,accel=tcg Don Slutz
2012-09-17 13:39   ` [Qemu-devel] [PATCH v3 07/17] target-i386: Use Hypervisor level in -machine pc, accel=tcg Don Slutz
2012-09-17 13:39 ` [PATCH v3 08/17] target-i386: Add Hypervisor vendor Don Slutz
2012-09-17 13:39   ` [Qemu-devel] " Don Slutz
2012-09-17 13:39 ` Don Slutz
2012-09-17 13:39   ` [Qemu-devel] " Don Slutz
2012-09-17 13:39 ` [PATCH v3 09/17] target-i386: Add cpu object access routines for " Don Slutz
2012-09-17 13:39   ` [Qemu-devel] " Don Slutz
2012-09-17 13:39 ` [PATCH v3 10/17] target-i386: Use Hypervisor vendor in -machine pc,accel=kvm Don Slutz
2012-09-17 13:39   ` [Qemu-devel] [PATCH v3 10/17] target-i386: Use Hypervisor vendor in -machine pc, accel=kvm Don Slutz
2012-09-17 13:39 ` [PATCH v3 10/17] target-i386: Use Hypervisor vendor in -machine pc,accel=kvm Don Slutz
2012-09-17 13:39   ` [Qemu-devel] [PATCH v3 10/17] target-i386: Use Hypervisor vendor in -machine pc, accel=kvm Don Slutz
2012-09-17 13:39 ` [PATCH v3 11/17] target-i386: Use Hypervisor vendor in -machine pc,accel=tcg Don Slutz
2012-09-17 13:39   ` [Qemu-devel] [PATCH v3 11/17] target-i386: Use Hypervisor vendor in -machine pc, accel=tcg Don Slutz
2012-09-17 13:39 ` [PATCH v3 11/17] target-i386: Use Hypervisor vendor in -machine pc,accel=tcg Don Slutz
2012-09-17 13:39   ` [Qemu-devel] [PATCH v3 11/17] target-i386: Use Hypervisor vendor in -machine pc, accel=tcg Don Slutz
2012-09-17 13:39 ` [PATCH v3 12/17] target-i386: Add some known names to Hypervisor vendor Don Slutz
2012-09-17 13:39   ` [Qemu-devel] " Don Slutz
2012-09-17 13:39 ` [PATCH v3 13/17] target-i386: Add optional Hypervisor leaf extra Don Slutz
2012-09-17 13:39   ` [Qemu-devel] " Don Slutz
2012-09-17 13:39 ` [PATCH v3 14/17] target-i386: Add cpu object access routines for " Don Slutz
2012-09-17 13:39   ` [Qemu-devel] " Don Slutz
2012-09-17 13:39 ` [PATCH v3 15/17] target-i386: Add setting of Hypervisor leaf extra for known vmare4 Don Slutz
2012-09-17 13:39   ` [Qemu-devel] " Don Slutz
2012-09-17 13:39 ` [PATCH v3 16/17] target-i386: Use Hypervisor leaf extra in -machine pc,accel=kvm Don Slutz
2012-09-17 13:39   ` [Qemu-devel] [PATCH v3 16/17] target-i386: Use Hypervisor leaf extra in -machine pc, accel=kvm Don Slutz
2012-09-17 13:39 ` [PATCH v3 17/17] target-i386: Use Hypervisor leaf extra in -machine pc,accel=tcg Don Slutz
2012-09-17 13:39   ` [Qemu-devel] [PATCH v3 17/17] target-i386: Use Hypervisor leaf extra in -machine pc, accel=tcg Don Slutz
2012-09-17 14:00 ` [PATCH v3 00/17] Allow changing of Hypervisor CPUIDs Don Slutz
2012-09-17 14:00   ` [Qemu-devel] " Don Slutz
2012-09-17 19:27   ` Eduardo Habkost
2012-09-17 19:27     ` Eduardo Habkost
2012-09-17 14:00 ` [PATCH v3 01/17] target-i386: Allow tsc-frequency to be larger then 2.147G Don Slutz
2012-09-17 14:00   ` [Qemu-devel] " Don Slutz
2012-09-17 14:00 ` [PATCH v3 02/17] target-i386: Add missing kvm bits Don Slutz
2012-09-17 14:00   ` [Qemu-devel] " Don Slutz
2012-09-17 14:00 ` [PATCH v3 03/17] target-i386: Add Hypervisor level Don Slutz
2012-09-17 14:00   ` [Qemu-devel] " Don Slutz
2012-09-17 14:00 ` [PATCH v3 04/17] target-i386: Add cpu object access routines for " Don Slutz
2012-09-17 14:00   ` [Qemu-devel] " Don Slutz
2012-09-17 14:00 ` [PATCH v3 05/17] target-i386: Add x86_set_hyperv Don Slutz
2012-09-17 14:00   ` [Qemu-devel] " Don Slutz
2012-09-19 19:32   ` Eduardo Habkost
2012-09-19 19:32     ` Eduardo Habkost
2012-09-19 21:26     ` Don Slutz
2012-09-19 21:26       ` Don Slutz
2012-09-20 16:20       ` Eduardo Habkost
2012-09-20 16:20         ` [Qemu-devel] " Eduardo Habkost
2012-09-17 14:00 ` [PATCH v3 06/17] target-i386: Use Hypervisor level in -machine pc,accel=kvm Don Slutz
2012-09-17 14:00   ` [Qemu-devel] [PATCH v3 06/17] target-i386: Use Hypervisor level in -machine pc, accel=kvm Don Slutz
2012-09-17 14:00 ` [PATCH v3 07/17] target-i386: Use Hypervisor level in -machine pc,accel=tcg Don Slutz
2012-09-17 14:00   ` [Qemu-devel] [PATCH v3 07/17] target-i386: Use Hypervisor level in -machine pc, accel=tcg Don Slutz
2012-09-17 14:00 ` [PATCH v3 08/17] target-i386: Add Hypervisor vendor Don Slutz
2012-09-17 14:00   ` [Qemu-devel] " Don Slutz
2012-09-17 14:00 ` [PATCH v3 09/17] target-i386: Add cpu object access routines for " Don Slutz
2012-09-17 14:00   ` [Qemu-devel] " Don Slutz
2012-09-17 14:01 ` [PATCH v3 10/17] target-i386: Use Hypervisor vendor in -machine pc,accel=kvm Don Slutz
2012-09-17 14:01   ` [Qemu-devel] [PATCH v3 10/17] target-i386: Use Hypervisor vendor in -machine pc, accel=kvm Don Slutz
2012-09-17 14:01 ` [PATCH v3 11/17] target-i386: Use Hypervisor vendor in -machine pc,accel=tcg Don Slutz
2012-09-17 14:01   ` [Qemu-devel] [PATCH v3 11/17] target-i386: Use Hypervisor vendor in -machine pc, accel=tcg Don Slutz
2012-09-17 14:01 ` [PATCH v3 12/17] target-i386: Add some known names to Hypervisor vendor Don Slutz
2012-09-17 14:01   ` [Qemu-devel] " Don Slutz
2012-09-17 14:01 ` [PATCH v3 13/17] target-i386: Add optional Hypervisor leaf extra Don Slutz
2012-09-17 14:01   ` [Qemu-devel] " Don Slutz
2012-09-17 14:01 ` [PATCH v3 14/17] target-i386: Add cpu object access routines for " Don Slutz
2012-09-17 14:01   ` [Qemu-devel] " Don Slutz
2012-09-17 14:01 ` [PATCH v3 15/17] target-i386: Add setting of Hypervisor leaf extra for known vmare4 Don Slutz
2012-09-17 14:01   ` [Qemu-devel] " Don Slutz
2012-09-17 14:01 ` [PATCH v3 16/17] target-i386: Use Hypervisor leaf extra in -machine pc,accel=kvm Don Slutz
2012-09-17 14:01   ` [Qemu-devel] [PATCH v3 16/17] target-i386: Use Hypervisor leaf extra in -machine pc, accel=kvm Don Slutz
2012-09-17 14:01 ` [PATCH v3 17/17] target-i386: Use Hypervisor leaf extra in -machine pc,accel=tcg Don Slutz
2012-09-17 14:01   ` [Qemu-devel] [PATCH v3 17/17] target-i386: Use Hypervisor leaf extra in -machine pc, accel=tcg Don Slutz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1347889189-9199-4-git-send-email-Don@CloudSwitch.com \
    --to=don@cloudswitch.com \
    --cc=afaerber@suse.de \
    --cc=anthony@codemonkey.ws \
    --cc=avi@redhat.com \
    --cc=ehabkost@redhat.com \
    --cc=foliveira@cloudswitch.com \
    --cc=imammedo@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=mtosatti@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.