On 1/2/24 05:57, Richard Henderson wrote:
The host SHMLBA is by definition a multiple of the host page size.
Thus the remaining component of qemu_host_page_size is the
target page size.

Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
---
  linux-user/elfload.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/linux-user/elfload.c b/linux-user/elfload.c
index f3c1e0fb64..4df565d64b 100644
--- a/linux-user/elfload.c
+++ b/linux-user/elfload.c
@@ -2892,7 +2892,7 @@ static bool pgb_addr_set(PGBAddrs *ga, abi_ulong 
guest_loaddr,
/* Add any HI_COMMPAGE not covered by reserved_va. */
      if (reserved_va < HI_COMMPAGE) {
-        ga->bounds[n][0] = HI_COMMPAGE & qemu_host_page_mask;
+        ga->bounds[n][0] = HI_COMMPAGE & -qemu_real_host_page_size();
          ga->bounds[n][1] = HI_COMMPAGE + TARGET_PAGE_SIZE - 1;
          n++;
      }
@@ -3074,7 +3074,7 @@ void probe_guest_base(const char *image_name, abi_ulong 
guest_loaddr,
                        abi_ulong guest_hiaddr)
  {
      /* In order to use host shmat, we must be able to honor SHMLBA.  */
-    uintptr_t align = MAX(SHMLBA, qemu_host_page_size);
+    uintptr_t align = MAX(SHMLBA, TARGET_PAGE_SIZE);
/* Sanity check the guest binary. */
      if (reserved_va) {

Reviewed-by: Pierrick Bouvier <pierrick.bouv...@linaro.org>

Reply via email to