Re: [edk2] Question about PEX boot on Xen with OVMF as bios

2015-06-09 Thread Anthony PERARD
Xen PV disk driver in OVMF. > I think the infrastructure needed to port netfront driver is already > there. Just that the driver is not yet there. > > Anthony, am I right? That correct. But the infrastructure might need some refactori

Re: [edk2] [PATCH v5 23/29] Ovmf/Xen: port XenBusDxe to other architectures

2015-02-25 Thread Anthony PERARD
() using the same 16-bit compare and exchange, making > this module fully architecture agnostic. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Ard Biesheuvel Reviewed-by: Anthony PER

Re: [edk2] [PATCH v4 23/29] Ovmf/Xen: port XenBusDxe to other architectures

2015-02-23 Thread Anthony PERARD
On Mon, Feb 23, 2015 at 05:54:03PM +, Ard Biesheuvel wrote: > On 23 February 2015 at 15:16, Anthony PERARD > wrote: > > On Thu, Feb 12, 2015 at 07:19:15PM +0800, Ard Biesheuvel wrote: > >> This patch updates XenBusDxe to use the 16-bit compare and exchange > >>

Re: [edk2] [PATCH v4 23/29] Ovmf/Xen: port XenBusDxe to other architectures

2015-02-23 Thread Anthony PERARD
inding.h >ComponentName.c >ComponentName.h > - InterlockedCompareExchange16.c > - InterlockedCompareExchange16.h >GrantTable.c >GrantTable.h >EventChannel.c > @@ -45,14 +43,7 @

Re: [edk2] [PATCH v4 21/29] Ovmf/Xen: move XenBusDxe to abstract XENIO_PROTOCOL

2015-02-20 Thread Anthony PERARD
., as a UEFI Driver Model implementation for a PCI > device. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Acked-by: Laszlo Ersek > Signed-off-by: Ard Biesheuvel Reviewed-by: Anthony

Re: [edk2] [PATCH v4 20/29] Ovmf/Xen: add separate driver for Xen PCI device

2015-02-19 Thread Anthony PERARD
gt; abstract XENIO_PROTOCOL for XenBusDxe to bind against. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Reviewed-by: Laszlo Ersek > Signed-off-by: Ard Biesheuvel Reviewed-by: Anthony PERARD -- Anthony PERARD

Re: [edk2] [PATCH v4 18/29] Ovmf/Xen: move XenBusDxe hypercall code to separate library

2015-02-19 Thread Anthony PERARD
> of the code, such as a Xen console SerialPortLib driver. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Reviewed-by: Laszlo Ersek > Signed-off-by: Ard Biesheuvel Reviewed-by: Anthony PER

Re: [edk2] [PATCH v4 19/29] Ovmf/Xen: introduce XENIO_PROTOCOL

2015-02-19 Thread Anthony PERARD
that it can support non-PCI Xen implementations > such as Xen on ARM. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Reviewed-by: Laszlo Ersek > Signed-off-by: Ard Biesheuvel Reviewed-by: Anthony P

Re: [edk2] [PATCH v4 17/29] Ovmf/Xen: refactor XenBusDxe hypercall implementation

2015-02-19 Thread Anthony PERARD
v, >IN INTN Operation, >IN OUT VOID *Arguments >) > { > - ASSERT (Dev->Hyperpage != NULL); > - return XenHypercall2 ((UINT8*)Dev->Hyperpage + > __HYPERVISOR_event_channel_op * 32, > + return XenHypercall2 (__HYPERVISOR_event_channel_op, >

Re: [edk2] [PATCH v4 16/29] Ovmf/Xen: fix pointer to int cast in XenBusDxe

2015-02-19 Thread Anthony PERARD
es on 32-bit architectures, which may > require runtime library support. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Reviewed-by: Stefano Stabellini > Reviewed-by: Laszlo Ersek > Signed-off-by: Ard Biesheuvel Reviewed-by: Antho

Re: [edk2] [PATCH v4 15/29] Ovmf/Xen: move Xen interface version to

2015-02-19 Thread Anthony PERARD
ntribution Agreement 1.0 > Reviewed-by: Stefano Stabellini > Acked-by: Laszlo Ersek > Signed-off-by: Ard Biesheuvel Reviewed-by: Anthony PERARD -- Anthony PERARD -- Download BIRT iHub F-Type - The Free Ente

Re: [edk2] [PATCH 0/4] OvmfPkg/Xen* build fix for VS2010

2014-11-14 Thread Anthony PERARD
On Thu, Nov 13, 2014 at 05:27:08PM +, Anthony PERARD wrote: > Hi, > > I've took ovmf-ia32-msft-xen.patch from Scott Duplichan and made a few > changes. > > So the first two patchs of this series changes the type of some > variables/arguments/fields in order to avo

Re: [edk2] [PATCH v2 6/6] OvmfPkg XenBusDxe: Convert X64/TestAndClearBit.asm to NASM

2014-11-13 Thread Anthony PERARD
On Thu, Nov 13, 2014 at 07:00:18PM +0100, Laszlo Ersek wrote: > On 11/13/14 18:55, Anthony PERARD wrote: > > On Thu, Nov 13, 2014 at 12:13:23PM +0100, Laszlo Ersek wrote: > > >> But nasm emits the same warning. Any ideas? > > > > Yes, put lock and btr

[edk2] [PATCH 5/4] OvmfPkg/XenBusDxe: Fix a nasm warning about instruction not lockable.

2014-11-13 Thread Anthony PERARD
The fix, having "lock" and the locked instruction on the same line in the source. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- OvmfPkg/XenBusDxe/X64/TestAndClearBit.nasm | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

Re: [edk2] [PATCH v2 6/6] OvmfPkg XenBusDxe: Convert X64/TestAndClearBit.asm to NASM

2014-11-13 Thread Anthony PERARD
On Thu, Nov 13, 2014 at 12:13:23PM +0100, Laszlo Ersek wrote: > Anthony, > > On 11/06/14 12:24, Anthony PERARD wrote: > > The BaseTools/Scripts/ConvertMasmToNasm.py script was used to convert > > X64/TestAndClearBit.asm to X64/TestAndClearBit.nasm > > >

[edk2] [PATCH 2/4] OvmfPkg/XenBusDxe: Fix some types.

2014-11-13 Thread Anthony PERARD
This patch replace some types in GrantTable and the argument Index of XenHypercallHvmGetParam to what the types should be. This avoid to have type cast in code. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- OvmfPkg/XenBusDxe/GrantTable.c | 4

[edk2] [PATCH 0/4] OvmfPkg/Xen* build fix for VS2010

2014-11-13 Thread Anthony PERARD
cott patch and the many cast which I think are correct and cannot really be avoided. Thanks, Anthony PERARD (2): OvmfPkg/XenBusDxe: In XenStore, replace type of Len from UINTN to UINT32. OvmfPkg/XenBusDxe: Fix some types. Scott Duplichan (2): OvmfPkg/XenBusDxe: fix VS2010 build failures

[edk2] [PATCH 3/4] OvmfPkg/XenBusDxe: fix VS2010 build failures

2014-11-13 Thread Anthony PERARD
From: Scott Duplichan This patch contain only type cast. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Scott Duplichan Signed-off-by: Anthony PERARD --- OvmfPkg/XenBusDxe/EventChannel.c | 6 +++--- OvmfPkg/XenBusDxe/GrantTable.c | 4 ++-- OvmfPkg/XenBusDxe

[edk2] [PATCH 4/4] OvmfPkg/XenPvBlkDxe: fix VS2010 build failures

2014-11-13 Thread Anthony PERARD
From: Scott Duplichan This patch contain type casts and replace one * operation by a MultU64x32() call. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Scott Duplichan Signed-off-by: Anthony PERARD --- OvmfPkg/XenPvBlkDxe/BlockFront.c | 16 OvmfPkg

[edk2] [PATCH 1/4] OvmfPkg/XenBusDxe: In XenStore, replace type of Len from UINTN to UINT32.

2014-11-13 Thread Anthony PERARD
type cast were it does not make sense to have them. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- CC: Xen Devel --- OvmfPkg/XenBusDxe/XenStore.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/OvmfPkg/XenBusDxe

Re: [edk2] [PATCH v5 2/9] OvmfPkg: PlatformPei: Platform specific ACPI power management setup

2014-11-13 Thread Anthony PERARD
, > > and simply shutting up compiler warnings would be low on my list of > > suspects :) > > > > I think I'll just go for this instead: > > > > > UINTN PmCmd; > > > UINTN Pmba; > > > UINTN PmRegMisc; > > > > >

[edk2] [PATCH] OvmfPkg/XenBusDxe: Avoid typedef XENBUS_DEVICE twice.

2014-11-12 Thread Anthony PERARD
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- OvmfPkg/XenBusDxe/XenHypercall.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/OvmfPkg/XenBusDxe/XenHypercall.h b/OvmfPkg/XenBusDxe/XenHypercall.h index 3627b18..088037b 100644 --- a

Re: [edk2] Xcode build failure on OVMF

2014-11-12 Thread Anthony PERARD
k/src/edk2/OvmfPkg/XenBusDxe/XenBusDxe.h:86:31: note: > previous definition is here > typedef struct _XENBUS_DEVICE XENBUS_DEVICE; I'm going to send a patch to try to fix this, hopefully that will be enough. -- Anthony PERARD -

Re: [edk2] [PATCH v2] OvmfPkg/XenPvBlkDxe: Don't include system inttypes.h

2014-11-12 Thread Anthony PERARD
On Tue, Nov 11, 2014 at 04:12:24PM -0800, Jordan Justen wrote: > EDK II code should not include system include files. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Jordan Justen > Cc: Anthony PERARD > Cc: Scott Duplichan > --- > OvmfPkg/

[edk2] [PATCH v2 3/6] OvmfPkg XenBusDxe: Convert Ia32/TestAndClearBit.asm to NASM

2014-11-06 Thread Anthony PERARD
The BaseTools/Scripts/ConvertMasmToNasm.py script was used to convert Ia32/TestAndClearBit.asm to Ia32/TestAndClearBit.nasm Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- OvmfPkg/XenBusDxe/Ia32/TestAndClearBit.S| 13

[edk2] [PATCH v2 6/6] OvmfPkg XenBusDxe: Convert X64/TestAndClearBit.asm to NASM

2014-11-06 Thread Anthony PERARD
The BaseTools/Scripts/ConvertMasmToNasm.py script was used to convert X64/TestAndClearBit.asm to X64/TestAndClearBit.nasm Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- OvmfPkg/XenBusDxe/X64/TestAndClearBit.S | 12

[edk2] [PATCH v2 5/6] OvmfPkg XenBusDxe: Convert X64/InterlockedCompareExchange16.asm to NASM

2014-11-06 Thread Anthony PERARD
The BaseTools/Scripts/ConvertMasmToNasm.py script was used to convert X64/InterlockedCompareExchange16.asm to X64/InterlockedCompareExchange16.nasm Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- OvmfPkg/XenBusDxe/X64/InterlockedCompareExchange16.S

[edk2] [PATCH v2 0/6] OvmfPkg/XenBusDxe: Convert *.asm to NASM.

2014-11-06 Thread Anthony PERARD
Conversion done using the tools BaseTools/Scripts/ConvertMasmToNasm.py Change in V2: - one commit per assembly source files. Anthony PERARD (6): OvmfPkg XenBusDxe: Convert Ia32/hypercall.asm to NASM OvmfPkg XenBusDxe: Convert Ia32/InterlockedCompareExchange16.asm to NASM OvmfPkg

[edk2] [PATCH v2 2/6] OvmfPkg XenBusDxe: Convert Ia32/InterlockedCompareExchange16.asm to NASM

2014-11-06 Thread Anthony PERARD
The BaseTools/Scripts/ConvertMasmToNasm.py script was used to convert Ia32/InterlockedCompareExchange16.asm to Ia32/InterlockedCompareExchange16.nasm Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- OvmfPkg/XenBusDxe/Ia32/InterlockedCompareExchange16.S

[edk2] [PATCH v2 1/6] OvmfPkg XenBusDxe: Convert Ia32/hypercall.asm to NASM

2014-11-06 Thread Anthony PERARD
The BaseTools/Scripts/ConvertMasmToNasm.py script was used to convert Ia32/hypercall.asm to Ia32/hypercall.nasm Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- OvmfPkg/XenBusDxe/Ia32/hypercall.S | 22 -- .../Ia32

[edk2] [PATCH v2 4/6] OvmfPkg XenBusDxe: Convert X64/hypercall.asm to NASM

2014-11-06 Thread Anthony PERARD
The BaseTools/Scripts/ConvertMasmToNasm.py script was used to convert X64/hypercall.asm to X64/hypercall.nasm Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- OvmfPkg/XenBusDxe/X64/hypercall.S | 22 -- .../X64

Re: [edk2] [PATCH] OvmfPkg/XenBusDxe: Convert *.asm to NASM.

2014-11-05 Thread Anthony PERARD
g > then the script should produce individual commits for each assembly > file. I did not know about it :(, Thanks. I haven't try to run the script with -h or --help ..., but only once without any parameter, which did not display

[edk2] [PATCH] OvmfPkg/XenBusDxe: Convert *.asm to NASM.

2014-11-05 Thread Anthony PERARD
The BaseTools/Scripts/ConvertMasmToNasm.py script was used to convert all *.asm to *.nasm. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- .../XenBusDxe/Ia32/InterlockedCompareExchange16.S | 15 --- ...nge16.asm

[edk2] [PATCH 4/4] OvmfPkg/XenBusDxe: Fix initialisation of gXenBusDevicePathTemplate

2014-10-31 Thread Anthony PERARD
.. to avoid the use .member = value syntax as VS does not support it. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- OvmfPkg/XenBusDxe/XenBus.c | 20 +--- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/OvmfPkg/XenBusDxe

[edk2] [PATCH 0/4] OvmfPkg/Xen*: Try to fix building with VS

2014-10-31 Thread Anthony PERARD
Tested only with GCC. Anthony PERARD (4): OvmfPkg/Include/...Xen: Convert __i386__/__x86_64__ to MDE_CPU_IA32/MDE_CPU_X64. OvmfPkg/Xen*: Pass struct XENSTORE_TRANSACTION argument as a pointer OvmfPkg/XenBusDxe: Fix initialisation of gXenBusPrivateData OvmfPkg/XenBusDxe: Fix

[edk2] [PATCH 1/4] OvmfPkg/Include/...Xen: Convert __i386__/__x86_64__ to MDE_CPU_IA32/MDE_CPU_X64.

2014-10-31 Thread Anthony PERARD
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- OvmfPkg/Include/IndustryStandard/Xen/arch-x86/xen.h | 8 OvmfPkg/Include/IndustryStandard/Xen/hvm/params.h | 2 +- OvmfPkg/Include/IndustryStandard/Xen/io/blkif.h | 6 +++--- OvmfPkg/Include

[edk2] [PATCH 2/4] OvmfPkg/Xen*: Pass struct XENSTORE_TRANSACTION argument as a pointer

2014-10-31 Thread Anthony PERARD
As EDK II does not allow calls with a struct. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- OvmfPkg/Include/Protocol/XenBus.h | 14 +++--- OvmfPkg/XenBusDxe/XenBus.c| 2 +- OvmfPkg/XenBusDxe/XenStore.c | 36

[edk2] [PATCH 3/4] OvmfPkg/XenBusDxe: Fix initialisation of gXenBusPrivateData

2014-10-31 Thread Anthony PERARD
.. to avoid the use .member = value syntax as VS does not support it. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- OvmfPkg/XenBusDxe/XenBus.c | 53 +- 1 file changed, 29 insertions(+), 24 deletions(-) diff

Re: [edk2] [Xen-devel] [PATCH v4 00/19] Introducing Xen PV block driver to OVMF

2014-10-27 Thread Anthony PERARD
asm.py can you look at > converting the assembly to NASM with that script? Yes, that should not be a problem. Thank you. -- Anthony PERARD -- ___ edk2-devel mailing list

[edk2] [PATCH v4 13/19] OvmfPkg/XenBusDxe: Add an helper AsciiStrDup.

2014-10-23 Thread Anthony PERARD
.. because we need it in the patch titled: "OvmfPkg/XenBusDxe: Indroduce XenBus support itself." Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD Reviewed-by: Konrad Rzeszutek Wilk --- OvmfPkg/XenBusDxe/Helpers.c | 9 + OvmfPkg

[edk2] [PATCH v4 17/19] OvmfPkg/XenPvBlkDxe: Xen PV Block device, initial skeleton

2014-10-23 Thread Anthony PERARD
A ParaVirtualize block driver. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V4: - Replace the license by the commonly used file header text. - Add brief description for the driver. Change in V3: - enable compilation for Ia32 and Ia32X64

[edk2] [PATCH v4 11/19] OvmfPkg/XenBusDxe: Add TestAndClearBit.

2014-10-23 Thread Anthony PERARD
This atomically test's and clear's a bit. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD Reviewed-by: Konrad Rzeszutek Wilk --- Change in V3: - adding IA32 support. (not yet reviewed) both XenBusDxe/Ia32/TestAndClearBit.{S,asm} are new Ch

[edk2] [PATCH v4 15/19] OvmfPkg/XenBusDxe: Indroduce XenBus support itself.

2014-10-23 Thread Anthony PERARD
-by: Anthony PERARD --- Change in V4: - Replace the license by the commonly used file header text. - Clean XenBus.h header (remove copyright that does not belong to the file anymore; and rewrite the brief description of the file) - Fix description on the function Change in V3: - Insert to

[edk2] [PATCH v4 14/19] OvmfPkg/XenBusDxe: Add XenStore function into the XenBus protocol

2014-10-23 Thread Anthony PERARD
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD Reviewed-by: Konrad Rzeszutek Wilk --- Change in V3: - Have XenStoreWaitWatch/XenBusWaitForWatch return a XENSTORE_STATUS instead of VOID. - Add description of the introducted member of the protocol

[edk2] [PATCH v4 02/19] OvmfPkg: Add public headers from Xen Project.

2014-10-23 Thread Anthony PERARD
\2/g s/([^a-zA-Z0-9_]|^)char([^a-zA-Z0-9_]|$)/\1CHAR8\2/g s/([^a-zA-Z0-9_]|^)unsigned long([^a-zA-Z0-9_]|$)/\1UINTN\2/g s/([^a-zA-Z0-9_]|^)long([^a-zA-Z0-9_]|$)/\1INTN\2/g License: This patch adds many files under the MIT licence. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-of

[edk2] [PATCH v4 12/19] OvmfPkg/XenBusDxe: Add XenStore client implementation

2014-10-23 Thread Anthony PERARD
Signed-off-by: Anthony PERARD Reviewed-by: Konrad Rzeszutek Wilk --- Change in V3: - moving xs_wire.h from patch #1 to this patch - fix return value of XenStoreListDirectory - Use a timeout to print a debug message if the other side of the xenstore ring does not notify through the event channel

[edk2] [PATCH v4 18/19] OvmfPkg/XenPvBlkDxe: Add BlockFront client.

2014-10-23 Thread Anthony PERARD
are under the MIT license. Signed-off-by: Samuel Thibault Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V4: - add file header to BlockFront.h (license, copyright, brief desc) Change in V3: - Improve comment of XenBusReadUint64. - Moving

[edk2] [PATCH v4 19/19] OvmfPkg/XenPvBlkDxe: Add BlockIo.

2014-10-23 Thread Anthony PERARD
Implement the BlockIo protocol. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V4: - Replace the license by the commonly used file header text. Change in V3: - assert(Media->BlockSize % 512 == 0) - Use Sector instead of Offset to issue

[edk2] [PATCH v4 05/19] OvmfPkg/XenBusDxe: Add support to make Xen Hypercalls.

2014-10-23 Thread Anthony PERARD
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD Reviewed-by: Konrad Rzeszutek Wilk --- Change in V4: - Replace the license by the commonly used file header text. - add file header to XenHypercall.h (license, copyright, brief desc) Change in V3: - adding

[edk2] [PATCH v4 06/19] OvmfPkg/XenBusDxe: Open PciIo protocol.

2014-10-23 Thread Anthony PERARD
The PciIo interface will be used in "OvmfPkg/XenBusDxe: Add Grant Table functions" to get the memory address of the BAR 1 and use the space to map shared memory. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V3: - add a commit d

[edk2] [PATCH v4 09/19] OvmfPkg/XenBusDxe: Add Grant Table functions.

2014-10-23 Thread Anthony PERARD
There are used to grant access of pages to other Xen domains. This code originaly comes from the Xen Project, and more precisely from MiniOS. Signed-off-by: Steven Smith Signed-off-by: Grzegorz Milos Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD

[edk2] [PATCH v4 00/19] Introducing Xen PV block driver to OVMF

2014-10-23 Thread Anthony PERARD
vmfPkg: Add the MIT license to License.txt". Other changes have been listed after a '---' line in every patches. Anthony PERARD (19): OvmfPkg: Add the MIT license to License.txt. OvmfPkg: Add public headers from Xen Project. OvmfPkg: Add basic skeleton for the XenBus bus driver.

[edk2] [PATCH v4 10/19] OvmfPkg/XenBusDxe: Add Event Channel Notify.

2014-10-23 Thread Anthony PERARD
This first function is used to notify the other side that there is something to do. The other side is another Xen domain. License: This patch adds event_channel.h which is under MIT licence. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V4

[edk2] [PATCH v4 04/19] OvmfPkg/XenBusDxe: Add device state struct and create an ExitBoot services event.

2014-10-23 Thread Anthony PERARD
The ExitBoot event is used to disconnect from the device before the next operating system start using them. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V3: - use the variable mMyDevice to prevent the driver from starting twice (if there

[edk2] [PATCH v4 03/19] OvmfPkg: Add basic skeleton for the XenBus bus driver.

2014-10-23 Thread Anthony PERARD
This includes Component Name and Driver Binding. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD Reviewed-by: Konrad Rzeszutek Wilk --- Change in V4: - Replace the license by the commonly used file header text. - Add brief description for the driver

[edk2] [PATCH v4 01/19] OvmfPkg: Add the MIT license to License.txt.

2014-10-23 Thread Anthony PERARD
: Anthony PERARD --- Change in V3: New patch --- OvmfPkg/License.txt | 25 + 1 file changed, 25 insertions(+) diff --git a/OvmfPkg/License.txt b/OvmfPkg/License.txt index be68999..e2eff41 100644 --- a/OvmfPkg/License.txt +++ b/OvmfPkg/License.txt @@ -23,3 +23,28

[edk2] [PATCH v4 16/19] OvmfPkg/XenBusDxe: Add Event Channel into XenBus protocol.

2014-10-23 Thread Anthony PERARD
: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V3: - eventchannel, update protocol to return error code. - expand patch description - Add comments in the XenBus Protocol header. Change in V2: - coding style - adding comment to functions - Rename Xenbus to XenBus

[edk2] [PATCH v4 07/19] OvmfPkg: Introduce XenBus Protocol.

2014-10-23 Thread Anthony PERARD
This protocol will be used for communication between a PV driver (like a PV block driver) and the XenBus/XenStore. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD -- Change in V3: - Add disclaimer about the volatile nature of the protocol. - Add a

[edk2] [PATCH v4 08/19] OvmfPkg/XenBusDxe: Add InterlockedCompareExchange16.

2014-10-23 Thread Anthony PERARD
This patch is inspired by InterlockedCompareExchange32 from the BaseSynchronizationLib. The function will be used in the "OvmfPkg/XenBusDxe: Add Grant Table functions" patch. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V3: -

Re: [edk2] [PATCH v3 15/19] OvmfPkg/XenBusDxe: Indroduce XenBus support itself.

2014-10-20 Thread Anthony PERARD
On Sun, Oct 19, 2014 at 05:20:45PM -0700, Jordan Justen wrote: > On 2014-10-17 10:03:58, Anthony PERARD wrote: > > This is a bus-like on top of XenStore. It will look for advertised > > ParaVirtualized devices and initialize them by producing XenBus > > protocol. > >

Re: [edk2] [PATCH v3 08/19] OvmfPkg/XenBusDxe: Add InterlockedCompareExchange16.

2014-10-20 Thread Anthony PERARD
On Sun, Oct 19, 2014 at 05:00:13PM -0700, Jordan Justen wrote: > On 2014-10-17 10:03:51, Anthony PERARD wrote: > > This patch is inspired by InterlockedCompareExchange32 from the > > BaseSynchronizationLib. > > Why not attempt to add it to BaseSynchronizationLib? The BaseSy

Re: [edk2] [PATCH v3 03/19] OvmfPkg: Add basic skeleton for the XenBus bus driver.

2014-10-20 Thread Anthony PERARD
On Sun, Oct 19, 2014 at 04:27:11PM -0700, Jordan Justen wrote: > On 2014-10-17 10:03:46, Anthony PERARD wrote: > > This includes Component Name and Driver Binding. > > > > Contributed-under: TianoCore Contribution Agreement 1.0 > > Signed-off-by: Anthony PERARD > &

Re: [edk2] [PATCH v3 02/19] OvmfPkg: Add public headers from Xen Project.

2014-10-20 Thread Anthony PERARD
On Sun, Oct 19, 2014 at 04:47:32PM -0700, Jordan Justen wrote: > On 2014-10-17 10:03:45, Anthony PERARD wrote: > > This patch imports publics headers in order to use features from Xen > > like XenStore, PV Block... There is only the necessary header files and > > there are onl

[edk2] [PATCH v3 19/19] OvmfPkg/XenPvBlkDxe: Add BlockIo.

2014-10-17 Thread Anthony PERARD
Implement the BlockIo protocol. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V3: - assert(Media->BlockSize % 512 == 0) - Use Sector instead of Offset to issue IOs. Change in V2: - Remove blockIo2 headers. - Fix few comment. - file hea

[edk2] [PATCH v3 14/19] OvmfPkg/XenBusDxe: Add XenStore function into the XenBus protocol

2014-10-17 Thread Anthony PERARD
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD Reviewed-by: Konrad Rzeszutek Wilk --- Change in V3: - Have XenStoreWaitWatch/XenBusWaitForWatch return a XENSTORE_STATUS instead of VOID. - Add description of the introducted member of the protocol

[edk2] [PATCH v3 10/19] OvmfPkg/XenBusDxe: Add Event Channel Notify.

2014-10-17 Thread Anthony PERARD
This first function is used to notify the other side that there is something to do. The other side is another Xen domain. License: This patch adds event_channel.h which is under MIT licence. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V3

[edk2] [PATCH v3 16/19] OvmfPkg/XenBusDxe: Add Event Channel into XenBus protocol.

2014-10-17 Thread Anthony PERARD
: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V3: - eventchannel, update protocol to return error code. - expand patch description - Add comments in the XenBus Protocol header. Change in V2: - coding style - adding comment to functions - Rename Xenbus to XenBus

[edk2] [PATCH v3 15/19] OvmfPkg/XenBusDxe: Indroduce XenBus support itself.

2014-10-17 Thread Anthony PERARD
-by: Anthony PERARD --- Change in V3: - Insert to ChildList later, once populated. - Remove XENBUS_XENSTORE_NODE macro. - add comment to XenBusAddDevice and XenBusEnumerateBus about concurrency calls. - Add a description to the introduced member to the protocol. Change in V2: - comment, file

[edk2] [PATCH v3 17/19] OvmfPkg/XenPvBlkDxe: Xen PV Block device, initial skeleton

2014-10-17 Thread Anthony PERARD
A ParaVirtualize block driver. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V3: - enable compilation for Ia32 and Ia32X64 - fix version (driver binding) Change in V2: - Add minimal support for controller name - Remove stuff about BlockIo2

[edk2] [PATCH v3 13/19] OvmfPkg/XenBusDxe: Add an helper AsciiStrDup.

2014-10-17 Thread Anthony PERARD
.. because we need it in the patch titled: "OvmfPkg/XenBusDxe: Indroduce XenBus support itself." Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD Reviewed-by: Konrad Rzeszutek Wilk --- OvmfPkg/XenBusDxe/Helpers.c | 9 + OvmfPkg

[edk2] [PATCH v3 12/19] OvmfPkg/XenBusDxe: Add XenStore client implementation

2014-10-17 Thread Anthony PERARD
Signed-off-by: Anthony PERARD Reviewed-by: Konrad Rzeszutek Wilk --- Change in V3: - moving xs_wire.h from patch #1 to this patch - fix return value of XenStoreListDirectory - Use a timeout to print a debug message if the other side of the xenstore ring does not notify through the event channel

[edk2] [PATCH v3 11/19] OvmfPkg/XenBusDxe: Add TestAndClearBit.

2014-10-17 Thread Anthony PERARD
This atomically test's and clear's a bit. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD Reviewed-by: Konrad Rzeszutek Wilk --- CC: Konrad Rzeszutek Wilk Change in V3: - adding IA32 support. (not yet reviewed) both XenBusDxe/Ia32/TestAndC

[edk2] [PATCH v3 18/19] OvmfPkg/XenPvBlkDxe: Add BlockFront client.

2014-10-17 Thread Anthony PERARD
are under the MIT license. Signed-off-by: Samuel Thibault Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V3: - Improve comment of XenBusReadUint64. - Moving blkif.h to this patch with the necessary #pragma pack(4) applied for Ia32. - Add a

[edk2] [PATCH v3 05/19] OvmfPkg/XenBusDxe: Add support to make Xen Hypercalls.

2014-10-17 Thread Anthony PERARD
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD Reviewed-by: Konrad Rzeszutek Wilk --- CC: Konrad Rzeszutek Wilk Change in V3: - adding IA32 support. (not reviewed yet) both XenBusDxe/Ia32/hypercall.{S,asm} file are new Change in V2: - file header

[edk2] [PATCH v3 03/19] OvmfPkg: Add basic skeleton for the XenBus bus driver.

2014-10-17 Thread Anthony PERARD
This includes Component Name and Driver Binding. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD Reviewed-by: Konrad Rzeszutek Wilk --- Change in V3: - enable compilation for Ia32 and Ia32X64 - fix version (driver binding) Change in V2: - Simple support

[edk2] [PATCH v3 08/19] OvmfPkg/XenBusDxe: Add InterlockedCompareExchange16.

2014-10-17 Thread Anthony PERARD
This patch is inspired by InterlockedCompareExchange32 from the BaseSynchronizationLib. The function will be used in the "OvmfPkg/XenBusDxe: Add Grant Table functions" patch. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V3: -

[edk2] [PATCH v3 02/19] OvmfPkg: Add public headers from Xen Project.

2014-10-17 Thread Anthony PERARD
-zA-Z0-9_]|$)/\1UINTN\2/g s/([^a-zA-Z0-9_]|^)long([^a-zA-Z0-9_]|$)/\1INTN\2/g License: This patch adds many files under the MIT licence. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V3: - Remove unused header sched.h - moving xs_wire.h in

[edk2] [PATCH v3 01/19] OvmfPkg: Add the MIT license to License.txt.

2014-10-17 Thread Anthony PERARD
: Anthony PERARD --- Change in V3: New patch --- OvmfPkg/License.txt | 25 + 1 file changed, 25 insertions(+) diff --git a/OvmfPkg/License.txt b/OvmfPkg/License.txt index be68999..e2eff41 100644 --- a/OvmfPkg/License.txt +++ b/OvmfPkg/License.txt @@ -23,3 +23,28

[edk2] [PATCH v3 06/19] OvmfPkg/XenBusDxe: Open PciIo protocol.

2014-10-17 Thread Anthony PERARD
The PciIo interface will be used in "OvmfPkg/XenBusDxe: Add Grant Table functions" to get the memory address of the BAR 1 and use the space to map shared memory. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V3: - add a commit d

[edk2] [PATCH v3 00/19] Introducing Xen PV block driver to OVMF

2014-10-17 Thread Anthony PERARD
Other changes have been listed after a '---' line in every patches. Anthony PERARD (19): OvmfPkg: Add the MIT license to License.txt. OvmfPkg: Add public headers from Xen Project. OvmfPkg: Add basic skeleton for the XenBus bus driver. OvmfPkg/XenBusDxe: Add device state struct a

[edk2] [PATCH v3 09/19] OvmfPkg/XenBusDxe: Add Grant Table functions.

2014-10-17 Thread Anthony PERARD
There are used to grant access of pages to other Xen domains. This code originaly comes from the Xen Project, and more precisely from MiniOS. Signed-off-by: Steven Smith Signed-off-by: Grzegorz Milos Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD

[edk2] [PATCH v3 07/19] OvmfPkg: Introduce XenBus Protocol.

2014-10-17 Thread Anthony PERARD
This protocol will be used for communication between a PV driver (like a PV block driver) and the XenBus/XenStore. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD -- Change in V3: - Add disclaimer about the volatile nature of the protocol. - Add a

[edk2] [PATCH v3 04/19] OvmfPkg/XenBusDxe: Add device state struct and create an ExitBoot services event.

2014-10-17 Thread Anthony PERARD
The ExitBoot event is used to disconnect from the device before the next operating system start using them. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V3: - use the variable mMyDevice to prevent the driver from starting twice (if there

Re: [edk2] [Xen-devel] [PATCH v2 18/18] OvmfPkg/XenPvBlkDxe: Add BlockIo.

2014-09-18 Thread Anthony PERARD
On Fri, Sep 12, 2014 at 10:57:53AM -0400, Konrad Rzeszutek Wilk wrote: > On Thu, Sep 04, 2014 at 05:51:13PM +0100, Anthony PERARD wrote: > > Install the BlockIo protocol. > > Implement. Perhaps point out that the blkif.h is a good literature > to understand it? Maybe not in

Re: [edk2] [Xen-devel] [PATCH v2 17/18] OvmfPkg/XenPvBlkDxe: Add BlockFront client.

2014-09-18 Thread Anthony PERARD
"XenPvBlk: unrecognized block operation %d response > > (status %d)\n", > > +Response->operation, Status)); > > +break; > > + } > > + > > + Dev->Ring.rsp_cons = ++ConsumerIndex; > > + if (IoData != NU

Re: [edk2] [Xen-devel] [PATCH v2 16/18] OvmfPkg/XenPvBlkDxe: Xen PV Block device, initial skeleton

2014-09-18 Thread Anthony PERARD
kg/OvmfPkg.dec > > + > > +[Sources] > > + XenPvBlkDxe.h > > + XenPvBlkDxe.c > > + ComponentName.c > > + ComponentName.h > > + > > + > > +[LibraryClasses] > > + UefiDriverEntryPoint > > + UefiBootServicesTableLib > > + MemoryAllocationLib >

Re: [edk2] [Xen-devel] [PATCH v2 15/18] OvmfPkg/XenBusDxe: Add Event Channel into XenBus protocol.

2014-09-18 Thread Anthony PERARD
On Fri, Sep 12, 2014 at 09:58:53AM -0400, Konrad Rzeszutek Wilk wrote: > On Thu, Sep 04, 2014 at 05:51:10PM +0100, Anthony PERARD wrote: > > Contributed-under: TianoCore Contribution Agreement 1.0 > > Signed-off-by: Anthony PERARD > > While the code is fine, please expand t

Re: [edk2] [Xen-devel] [PATCH v2 14/18] OvmfPkg/XenBusDxe: Indroduce XenBus support itself.

2014-09-18 Thread Anthony PERARD
On Thu, Sep 11, 2014 at 01:10:14PM -0400, Konrad Rzeszutek Wilk wrote: > On Thu, Sep 04, 2014 at 05:51:09PM +0100, Anthony PERARD wrote: > > This is a bus-like on top of XenStore. It will look for advertised > > ParaVirtualized devices and initialize them by producing XenB

Re: [edk2] [Xen-devel] [PATCH v2 11/18] OvmfPkg/XenBusDxe: Add XenStore client implementation

2014-09-18 Thread Anthony PERARD
On Wed, Sep 10, 2014 at 04:48:50PM -0400, Konrad Rzeszutek Wilk wrote: > On Thu, Sep 04, 2014 at 05:51:06PM +0100, Anthony PERARD wrote: > > XenStore is a key/value database, which is running on another virtual > > machine. It can be accessed through shared memory. T

Re: [edk2] [Xen-devel] [PATCH v2 09/18] OvmfPkg/XenBusDxe: Add Event Channel Notify.

2014-09-18 Thread Anthony PERARD
On Wed, Sep 10, 2014 at 04:15:03PM -0400, Konrad Rzeszutek Wilk wrote: > On Thu, Sep 04, 2014 at 05:51:04PM +0100, Anthony PERARD wrote: > > This first function is used to notify the other side that there is > > something to do. The other side is another Xen domain. > >

Re: [edk2] [Xen-devel] [PATCH v2 07/18] OvmfPkg/XenBusDxe: Add InterlockedCompareExchange16.

2014-09-18 Thread Anthony PERARD
On Wed, Sep 10, 2014 at 04:09:43PM -0400, Konrad Rzeszutek Wilk wrote: > On Thu, Sep 04, 2014 at 05:51:02PM +0100, Anthony PERARD wrote: > > This is a copy of InterlockedCompareExchange32 from the > > BaseSynchronizationLib. > > > > The function will be used in the ne

Re: [edk2] [Xen-devel] [PATCH v2 03/18] OvmfPkg/XenBusDxe: Add device state struct and create an ExitBoot services event.

2014-09-18 Thread Anthony PERARD
On Wed, Sep 10, 2014 at 03:58:16PM -0400, Konrad Rzeszutek Wilk wrote: > On Thu, Sep 04, 2014 at 05:50:58PM +0100, Anthony PERARD wrote: > > The ExitBoot event is used to disconnect from the device before the > > next operating system start using them. > > > > C

Re: [edk2] [Xen-devel] [PATCH v2 00/18] Introducing Xen PV block driver to OVMF

2014-09-08 Thread Anthony PERARD
On Fri, Sep 05, 2014 at 12:30:05PM -0700, Jordan Justen wrote: > On Fri, Sep 5, 2014 at 6:34 AM, Anthony PERARD > wrote: > > On Thu, Sep 04, 2014 at 08:01:07PM +0200, Laszlo Ersek wrote: > >> On 09/04/14 18:50, Anthony PERARD wrote: > >> > Hi all, > >>

Re: [edk2] [PATCH v2 00/18] Introducing Xen PV block driver to OVMF

2014-09-05 Thread Anthony PERARD
On Thu, Sep 04, 2014 at 08:01:07PM +0200, Laszlo Ersek wrote: > On 09/04/14 18:50, Anthony PERARD wrote: > > Hi all, > > > > This patch series is implementing the necessary in order to access a PV > > block > > device. For that, one need a XenStore client, a Xen

[edk2] [PATCH v2 15/18] OvmfPkg/XenBusDxe: Add Event Channel into XenBus protocol.

2014-09-04 Thread Anthony PERARD
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V2: - coding style - adding comment to functions - Rename Xenbus to XenBus. --- OvmfPkg/Include/Protocol/XenBus.h | 27 +++ OvmfPkg/XenBusDxe/EventChannel.c | 55

[edk2] [PATCH v2 18/18] OvmfPkg/XenPvBlkDxe: Add BlockIo.

2014-09-04 Thread Anthony PERARD
Install the BlockIo protocol. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V2: - Remove blockIo2 headers. - Fix few comment. - file header, copyright - Rewrite few comment and error messages - No more callback - Improving block read/write

[edk2] [PATCH v2 17/18] OvmfPkg/XenPvBlkDxe: Add BlockFront client.

2014-09-04 Thread Anthony PERARD
-off-by: Anthony PERARD --- Change in V2: - trigger CoW is probably not needed on OVMF (as opposed to Mini-OS), removed the test. - comments - renamed XenbusReadInteger to XenBusReadUint64 - remove callback from IoData, use simple status instead - return a status from the synchronus io - Close

[edk2] [PATCH v2 12/18] OvmfPkg/XenBusDxe: Add an helper AsciiStrDup.

2014-09-04 Thread Anthony PERARD
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- OvmfPkg/XenBusDxe/Helpers.c | 9 + OvmfPkg/XenBusDxe/XenBusDxe.h | 5 + 2 files changed, 14 insertions(+) create mode 100644 OvmfPkg/XenBusDxe/Helpers.c diff --git a/OvmfPkg/XenBusDxe

[edk2] [PATCH v2 10/18] OvmfPkg/XenBusDxe: Add TestAndClearBit.

2014-09-04 Thread Anthony PERARD
This atomically test and clear a bit. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V2: - Adding .asm version - Comment the function --- OvmfPkg/XenBusDxe/X64/TestAndClearBit.S | 13 + OvmfPkg/XenBusDxe/X64/TestAndClearBit.asm

[edk2] [PATCH v2 11/18] OvmfPkg/XenBusDxe: Add XenStore client implementation

2014-09-04 Thread Anthony PERARD
XenStore is a key/value database, which is running on another virtual machine. It can be accessed through shared memory. This is a client implementation. Origin: FreeBSD 10.0 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Anthony PERARD --- Change in V2: - Change

  1   2   >