[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/13] linux-user: Use memcpy in get_user/put_us
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 01/13] linux-user: Use memcpy in get_user/put_user. |
Date: |
Mon, 17 Sep 2012 17:06:23 +0100 |
On 17 September 2012 16:28, Richard Henderson <address@hidden> wrote:
> Not sure whether this is a gcc bug, but on sparc we wind up
> trying to use std to an unaligned address. The patch does
> work around the problem.
>
> Signed-off-by: Richard Henderson <address@hidden>
As far as the code is concerned:
Reviewed-by: Peter Maydell <address@hidden>
but please rewrite the commit message to be more sure of itself :-)
Specifically, this is required because the guest's alignment
requirements may be laxer than those of the host, so we can't
take a hptr for a 16/32/64 bit type (which only has the alignment
restrictions of the guest) and do a gcc pointer dereference on it
(which may assume the alignment restrictions of the host).
Does anybody know the semantics of the kernel's copy_from_user()?
Does it fail on misaligned userspace structs or does it allow
them?
-- PMM
- [Qemu-devel] [PATCH] fix gcc warnings when RESERVED_VA is 0, Mike Frysinger, 2012/09/15
- Re: [Qemu-devel] [PATCH] fix gcc warnings when RESERVED_VA is 0, Peter Maydell, 2012/09/16
- [Qemu-devel] [PATCH 02/13] tcg-sparc: Hack in qemu_ld/st64 for 32-bit., Richard Henderson, 2012/09/17
- [Qemu-devel] [PATCH 06/13] tcg-sparc: Support GUEST_BASE., Richard Henderson, 2012/09/17
- [Qemu-devel] [PATCH 03/13] tcg-sparc: Fix ADDX opcode., Richard Henderson, 2012/09/17
- [Qemu-devel] [PATCH 04/13] tcg-sparc: Assume v9 cpu always, i.e. force v8plus in 32-bit mode., Richard Henderson, 2012/09/17
- Re: [Qemu-devel] [PATCH 04/13] tcg-sparc: Assume v9 cpu always, i.e. force v8plus in 32-bit mode., Andreas Färber, 2012/09/17
- Re: [Qemu-devel] [PATCH 04/13] tcg-sparc: Assume v9 cpu always, i.e. force v8plus in 32-bit mode., Richard Henderson, 2012/09/17
- Re: [Qemu-devel] [PATCH 04/13] tcg-sparc: Assume v9 cpu always, i.e. force v8plus in 32-bit mode., Blue Swirl, 2012/09/17
- Re: [Qemu-devel] [PATCH 04/13] tcg-sparc: Assume v9 cpu always, i.e. force v8plus in 32-bit mode., Richard Henderson, 2012/09/17
- [Qemu-devel] [PATCH 09/13] tcg-sparc: Mask shift immediates to avoid illegal insns., Richard Henderson, 2012/09/17
- [Qemu-devel] [PATCH 05/13] tcg-sparc: Fix qemu_ld/st to handle 32-bit host., Richard Henderson, 2012/09/17