Message ID | 20110408133245.2975.97973.malonedeb@wampee.canonical.com |
---|---|
State | New |
Headers | show |
Hi, Thanks for reporting this issue, and the investigation. I don't really understand the rationale for the change, so I can't help much. This change appears to be from 5cd2c5b6ad75c46d40118ac67c0c09d4e7930a65. I think input from Richard Henderson (the author of the change) would be very useful. Brad
Looking through old bug tickets... is this still an issue with the latest version of QEMU? Or could we close this ticket nowadays? ** Changed in: qemu Status: New => Incomplete
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired
--- qemu-0.14.0/exec.c 2011-04-08 17:26:00.524464002 +0400 +++ exec.c 2011-04-08 17:26:09.800464003 +0400 @@ -2340,7 +2340,7 @@ int walk_memory_regions(void *priv, walk data.prot = 0; for (i = 0; i < V_L1_SIZE; i++) { - int rc = walk_memory_regions_1(&data, (abi_ulong)i << V_L1_SHIFT, + int rc = walk_memory_regions_1(&data, (abi_ulong)i << (V_L1_SHIFT + TARGET_PAGE_BITS), V_L1_SHIFT / L2_BITS - 1, l1_map + i); if (rc != 0) { return rc;