From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mail-pf1-f171.google.com (mail-pf1-f171.google.com [209.85.210.171])
 by mx.groups.io with SMTP id smtpd.web11.4787.1673037591052708538
 for <devel@edk2.groups.io>;
 Fri, 06 Jan 2023 12:39:51 -0800
Authentication-Results: mx.groups.io;
 dkim=pass header.i=@gmail.com header.s=20210112 header.b=V9DWX3/W;
 spf=pass (domain: gmail.com, ip: 209.85.210.171, mailfrom: pedro.falcato@gmail.com)
Received: by mail-pf1-f171.google.com with SMTP id x26so1907147pfq.10
        for <devel@edk2.groups.io>; Fri, 06 Jan 2023 12:39:50 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=gmail.com; s=20210112;
        h=cc:to:subject:message-id:date:from:in-reply-to:references
         :mime-version:from:to:cc:subject:date:message-id:reply-to;
        bh=M4uck9Wy7FdrA5yiI7CkLiyXrPPt2gmkYC7TZJCdlrQ=;
        b=V9DWX3/WNrtihcy4x+nd/JdOMbkWADaH3tebtDZOSmfY8l6fY6vX5XLIPPa7whnYXw
         Y0EW36va1Zar9OUnWhXMgzpLqbNiB6tAF2jYqfnIlti3tAijRof0iHqjk4sWDFTga2XU
         yrpbUrd7ULvJOjJMemViaRezX2e/pWKZGBWOUHzCyddCnofbhIxzCgIbxeHYhvOCZVcG
         4/8bkyIC++O3boCq4H98atSL+obBSmT8GLA8jqa3ITmYSDw6uvjpZfs/cwXjU/hPxFNy
         xJJU9DY1COJ3vPFW6fai3JL5B7168gNohpqAw17Fxd5Z0PnmWgoM47sftca/eLl7CJOE
         1/Kg==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=1e100.net; s=20210112;
        h=cc:to:subject:message-id:date:from:in-reply-to:references
         :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id
         :reply-to;
        bh=M4uck9Wy7FdrA5yiI7CkLiyXrPPt2gmkYC7TZJCdlrQ=;
        b=inzYapMOSauCqKiUnxeYkGW/MLInLSiAwoFDJm84+YQtc56Zrc06Sf0NqJ67KtEof9
         l7uihfoViKRvLFCAAX7m6Bv+PfSOO98hVMX7EZiz1pQJfPQcvVGnRbJJVUl7gbQVGqn8
         uc6PFcmmAGXONcY4fQCBMUnW4Hig5gGJP+dpd1sDqcUvoG60mxbxRdWMNXZJHynkeED6
         VW+DoNXAXjfPY3ktwryWGceG8sOKUOd2DKiTCyz8K5gJZd7BE5Hy762BF4aCsIusm7Es
         BuKdWge0rrgQfPJTAo59HIrzuwlb+LuyNtIOc3vyWcOAEvG1++1wiixUVHI8w/+KU7mX
         XjAQ==
X-Gm-Message-State: AFqh2kogGuAnssHgtmIR8BjPDS59aiLQ+J44lszyFLmJzG5RjwUCUSxT
	2hWZlqNmRaX6By/MZ3qG3kwTo5MkKkudVDda2RuQyL9QOHmhbw==
X-Google-Smtp-Source: AMrXdXu1bQIM8qz7xkeIXPnGu+/XvgsNJ+X8BX1V+mKvLSXVeFAC5bJvNH7LRc8cvkrYy/Ur3WWHNksLD4ijwRgUClM=
X-Received: by 2002:a62:30c6:0:b0:581:6059:b7c8 with SMTP id
 w189-20020a6230c6000000b005816059b7c8mr2974841pfw.26.1673037590304; Fri, 06
 Jan 2023 12:39:50 -0800 (PST)
MIME-Version: 1.0
References: <20230106140403.2889131-1-kraxel@redhat.com> <20230106140403.2889131-3-kraxel@redhat.com>
In-Reply-To: <20230106140403.2889131-3-kraxel@redhat.com>
From: "Pedro Falcato" <pedro.falcato@gmail.com>
Date: Fri, 6 Jan 2023 20:39:38 +0000
Message-ID: <CAKbZUD0RfG_LeSHwmFHLAn_=k+9RjBezPKZiFUhfHbb--fQi-Q@mail.gmail.com>
Subject: Re: [edk2-devel] [PATCH 2/2] OvmfPkg/PlatformInitLib: check 64bit mmio window for resource conflicts
To: devel@edk2.groups.io, kraxel@redhat.com
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>, =?UTF-8?B?TMOhc3psw7Mgw4lyc2Vr?= <lersek@redhat.com>, 
	Jordan Justen <jordan.l.justen@intel.com>, Pawel Polawski <ppolawsk@redhat.com>, 
	Oliver Steffen <osteffen@redhat.com>, Jiewen Yao <jiewen.yao@intel.com>
Content-Type: multipart/alternative; boundary="000000000000441f7005f19e6b5f"

--000000000000441f7005f19e6b5f
Content-Type: text/plain; charset="UTF-8"

On Fri, Jan 6, 2023 at 2:04 PM Gerd Hoffmann <kraxel@redhat.com> wrote:

> Add new operation mode to PlatformScanOrAdd64BitE820Ram() which checks
> whenever any reservations from the qemu e820 table overlap with the mmio
> window.  Should that be the case move it to avoid the conflict.
>
Hi,

In what case could this happen? Does QEMU ever place e820 entries inside
the PCI MMIO window? Why?
(In fact, when does QEMU even add a reservation in the e820? I've never
seen one.)
Just checking for QemuOpenBoardPkg.

Thanks,
Pedro

--000000000000441f7005f19e6b5f
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div dir=3D"ltr">On Fri, Jan 6, 2023 at 2:04 PM Gerd Hoffm=
ann &lt;<a href=3D"mailto:kraxel@redhat.com">kraxel@redhat.com</a>&gt; wrot=
e:</div><div class=3D"gmail_quote"><blockquote class=3D"gmail_quote" style=
=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding=
-left:1ex">Add new operation mode to PlatformScanOrAdd64BitE820Ram() which =
checks<br>
whenever any reservations from the qemu e820 table overlap with the mmio<br=
>
window.=C2=A0 Should that be the case move it to avoid the conflict.<br></b=
lockquote><div>Hi,</div><div>=C2=A0<br></div><div>In what case could this h=
appen? Does QEMU ever place e820 entries inside the PCI MMIO window? Why?</=
div><div>(In fact, when does QEMU even add a reservation in the e820? I&#39=
;ve never seen one.)<br></div><div>Just checking for QemuOpenBoardPkg.</div=
><div><br></div><div>Thanks,</div><div>Pedro</div></div></div>

--000000000000441f7005f19e6b5f--