[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] sparc64: fix helper_st_asi little endian case t
From: |
Stuart Brady |
Subject: |
Re: [Qemu-devel] [PATCH] sparc64: fix helper_st_asi little endian case typo |
Date: |
Sat, 11 Jul 2009 21:43:56 +0100 |
User-agent: |
Mutt/1.5.13 (2006-08-11) |
On Sat, Jul 11, 2009 at 10:22:18PM +0400, Igor Kovalenko wrote:
> It is clear that intention is to byte-swap value to be written, not
> the target address.
@@ -1949,13 +1949,13 @@ void helper_st_asi(target_ulong addr, ta
case 0x89: // Secondary LE
switch(size) {
case 2:
- addr = bswap16(addr);
+ addr = bswap16(val);
^^^^
Shouldn't that be 'val = bswap16(val)' (and likewise for the 32-bit and
64-bit cases)? Also needs a 'signed-off-by:'...
Cheers,
--
Stuart Brady