We ported MIPS64 r2 user mode emulation. When a binary is given to
qemu-mips64, our code first check whether it is Octeon binary or not if yes
it enable Octeon specific Instructions for. The following code snippet do
this job:
>
> > diff --git a/linux-user/elfload.c b/linux-user/elfload.c
> > ind
This series finally fixes -icount with iothread and avoids deadlocks
due to the vm_clock not making progress when the VM is stopped.
The crux of the fix is in patch 1, while patch 2 implements the
"clock warping" that fixes deadlocks in v2. Clock warping uses
the nanosecond resolution rt_clock tim
The previous patch however is not enough, because if the virtual CPU
goes to sleep waiting for a future timer interrupt to wake it up, qemu
deadlocks. The timer interrupt never comes because time is driven by
icount, but the vCPU doesn't run any insns.
You could say that VCPUs should never go to
It is purely for icount-based virtual timers. And now that we got the
code right, rename the function to clarify the intended scope.
Signed-off-by: Paolo Bonzini
Tested-by: Edgar E. Iglesias
---
cpus.c |4 ++--
qemu-timer.c | 13 -
qemu-timer.h |2 +-
3 files change
The correct fix for -icount is to consider the biggest difference
between iothread and non-iothread modes. In the traditional model,
CPUs run _before_ the iothread calls select (or WaitForMultipleObjects
for Win32). In the iothread model, CPUs run while the iothread
isn't holding the mutex, i.e.
This reverts commits 225d02cd and c9f7383c. While some parts of
the latter could be saved, I preferred a smooth, complete revert.
Signed-off-by: Paolo Bonzini
Tested-by: Edgar E. Iglesias
---
qemu-timer.c | 66 +++--
1 files changed, 36 ins
Use get_option_parameter() to instead of duplicating the loop, and use
BDRV_SECTOR_SIZE to instead of 512
Signed-off-by: Mitnick Lyu
---
block/vpc.c |8 ++--
1 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/block/vpc.c b/block/vpc.c
index 7b025be..56865da 100644
--- a/bloc
1 fixes spellos in the mass-storage driver
2-4 fix issues in Linux usb pass-through code.
Signed-off-by: Brad Hards
---
hw/usb-msd.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/usb-msd.c b/hw/usb-msd.c
index 76f5b02..947fd3f 100644
--- a/hw/usb-msd.c
+++ b/hw/usb-msd.c
@@ -33,7 +33,7 @@ do { printf("usb-msd: " fmt , ## __VA_ARGS__); } while (0)
en
This isn't used, but leaving it empty causes valgrind noise.
Signed-off-by: Brad Hards
---
usb-linux.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/usb-linux.c b/usb-linux.c
index 255009f..d958853 100644
--- a/usb-linux.c
+++ b/usb-linux.c
@@ -344,6 +344,7 @@ static i
Signed-off-by: Brad Hards
---
usb-linux.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/usb-linux.c b/usb-linux.c
index d958853..1f33c2c 100644
--- a/usb-linux.c
+++ b/usb-linux.c
@@ -107,7 +107,7 @@ enum {
/*
* Control transfer state.
* Note that 'buffer' _must
Signed-off-by: Brad Hards
---
usb-linux.c | 16
1 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/usb-linux.c b/usb-linux.c
index 1f33c2c..b02a0f9 100644
--- a/usb-linux.c
+++ b/usb-linux.c
@@ -233,8 +233,8 @@ static void async_complete(void *opaque)
Am 12.04.2011 18:06, schrieb Amit Shah:
> The GET_EVENT_STATUS_NOTIFICATION ATAPI command is listed as a
> mandatory command in the spec but we don't really implement it any of
> its sub-commands.
>
> The commit message for the last commit explains why implementing just
> the media subcommand is h
Am 13.04.2011 11:30, schrieb Mitnick Lyu:
> Use get_option_parameter() to instead of duplicating the loop, and use
> BDRV_SECTOR_SIZE to instead of 512
>
> Signed-off-by: Mitnick Lyu
Thanks, applied to the block branch.
Kevin
On Tue, 12 Apr 2011, Jan Kiszka wrote:
> Well, either you have a use for the VCPU state (how do you do migration
> in Xen without it?), or you should probably teach QEMU in a careful &
> clean way to run its device model without VCPUs - and without any
> TCG-related memory consumption. For the latt
On 2011-04-13 12:56, Stefano Stabellini wrote:
> On Tue, 12 Apr 2011, Jan Kiszka wrote:
>> Well, either you have a use for the VCPU state (how do you do migration
>> in Xen without it?), or you should probably teach QEMU in a careful &
>> clean way to run its device model without VCPUs - and withou
Signed-off-by: Alon Levy
---
libcacard/vcard_emul_nss.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/libcacard/vcard_emul_nss.c b/libcacard/vcard_emul_nss.c
index 71f2ba3..baada52 100644
--- a/libcacard/vcard_emul_nss.c
+++ b/libcacard/vcard_emul_nss.c
@@ -955,7 +95
On Wed, 13 Apr 2011, Jan Kiszka wrote:
> On 2011-04-13 12:56, Stefano Stabellini wrote:
> > On Tue, 12 Apr 2011, Jan Kiszka wrote:
> >> Well, either you have a use for the VCPU state (how do you do migration
> >> in Xen without it?), or you should probably teach QEMU in a careful &
> >> clean way t
> Were you able to replicate the problem with the steps that I had mentioned ?
> The key thing is is if you don't set breakpoint at 0x4 or 0x8 just follow
> the execution flow using "si" command of GDB.
> You will definitely hit the problem.
Ah, I had to find a different gdb to reproduce this with
The following changes since commit 9df38c47d01eb1fd7eb9d60ac70a4170e638b4a2:
target-arm: Detect tininess before rounding for FP operations (2011-04-12
23:33:33 +0200)
are available in the git repository at:
git://repo.or.cz/qemu/kevin.git for-anthony
Amit Shah (7):
atapi: Drives can b
From: Stefan Hajnoczi
Zero data clusters are a space-efficient way of storing zeroed regions
of the image.
Signed-off-by: Stefan Hajnoczi
Signed-off-by: Kevin Wolf
---
docs/specs/qed_spec.txt |8
1 files changed, 8 insertions(+), 0 deletions(-)
diff --git a/docs/specs/qed_spec.t
From: Amit Shah
Table 629 of the MMC-5 spec mentions two different error conditions when
a CDROM eject is requested: a) while a disc is inserted and b) while a
disc is not inserted.
Ensure we return the appropriate error for the present condition of the
drive and disc status.
Signed-off-by: Ami
From: Anthony Liguori
Zero clusters are similar to unallocated clusters except instead of reading
their value from a backing file when one is available, the cluster is always
read as zero.
This implements read support only. At this stage, QED will never write a
zero cluster.
Signed-off-by: Ant
From: Amit Shah
Drivers are free to lock drives without any media present. Such a
condition should not result in an error condition.
See Table 341 in MMC-5 spec for details.
Signed-off-by: Amit Shah
Signed-off-by: Kevin Wolf
---
hw/ide/core.c |9 ++---
1 files changed, 2 insertions(
From: Amit Shah
Handle GET_EVENT_STATUS_NOTIFICATION's No Event Available response in a
generic way so that future additions to the code to handle other
response types is easier.
Signed-off-by: Amit Shah
Acked-by: Jes Sorensen
Acked-by: Paolo Bonzini
Signed-off-by: Kevin Wolf
---
hw/ide/cor
From: Mitnick Lyu
Use get_option_parameter() to instead of duplicating the loop, and
use BDRV_SECTOR_SIZE to instead of 512
Signed-off-by: Mitnick Lyu
Signed-off-by: Kevin Wolf
---
block/vpc.c |8 ++--
1 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/block/vpc.c b/block/
From: Amit Shah
After a media change, the only commands allowed from the guest were
REQUEST_SENSE and INQUIRY. The guest may also issue
GET_EVENT_STATUS_NOTIFICATION commands to get media
changed notification.
Signed-off-by: Amit Shah
Acked-by: Jes Sorensen
Acked-by: Paolo Bonzini
Signed-off
From: Amit Shah
Instead of using magic numbers, use structs that are more descriptive of
the fields being used.
Signed-off-by: Amit Shah
Acked-by: Jes Sorensen
Acked-by: Paolo Bonzini
Signed-off-by: Kevin Wolf
---
hw/ide/core.c | 16 ++--
1 files changed, 14 insertions(+), 2 d
From: Amit Shah
Implement the 'media' sub-command of the GET_EVENT_STATUS_NOTIFICATION
command. This helps us report tray open, tray closed, no media, media
present states to the guest.
Newer Linux kernels (2.6.38+) rely on this command to revalidate discs
after media change.
This patch also s
From: Amit Shah
This makes the code more readable.
Also, there's a block like:
if () {
...
} else {
...
}
Split that into
if () {
...
return;
}
...
Signed-off-by: Amit Shah
Acked-by: Jes Sorensen
Acked-by: Paolo Bonzini
Signed-off-by: Kevin Wolf
---
hw/ide/core.c | 37
On Wed, Apr 13, 2011 at 10:45 AM, Brad Hards wrote:
> Signed-off-by: Brad Hards
> ---
> usb-linux.c | 16
> 1 files changed, 8 insertions(+), 8 deletions(-)
Some of these printfs look like they might be useful given that the
current USB support is known to be imperfect and fr
On Wed, Apr 13, 2011 at 10:45 AM, Brad Hards wrote:
> 1 fixes spellos in the mass-storage driver
> 2-4 fix issues in Linux usb pass-through code.
I left a comment on patch 4 and will wait for activity on that email
thread. Otherwise these could go via the trivial-patches tree and
I'll add them o
gcc 4.5.2 correctly complains that r is potentially uninitialized in this
function.
Signed-off-by: Anthony Liguori
---
hw/milkymist-pfpu.c |2 +-
roms/seabios|2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/milkymist-pfpu.c b/hw/milkymist-pfpu.c
index 4831
Hello Kevin, Stefan and all
There is a sentence "The maximum size of a dynamic hard disk is 2040 GB.
The actual size is restricted by the underlying disk hardware protocol. For
example,
ATA hard disks have a 127-GB limit" in VHD specification. This limitation
infected the
algorithm of block/vpc.c:
Hi,
On 04/13/2011 11:45 AM, Brad Hards wrote:
Signed-off-by: Brad Hards
---
usb-linux.c | 16
1 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/usb-linux.c b/usb-linux.c
index 1f33c2c..b02a0f9 100644
--- a/usb-linux.c
+++ b/usb-linux.c
@@ -233,8 +233,8 @@ stati
This is the result of an of the list discussion with
me, and I can confirm this fixes the issues I was seeing:
Tested-by: Hans de Goede
On 04/13/2011 01:42 PM, Alon Levy wrote:
Signed-off-by: Alon Levy
---
libcacard/vcard_emul_nss.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions
Am 13.04.2011 14:47, schrieb Lyu Mitnick:
> Hello Kevin, Stefan and all
>
> There is a sentence "The maximum size of a dynamic hard disk is 2040 GB.
> The actual size is restricted by the underlying disk hardware protocol.
> For example,
> ATA hard disks have a 127-GB limit" in VHD specification
On Wed, Apr 13, 2011 at 1:43 PM, Anthony Liguori wrote:
> diff --git a/roms/seabios b/roms/seabios
> index cc97564..06d0bdd 16
> --- a/roms/seabios
> +++ b/roms/seabios
> @@ -1 +1 @@
> -Subproject commit cc975646af69f279396d4d5e1379ac6af80ee637
> +Subproject commit 06d0bdd9e2e20377b3180e4986b1
On 2011-04-13 13:49, Stefano Stabellini wrote:
> On Wed, 13 Apr 2011, Jan Kiszka wrote:
>> On 2011-04-13 12:56, Stefano Stabellini wrote:
>>> On Tue, 12 Apr 2011, Jan Kiszka wrote:
Well, either you have a use for the VCPU state (how do you do migration
in Xen without it?), or you should p
On Tue, 12 Apr 2011 21:31:18 +0300
Blue Swirl wrote:
> On Tue, Apr 12, 2011 at 10:52 AM, Avi Kivity wrote:
> > On 04/11/2011 08:15 PM, Blue Swirl wrote:
> >>
> >> On Mon, Apr 11, 2011 at 10:01 AM, Markus Armbruster
> >> wrote:
> >> > Avi Kivity writes:
> >> >
> >> >> On 04/08/2011 12:41 AM,
On 04/13/2011 08:04 AM, Stefan Hajnoczi wrote:
On Wed, Apr 13, 2011 at 1:43 PM, Anthony Liguori wrote:
diff --git a/roms/seabios b/roms/seabios
index cc97564..06d0bdd 16
--- a/roms/seabios
+++ b/roms/seabios
@@ -1 +1 @@
-Subproject commit cc975646af69f279396d4d5e1379ac6af80ee637
+Subproject
From: Anthony Liguori
QEMU can drop a backing file so that an image file no longer depends on
the backing file, but this feature has not been exposed in qemu-img.
This is useful in an image streaming usecase or when an image file has
been fully allocated and no reads can hit the backing file anym
On 12 April 2011 22:59, Aurelien Jarno wrote:
> Use float64_unordered() in helper_cmptun() instead of doing the
> the comparison manually. This also fixes the wrong behaviours with
> sNaNs.
>
> Signed-off-by: Aurelien Jarno
> ---
> target-alpha/op_helper.c | 5 +++--
> 1 files changed, 3 inse
On 12 April 2011 22:59, Aurelien Jarno wrote:
> On alpha, all NaN should trap during a comparison, not only sNaN. Fix
> this by using float64_eq() instead of float64_eq_quiet().
The Compiler Writer's Guide disagrees with you:
www.compaq.com/cpq-alphaserver/technology/literature/cmpwrgd.pdf
page D
On Wed, 13 Apr 2011, Jan Kiszka wrote:
> On 2011-04-13 13:49, Stefano Stabellini wrote:
> > On Wed, 13 Apr 2011, Jan Kiszka wrote:
> >> On 2011-04-13 12:56, Stefano Stabellini wrote:
> >>> On Tue, 12 Apr 2011, Jan Kiszka wrote:
> Well, either you have a use for the VCPU state (how do you do mi
Thanks, Jan. I split my patch into three and started a new thread. I also
put all options in -net user. Yes, TCP firewall is coming. You'll see some
of the added functions will be shared.
Dai
On Tue, Apr 12, 2011 at 9:38 AM, Jan Kiszka wrote:
> On 2011-04-12 18:19, Daisuke Nojiri wrote:
> > Thi
On 13 April 2011 16:38, Richard Henderson wrote:
> [ Odd, the original thread doesn't seem to have arrived here. ]
>
> On 04/13/2011 07:52 AM, Peter Maydell wrote:
>> So I think you want the _quiet version here. (And helper_cmpteq
>> needs to use float64_eq_quiet rather than float64_eq.)
>
> Yes,
On 12 April 2011 22:59, Aurelien Jarno wrote:
> This patch series started with the goal of improving the build of
> target-i386 with softfloat (instead of softfloat-native), but it slowly
> became a collection of fixes and improvements with regard to softfloat
> and targets FPU.
For patches 01 02
I think you are right. This seems to be more of a GDB issue.
Any ways thanks for your support.
--Anup
On Wed, Apr 13, 2011 at 5:24 PM, Peter Maydell
wrote:
> > Were you able to replicate the problem with the steps that I had
> mentioned ?
> > The key thing is is if you don't set breakpoint at 0
> I think you are right. This seems to be more of a GDB issue.
The debug stub is still part of QEMU, so let's not close this bug just
yet :-)
** Summary changed:
- Undefined instruction exception starts at offset 0x8 instead of 0x4
+ ARM: singlestepping insn which UNDEFs should stop at UNDEF ve
Signed-off-by: Lluís Vilanova
---
target-arm/helper.c| 246 ++--
target-arm/iwmmxt_helper.c | 28 +++--
target-arm/translate.c | 26 ++---
3 files changed, 150 insertions(+), 150 deletions(-)
diff --git a/target-arm/helper.c b/target-arm/h
Signed-off-by: Lluís Vilanova
---
target-m68k/translate.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/target-m68k/translate.c b/target-m68k/translate.c
index 038c0af..15a650e 100644
--- a/target-m68k/translate.c
+++ b/target-m68k/translate.c
@@ -3022,7 +3022,7 @@ gen
This provides a consistent naming scheme across all targets.
Signed-off-by: Lluís Vilanova
---
target-m68k/helper.c|2 +-
target-m68k/helper.h| 54 +++
target-m68k/helpers.h | 54 ---
target
This provides a consistent naming scheme across all targets.
Signed-off-by: Lluís Vilanova
---
target-arm/helper.c|2
target-arm/helper.h| 475
target-arm/helpers.h | 475
targe
On 13 April 2011 17:38, Lluís wrote:
> Signed-off-by: Lluís Vilanova
> ---
> target-arm/helper.c | 246
> ++--
> target-arm/iwmmxt_helper.c | 28 +++--
> target-arm/translate.c | 26 ++---
> 3 files changed, 150 insertions(+), 150 deletion
On 04/13/2011 08:42 AM, Peter Maydell wrote:
> On 13 April 2011 16:38, Richard Henderson wrote:
>> [ Odd, the original thread doesn't seem to have arrived here. ]
>>
>> On 04/13/2011 07:52 AM, Peter Maydell wrote:
>>> So I think you want the _quiet version here. (And helper_cmpteq
>>> needs to use
[ Odd, the original thread doesn't seem to have arrived here. ]
On 04/13/2011 07:52 AM, Peter Maydell wrote:
> So I think you want the _quiet version here. (And helper_cmpteq
> needs to use float64_eq_quiet rather than float64_eq.)
Yes, the _quiet version is what's needed for all comparisons.
Fo
Public bug reported:
The latest git version of qemu (commit
9df38c47d01eb1fd7eb9d60ac70a4170e638b4a2) fails to boot the OpenSolaris
image from http://dlc.sun.com/osol/opensolaris/2009/06/osol-0906-ai-
sparc.iso.
qemu-img create opensolaris 3G
qemu-system-sparc -hda opensolaris -cdrom osol-0906-ai
On Tue, Apr 12, 2011 at 11:41:49PM +0100, Peter Maydell wrote:
> On 12 April 2011 22:59, Aurelien Jarno wrote:
> > I haven't look at the documentation, but for the neighbouring code it looks
> > clear that floatx80_eq() should be used instead of floatx80_eq_quiet().
>
> Actually I think it's irre
On Tue, Apr 12, 2011 at 07:40:33PM -0700, Nathan Froyd wrote:
> On Tue, Apr 12, 2011 at 11:59:29PM +0200, Aurelien Jarno wrote:
> > Given that float32_*() functions are IEEE754 compliant, the efscmp*()
> > functions are correctly implemented, while efstst*() are not. This
> > patch reverse the impl
On Wed, Apr 13, 2011 at 12:26:01PM +0500, Khansa Butt wrote:
> We ported MIPS64 r2 user mode emulation. When a binary is given to
> qemu-mips64, our code first check whether it is Octeon binary or not if yes
> it enable Octeon specific Instructions for. The following code snippet do
> this job:
>
On Wed, Apr 13, 2011 at 08:53:27AM -0700, Richard Henderson wrote:
> On 04/13/2011 08:42 AM, Peter Maydell wrote:
> > On 13 April 2011 16:38, Richard Henderson wrote:
> >> [ Odd, the original thread doesn't seem to have arrived here. ]
> >>
> >> On 04/13/2011 07:52 AM, Peter Maydell wrote:
> >>> S
On Wed, Apr 13, 2011 at 06:16:55PM +0100, Peter Maydell wrote:
> On 13 April 2011 17:38, Lluís wrote:
> > Signed-off-by: Lluís Vilanova
> > ---
> > target-arm/helper.c | 246
> > ++--
> > target-arm/iwmmxt_helper.c | 28 +++--
> > target-arm/tra
On Wed, Apr 13, 2011 at 4:08 PM, Luiz Capitulino wrote:
> On Tue, 12 Apr 2011 21:31:18 +0300
> Blue Swirl wrote:
>
>> On Tue, Apr 12, 2011 at 10:52 AM, Avi Kivity wrote:
>> > On 04/11/2011 08:15 PM, Blue Swirl wrote:
>> >>
>> >> On Mon, Apr 11, 2011 at 10:01 AM, Markus Armbruster
>> >> wrote:
>
Aurelien Jarno writes:
> On Wed, Apr 13, 2011 at 06:16:55PM +0100, Peter Maydell wrote:
>> On 13 April 2011 17:38, Lluís wrote:
>> > Signed-off-by: Lluís Vilanova
>> > ---
>> > target-arm/helper.c | 246
>> > ++--
>> > target-arm/iwmmxt_helper.c
Aurelien Jarno writes:
> On Wed, Apr 13, 2011 at 06:16:55PM +0100, Peter Maydell wrote:
>> On 13 April 2011 17:38, Lluís wrote:
>> > Signed-off-by: Lluís Vilanova
>> > ---
>> > target-arm/helper.c | 246
>> > ++--
>> > target-arm/iwmmxt_helper.c
On Mon, Apr 11, 2011 at 10:27:41PM +0200, Jan Kiszka wrote:
> These patches were posted before. They bring down the overhead of the
> io-thread mode for TCG here, specifically when emulating SMP.
>
> The major change in this version, besides rebasing, is the exclusion of
> KVM from the main loop p
gen_pc_load was introduced in commit
d2856f1ad4c259e5766847c49acbb4e390731bd4.
The only reason for parameter searched_pc was
a debug statement in target-i386/translate.c.
Remove searched_pc from the debug statement
and from the parameter list of gen_pc_load.
Signed-off-by: Stefan Weil
---
exec-
helpfull -> helpful
usefull -> useful
cotrol -> control
and a grammar fix.
Signed-off-by: Stefan Weil
---
qemu-options.hx |4 ++--
savevm.c|2 +-
target-arm/helper.c |2 +-
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/qemu-options.hx b/qemu-options.
On Tue, Apr 12, 2011 at 7:19 PM, Daisuke Nojiri wrote:
> This patch adds: -drop-udp, -allow-udp ADDR:PORT, -drop-log FILE
> e.g.) $ qemu -net user -drop-log qemu.drop -drop-udp -allow-udp
> 10.0.2.3:53
> -drop-udp enables usermode firewall for out-going UDP packats from a guest.
> All UDP packet
Hello Stefan,
I have a question about get_option_parameter(). I am wondering whether
get_option_parameter is suitable to use instead of doing the search by
myself
in the case like following:
/* Read out options */
while (options && options->name) {
if (!strcmp(options->name, BLOCK_OP
On 13 April 2011 21:38, Stefan Weil wrote:
> gen_pc_load was introduced in commit
> d2856f1ad4c259e5766847c49acbb4e390731bd4.
> The only reason for parameter searched_pc was
> a debug statement in target-i386/translate.c.
>
> Remove searched_pc from the debug statement
> and from the parameter lis
On Tue, Apr 12, 2011 at 11:31:20PM +0100, Peter Maydell wrote:
> On 12 April 2011 22:32, Aurelien Jarno wrote:
> > On Mon, Apr 11, 2011 at 04:32:08PM +0100, Peter Maydell wrote:
>
> >> @@ -1524,12 +1528,12 @@ uint64_t HELPER(neon_abdl_u16)(uint32_t a,
> >> uint32_t b)
> >> {
> >> uint64_t
On Wed, 13 Apr 2011 10:52:37 pm Hans de Goede wrote:
> > @@ -359,7 +359,7 @@ static int usb_host_claim_interfaces(USBHostDevice
> > *dev, int configuration)
> >
> > ret = ioctl(dev->fd, USBDEVFS_CLAIMINTERFACE,&interface);
> > if (ret< 0) {
> > if (errno == EBUSY)
This patch series started with the goal of improving the build of
target-i386 with softfloat (instead of softfloat-native), but it slowly
became a collection of fixes and improvements with regard to softfloat
and targets FPU.
Aurelien Jarno (19):
softfloat: use GCC builtins to count the leading
Add floatx_{add,mul,sub} defines, and use them instead of using direct
C operations.
Reviewed-by: Peter Maydell
Signed-off-by: Aurelien Jarno
---
target-i386/exec.h |6 ++
target-i386/op_helper.c | 18 --
2 files changed, 14 insertions(+), 10 deletions(-)
diff --
Use float unions from cpu-all.h instead of redefining new (wrong for arm)
ones in target-i386. This also allows building cpu-exec.o with softfloat.
Reviewed-by: Peter Maydell
Signed-off-by: Aurelien Jarno
---
target-i386/exec.h | 27 ++-
1 files changed, 2 insertions(+
Add a CPU_LDoubleU type, matching the floatx80 definition and the long
double type on x86 hosts.
Based on a patch from Laurent Vivier .
Cc: Laurent Vivier
Reviewed-by: Peter Maydell
Signed-off-by: Aurelien Jarno
---
cpu-all.h | 10 ++
1 files changed, 10 insertions(+), 0 deletions(-
SSE instructions CMPUNORDPS/D and CMPORDPS/D do not trigger an invalid
exception if operands are qNANs.
Reviewed-by: Peter Maydell
Signed-off-by: Aurelien Jarno
---
target-i386/ops_sse.h |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/target-i386/ops_sse.h b/target-i
Add float*_unordered_quiet() functions to march the softfloat versions.
As FPU status is not tracked with softfloat-native, they don't differ
from the signaling version.
Reviewed-by: Peter Maydell
Signed-off-by: Aurelien Jarno
---
fpu/softfloat-native.h | 15 ---
1 files changed,
I am not a big fan of code moving, but having the signaling version in
the middle of quiet versions and vice versa doesn't make the code easy
to read.
This patch is a simple code move, basically swapping locations of
float*_eq and float*_eq_quiet.
Reviewed-by: Peter Maydell
Signed-off-by: Aureli
As the softfloat comparison functions already test for NaN, there is no
need to always call the float*_unordered*() functions.
Reviewed-by: Peter Maydell
Signed-off-by: Aurelien Jarno
---
target-mips/op_helper.c | 72 +++---
1 files changed, 36 insertio
MIPS FPU instructions should start with a clean softfpu status. This
is done for the most instructions, but not for comparison ones.
Signed-off-by: Aurelien Jarno
---
target-mips/op_helper.c | 41 +
1 files changed, 25 insertions(+), 16 deletions(-)
dif
Softfloat has its own implementation to count the leading zeros. However
a lot of architectures have either a dedicated instruction or an
optimized to do that. When using GCC >= 3.4, this patch uses GCC builtins
instead of the handcoded implementation.
Note that I amware that QEMU_GNUC_PREREQ is d
Use CPU_LDoubleU in cpu_dump_state() instead of redefining a union for
doing the conversion.
Based on a patch from Laurent Vivier .
Cc: Laurent Vivier
Reviewed-by: Peter Maydell
Signed-off-by: Aurelien Jarno
---
target-i386/helper.c | 12 +++-
1 files changed, 3 insertions(+), 9 del
Contrary to cabs.ps.* instructions, c.ps.* should not compare the absolute
value of the operand, but directly the operands.
Signed-off-by: Aurelien Jarno
---
target-mips/op_helper.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/target-mips/op_helper.c b/target-m
Use float64_unordered_quiet() in helper_cmptun() instead of doing the
the comparison manually.
According to the "Alpha Compiler Writer's Guide", we should use the
_quiet version here, as CMPTUN and CMPTEQ should generate InvalidOp
for SNaNs but not for QNaNs.
Thanks to Peter Maydell and Richard
Make clear for all comparison functions which ones trigger an exception
for all NaNs, and which one only for sNaNs.
Reviewed-by: Peter Maydell
Signed-off-by: Aurelien Jarno
---
fpu/softfloat.c | 85 +++
1 files changed, 48 insertions(+), 37
Reviewed-by: Peter Maydell
Signed-off-by: Aurelien Jarno
---
target-mips/op_helper.c | 35 ---
1 files changed, 20 insertions(+), 15 deletions(-)
diff --git a/target-mips/op_helper.c b/target-mips/op_helper.c
index abcb6eb..0a62361 100644
--- a/target-mips/op_h
Use the new float*_unordered*() functions from softfloat instead of
redefining a private version.
Reviewed-by: Peter Maydell
Signed-off-by: Aurelien Jarno
---
target-mips/op_helper.c | 168 ---
1 files changed, 70 insertions(+), 98 deletions(-)
diff
float*_eq functions have a different semantics than other comparison
functions. Fix that by first renaming float*_quiet() into float*_eq_quiet().
Note that it is purely mechanical, and the behaviour should be unchanged.
That said it clearly highlight problems due to this different semantics,
they
Add float*_unordered() functions to softfloat, matching the softfloat-native
ones. Also add float*_unordered_quiet() functions to match the others
comparison functions.
This allow target-i386/ops_sse.h to be compiled with softfloat.
Reviewed-by: Peter Maydell
Signed-off-by: Aurelien Jarno
---
float*_eq_signaling functions have a different semantics than other
comparison functions. Fix that by renaming float*_quiet_signaling() into
float*_eq().
Note that it is purely mechanical, and the behaviour should be unchanged.
Reviewed-by: Peter Maydell
Signed-off-by: Aurelien Jarno
---
fpu/s
efstst*() functions are fast SPE funtions which do not take into account
special values (infinites, NaN, etc.), while efscmp*() functions are
IEEE754 compliant.
Given that float32_*() functions are IEEE754 compliant, the efscmp*()
functions are correctly implemented, while efstst*() are not. This
On Sat, Apr 2, 2011 at 19:36, Stefan Weil wrote:
> tcg_gen_exit_tb takes a parameter of type tcg_target_long,
> so the type casts of pointer to long should be replaced by
> type casts of pointer to tcg_target_long (suggested by Blue Swirl).
>
> These changes are needed for build environments where
Ping.
Gleb, The issue of specifying both data and file is addressed.
Do you have any other comments? Otherwise can you provide your acked-by?
On Tue, Apr 05, 2011 at 10:53:16PM +0400, Michael Tokarev wrote:
> This patch almost rewrites acpi_table_add() function
> (but still leaves it using old get
Am 14.04.2011 03:42, schrieb TeLeMan:
On Sat, Apr 2, 2011 at 19:36, Stefan Weil wrote:
tcg_gen_exit_tb takes a parameter of type tcg_target_long,
so the type casts of pointer to long should be replaced by
type casts of pointer to tcg_target_long (suggested by Blue Swirl).
These changes are nee
Add very basic implementation of collie PDA emulation. The system lacks
LoCoMo and graphics/sound emulation. Linux kernel boots up to mounting
rootfs (theoretically it can be provided in pflash images).
Signed-off-by: Dmitry Eremin-Solenikov
---
Makefile.target |1 +
hw/collie.c | 69 +
Basic implementation of DEC/Intel SA-1100/SA-1110 chips emulation.
Implemented:
- IRQs
- GPIO
- PPC
- RTC
- UARTs (no IrDA/etc.)
- OST reused from pxa25x
Everything else is TODO (esp. PM/idle/sleep!) - see the todo in the
hw/strongarm.c
V5:
* syntax fixup
V4:
* use bitnames to access R
Blue Swirl writes:
> On Wed, Apr 13, 2011 at 4:08 PM, Luiz Capitulino
> wrote:
>> On Tue, 12 Apr 2011 21:31:18 +0300
>> Blue Swirl wrote:
>>
>>> On Tue, Apr 12, 2011 at 10:52 AM, Avi Kivity wrote:
>>> > On 04/11/2011 08:15 PM, Blue Swirl wrote:
>>> >>
>>> >> On Mon, Apr 11, 2011 at 10:01 AM,
100 matches
Mail list logo