Re: [Qemu-devel] [Bug 1095531] Re: sparc32plus (and others?) has x86 code generation errors on 64bit hosts

2017-07-11 Thread Samuel Seay
I no longer have a setup to test this so I can only say to close it. On Jul 11, 2017 4:15 PM, "Thomas Huth" <1095...@bugs.launchpad.net> wrote: > Can you still reproduce this problem wit the latest release of QEMU > (currently version 2.9.0), or could we close this bug nowadays? > > ** Changed in

[Qemu-devel] [PATCH] Modification to remove a h2g() call and calculate properly instead

2013-01-04 Thread Samuel Seay
Peter Maydell recommended the change to be more proper. The result was tested and shows coming up with the same proper value. Signed-off-by: Samuel Seay --- linux-user/signal.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/linux-user/signal.c b/linux-user/signal.c index

[Qemu-devel] [Bug 1095531] Re: sparc32plus (and others?) has x86 code generation errors on 64bit hosts

2013-01-02 Thread Samuel Seay
I forgot to add, this is on the latest master branch as of this post. I am doing a static compile with debug enabled. My test is doing a chroot of a sparc system built from debootstrap. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU.

[Qemu-devel] [Bug 1095531] [NEW] sparc32plus (and others?) has x86 code generation errors on 64bit hosts

2013-01-02 Thread Samuel Seay
Public bug reported: On 64bit hosts, the load and store functions compile improperly. The issue is the call to gen_address_mask() under the ld and st functions in target-sparc/translate.c. Below are some snips from the log file. Doing a gdb debug, this results in constant access violation errors w

[Qemu-devel] [PATCH v3] Modifications to fix PowerPC on 64bit host, bug 1052857

2013-01-02 Thread Samuel Seay
gt; v3, removed unneeded casting in the put_user call left over from __put_user Signed-off-by: Samuel Seay --- linux-user/signal.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/linux-user/signal.c b/linux-user/signal.c index 95e2ffa..c43b8ac 100644 --- a/linux-user/si

[Qemu-devel] [PATCH v2] Modifications to fix PowerPC on 64bit host, bug #1052857

2013-01-02 Thread Samuel Seay
Removed h2g() macro around the ka->_sa_handler due to _sa_handler being a guest value Changed the __put_user macro as it was attempting to put a value to a guest memory location without converting to the guest area Signed-off-by: Samuel Seay --- linux-user/signal.c |4 ++-- 1 file chan

Re: [Qemu-devel] [PATCH] Change to correct PowerPC on a 64bit host

2013-01-02 Thread Samuel Seay
net access in the VM, any suggestions to make everyone happy on the patch submission? I might be able to redo the git setup on my mac and do it from there. Samuel On Wed, Jan 2, 2013 at 9:02 AM, Peter Maydell wrote: > On 2 January 2013 13:01, Samuel Seay wrote: > > The VM I did the w

Re: [Qemu-devel] [PATCH] Change to correct PowerPC on a 64bit host

2013-01-02 Thread Samuel Seay
to write the value of env->gpr[1] into newsp, which is not valid in host. I can not speak to the line a bit further with h2g(sc). Samuel On Wed, Jan 2, 2013 at 7:00 AM, Peter Maydell wrote: > On 2 January 2013 04:58, Samuel Seay wrote: > > Attached is a patch for fixing bug #105

[Qemu-devel] [Bug 1052857] Re: qemu-user compiled static for ppc fails on 64bit hosts

2013-01-01 Thread Samuel Seay
I just submitted a patch to the dev mailing list. Just in case there is an issue with the submitted patch, or if Erik wants it sooner, I attached the patch I submitted. ** Patch added: "0001-Change-to-correct-PowerPC-on-64bit-host.patch" https://bugs.launchpad.net/qemu/+bug/1052857/+attachment

[Qemu-devel] [PATCH] Change to correct PowerPC on a 64bit host

2013-01-01 Thread Samuel Seay
Attached is a patch for fixing bug #1052857. My local tests show it working properly on 32 and 64bit. Signed-off-by: Samuel Seay 0001-Change-to-correct-PowerPC-on-64bit-host.patch Description: Binary data

[Qemu-devel] [Bug 1052857] Re: qemu-user compiled static for ppc fails on 64bit hosts

2013-01-01 Thread Samuel Seay
** Summary changed: - qemu-user compiled static for ppc fails + qemu-user compiled static for ppc fails on 64bit hosts -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1052857 Title: qemu-user compil

[Qemu-devel] [Bug 1052857] Re: qemu-user compiled static for ppc fails

2013-01-01 Thread Samuel Seay
I ran into this issue also and did a bit of investigating. This is only an issue when ran on a 64bit host. The actual problem line is err |= __put_user(h2g(ka->_sa_handler), &sc->handler); inside of linux_user/signal.c. What I am unsure of is when the h2g() macro, the cause of the assert, is vali