On Mon, Apr 15, 2019 at 03:49:25PM +0200, Laszlo Ersek wrote:
> On 04/15/19 15:33, Laszlo Ersek wrote:
> > On 04/09/19 13:08, Anthony PERARD wrote:
> >> diff --git a/OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c
> >> b/OvmfPkg/Library/PlatformBootManagerL
On Fri, Apr 12, 2019 at 01:15:48PM +0200, Laszlo Ersek wrote:
> On 04/09/19 13:08, Anthony PERARD wrote:
> > Rework XenPublishRamRegions for PVH, handle the Reserve type and explain
> > about the ACPI type. MTRR settings aren't modified anymore, on HVM, it's
> > already done
go through? The Xen one?
>
> Fine with me. I could also include it in a "miscellaneous cleanup" pull
> request along with other cleanup patches I got in flight.
Markus, I don't have any other Xen patches, so could you include this
one in your pull request?
Thanks,
--
An
hat without the ELFNOTE, libxc will load an ELF but with
the plain ELF loader, which doesn't check for shstrtab.
Signed-off-by: Anthony PERARD
---
Notes:
v2: Rework commit message, explain why the check is unnecessary.
tools/libxc/xc_dom_elfloader.c | 9 -
1 file changed, 9 deletion
On Thu, May 16, 2019 at 10:38:54PM +0100, Julien Grall wrote:
> Hi Anthony,
>
> Thank you for CCing me.
>
> On 5/16/19 11:37 AM, Anthony PERARD wrote:
> > On Wed, May 15, 2019 at 07:48:17PM +, osstest service owner wrote:
> > > flight 136184 qemu-upstream-4.11-
Ping?
On Tue, Apr 09, 2019 at 05:45:33PM +0100, Anthony PERARD wrote:
> This patch series fix libxl_cdrom_insert to work with a depriviledge QEMU. For
> that, we need to use libxl__ev_qmp. For that, we need a new lock because
> userdata_lock can't be used anymore.
>
> Patch s
On Thu, May 23, 2019 at 10:41:48AM +0100, Anthony PERARD wrote:
> On Wed, May 22, 2019 at 07:54:42PM +0200, Olaf Hering wrote:
> > Am Wed, 22 May 2019 15:51:40 +0100
> > schrieb Anthony PERARD :
> >
> > > Can you give it a try with one of the affected qemu? (qemu-x
y a patch to the
qemu-xen tree:
https://lists.xenproject.org/archives/html/xen-devel/2019-05/msg01840.html
--
Anthony PERARD
___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
On Tue, Jun 04, 2019 at 06:29:14PM +0100, Ian Jackson wrote:
> Anthony PERARD writes ("[PATCH 6/9] libxl_disk: Cut libxl_cdrom_insert into
> steps .."):
> > This patch cut libxl_cdrom_insert into different step/function but there
> > are still called synchronously.
Set-up page permission for Xen mappings earlier
> on"
This commit alone, cherry-picked on top of stable-4.11 makes xen boot on
rochester.
>- f60658c6ae "xen/arm: Stop relocating Xen"
With that commit applied, xen doesn't build. So I couldn't try to boot
it. (mm.c: In
On Tue, Jun 04, 2019 at 06:47:32PM +0100, Ian Jackson wrote:
> Anthony PERARD writes ("[PATCH 9/9] libxl_disk: Implement missing timeout for
> libxl_cdrom_insert"):
> > Since the previous patch "libxl_disk: Use ev_qmp in libxl_cdrom_insert",
> > there are no
On Tue, Jun 04, 2019 at 06:11:23PM +0100, Ian Jackson wrote:
> Anthony PERARD writes ("[PATCH 4/9] libxl_internal: Create new lock for
> devices hotplug via QMP"):
> > The current lock `domain_userdata_lock' can't be used when modification
> > to a guest is don
Update to the latest stable tag.
Signed-off-by: Anthony PERARD
---
Config.mk | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Config.mk b/Config.mk
index 417039d7f6..0fa4591379 100644
--- a/Config.mk
+++ b/Config.mk
@@ -274,7 +274,7 @@ QEMU_TRADITIONAL_URL ?=
git
On Mon, Jun 17, 2019 at 05:15:51PM +0100, Paul Durrant wrote:
> > -Original Message-
> > From: Anthony PERARD [mailto:anthony.per...@citrix.com]
> > Sent: 17 June 2019 16:41
> > To: qemu-de...@nongnu.org
> > Cc: xen-devel@lists.xenproject.org; Anthony Pe
Hi,
Fix the build in osstest and some cleanup
For reference:
Recent flight failure:
https://lists.xenproject.org/archives/html/xen-devel/2019-06/msg01022.html
Bisect result:
https://lists.xenproject.org/archives/html/xen-devel/2019-06/msg01029.html
Thanks.
Anthony PERARD (4):
xen: Fix
ce/" along with blkif.h and
usbif.h as those describe interface (and would allow to also import a
xen.h header if that was necessary).
xen-mapcache.c doesn't needs grant_table.h.
Signed-off-by: Anthony PERARD
---
hw/9pfs/xen-9pfs.h | 2 +-
hw/block/xen_blkif.h
d up a bit while importing them. The header
guard symbols are changed to match QEMU's coding style, some other
part of the file that QEMU doesn't use are removed.
Signed-off-by: Anthony PERARD
---
hw/9pfs/xen-9pfs.h |2 +-
hw/block/xen_blkif.h|2 +
Avoid using a variable length array.
Signed-off-by: Anthony PERARD
---
Notes:
Was suggested by Peter here:
"should we try to stop using variable length arrays?"
hw/i386/xen/xen-hvm.c | 11 ---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/hw/i3
re removed.
xen-mapcache.c doesn't needs params.h, so remove the include.
Signed-off-by: Anthony PERARD
---
hw/i386/xen/xen-hvm.c | 6 +-
hw/i386/xen/xen-mapcache.c| 2 -
include/hw/xen/interface/hvm/e820.h | 28 +++
include/hw/xen/interface/hvm/iore
On Mon, Jun 17, 2019 at 05:45:44PM +0100, Anthony PERARD wrote:
> On Mon, Jun 17, 2019 at 05:15:51PM +0100, Paul Durrant wrote:
> > AFAICT the only place (apart from legacy code in xen_common.h) that
> > params.h is necessary is in xen_suspend_notifier(). I wonder whether
> >
tmap when
`log_for_dirtybit' is cleared.
Thanks,
--
Anthony PERARD
___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
: xen.next
Thanks.
Anthony PERARD (2):
Revert xen/io/ring.h of "Clean up a few header guard symbols"
xen: Import other xen/io/*.h
hw/9pfs/xen-9pfs.h |4 +-
hw/block/xen_blkif.h |5 +-
hw/char/xen_console.c|2 +-
en headers have been imported (by
f65eadb639 "xen: import ring.h from xen") but not all. With the change
in the guards symbole, the ring.h header start to be imported twice.
Signed-off-by: Anthony PERARD
---
CC: Markus Armbruster
---
Notes:
v3:
- new patch, replace "xen:
On Tue, Jun 18, 2019 at 01:14:08PM +0100, Daniel P. Berrangé wrote:
> On Tue, Jun 18, 2019 at 12:23:38PM +0100, Anthony PERARD wrote:
> > Following 37677d7db3 "Clean up a few header guard symbols", QEMU start
> > to fail to build:
> >
> > In file included fro
://winpvdrvbuild.xenproject.org:8080/job/XENVBD-master/126
Signed-off-by: Paul Durrant
Reviewed-by: Anthony PERARD
Message-Id: <20190409164038.25484-1-paul.durr...@citrix.com>
[Edited error message]
Signed-off-by: Anthony PERARD
---
hw/block/dataplane/xen-block.c | 25
en headers have been imported (by
f65eadb639 "xen: import ring.h from xen") but not all. With the change
in the guards symbole, the ring.h header start to be imported twice.
Signed-off-by: Anthony PERARD
Reviewed-by: Daniel P. Berrangé
Reviewed-by: Paul Durrant
Message-Id: <20190621105441
/close in xen_device_realize/unrealize().
NOTE: This patch does not add an AioContext parameter to
xen_device_bind_event_channel(). That will be done in a subsequent
patch.
Signed-off-by: Paul Durrant
Reviewed-by: Anthony PERARD
Message-Id: <20190408151617.13025-2-paul.d
for doing a deferred poll of the shared ring should be because
there were previously insufficient resources to fully complete a previous
poll.
Signed-off-by: Paul Durrant
Reviewed-by: Anthony PERARD
Message-Id: <20190408151617.13025-4-paul.durr...@citrix.com>
Signed-off-by: Anthony PERARD
-
From: Paul Durrant
This patch adds an AioContext parameter to xen_device_bind_event_channel()
and then uses aio_set_fd_handler() to set the callback rather than
qemu_set_fd_handler().
Signed-off-by: Paul Durrant
Reviewed-by: Anthony PERARD
Message-Id: <20190408151617.13025-3-paul.d
.
(xenctrl.h does include params.h)
Signed-off-by: Anthony PERARD
Reviewed-by: Paul Durrant
Message-Id: <20190618112341.513-4-anthony.per...@citrix.com>
---
hw/i386/xen/xen-hvm.c | 1 -
hw/i386/xen/xen-mapcache.c | 2 --
2 files changed, 3 deletions(-)
diff --git a/hw/i386/xen/xen-h
* xen-block: Support IOThread polling for PV shared rings
* Avoid usage of a VLA
* Cleanup Xen headers usage
Anthony PERARD (4):
xen: Avoid VLA
xen: Drop includes of xen/hvm/params.h
Revert xen/io/ring.h of "Cle
Avoid using a variable length array.
We allocate the `dirty_bitmap' buffer only once when we start tracking
for dirty bits.
Signed-off-by: Anthony PERARD
Reviewed-by: Paul Durrant
Reviewed-by: Philippe Mathieu-Daudé
Message-Id: <20190618112341.513-5-anthony.per...@citrix.com>
---
hw/i3
d up a bit while importing them. The header
guard symbols are changed to match QEMU's coding style, some other
part of the file that QEMU doesn't use are removed.
Signed-off-by: Anthony PERARD
Reviewed-by: Paul Durrant
---
hw/9pfs/xen-9pfs.h |2 +-
hw/block/x
ce/" along with blkif.h and
usbif.h as those describe interface (and would allow to also import a
xen.h header if that was necessary).
xen-mapcache.c doesn't needs grant_table.h.
Signed-off-by: Anthony PERARD
Reviewed-by: Paul Durrant
---
hw/9pfs/xen-9pfs.h | 2 +-
hw/blo
Hi,
Fix the build in osstest and some cleanup
For reference:
Recent flight failure:
https://lists.xenproject.org/archives/html/xen-devel/2019-06/msg01022.html
Bisect result:
https://lists.xenproject.org/archives/html/xen-devel/2019-06/msg01029.html
Thanks.
Anthony PERARD (4):
xen: Fix
Avoid using a variable length array.
We allocate the `dirty_bitmap' buffer only once when we start tracking
for dirty bits.
Signed-off-by: Anthony PERARD
---
Notes:
v2:
- Allocate the bitmap buffer only once when we start tracking dirty bits.
(instead of at every function call
.
(xenctrl.h does include params.h)
Signed-off-by: Anthony PERARD
---
Notes:
v2:
- new patch replacing "xen: Import Xen public headers used by xen-hvm.c"
- only drop includes of params.h, and don't import the other two;
ioreq.h is an interface with Xen; e820.h is just us
On Tue, Jun 18, 2019 at 08:55:53AM +0100, Paul Durrant wrote:
> > -Original Message-
> > From: Anthony PERARD [mailto:anthony.per...@citrix.com]
> >
> > On Mon, Jun 17, 2019 at 05:45:44PM +0100, Anthony PERARD wrote:
> > > On Mon, Jun 17, 2019 at 05:
c = ERROR_FAIL;
goto out;
}
rc=0;
out:
GC_FREE;
return rc;
> +}
> +
Thanks,
--
Anthony PERARD
___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
itignore
> index 26bc583f74..3822bb75ba 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -30,6 +30,7 @@ cscope.out
> cscope.po.out
> .config
> .vimrc
> +.vscode
Shouldn't this go in "~/.config/git/ignore" instead? Or whatever `git
config core.excludesFile` poin
and 3.1
Signed-off-by: Anthony PERARD
---
Hi Olaf, Roger, Wei,
Instead of a fix in libxl (with the reverted patch "libxl: fix
migration of PV and PVH domUs with and without qemu"), what do you
think of this QEMU patch? It is much simpler that trying to work
around the issue in libxl,
a softlink
"python-config" to "python2-config" as well and the "checking for
PyArg_ParseTuple" is more likely to succeed.
Cheers,
--
Anthony PERARD
___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
On Wed, May 22, 2019 at 07:54:42PM +0200, Olaf Hering wrote:
> Am Wed, 22 May 2019 15:51:40 +0100
> schrieb Anthony PERARD :
>
> > Can you give it a try with one of the affected qemu? (qemu-xen-4.10 or
> > qemu-xen-4.11)
>
> Thanks for the patch. Unfortunately ther
that the -qcow2 test which now keeps failing isn't blocking?
Anyway, the *arm64* tests succeed, and the *qcow2 test keeps failing.
But the qcow2 issue seems to be a guest kernel thread failing to
suspend, so not a bug in qemu, but a regression the guest used for
testing, like in xen
u know what could cause xen to fail to boot?
I don't believe a few more patch on top of qemu-xen would.
Thanks,
--
Anthony PERARD
___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
't matter
to qemu-upstream-* flights.
Thanks,
--
Anthony PERARD
___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
On Wed, May 15, 2019 at 01:07:03PM +0100, Andrew Cooper wrote:
> On 15/05/2019 12:40, Anthony PERARD wrote:
> > This was probably useful to load ELF Note, but now ELF notes
> > "should live in a PT_NOTE segment" (elfnote.h).
> >
> > With notes living in segmen
are a blob as compaptible with the PVH
boot ABI.
Note that without the ELFNOTE, libxc will load an ELF but with
the plain ELF loader, which doesn't check for shstrtab.
Signed-off-by: Anthony PERARD
---
tools/libxc/xc_dom_elfloader.c | 9 -
1 file changed, 9 deletions(-)
diff --git a/t
s, Device, Function, PCI_COMMAND_OFFSET)
> > + );
> > +
> >for (Offset = BarOffsetBase; Offset < BarOffsetEnd; Offset += sizeof
> > (UINT32)) {
> > PcatPciRootBridgeBarExisted (
> >PCI_LIB_ADDRESS (Bus, Device, Function, Offset),
> >
&
-amd64
>build-arm64
>build-armhf
>build-i386
> change
>revision_ovmf to the output of ./ap-fetch-version-baseline ovmf
>tree_ovmf to git://xenbits.xen.org/osstest/ovmf.git
> (each being previously set to the empty strin
-off-by: Anthony PERARD
---
This patch would unblock the qemu-mainline branch in osstest.
---
tools/Makefile | 1 -
1 file changed, 1 deletion(-)
diff --git a/tools/Makefile b/tools/Makefile
index c903d6a63e..99cbc950dc 100644
--- a/tools/Makefile
+++ b/tools/Makefile
@@ -246,7 +246,6 @@ subdir
This new XenHvmloaderDetected() return true if the hvmloader firmware
has runned before OVMF.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
Acked-by: Laszlo Ersek
---
Notes:
v3:
- Added one sentence in the commit message.
OvmfPkg
aren't modified anymore, on HVM, it's
already done by hvmloader, on PVH it is supposed to have sane default.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
Acked-by: Laszlo Ersek
---
Notes:
Comment for Xen people:
About MTRR, should we redo the setting
nt
Signed-off-by: Anthony PERARD
---
xen/include/public/io/ring.h | 15 +++
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git a/xen/include/public/io/ring.h b/xen/include/public/io/ring.h
index c5d53e3103..5610ae14f8 100644
--- a/xen/include/public/io/ring.h
+++ b/xen/include
The purpose of XenPlatformLib is to regroup the few functions that are
used in several places to detect if Xen is detected, and to get the
XenInfo HOB.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
---
Notes:
v3:
- use SPDX
- add
Use the already checked pointer mXenHvmloaderInfo to retrieve the E820
table produced by hvmloader.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
Acked-by: Laszlo Ersek
---
OvmfPkg/XenPlatformPei/Xen.c | 18 +-
1 file changed, 9
When the device ID of the host bridge is unknown, check if we are
running as a PVH guest as there is no PCI bus in that case.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
---
Notes:
v3:
- Remove use of XEN_PVH_PCI_HOST_BRIDGE_DEVICE_ID
ne.
Based on the "PcAtChipsetPkg/8254TimerDxe" implementation.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
---
Notes:
v3:
- rebased, SPDX, copyright
v2:
- Use InitializeApicTimer instead of WriteLocalApicReg
- rework comments (re
: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
Acked-by: Laszlo Ersek
---
OvmfPkg/XenPlatformPei/Platform.h | 5 +
OvmfPkg/XenPlatformPei/Platform.c | 2 ++
OvmfPkg/XenPlatformPei/Xen.c | 23 ---
3 files changed, 19 insertions(+), 11
This patch replace the XenDetected() function by the one in
XenPlatformLib.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
---
Notes:
v3:
- new patch, splited from the next patch
(which was OvmfPkg/AcpiPlatformDxe: Use PVH RSDP if exist
If the firmware have been started via the Xen PVH entry point, a RSDP
pointer would have been provided. Use it.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
---
Notes:
v3:
- patch splited from the previous one
- Fix DEBUG format string, use
://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
Acked-by: Laszlo Ersek
---
Notes:
v3:
- fix coding style
- fix commit message
OvmfPkg/Include/Guid/XenInfo.h | 4
OvmfPkg/PlatformPei/Xen.c | 3 ---
OvmfPkg/XenPlatformPei/Xen.c | 25
Replace the XenDetected() implementation by the one from
XenPlatformLib.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
---
Notes:
v3:
- new patch
.../PlatformBootManagerLib.inf| 1 +
.../PlatformBootManagerLib/BdsPlatform.c
.
Have PlatformBootManagerLib look for the new console, when running as a
Xen guest.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
---
Notes:
v3:
- removed PciSioSerialDxe and IsaSerialDxe from OvmfXen, since they
would not be used, maybe
?id=1689
Signed-off-by: Anthony PERARD
---
Notes:
v3:
- Instead of checking for a false value,
XEN_PVH_PCI_HOST_BRIDGE_DEVICE_ID, simply check if we are running xen
when the HostBridge device ID isn't recognised.
OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c | 6
: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
Acked-by: Laszlo Ersek
---
Notes:
v3:
- moved RealTimeClockLib|*/XenRealTimeClockLib.inf to the global
[LibraryClasses]
OvmfPkg/OvmfXen.dsc | 3 ++-
OvmfPkg/OvmfXen.fdf | 2 +-
2 files changed, 3
Signed-off-by: Anthony PERARD
Acked-by: Laszlo Ersek
---
Notes:
v3:
- fix commit message
- add 'm' prefix to the global variables
and make them static
OvmfPkg/XenPlatformPei/Xen.c | 46 +++-
1 file changed, 45 insertions(+), 1 deletion(-)
diff
XenPvhDetected() can be used to figure out if OVMF has started via the
Xen PVH entry point.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
Acked-by: Laszlo Ersek
---
OvmfPkg/XenPlatformPei/Platform.h | 5 +
OvmfPkg/XenPlatformPei/Xen.c | 13
always succeed.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
---
Notes:
v3:
- only modify XenHypercallLib, and to the modification of XenPlatformPei
in a separated patch.
- Allow XenHypercallLibInit to be called outside the library instead
://lists.xenproject.org/archives/html/xen-devel/2019-03/msg02139.html
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
Acked-by: Laszlo Ersek
---
Notes:
v3:
- fix commit message
OvmfPkg/XenPlatformPei/Xen.c | 16
1 file changed, 16
for the Grant Tables.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
---
Notes:
v3:
- downgrade type to DXE_DRIVER
- use SPDX
- rework InitializeXenIoPvhDxe, and handle errors properly.
- Free the reserved allocation in ExitBootServices
The XenPlatformPei needs to make hypercalls, but the XenHypercallLib was
initialised before the HyperPage was ready. Now that XenPlatformPei has
initialised the HyperPage, reinitialise the XenHypercallLib.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
PcdFSBClock is used by SecPeiDxeTimerLibCpu, the TimerLib
implementation. It will also be used by XenTimerDxe. Override
PcdFSBClock to match Xen vLAPIC timer frequency.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
Acked-by: Laszlo Ersek
---
Notes
Check if there's a start of the day struct provided to PVH guest, save
the ACPI RSDP address for later.
This patch import import arch-x86/hvm/start_info.h from xen.git.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
Acked-by: Laszlo Ersek
---
Notes
Move XenRealTimeClockLib from ArmVirtPkg to OvmfPkg so it can be used
from the OvmfPkg by the following patch, "OvmfPkg/OvmfXen: use
RealTimeClockRuntimeDxe from EmbeddedPkg"
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
Reviewed-by: La
Signed-off-by: Anthony PERARD
---
Notes:
v3:
- new patch
.../Library/XenPlatformLib/XenPlatformLib.c| 18 +++---
1 file changed, 15 insertions(+), 3 deletions(-)
diff --git a/OvmfPkg/Library/XenPlatformLib/XenPlatformLib.c
b/OvmfPkg/Library/XenPlatformLib
to:
include/hw/xen/io/ring.h:47:5: error: "__XEN_INTERFACE_VERSION__"
is not defined, evaluates to 0 [-Werror=undef]
Cleanup ring.h. The xen_*mb() macros are already defined in xenctrl.h
which is included in xen_common.h.
Reported-by: Markus Armbruster
Signed-off-by: Anth
Reserve hvmloader's memory only when it has run.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
Acked-by: Laszlo Ersek
---
Notes:
v3:
- fix empty commit message body
OvmfPkg/XenPlatformPei/Xen.c | 4 +++-
1 file changed, 3 insertions(+), 1
The informations to make a XENMEM_memory_map hypercall is copied over
from the public header of the Xen Project, with the type name modified
to build on OVMF.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
Acked-by: Laszlo Ersek
---
Notes:
v3
module variable mXen, as it should be always true.
- Have the platform PEI initialization fails if Xen has not been
detected.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
Reviewed-by: Laszlo Ersek
---
Notes:
v3:
- fix coding style in new code
to TPM2_ENABLE==true
- changed: PcdPciDisableBusEnumeration dynamic default flipped to TRUE
- changed: default FD_SIZE_IN_KB to 2M.
- reverted d272449d9e1e, "OvmfPkg: raise DXEFV size to 11 MB"
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
---
Not
.
To build and boot:
To build, simply run OvmfPkg/build.sh -p OvmfPkg/OvmfXen.dsc
Then use OVMF.fd as a kernel of a pvh guest config file (with xl/libxl).
Patch series available in this git branch:
https://xenbits.xen.org/git-http/people/aperard/ovmf.git br.platform-xen-pvh-v3
Anthony PERARD (35
signature (for "Xen PVH").
[1] https://xenbits.xenproject.org/docs/unstable/misc/pvh.html
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
Acked-by: Laszlo Ersek
---
Notes:
v3:
- rename PcdXenStartOfDay* to PcdXenPvhStartOfDay*
- Use f
?id=1689
Signed-off-by: Anthony PERARD
Acked-by: Laszlo Ersek
---
Notes:
v3:
- rebased, SPDX
- fix commit message
.../XenResetVector/Ia16/Real16ToFlat32.asm| 3 +
.../XenResetVector/Ia32/Flat32ToFlat64.asm| 68 +++
.../XenResetVector/Ia32/SearchForBfvBase.asm
Add missing dependency on PciLib
and remove extra includes of OvmfPlatforms.h.
Signed-off-by: Anthony PERARD
Reviewed-by: Laszlo Ersek
---
Notes:
v2:
- also add PciLib.h include to the .c
- and remove extra include of OvmfPlatforms.h
OvmfPkg/Library/ResetSystemLib
-off-by: Anthony PERARD
---
Notes:
v3:
- added license to generate_elf_header.c
which is renamed to OvmfPkg/OvmfXenElfHeaderGenerator.c
- added an ELF NOTE into the header
OvmfPkg/OvmfXen.fdf | 101 +++-
OvmfPkg/OvmfXenElfHeaderGenerator.c | 140
r complete if the OS have
disabled the Local APIC Timer.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
---
Notes:
v3:
- cleanup .dsc, leave only one TimerLib resolution
- Added a note in the commit message regarding the use of the
commit "OvmfPkg/XenResetVector: Add new entry point
for Xen PVH"
Value FILE_GUID of XenResetVector have not changed compare to ResetVector
because it is a special value (gEfiFirmwareVolumeTopFileGuid).
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689
Signed-off-by: Anthony PERARD
-by: Anthony PERARD
---
Notes:
v3:
- rebased, SPDX
- remove `cli' as via PVH the interrupts are guaranteed to be off
- rewrite some comments
.../XenResetVector/Ia16/ResetVectorVtf0.asm | 81 +++
OvmfPkg/XenResetVector/Ia32/XenPVHMain.asm| 49 +++
OvmfPkg
recent Windows PV XENVBD
> > driver [2] using a xen-disk device with a 4kB logical block size.
> >
> > [1]
> > http://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=67e1c050e36b2c9900cca83618e56189effbad98
> > [2] https://winpvdrvbuild.xenproject.org:8080/job/XENVBD-ma
moval may not be done
>in device model's side.
Reviewed-by: Anthony PERARD
Thanks,
--
Anthony PERARD
___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
ead" could you do something
like the following?
char *colo_compare_iothread_id = GCSPRINTF("colo-compare-iothread-%d", i);
flexarray_append_pair(dm_args, "-object",
GCSPRINTF("iothread,id=%s", colo_compare_iothread_id);
// then us
eturn that error.
Thanks.
> +break;
> + }
> +LOGD(DEBUG, qmp->domid,
> + "Waiting for completion of deleting device %s", id);
> +sleep(1);
> +} while (retry++ < 5);
> +}
--
Anthony PERARD
___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
On Mon, Jun 17, 2019 at 05:10:50PM +0100, Anthony PERARD wrote:
> Update to the latest stable tag.
>
> Signed-off-by: Anthony PERARD
Hi,
Could one of you commit this patch?
Thanks,
--
Anthony PERARD
___
Xen-devel mailing list
On Mon, Jul 01, 2019 at 03:19:29PM +, Jan Beulich wrote:
> On 01.07.2019 17:04, Anthony PERARD wrote:
> > On Mon, Jun 17, 2019 at 05:10:50PM +0100, Anthony PERARD wrote:
> >> Update to the latest stable tag.
> >>
> >> Signed-off-by: Anthony PERARD
> &g
bh_schedule()
> is moved into xen_block_complete_aio(), which is more intuitive since the
> only reason for doing a deferred poll of the shared ring should be because
> there were previously insufficient resources to fully complete a previous
> poll.
>
> Signed-off-by: Paul Durrant
> ---
Revie
"logical_block_size != %u not supported",
Maybe add "by frontend" to the error message?
> + XEN_BLKIF_SECTOR_SIZE);
> +return;
> +}
> +
With the question answered:
Reviewed-by: Anthony PERARD
Thanks,
--
Anthony PERARD
___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
On Wed, Apr 10, 2019 at 04:20:05PM +0100, Paul Durrant wrote:
> > -Original Message-
> > From: Anthony PERARD [mailto:anthony.per...@citrix.com]
> > Sent: 10 April 2019 13:57
> > To: Paul Durrant
> > Cc: qemu-de...@nongnu.org; qemu-bl...@nongnu.org;
>
sg02782.html
I had to add the `live' parameter to fix the lock issue.
> During a migration the disks have to be released either way. During "xl save"
> the domU may continue to run, if '-c' was specified.
xl save -c is a snapshot, right? There isn't a second QEMU process that
is goi
ne.
Based on the "PcAtChipsetPkg/8254TimerDxe" implementation.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Anthony PERARD
---
Notes:
v2:
- Use InitializeApicTimer instead of WriteLocalApicReg
- rework comments (remove many that don't apply)
- remove unuse
601 - 700 of 2950 matches
Mail list logo